设为首页 - 加入收藏 ASP站长网(Aspzz.Cn)- 科技、建站、经验、云计算、5G、大数据,站长网!
热搜: 重新 试卷 创业者
当前位置: 首页 > 运营中心 > 建站资源 > 优化 > 正文

代码写成这样,老夫无可奈何!(2)

发布时间:2019-10-17 14:33 所属栏目:21 来源:栈长
导读:8、看花眼的参数 Orderorder=newOrder(orderNo,1678.13,CNY,1,3,wx,5,1,0,8,0,1,newDate()); 看到这样的代码,我也是醉醉的了。 我就见过同事写出这样的代码,是赶时间呢,还是真的不会设计模式,一个构造函数参数

8、看花眼的参数

  1. Order order = new Order(orderNo, 1678.13, "CNY", 1, 3, "wx", 5, "1", 0, "8", 0, 1, new Date()); 

看到这样的代码,我也是醉醉的了。

我就见过同事写出这样的代码,是赶时间呢,还是真的不会设计模式,一个构造函数参数能写这么多,且不说魔法值,这么长,看一眼都觉得头大。

9、消失的异常

  1. try { 
  2.     ... 
  3. } catch (Exception e) {} 

程序捕捉了异常,却不打印。

  1. try { 
  2.     ... 
  3. } catch (Exception e) { 
  4.     e.printStackTrace(); 

错误是输出来了,但没有输出到日志文件。

  1. try { 
  2.     ... 
  3. } catch (Exception e) { 
  4.     logger.info("...", e); 

日志级别用的不对,去错误日志文件里面死活找不着错误日志。

上面的种种不规范的写法,在生产环境出现问题时,一方面导致排查问题艰难,另一方面,监控系统也不能及时地监控到异常。

10、神奇的if

最后一个了,压轴了。。

来,我们一起赏析下这段神奇的 if 代码:

  1. if(user.getFirstName() != null) 
  2.     if(user.getMiddleName() != null) 
  3.         if(user.getLastName() != null) 
  4.             if(user.getNickName() != null) 
  5.                 if(user.getShortName() != null) 
  6.                     if(user.getFullName() != null) 
  7.                         if(user.getFamilyName() != null) 
  8.                             if(user.getEnglishName() != null) 
  9.                                 ... 
  10.                                 return true; 
  11. return false;       

这段代码够奇葩了吧,一个 if 能搞定的非要写这么长,还缩进得这么有层次感。。

小结

写神逻辑代码的人才这么多,找问题、解决问题真是太难了。

(编辑:ASP站长网)

网友评论
推荐文章
    热点阅读