Code Smell - Le rasoir de Hanlon-js tutoriel-php.cn

Code Smell - Le rasoir de Hanlon

PHPz

Libérer： 2024-08-18 07:23:32

original

1027 Les gens l'ont consulté

Ne compliquez pas trop : restez simple

TL;DR : Un code trop défensif conduit à une complexité inutile.

Problèmes

Complexité inutile
Logique déroutante
Bogues cachés
Entretien plus dur
Performances plus lentes
Code encombré

Solutions

Simplifier les contrôles
Faites confiance à votre logique
Concentrez-vous sur l'essentiel
Suivez le K.I.S.S. principe
Refactor régulièrement

Contexte

Réfléchir et concevoir trop votre code peut conduire à une complexité inutile.

Vous devrez peut-être vous défendre contre tous scénarios possibles, mais cette approche produit souvent un code volumineux et déroutant.

Hanlon's Razor suggère que vous ne devriez pas présumer de la méchanceté lorsque de simples erreurs ou malentendus sont plus probables.

Évitez une programmation trop défensive et concentrez-vous sur une logique claire et directe.

Vous pourriez anticiper des problèmes futurs qui pourraient ne jamais se produire ou essayer de rendre votre code trop flexible.

Un code simple est plus facile à maintenir, à déboguer et à comprendre.

Exemple de code

Faux

function processData(data) {
    if (typeof data === 'undefined') {
        throw new Error('Data is undefined');
    }

    if (typeof data !== 'object') {
        throw new Error('Data is not an object');
    }

    if (data === null) {
        throw new Error('Data is null');
    }

    if (Array.isArray(data)) {
        throw new Error('Data should not be an array');
    }

    if (!data.hasOwnProperty('items')) {
        return [];
    }

    if (!Array.isArray(data.items)) {
        throw new Error('Items should be an array');
    }

    if (data.items.length === 0) {
        return []; 
    }

    let processedItems = [];
    for (let item of data.items) {
        if (typeof item === 'undefined') {
            continue; // Skip undefined items
        }

        if (typeof item !== 'object') {
            continue; // Skip non-object items
        }

        if (item === null) {
            continue; // Skip null items
        }

        processedItems.push(processItem(item));
    }

    return processedItems;
}

Copier après la connexion

Droite

function processData(data) {
    if (!Array.isArray(data.items)) {
        throw new Error('Invalid data');
    }

    return data.items
        .filter(item => typeof item === 'object' && item !== null)
        .map(item => processItem(item));
}

Copier après la connexion

Détection

[X] Manuel

Le code compliqué comporte généralement plus de lignes et les méthodes longues sont un indice possible.

Balises

Bouffants

Niveau

[x] Intermédiaire

Génération d'IA

Les générateurs d'IA peuvent introduire cette odeur lorsqu'ils tentent de prendre en compte tous les cas extrêmes possibles.

Par exemple, gérer les NULL n'est pas nécessaire si vous les évitez complètement.

Détection IA

Les outils d'IA peuvent aider à détecter un code trop défensif en analysant la logique et en suggérant des simplifications avec des conseils appropriés.

Ces outils recommandent souvent de supprimer les vérifications inutiles ou de les combiner pour plus de clarté.