一個值得記住的教訓啊

昨天下午實現一段代碼,如果一說的話非常簡單,非常低級的問題,但當時測試好幾次,包括兩個人review了,都沒有發現問題.

結果集成到應用發布的時候出現BUG了,只好復原.好在我們的應用都有預發而機制,沒有影響線上的真實應用.

 

情境是這樣的:

 

switch(x)｛

    case 1:

    case 2:

    case 3: // nothing

               break;

    case 4:

              dosomething;break;

  

    case 5:

              dosomething;break;

    default:

              dosomething;break;

｝

 

昨天要實現case 3的邏輯,是一個URL編碼.結果改成:

switch(x)｛

    case 1:

    case 2:

    case 3: // nothing

               urlencode(url);

               break;

    case 4:

              dosomething;break;

  

    case 5:

              dosomething;break;

    default:

              dosomething;break;

｝

 

當時測試的時候唯寫了個URL傳進來,正確輸出了.

沒想到case 1和case 2的情況穿越到 case 3了,結果是case 1和case 2的字串都進入了urlencode的邏輯.

這麼低級有錯誤,說出來誰都明白,或者如果整個代碼是自己一次性寫的,絕對不會犯這種低級錯誤,但在修改別人原有代碼時

卻忘記了在上面加個break來阻止穿越.

 

這樣的代碼如果考慮到以後別人可能修改和維護,其實不如費點事每個case直接加上break;否則別人很容易在增加處理狀態時忘記

上面的穿越.

 

小心啊,小心................................