For investors
股价:
5.36 美元 %For investors
股价:
5.36 美元 %认真做教育 专心促就业
代码审查是程序员在开发软件的时候会经历的一个环节,下面我们就通过案例分析来了解一下,软件开发代码审查需要关注哪些问题。
搞清楚上下文还可以让你更好地理解之前工程师的命名、分组以及其他决定。而在另一方面,我们也要记住,上下文是一柄双刃剑,它可能会让我们思路闭塞。了解上下文的情况下,一切都是清楚明了的,但我们也会因此忽视一些单独去看才会发现的问题。牢记这一点,在找出佳类和变量名之前,尽量多站在“新人”的角度上看问题。
不要误会我的意思,但仅从“整洁代码”的层面来考虑代码审查的话,我们所做的可能也只是一些表面的打磨工作。指出方法中的嵌套条件或者多参数问题的确是有意义的,我们也应当继续这样做下去,但说实话,这些并不是代码审查的大价值所在。即使有上面列的这些错误,程序也还是会正常运行的,如果能有位经验丰富的程序员来做维护,它基本就不会出什么问题。更进一步讲,几个嵌套的“if”语句对于一般程序员来说可远比装饰器模式更容易理解。我们的关注点应当放在经过审查的代码所处的上下文中,并在那里排查潜在问题。
架构是软件中关键的部分之一,其应当能反映业务逻辑,也是代码审查中的重点关注对象。如果我们能在这个核心中发现问题并迅速做出反应,就会为产品带来大的价值。
我们绝不应以咄咄逼人的姿态来审查代码。立志找出代码中所有错误并在评论中毫不客气地指出它们,通常是新手程序员才会干得出来的事情,也有可能是那些在“伺机报复”的程序员的行为。不过如果后一无所获那可就是太惨了。这种行为会让我们更多去思考一些小细节,结果失去了对所审查代码所处架构的大局观念。从具体场景中开始讨论可能的用例、问题和扩展需求,总是要比见缝插针地强塞设计模式要好。
【免责声明】本文系本网编辑部分转载,转载目的在于传递更多信息,并不代表本网赞同其观点和对其真实性负责。如涉及作品内容、版权和其它问题,请在30日内与管理员联系,我们会予以更改或删除相关文章,以保证您的权益!更多内容请加danei0707学习了解。欢迎关注“达内在线”参与分销,赚更多好礼。