看到别人的代码,不想去改咋办?

2018-04-03 15:58:45 +08:00
 java253738191

求问我这种是强迫症吗?怎么解决??

8402 次点击
所在节点    程序员
68 条回复
Tyris
2018-04-04 14:24:09 +08:00
@Midnight 可以,省了找混淆器的麻烦:doge:
whatTheGhost
2018-04-04 17:05:31 +08:00
线上代码强制 core 这个不一定是黑点。比如我用了 libuv,libuv Release 编译也有 assert 强制 core,这个还改不了。有时候已经是错误了,还不如早崩。

本人之前的同事,工作三年甚至还没养成代码缩进习惯(毕业于某 CS 专业比较好的 985 ),喜欢到处复制粘贴代码(比如需求改了,把 xxx 函数内代码复制一下,改名 xxx2 ),完全没有时间 /空间复杂度概念,喜欢到处飞全局变量,用全局变量来做线程同步……
gjquoiai
2018-04-04 17:08:20 +08:00
https://imgur.com/a/iJcc0
凑合活着呗 ┑( ̄Д  ̄)┍
gjquoiai
2018-04-04 17:09:14 +08:00
gjquoiai
2018-04-04 17:10:52 +08:00
@gjquoiai
二连失败 _(:з)∠)_
sampeng
2018-04-04 17:19:29 +08:00
1。如果这是一个一个人开发的项目,这么写未尝不可。工具是服务人的,不是人去完美服从工具。

2.拼写错误不能忍,这个倒是可以吐槽一下。。但 a。a。b 这样的。看情况。如果是在非常小的函数里面。比如就 5-6 行。。我经常写 tmp 啊。tmp1.tmp2 之类的。以前我也很反感。后来发现真没那么多词去形容中间态或者临时用的。大函数除外。。大的主业务逻辑绝逼不能这么搞。。会死人的。。。其实你去看看其他源码。比如 git。。说实话,我简直看得想一头撞死在墙上。。第一版功能少,勉强还能看。看最新版。。。真的心里在呐喊,tm 什么玩意。。。到处都神奇数字,神奇变量,全局变量到处飞到处初始化。。。

3.看情况,assert 如字面含义,绝逼 100%不可能会是异常的。如果是异常了。直接崩溃未尝不可。你得赶紧处理。但如果要求 100%不能崩的。。所以看业务。最近写 rust。。原先特别担心写 unwrap。。后来发现其实没那么糟糕。。就随心所欲了。。。

4.依然看情况,老代码。可能是一个开发用几天时间赶工赶出来的。根本没时间去处理。。。

5.不要主动去修改任何别人的代码!!除非你提前跟他说,你需要重构,怎么重构巴拉巴拉。否则就是自讨苦吃。。。
wjm2038
2018-04-05 06:33:43 +08:00
@blaxmirror 不明所以的赋值吧,我是这么理解的
whoisghost
2018-04-08 22:02:37 +08:00
@blaxmirror 呃,那成都就可能不会像你们 team 那么好了,毕竟是团队,不是整个公司。

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://tanronggui.xyz/t/443976

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX