укупорочные Javascript, сохранить внешний объект

Question

Я использую Javascript затворов, чтобы защитить переменные, сделав их локальными для возвращенных функций, таких как:

closure = function() { 
    var secretVar = 'Secret'; 

    return { 
     "msg" : function() { 
      console.log(secretVar); 
     } 
    }; 
}(); 
console.log(closure.secretVar); // Undefined 
closure.msg(); // Secret 

Я чувствую, что у меня есть довольно хорошее представление о том, что, давая мне возможность контролировать доступ к внутренним переменным, если это вообще возможно.

Я сейчас работаю в этой проблему

closure = function() { 
    var secretVar = ['Secret One','Secret Two']; 

    return { 
     "del" : function(modMe) { 
      modMe = secretVar; 
      modMe.slice(1,1); 
      console.log(modMe); 
     }(secretVar), 
     "secretVar" : function() { 
      console.log(secretVar); 
     } 
    }; 
}(); 
closure.del(); // Secret One 
closure.secretVar(); // Secret One 

Я хочу closure.del(), чтобы вернуть секретные, но я хочу, чтобы secretVar остались нетронутыми, однако, это не так. Функция del() изменяет ссылку, а не копию, и я не уверен, как заставить ее копировать secretVar и модифицировать ее.

Я предполагаю, что это будет в виде

(function(y) { 
    //..Body that changes y 
})(secretVar) 

, но я не смог получить, чтобы работать. Есть идеи?

Answer 1

Ваша проблема не имеет ничего общего с закрытием. Когда вы делаете:

modMe = secretVar; 

Вы просто создать новую переменную, указывающую на же массив. То, что вы делаете с одним, будет отражено в обеих переменных, поскольку они указывают на одно и то же.

Если вы хотите, чтобы выполнить какое-то изменения на массиве (при сохранении оригинала), вам нужно скопировать его первым:

var copied = []; 
for(var i = 0; i < secretVar.length; i++){ 
    copied.push(secretVar[i]); 
} 

Edit: Как и в стороне, когда вы говорите, что вы используете замыкание для «защиты переменных» вы не защищаете их от изменения возвращаемыми функциями, которые выполняли закрытие. Вы просто сделали это так, чтобы переменная была недоступна из-за пределов этих функций. Но внутри области, например, когда вы делаете свой фрагмент, переменная существует и доступна для этой функции и не является «защищенной» только потому, что она является закрытием.

Edit 2: Если вы собираетесь копировать массив часто, вы можете устранить некоторые раздражения итерации, создавая closured функцию, чтобы сделать копирование для вас:

closure = function() { 
    var secretVar = 'Secret'; 
    var clone = function(){ 
     var clonedArr = []; 
     var length = secretVar.length; 
     for(var i = 0; i < length; i++){ 
      clonedArr.push(secretVar[i]); 
     } 
     return clonedArr; 
    } 

    return { 
     "msg" : function() { 
      var duplicate = clone(); 
     } 
    }; 
}(); 

Answer 2

Роберт сделал Хороший комментарий, но также обратите внимание, что closure.del не является функцией, но не определено. Еще раз посмотрите на свой код. Функция автоматически выполняется и ничего не возвращает, поэтому del будет ничем.

var del = function(modMe) { 
    modMe = secretVar; 
    modMe.slice(1,1); 
    console.log(modMe); 
}(secretVar); 

Это вообще такая форма:

var a = function(b){ 
    console.log(b) 
}('input'); 

console.log(typeof a); // undefined 
a(); // error: a is not a function 

Answer 3

Ваше выражение function(modMe) { ... }(secretVar) фактически возвращает значение, а не функцию, так как вы выполняете функцию. В сочетании с ответом Мэтта, что вам нужно:

1. Удалите (secretVar) компонент из выражения, а также параметр modMe - функция уже имеет доступ к secretVar в любом случае.
2. На самом деле скопируйте массив, прежде чем над ним работать. См. Ответ Мэтта.

Answer 4

Очень близко - форма:

(function(y) { 
    return function() { 
     // that changes y 
    }; 
})(secretVar) 

Примечание: Это будет по-прежнему передавать ссылку, и любые деструктивные или изменяющие операции по-прежнему влияют на secretVar. Вам нужно будет сделать глубокую копию secretVar, если вы хотите избежать ее изменения.

SEE: http://jsfiddle.net/B3ryr/2/