Bau Kod - Pisau Cukur Hanlon-tutorial js-php.cn

Bau Kod - Pisau Cukur Hanlon

PHPz

Lepaskan： 2024-08-18 07:23:32

asal

991 orang telah melayarinya

Jangan Terlalu Rumit: Pastikan Ia Mudah

TL;DR: Kod overdefensive membawa kepada kerumitan yang tidak perlu.

Masalah

Kerumitan yang tidak perlu
Logik mengelirukan
Pepijat tersembunyi
Penyelenggaraan yang lebih keras
Prestasi yang lebih perlahan
Kod Berterabur

Penyelesaian

Memudahkan semakan
Percaya logik anda
Fokus pada perkara penting
Ikuti K.I.S.S. prinsip
Refactor selalu

Konteks

Terlalu berfikir dan mereka bentuk kod anda boleh membawa kepada kerumitan yang tidak perlu.

Anda mungkin perlu bertahan daripada setiap senario yang mungkin, tetapi pendekatan ini selalunya menghasilkan kod yang mengelirukan.

Hanlon's Razor mencadangkan agar anda tidak menganggap niat jahat apabila kesilapan mudah atau salah faham lebih berkemungkinan besar.

Elakkan pengaturcaraan yang terlalu defensif dan fokus pada logik yang jelas dan jelas.

Anda mungkin menjangkakan masalah masa depan yang mungkin tidak akan berlaku atau cuba menjadikan kod anda terlalu fleksibel.

Kod ringkas lebih mudah diselenggara, nyahpepijat dan difahami.

Kod Contoh

salah

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;
}

Salin selepas log masuk

Betul

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));
}

Salin selepas log masuk