[讨论] 会手痒想动前人的程式吗?

楼主: sec5566 (sec)   2019-01-11 15:08:43
一个系统当然不要去改,
这样才稳定,
因为你不知道会不会突然冒出bug,
可是有时候接受前人程式,
会看到一些违反dry原则的,
或是命名规则有问题的,
像函式用大驼峰,类别用小驼峰,
或很奇怪的名称之类,
不然就是排版很乱的,
这种大家会手痒去改吗?
改下去又是大工程了,结果工作越做越多
另外如果要扩充新函式,
大家会继续照他的命名规则写,
还是用正规的?
作者: cuteSquirrel (松鼠)   2019-01-11 15:17:00
得到重构批准再改
作者: THEWORLDS (天下)   2019-01-11 15:19:00
因为你不知道会不会突然冒出bug 那就是没看懂还想硬改?
作者: steve1012 (steve)   2019-01-11 15:25:00
没unit test会加 太长会改 命名照codebase convention
作者: yaya517 (Abby)   2019-01-11 15:28:00
在可以运行的最低限度下修改 有些深根蒂固也很难改 除非bug已经出来了 就稍微重构干脆顺便把他改好
作者: steve1012 (steve)   2019-01-11 15:29:00
不过这没有共识 你只是想要战吧 zzz
作者: APTON (玮玮)   2019-01-11 15:37:00
我同事用goto,我都没去改了...
作者: final01 (牛顿运动定律)   2019-01-11 15:46:00
大哥,你说的是团队一致的问题吧
作者: DCTmaybe (竹竹人)   2019-01-11 15:49:00
嘘id
作者: benqm300 (人生苦短)   2019-01-11 16:02:00
逻辑或写法有问题才改,命名问题是管理问题,除非轮到自己负责不然下班比较重要
作者: pttworld (批踢踢世界)   2019-01-11 16:13:00
达到融入不同专案风格的境界,甚至不同程式语言
作者: diabloevagto (wi)   2019-01-11 16:25:00
问你主管,谁知道你公司是不是就用旧程式的规则
作者: xxtuoo (浪费时间不好QQ)   2019-01-11 16:35:00
命名会看不下去的代表还太浅Zzz
作者: Noahdot (驳二宫和也)   2019-01-11 16:52:00
你还来啊?
作者: testPtt (测试)   2019-01-11 18:33:00
太旧的东西我会先想到重写
作者: MOONY135 (谈无欲)   2019-01-11 19:07:00
我觉得当你的同事...有点悲伤
作者: alihue (wanda wanda)   2019-01-11 19:24:00
886
作者: YahooTaiwan (超可爱南西我老婆)   2019-01-11 19:55:00
命名规则又没有标准答案...
作者: joery (Lin)   2019-01-11 19:58:00
你确定牵一发不会动全身,你确定整个系统你改一个地方有百分百,确定,绝对把握,再次确认一定不会影响其他地方或造成其他地方出问题。
作者: lonelytea (霸气逼人)   2019-01-11 19:58:00
一定会被你改出问题啦 不用想了
作者: joery (Lin)   2019-01-11 19:59:00
你确定牵一发不会动全身,你确定整个系统你改一个地方有百分百,确定,绝对把握,再次确认一定不会影响其他地方或造成其他地方出问题。不然如果真的出问题只会动目前影响的,而且看了好几遍确认他流程逻辑找到可以下手的点改,顺便重构该部份程式就好。不然万一系统有问题,你就有得哭了
作者: stupid0319 (征女友)   2019-01-11 21:08:00
没有bug的话,不要改
作者: ChiangKeith (Wait)   2019-01-11 22:53:00
命名不要太初凡入圣根本不重要吧
作者: s001582000 (仁傑)   2019-01-11 23:22:00
我接到风行天的code都会重写
作者: ChungLi5566 (中坜56哥)   2019-01-11 23:53:00
会这样写一定有原因 不熟的话别乱动例如弱扫没过 多绕点路骗过弱扫软件
作者: CloudyWing (孤单ㄉ翼)   2019-01-12 00:07:00
题外话C# public method 大驼峰反而是标准 XD现在的别人写得就算看不下去,但只要稳定就不去动,除非被反应问题太多或是有些架构要去调整,才会整个整理
作者: y3k (激流を制するは静水)   2019-01-12 00:18:00
写太难看改不动或有bug再改 因为也有可能是你没能力...
作者: cphe (魔鬼藏在垃圾筒里)   2019-01-12 00:31:00
改排版改命名没什么意义吧 这东西不同人写就是不同风格除非一进公司就有明确规定,甚至coding style的script在扫
作者: shyangs (厚呦)   2019-01-12 02:10:00
方法 C# 大驼峰, Java 小驼峰; 一天写个语言我常弄错.
作者: wadechen (忙)   2019-01-12 02:33:00
一直执著在这问题 不觉得很辛苦吗
作者: william81615 (别这样玩我)   2019-01-12 06:16:00
楼上 这不就是典型上班没事干找碴吗XD
作者: layolayo (蓝天晴雨)   2019-01-12 07:47:00
吃饱太闲喔...通常半瓶水的人很容易觉得别人的code都是垃圾
作者: MOONY135 (谈无欲)   2019-01-12 10:05:00
我只要能抄队友的code我就一定不改
作者: ymcheung (ymc)   2019-01-12 10:11:00
但是前人留下的 CSS 不管动哪行都会改到其他页面
作者: ab4daa (nooooooooooooooooooo)   2019-01-12 10:34:00
if it works, don't fix it!
作者: IhateOGC (我讨厌)   2019-01-12 11:21:00
goto谁说不能用?该好好update了还停在70年代教科书?
作者: loadingN (sarsaparilla)   2019-01-12 12:10:00
你上班没有其他更重要的事了吗?
作者: tedmax100 (Nathan-Lu)   2019-01-12 13:16:00
以不变应万变; 敌不动,我不动
作者: hanshsu (小肉呆)   2019-01-12 14:22:00
linux kernel 一堆 goto, 唉 怎么办?
作者: prag222 (prag)   2019-01-12 16:44:00
我写按键精灵都用goto ....不过现在没写了
作者: Ghamu (猫丸)   2019-01-12 17:57:00
千万不要改 烂code 烂命名 烂排版 就是让老板多请你进公司工作的主因 就是有这么多拉低产能不是生产的杂活 才需要一直请工程师进来你是领钱办事的 达则兼善天下 穷则独善其身
作者: samblow2000 (猫哥大大)   2019-01-12 18:55:00
劝你不要改...自己写都不敢保证没问题了...
作者: doranako (真爱无限)   2019-01-12 19:29:00
有时间再改
作者: knives   2019-01-13 15:52:00
请问有出错吗,有主管指示吗
作者: shooter555 (shooter)   2019-01-13 20:38:00
觉得goto有时候能让程式码简洁0.0 还不错用啊
作者: y3k (激流を制するは静水)   2019-01-14 19:22:00
goto是那种有能力用好的人不问 问的人一定不能让他用的东西另外linux kernel是怎样小弟不清楚 但是跟一堆asm()比起来我好奇大家会认为谁优谁劣XD
作者: fruithuang (黄水果)   2019-01-14 21:09:00
不出问题可以改,出问题就赖在你身上。系统忽然不稳都可赖说是你改的问题......
作者: a9228097 (逞强是我的专长)   2019-02-02 16:42:00
如果只改naming 跟format就用tool改一下xd

Links booklink

Contact Us: admin [ a t ] ucptt.com