编写可读代码的艺术读书笔记--简化和重新组织代码

第二部分：简化循环和逻辑

把控制流变得易读

关键思想：把条件、循环以及其他队控制流的改变做得越“自然”越好，运用一种方式使读者不用停下来重读你的代码。

拆分超长的表达式

关键思想：将超长的表达式拆分成容易理解的小块

例如：

if(request.user.id == document.owner_id){

//user can edit this document

}

…

If(request.user.id != document.owner_id){

//document is read-only…

这段代码的主要概念是“该用户拥有此文档吗？”这个概念可以通过增加一个总结变量来表达得更清楚。

Final Boolean user_owns_document =(request.user.id == document.owner_id);

If(user_owns_document){

//user can edit this document

}

..

If(!user_owns_document){

//document is read-only

}

变量与可读性

假设你有一个网页，上面由几个文本输入字段，布置如下：

<input type = “text” id = “input1” value= “Dustin”>

<input type = ”text” id = “input2”value=”Trevor”>

<input type = ”text” id = “input3” value= “”>

<input type = “text” id = “input4”value=”Melissa”>

你的工作是写一个叫setFirstEmptyInput()的函数，它接受一个字符串并把它放在页面上第一个空的<input>字段中，返回已更新的元素（如果没有剩下任何空字段则返回null）。

var setFirstEmptyInput = function(new_value){

     var found = false;

     var i = 1;

     var elem = document.getElementById(‘input’+ i);

     while(elem != null){

         if(elem.value== ‘’){

                   found= true;

                   break;

         }

           i++;

           elem= document.getElementById(‘input’ + i);

      }

     if(found) elem.value = new _value;

          return elem;

}

三个变量存在于整个函数中，并且写入了多次。中间变量（found）可以通过提前返回来消除，elem在整个代码中以循环的方式多次用到，这段代码让我们觉得elem就是在迭代的值，实际上只是在累加i。所以把while循环重写成对i的for循环。

var setFirstEmptyInput =function(new_value){

         for(varI = 1; true; i++){

                   varelem = document.getElementById(‘input’ + i)；

                   if(elem== null)

                            returnnull;//Search failed. No Empty input found.

                   if(elem.value==’’){

                            elem.value= new_vlaue;

                            returnelem;

                   }       

        }

}

第三部分：重新组织代码

抽取不相关的子问题

积极地发现并抽取出不相关的子逻辑。

1.      看看某个函数或代码块，问问自己，这段代码高层次的目标是什么？

2.      对于每一行代码，问一下，它是直接为了目标而工作吗？这段代码高层次的目标是什么？

3.      如果足够的行数在解决不相关的子问题，抽取代码到独立的函数中。