JSLint - Non si modifica il parametro <x> quando si usano 'argomenti'?

Question

Tutti,

Io uso JSLint per convalidare i miei file JS. Nel mio progetto più recente, sto usando il seguente formato per impostare i valori predefiniti per una serie di funzioni JavaScript (ulteriormente dettagliato here):

function(a, b, option) { 
    option = arguments.length > 2 ? option : "some default value"; 
    // ... 
} 

Questo però fa sì che l'ultima build di JSLint per produrre il seguente errore:

"Do not mutate parameter 'option' when using 'arguments'." 

Sono consapevole del fatto che l'utilizzo del metodo più comune per l'assegnazione dei valori predefiniti (ovvero) sopprime l'errore; tuttavia, questo produrrà un comportamento errato se intendo passare un valore false a option.

È l'unica soluzione a questo problema per introdurre una nuova variabile? es .:

var option2 = arguments.length > 2 ? option : "some default value"; 

Answer 1

Credo JSLint si avverte dal momento che si tenta di modificare uno degli argomenti di ingresso utilizzando un assegno con la parola chiave argomenti. JSHint, tuttavia, mi dà alcun avvertimento quando provo il tuo codice.

Una soluzione al problema potrebbe essere quella di verificare se option è definito o meno, in questo modo si va in giro il problema con l'invio dei valori falsy:

function(a, b, option) { 
    if(typeof option === "undefined") { 
     option = "some default value"; 
    } 
    // ... 
} 

Se si scopre che è ingombrante per scrivere questo typeof verifica ogni volta, creare una funzione isDef:

function isDef(param) { 
    return typeof param !== "undefined"; 
} 
function(a, b, option) { 
    option = isDef(option) ? option : "some default value"; 
    // ... 
} 

// Simon A.

Answer 2

Nella maggior parte situazione s si sarebbe meglio solo facendo:

function(a, b, option) { 
    option = option || "some default value"; 
    // ... 
} 

o

function(a, b, option) { 
    if (!option || typeof option !== 'string') { 
     option = "some default value"; 
    } 
    // ... 
}