百科问答小站 logo
百科问答小站 font logo



项目组里的代码审查员要我把代码写的很啰嗦,怎么办? 第1页

  

user avatar   yao-dong-27 网友的相关建议: 
      

我有时也会写成这样

       boolean flag = getMboValue().isNull(); if (flag) {     this.getMboValue("vendor").setReadOnly(false); } else {     this.getMboValue("vendor").setReadOnly(true); }     

原因是这样方便下断点调试,比如 flag是true为正常,false是异常情况需要调试。写成一行断点就不好下了。

我也不反对这种情况直接写成一行,只要命名合理逻辑清晰可读就好

我反对以下这样一行的写法:

       foo.setDisable( ! (bar.isEnable()  && this.getValue().isNull()) );     

肯定、否定、并且、取反来回兜几个圈后我脑子就冒烟了,智商欠费。


user avatar   Ivony 网友的相关建议: 
      

首先,很显然的代码审查员的建议写法除了增加代码量之外,在可读性方面并没有太多的提升,甚至因为代码量的提升带来更多的心智负担和出错的概率。

例如因为某个奇怪的失误把代码写成了这样:

       boolean flag = getMboValue().isNull(); if (flag) { this.getMboValue("vendor").setReadOnly(false); } else { this.getMboValue("vendor").setReadOnly(false); }     

所以,原本一个可能出现问题的地方现在变成了四五个。


除此之外,flag的命名不够meaning,而我也倾向于移除这个变量,毕竟这个表达式并不长。但与上面不同的是,我个人会建议完全弃用 ! 运算符,原因是这个运算符实在是太不够明显了。所以我个人建议的写法是:

       this.getMboValue("vendor").setReadOnly( getMboValue().isNotNull() );     

增加一个isNotNull方法来使得语义更为清晰。

或者:

       this.getMboValue("vendor").setReadOnly( getMboValue().hasValue() );     

在无法增加方法的情况下,我也会建议用 == false来代替 ! ,因为在字面上这样会更难以被忽略:

       this.getMboValue("vendor").setReadOnly( getMboValue().isNull() == false );     

比较一下:

       this.getMboValue("vendor").setReadOnly( !getMboValue().isNull() );     

很显然上面的写法在提醒看的人,嘿,哥们儿注意点,我这里是反的。




当然,抛开代码审查员的问题,作为一个一无所知的程序员来重新审视这段代码,我会发现Mbo这个缩写其实会非常的令人费解,而且getMboValue这个方法实在是出现了太多次,另外为代码中一个有this,另一个没有this,让这段逻辑更加离奇。


最后,童鞋你写大括号的习惯我很欣赏,趁着你们的代码审查员还没有让你把所有的大括号都变成半展开之前,赶紧换工作投入大C#的怀抱吧……到时候同样的代码可能只需要写成这样就可以了:

mbo["vendor"].ReadOnly = mbo["vendor"] != null;


user avatar   qin.chao 网友的相关建议: 
      

景甜:抱歉,是我选的他。




  

相关话题

  你目前写出的最大的 Bug 是怎样的? 
  点语法最早是在哪个编程语言中出现的? 
  win7笔记本,外接扩展显示器,如何在不缩小外接显示器像素的同时去除外接显示器的桌面背景黑边? 
  PHP、Java、Python、C、C++ 这几种编程语言都各有什么特点或优点? 
  这种代码命名规范,到底好不好? 
  如何写好业务代码? 
  JavaScript 如果拥有多线程能力会怎样? 
  在业务需求完全一致,技术指标要求不一致时,是否应该为此提供使用不同优化算法的接口? 
  你所见过最恶心的代码片段是怎样的? 
  为什么全网甚至全中国找不到一个能轻松,清晰,简洁明了把java编程讲清楚的人? 

前一个讨论
维基百科比百度百科差在哪里?
下一个讨论
女性健身的常见误区和谣言有哪些?





© 2025-04-02 - tinynew.org. All Rights Reserved.
© 2025-04-02 - tinynew.org. 保留所有权利