首页   注册   登录

woodensail

V2EX 第 170847 号会员,加入于 2016-04-27 14:15:12 +08:00
woodensail 最近回复了
@keepeye 看了下,丧到极点了啊。
17 小时 3 分钟前
回复了 UnrealEngine 创建的主题 游戏 你们黑五都买了哪些游戏呀?
买了幽灵教义,相当于低配 xcom,玩儿起来还不错。
23 小时 0 分钟前
回复了 KunMinX 创建的主题 问与答 南方的冬天 防止冻手冻脚 的好招
https://item.jd.com/16845441133.html
这种不错,而且脚暖和后全身都暖和,手也不怕冷了。
@chaleaochexist 是时候重构了
@pence2019 加注释都不肯就比较麻烦了,没有威信的后果(doge)
其实我感觉楼主的公司就是想做重构又没时间,只好搞这种渐进式重构。但是渐进式重构需要对项目有很强的掌控能力,要不然各种翻车。比如遗漏影响范围导致依赖方挂了,忽略特殊场景导致部分业务场景异常等。
@pence2019 注释基本上强制推行没问题,写注释基本不会翻车。
但是后面几个就算是重构级别的了,动到哪儿了都得全面回归测试。事前也得做全面的分析,确定影响范围。
@pence2019 有,但是看具体项目,这边 code review 的第一追求是减少无用改动,其次才是代码风格。特别是对于老项目,不允许在不经测试的情况下私自改动。
实在忍不了就提重构,自己重新做,测试重新测。
对于老项目,一般原则是保持原有风格,尽量少动,同时新增代码也要尽量贴近原有风格。要不然一个项目经手几回就乱成一团糟了。
当然要是想当做一个长期项目来做,建议重构,不仅仅是代码风格,连项目架构一起按自己想要的方式来调。
关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   1577 人在线   最高记录 5043   ·     Select Language
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.3 · 15ms · UTC 00:47 · PVG 08:47 · LAX 16:47 · JFK 19:47
♥ Do have faith in what you're doing.