代碼氣味 - Hanlon&#s Razor-js教程-PHP中文網

代碼氣味 - Hanlon&#s Razor

PHPz

發布： 2024-08-18 07:23:32

原創

1065 人瀏覽過

不要太複雜：保持簡單

TL;DR：過度防禦的程式碼會導致不必要的複雜性。

問題

不必要的複雜性
邏輯混亂
隱藏的錯誤
更難維護
效能較慢
混亂的程式碼

解決方案

簡化檢查
相信你的邏輯
專注於必需品
跟隨 K.I.S.S.原則
定期重構

情境

過度思考和過度設計你的程式碼可能會導致不必要的複雜性。

您可能需要防禦每種可能的情況，但這種方法通常會產生臃腫、混亂的程式碼。

漢隆剃刀建議，當更可能發生簡單錯誤或誤解時，您不應抱持惡意。

避免過度防禦性程式設計並專注於清晰、直接的邏輯。

您可能會預見到未來可能永遠不會發生的問題，或者嘗試使您的程式碼過於靈活。

簡單的程式碼更容易維護、調試和理解。

範例程式碼

錯誤的

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

登入後複製

正確的

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

登入後複製