“写更少的代码” 是否真的有必要？

资讯频道 → 研发管理

7顶
2踩

2013-04-12 18:23 by 副主编 wangguo 评论(29) 有18385人浏览

代码行编程工作 LOC 项目复杂度

声明：ITeye资讯文章的版权属于ITeye网站所有，严禁任何网站转载本文，否则必将追究法律责任！

下面的两种代码编写风格，你更倾向于哪一种呢？

第1种：

public static void happyBirthday(int age)
{  
        if ((age == 16) || (age == 21) || ((age > 21) && (((age % 10) == 0) || ((age % 25) == 0))))        
    {
        System.out.println("Super special party, this year!");
    }
    else
    {
        System.out.println("One year older. Again.");
    }
}

第2种：

public static void happyBirthday(int age)
{
    boolean sweet_sixteen = (age == 16);
    boolean majority = (age == 21);
    boolean adult = (age > 21);
    boolean decade = (age % 10) == 0;
    boolean quarter = (age % 25) == 0;

    if (sweet_sixteen || majority || (adult && (decade || quarter)))
    {
        System.out.println("Super special party, this year!");
    }
    else
    {
        System.out.println("One year older. Again.");
    }
}

尽管这两段代码实现方式区别不大，但是单从代码行数方面考虑，第1种有效代码行数仅为第2种的一半。你更倾向于哪种风格呢？

这是问答网站Stackexchange（Stackoverflow扩展版）中的一个问题。提问者Ankit是一名Java开发者，他经常被代码审查者要求减少代码行数，审查者认为他应该专注于使用更少的代码来完成同样的工作，而Ankit则希望能够使代码更加清晰，即使这样会增加代码的行数，他认为这不是简单的去除冗余代码的问题，而是有关个人编程风格。

那么LOC（代码行）小的话，对于代码执行有什么影响？如果大的话，又有什么影响呢？下面来看看其他开发者怎么说的。

mattnz认为统计SLOC完全没有必要，与之相比，更应该去统计代码的重要性、可读性和复杂性：

引用

这涉及到软件项目度量的问题，不管目的如何，可以对项目中重要的代码进行衡量。

衡量SLOC（Source lines of code）是否真的重要？当然不，除了混淆编程竞赛外，对于商业组织来说，SLOC从来都不重要，未来也不会重要。

问自己一个简单的问题，减少SLOC是否可以使你的代码更好？SLOC往往被天真地用来衡量项目复杂度，这种情况下，SLOC少的代码也许是好的代码。你应该极力避免去统计那些容易统计的东西（比如SLOC），尝试去统计重要性、可读性、复杂性等难以统计的东西。

Erik Dietrich表示认同代码审查者的建议：

引用

你在代码中所写的每一个语句都是一个技术责任，也是一个潜在的故障点。如果解决同一个问题，你用了10句话，而你的同事只用5句话，那么从出错可能性上考虑，你同事的代码可能会被认为更好，因为你的代码可能更容易出错。

当然我指的是语句，不是代码行。比如下面的代码：

void someMethod() {   
someobject.doSomething(someSingleton.getInstance().with().a().lot().of().law().of().demeter().violations()).and().if().that().werent().enough().theres().more();
}

只有一行，但是其中可能会有大量的地方出问题。因此我想说的是，专注于使用最少量的语句来实现（注意是“语句”），而对于代码行，你需要写多少就写多少吧。我认为这也是你的代码审查者所希望的。

我个人认为，还需要取得一个平衡点。正如我说，你所写的每一条语句都是一处技术责任，但如果你使用描述性的名称来命名一个局部变量，使你的代码更加清晰、可读，这样也是好的。我认为你比较容易卷入人们对较小审美差异的争论中，但整体上，我比较认同你的代码审查者的观点，因为这样有利于最大限度地减少可能出错的东西。

Xion更喜欢简洁的解决方案：

引用

代码审查者的建议从字面上来说，不会对程序有什么提高的地方。我比较认同的是——让你的代码做更少的事情。

换句话说，就是要求代码简单。代码是一种责任，而不是资产，因此应该减少它的量。你可以通过一个更直接、更朴实的方式来解决问题。

就像Ken Thompson曾经说的——“我最有效率的一天是扔掉了1000行代码。”

jhewlett比较认同提问者所说的“使代码更加清晰，即使这样会增加代码的行数”：

引用

我见过很多简洁的程序，代码行数相当少，但是从这些代码中不能马上看出它们的意思。

可读性为王，因为其他开发者还要维护你的代码。

我认为最好是——将更短的方法作为目标，不要单单追求减少代码行，而是让代码“短”到只做一件单一的事情。

Joshua Volearix认为简洁、容易阅读的代码是开发中的最重要的因素：

引用

下面的代码我完全无法接受：

void someMethod() {   
someobject.doSomething(someSingleton.getInstance().with().a().lot().of().law().of().demeter().violations()).and().if().that().werent().enough().theres().more();
}

但是我发现如果将下面的代码：

if (boolean == true){
value.prop = option1;
}
else{
value.prop = option2;
}

改变如下：

value.prop =  boolean ? option1 : option2;

这样即减少了代码行数，又不牺牲可读性。

至于修改后如何影响代码执行？我反正没注意到性能的增加或减少。比起使用多少行代码来实现，更重要的是使用了多少过程来实现。我之前看到过一些很简洁的代码，但是它们的执行效率比不上那些拥有更好的设计流程但行数多的代码。

Vitim.us认为更少的代码更易读，但Joe提出了异议：

引用

我不认为更少的代码==代码更易读。然而现实中，大段的代码往往没有经过合理的设计。因此，我认为要求减少代码行，可以迫使程序员想出更好的设计。

更详细的讨论可以参阅原帖：How important is it to reduce the number of lines in code?

对于这个问题，你是怎么看待的呢？

分享到：

7
顶

2
踩

评论共 29 条请登录后发表评论

9 楼云上太阳 2013-04-13 10:46

这就需要找一个平衡点，平衡点的问题在programmer世界简直太常见了，然而把握这个平衡点的能力也是评价一个coder是否是一个优秀的programmer的参照点之一

8 楼 xieyongwei 2013-04-13 09:41

我认为编程必须需要考虑易读易维护、执行效率，其中包括 “写更少的代码” ！
当然这个 “写更少的代码” 看你是怎么实现的，最常用的方式封装和重构，做得好就更易懂和易维护。

7 楼 lg_asus 2013-04-13 08:18

I prefer 2nd style, reader or maintainer can't understand the meaning of conditional expresstion at first sight in the 1st style, so you need comments, the 2nd style is almost self-explanation. From the view of maintenance, if you are asked to add or modify the `conditional` expression in 1st style, you must be more careful.

6 楼 PetriNet 2013-04-13 00:42

没做过global deployment吧，宝强哥

5 楼 freezingsky 2013-04-12 20:37

代码清晰了，自然可以考虑更少的代码量！

4 楼 suneyejava 2013-04-12 20:20

最重要的是要逻辑清晰，一个封装要保持功能尽量单一，接口清晰。

3 楼 laogao3232 2013-04-12 19:42

我觉得第一段代码的可读性足够了。
第二段是把业务逻辑带出来了，这个没必要吧。
当然就是只对这个简单的代码而言。
如果是复杂的代码，还是要通过命名等，让每一段代码的意义都清晰明了。

2 楼 clxy 2013-04-12 19:22

必须是可读性优先啊。

之前有篇文章说得好，

引用

程序代码是写给人看的！

（编译给机器用的。）

不过这里讨论的“行数”，我认为毫无意义。
行数多少即不等于代码质量好坏，也不等于代码可读性好坏。

1 楼 ccheng123456 2013-04-12 18:43

[b][/b]. 板凳。代码还是易读的。不然维护的人头大了。

发表评论

您还没有登录,请您登录后再发表评论

7顶2踩