코드 냄새 - 한론의 면도기-JS 튜토리얼-php.cn

코드 냄새 - 한론의 면도기

PHPz

풀어 주다： 2024-08-18 07:23:32

원래의

1026명이 탐색했습니다.

너무 복잡하지 말고 단순하게 유지하세요

TL;DR: 과도한 방어 코드는 불필요한 복잡성을 초래합니다.

문제

불필요한 복잡성
혼란스러운 논리
숨겨진 버그
더욱 강화된 유지관리
성능 저하
복잡한 코드

솔루션

검사 단순화
당신의 논리를 믿으세요
필수에 집중
K.I.S.S를 팔로우하세요. 원칙
정기적으로 리팩터링

문맥

코드를 지나치게 생각하고 과도하게 설계하면 불필요한 복잡성이 발생할 수 있습니다.

가능한 모든 시나리오를 방어해야 할 수도 있지만 이 접근 방식은 종종 비대하고 혼란스러운 코드를 생성합니다.

Hanlon's Razor는 단순한 실수나 오해가 발생할 가능성이 높은 경우 악의를 품지 말라고 제안합니다.

지나치게 방어적인 프로그래밍을 피하고 명확하고 간단한 논리에 집중하세요.

앞으로 결코 일어나지 않을 문제를 예상하거나 코드를 너무 유연하게 만들려고 할 수도 있습니다.

간단한 코드는 유지 관리, 디버깅 및 이해가 더 쉽습니다.

샘플 코드

잘못된

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

로그인 후 복사