V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
ccw55
V2EX  ›  程序员

大鸟请进 Java 代码评审

  •  
  •   ccw55 · 2017-11-23 16:37:56 +08:00 · 1846 次点击
    这是一个创建于 2557 天前的主题,其中的信息可能已经有所发展或是发生改变。

    多个月之前,参与某项目开发,目前在进行代码评审,项目经理一直在吐槽以下几个问题,我资历尚浅,但仍感某些改进有些问题,请大侠们帮忙看看这评审意见是否可行。谢谢指导

    代码评审发现的一些问题 1、职责不清晰:包含资产查询、收益查询、组合交易账号的查询逻辑(这个我认,一开始业务不大清晰,只是查资产,把一些要用到的数据一起查出来了) 2、一堆的 get/set,非常影响阅读(某些 vo 互相转换而已,由于属性多,影响阅读) 3、map 的 key 值含义不清晰(说是 key 都是 String 类型,要进入方法里面才看得到是什么,影响阅读) 4、方法体内处理入参(说是把传进去的入参改了,这样不妥,但是我的实现是需要改入参,说这是一个不好的习惯) 5、缺少复杂方法的算法注释(说方法行数太多,得说明这个方法的实现逻辑) 6、方法体入参过多(建议将入参封装成对象。。。)

    觉得有些意见挺不靠谱的,希望能互相探讨互相学习一下。

    kimqcn
        1
    kimqcn  
       2017-11-23 16:42:09 +08:00
    项目经理能说出这些来是为你好。
    hcymk2
        2
    hcymk2  
       2017-11-23 16:43:20 +08:00
    这些个《重构:改善既有代码的设计》基本都说的了的。
    owenliang
        3
    owenliang  
       2017-11-23 17:01:35 +08:00
    模块边界都拆不清楚,这代码质量肯定烂。
    jadec0der
        4
    jadec0der  
       2017-11-23 17:37:57 +08:00
    找工作都说要找大牛指导,有人指导又觉得不服,咋办
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1102 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 22ms · UTC 19:00 · PVG 03:00 · LAX 11:00 · JFK 14:00
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.