這個例子中的if else也要重構掉嗎？

• 在多人開發的項目中，邏輯無序而不可控；數據格式有序而極易控制
• 在單元測試中，邏輯區塊必須進行測試，否則無法確保其正確性；而數據本身無需測試
• 邏輯代碼越少，邏輯複雜度就越少；邏輯代碼越多，邏輯複雜度就越高

• 邏輯與數據分離
• 邏輯修改成本巨大，數據修改成本極小
• 邏輯修改風險巨大，數據修改風險極小

• 數據來源靈活，數據改變靈活

第一個例子只有兩個分支的時候，可以不重構。但分支變多的時候，需要重構。

第二個例子確實需要重構，那種重構方式叫表格驅動。表格驅動最開始可能不習慣，但其實是很好讀很很容易修改的。

區間不要寫成(a, b)，而應該寫成 [a, b)，也就是說第一個數據的判斷不要使用 &<, 而應該使用 &<=：

if (score &> 74 score &< 90)

修改成：

if (score &>= 75 score &< 90)

使用[a, b) 作為區間的形式，可以將一串數自然分隔。比如將[a, c), 用 b 分成兩段，就變成：

[a, b)
[b, c)


前一個區的最後數字，剛好等於的後一個區的前面數字。另外，b - a 剛好等於數字個數。這樣分區會更自然，不容易出錯。

另外這種區間的判斷，應該將變數放在中間，將其從小到大排序，就更容易看出關係。

if (75 &<= score score &< 90)

整體將各個 if 的順序從小到大排列，添加上 Infinity 作為柵欄，這樣就修改成了：

var startNumber;
if (-Infinity &<= score score &< 7) { startNumber = 1; } else if (7 &<= score score &< 9) { startNumber = 2; } else if (9 &<= score score &< 11) { startNumber = 3; } else if (11 &<= score score &< 30) { startNumber = 4; } else if (30 &<= score score &< 45) { startNumber = 5; } else if (45 &<= score score &< 60) { startNumber = 6; } else if (60 &<= score score &< 75) { startNumber = 7; } else if (75 &<= score score &< 90) { startNumber = 8; } else if (90 &<= score score &< Infinity) { startNumber = 9; }

添加 Infinity 使得判斷的形式更統一。這時就算不抽取成表格驅動，也可以將判斷抽成函數：

if (isInRange(score, 7, 9))


更進一步，就是將分隔點抽取出來。

[-Infinity, 7, 9, 11, 30, 45, 60, 75, 90, Infinity]


就演化成表格驅動了。

你朋友本意是好的，但他的代碼有點小問題，就是沒有按大小順序排列區間，讀起來不容易看出區間之間的聯繫。另外因為區間的後一個數字是下一個區間的前一個數字，有個數字就是重複的，只需要抽取分隔點就成了。

因為 C 系列的語言數組從 0 開始數數。日常中往往從 1 開始數數。這樣可以在 0 的位置預先填一個柵欄值，消除代碼跟日常的不同。這樣就變成了。

function rank(score) {
var t = [-Infinity, 7, 9, 11, 30, 45, 60, 75, 90, Infinity]
return t.findIndex(function(val) {
return score &< val; }); }

其實這種隱式用數組的索引作為結果也不是很好的，最好還是將其顯式寫出來。

function client(int actionName) {
// 前置處理
switch (actionName) {
case "a":
preA();
break;
case "b":
preB();
break;
case "c":
...
}
// do somethings
switch (actionName) {
case "a":
postA();
break;
case "b":
...
}
}


重構為下面的代碼，就會更清晰，可維護性更高：

var actionMap = {
a: {pre: preA, post: postA},
b: {pre: preB, post: postB},
c: {pre: preC, post: postC}
};

function client(actionName) {
var actionInfo = actionMap[actionName];
actionInfo.pre();
// Do somethings.
actionInfo.post();
}


如果只是為了學習如何重構，那就玩一下咯。。

一般的if else是沒有辦法刪除的，無論你將其轉為switch語句，還是多態——你只能將其從一種形式轉換為另外一種形式。

在複雜的情況下應該抽象成那樣子會更容易理解。

但是在這個case里，這裡的if else都只是單層，沒有必要為了重構而去重構。相反地，在這個例子里，最需要重構的不是if語句，而是Magic Number。為7，9，11，30，45，60，75，90賦予一個變數值，會比重構if-else更有意義。像這裡的分數，不會有極端的需求出現，每個level都會在一定的範圍裡面。

基本贊同@huang phodal ，並且要補充，score的條件判斷並非是magic number，因為我們很容易看出其意義。而且score與starNumber的對應可能變化嗎？如果不太可能變化，就沒有必要提取出來。因為現有的代碼並沒有複雜到看不清楚其對應關係。

通過lookup表來分派代碼的方式適用於需要動態性、可擴展性的場合，但不是沒有代價的。代碼多了一層間接，其實理解成本是略微上升的。更重要的是使得代碼的靜態分析變困難——比如圈複雜度的分析，也使得無法應用某些性能優化（像分支順序調整）。

void avrisp_cmd(const uint8_t *buf, uint16_t len)
{
uint8_t cmd = *buf;
switch (cmd)
{
case 1:
// cmd_sign_on
break;
case 2:
// cmd_get_param
break;
// ...
default:
// cmd_unknown
break;
}
}


這樣的代碼既冗長，又難以維護，而且內存消耗（所有代碼塊共用同一個棧框）、時間消耗（代碼塊地址用的是 O(n) 的線性搜索）都成問題。如果這麼做：

#define HANDLER(x) void x(const uint8_t *buf, uint16_t len)
typedef HANDLER((*handler_t))
const PROGMEM handler_t handlers[256] = {NULL, cmd_sign_on, cmd_get_param. /* ... */};
HANDLER(cmd_unknown);

void avrisp_cmd(const uint8_t *buf, uint16_t len)
{
uint8_t cmd = *buf;
handler_t handler = (handler_t)pgm_read_ptr((uint16_t)*handlers[cmd]);
(handler ? handler : cmd_unknown)(buf, len);
}


看上去是不是簡單多了？handlers 變數的聲明又兼作 handler 函數列表了，內存（每個功能的代碼塊分開）、時間（指針引用 O(1) 效率）都大大減少了。

脫離實際情況談重構，這是在犯罪。

有的地方經常要橫向擴展。

想清楚，重構的目的是什麼在動手，不遲。

程序員不喜歡做重複的事情，所以會覺得這種if else「醜陋」，於是就去「抽象」，然後代碼其實是變「複雜」了。正是在這種過程中，編程語言才變成了普通人理解困難的語言，啊哈。

…不都是這樣寫的嗎，假如我有100個區間呢…這不叫重構，這叫寫的太少..

而且，如果想要構造一個對於等長或者簡單的可公式計算距離的區間(比如等比增長的區間距離)，加入一個function參數，輸入index，輸出計算後的區間距離length，更好擴展(等距就直接返回常數number就行)，JavaScript的動態性和函數對象讓這個非常好寫，不支持函數傳參的語言可能需要定義兩個類來處理長度變換和求值。

function rank(score, intervals) {
var obj = intervals.find(function(e, i, arr) {
var left = arr[i-1] ? arr[i-1].k : Infinity;
var right = e.k;
return score &> left score &<= right; }); return obj ? obj.v : -1; //if not find, return -1 } // genLength(index),genNum(index),index start from 1 function genInterval(start, step, genLength, genNum) { var arr = [{k: start,v: genNum(0)}]; for (i=1; i&<=step; i++) { var last = arr[i-1].k; var length = genLength(last); var obj = {}; obj.k = last + length; //save the score of right intervals obj.v = genNum(i); //save the num arr.push(obj); } return arr; } var intervals = genInterval(0, 10, (i) =&> 10, (i) =&> i);
//(0,10]=&>1,[11,20]=&>2,...(90,100]=&>10
var a = rank(60,intervals);
//6
console.log(a);

var ratio = genInterval(1, 10, (i) =&> i, (i) =&> 2*i);
//(1,2]=&>2,(2,4]=&>4,(4,8]=&>6...(512,1024]=&>20
var b = rank(1024,ratio);
//20
console.log(b)


如果把if else寫死了， 等到要添加更多的條件分支的時候，就得改寫方法了，更不要說，如果條件範圍是用戶輸入之後作為參數傳進來的情況。

有一些答案提到了重構為了加強可讀性，這點當然沒錯。但是其實除了可讀性和效率，重構也希望增強可維護性和可擴展性。

重構之後可以把數據範圍提到一個便於修改的地方，假設有一百個這種代碼，找死你（其實也挺快的

這個函數以後還能重用（說說罷了

根據之前開發的經驗最後就會被改成這樣了，別人是否一眼看得懂...我愛莫能助。

var doSomethingFactory = function(lookup, def) {
return function(x) {
try {
return (lookup[x] || def)();
} catch (e) { /* error handling */ }
};
};