大鸟请进 Java 代码评审 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
ccw55
V2EX    程序员

大鸟请进 Java 代码评审

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

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

    代码评审发现的一些问题 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     Solana     3809 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 35ms UTC 10:16 PVG 18:16 LAX 02:16 JFK 05:16
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86