Perché non bisogna sovrascrivere gli argomenti delle funzioni: prevenire confusione e problemi di debugging

Regola

Non sovrascrivere gli argomenti
delle funzioni. Riassegnare i parametri
può confondere chi chiama la
funzionee rendere difficile il debugging.

Linguaggi supportati: 45+

Introduzione

La riassegnazione dei parametri di una funzione modifica il valore degli argomenti all'interno della funzione stessa, rendendo incerto il valore effettivo del parametro in un dato momento. Durante il debugging, non è possibile dare per scontato che un parametro contenga ancora il valore passato dal chiamante. Questo genera confusione per chiunque legga il codice e rende più difficile individuare i bug, poiché il valore del parametro cambia nel corso dell'esecuzione.

Esempi di codice

❌ Non conforme:

function processUser(user) {
    if (!user) {
        user = { id: null, name: 'Guest' };
    }

    user = { ...user, processedAt: Date.now() };

    if (user.age < 18) {
        user = { ...user, restricted: true };
    }

    return user;
}

Perché è errato: Il parametro user viene riassegnato più volte, rendendo impossibile sapere cosa abbia passato il chiamante originale senza leggere l'intera funzione. Il debugging mostra valori modificati anziché l'input effettivo, ostacolando il tracciamento dei problemi alla fonte.

✅ Conforme:

function processUser(user) {
    const currentUser = user || { id: null, name: 'Guest' };

    const processedUser = {
        ...currentUser,
        processedAt: Date.now()
    };

    if (currentUser.age < 18) {
        return { ...processedUser, restricted: true };
    }

    return processedUser;
}

Perché questo è importante: Le nuove variabili (currentUser, processedUser) contengono valori intermedi mentre l'originale user il parametro rimane invariato. I debugger mostrano il valore di input effettivo e il codice traccia chiaramente le trasformazioni attraverso variabili denominate.

Conclusione

Usa nuove variabili per le trasformazioni anziché riassegnare i parametri. Questo rende esplicito il flusso dei dati e preserva l'input originale per il debug. La leggera verbosità delle dichiarazioni di variabili extra è ampiamente ripagata da una migliore leggibilità e manutenibilità in fase di debug.