マイナーな構文の傷を参照することにより、グローバル変数を渡すと、4から別に参照される変数

Question

行うための正しい方法は何

...

var array1 = [] 
var array2 = [] 

function doIt(arg){ 
    var myArray; 

    if(arg == 1){ 
     myArray = array1 
    }else if(arg == 2){ 
     myArray = array2 
    } 

    myArray.push('test'); 

} 

doIt(1); //array1 should now be ['test'] but it's empty 

Answer 1

を変更し、あなたのコードはOKです：http://jsfiddle.net/hk9Md/

var array1 = []; // <-- added semi-colon 
var array2 = []; // <-- added semi-colon 

function doIt(arg){ 
    var myArray; 

    if (arg == 1) { 
     myArray = array1; // <-- added semi-colon 
    } else if (arg == 2) { 
     myArray = array2; // <-- added semi-colon 
    } 
    myArray.push('test'); 
} 
doIt(1); 
alert(array1[0]); // produces 'test' 

Answer 2

なぜない：

var myArray = arg == 1? array1 : array2; 

2つ以上の選択肢がある場合は、

var source = { 
    '1': array1, 
    '2': array2, 
    '3': array3, 
    ... 
} 

またはゼロインデックス：

次いで

var source = [ ,array1, array2, array3, ...]; 

：

function doIt(arg) { 
    var myArray = source[arg]; 
    ... 
} 

あるいは：

var doIt = (function() { 
    var source = { 
    '1': array1, 
    '2': array2, 
    '3': array3, 
    ... 
    } 
    return function (arg) { 
    var myArray = source[arg]; 
    } 
}()); 

ヤーブーツを埋めます。 :-)

Answer 3

おそらく、これを行うための「正しい」方法は、副作用のある関数を書くことではありません。それは、配列の変更されたコピーを返すようにするには、関数を書くことができます：

var array1 = [1, 2, 3], 
    array2 = [4, 5, 6], 
    x = getUserInput(), 
    result = doIt(1, x == 1 ? array1 : array2); 

function doIt(arg, original_array){ 
    var copy = [].concat(original_array); 
    // ... 
    return copy; 
} 

時にはそれは避けられないですが、地元の関数はグローバルな状態を変更したときにプログラムがデバッグに非常に難しくなります。