Day 2- 好品味：優秀工程師的直覺

17th鐵人賽 javascript code smell maintainability solid principles

Sunny.Cat

2025-09-04 13:22:19

672 瀏覽

分享至

消除你程式碼的臭味 Day 2- 好品味：優秀工程師的直覺

很多人以為，程式設計的厲害在於演算法或是那些設計模式。
但還有一個更務實的概念一定要掌握。

Bad programmers worry about the code.
Good programmers worry about data structures and their relationships.
拙劣的工程師擔心程式碼，優秀的工程師關心資料結構與其間的關係。

一個工程師的層次，往往取決於他如何組織資料。

「好品味 Good Taste」，不是指用了多麼複雜的技巧或寫出花俏的程式碼，而是能從資料結構層面，看穿問題的本質，從根本上化繁為簡的直覺。

經典案例：鏈結串列 (Linked List) 的節點刪除

先假設我們有一個鏈結串列： 10 -> 20 -> 30 -> null。
需要刪除某個節點。

🔴 臭味程式碼處理（10 行，帶 if 判斷)

這段程式碼的作者，恐懼所有可能的邊界情況（Edge Case）。
他害怕串列是空的，也害怕要刪除的恰好是頭節點。
於是，他用層層的 if 條件來保護自己。

// 充滿防禦性檢查的程式碼
function removeNode(head, targetValue) {
    // 特殊情況 1: 害怕空串列
    if (head === null) {
        return null;
    }

    // 特殊情況 2: 害怕刪除的是頭節點
    if (head.value === targetValue) {
        return head.next;
    }

    // 一般情況：總算進入「正常」邏輯
    let current = head;
    while (current.next !== null) {
        if (current.next.value === targetValue) {
            current.next = current.next.next;
            return head; // 操作完成，提前返回
        }
        current = current.next;
    }

    return head; // 找不到目標
}

這是典型的壞味道（Code Smell）。
三段風格迥異的邏輯，處理的卻是同一個核心問題，程式碼因此變得複雜而不必要。

把程式碼變成流程圖：

🟢 好品味的程式碼（4 行，無條件分支）

有品味的程式設計師不處理特殊情況，他們消滅特殊情況。

// 透過改變資料結構簡化邏輯
function removeNode(head, targetValue) {
    // 建立一個 dummy 節點，它的 next 指向真正的 head
    const dummy = { next: head };
    let prev = dummy;
    let current = head;

    while (current !== null) {
        if (current.value === targetValue) {
            // 無論 current 是不是頭節點，操作都完全一樣
            prev.next = current.next;
            break; // 找到就完成任務，跳出迴圈
        }
        prev = current;
        current = current.next;
    }

    // 返回 dummy.next，這永遠是串列的「新」頭部
    return dummy.next;
}

這做法的流程圖簡單多了，只有唯一的出口返回：