入职新公司看到代码一大堆 warning,只有我会觉得痛苦吗

29 天前
 ciaoSora

大家接手别人代码看到成吨的 warning 会不会觉得非常痛苦?我自己是开发自己的项目时会非常注意 warning 的问题,一定不可能让 warning 出现,甚至 linter 的一些 warning 我都不会让他出现。

我的想法是,warning 就是潜在的 bug (比如声明新变量但是未使用),如果确实有合理的理由这样子做,那么可以显式地让编译器或 linter 忽略某处,这样子能让未来的自己或其他人知道此处是有合理理由的,而不是不小心的。

目前待过的所有公司里,我感觉好像大家都非常不在意编译器和 linter 的 warning ,CICD pipeline 以及 git hook 也不检查这些。难道是我过于学究了吗?

6178 次点击
所在节点    职场话题
72 条回复
jzhm
29 天前
@ciaoSora #6 就 C 语言来说,warning 其实可以找机会都消掉
burnsby
29 天前
是个人都会痛苦吧,不过稳定能跑才是第一要解决的
WillBeethoven
29 天前
跟着往上拉就完事儿了
8355
29 天前
有人用 ide 有人用 vscode
litchinn
29 天前
问题就是太多了,如果只有少量的话大部分人还是会看的,一旦多了就没人看然后越来越多
好的行为当然是解决掉,不过有时候太麻烦了也就放在那里了
其实 warning 就是个提醒,想想你自己在代码中打印 warning 日志的场景
woodfizky
29 天前
warning 被设计出来就是有用的,不是每个人都能避免写代码的时候犯低级错误的,保留并且认真的留意 IDE 的这些提示是一个减少潜在错误的很有效的手段。
当然前提是 IDE 的各种 warning 机制设计的比较合理。

我个人写 Python ,是要用 Pycharm 确保整个文件都监测不到最低级的问题的。
有些地方类型注解不对/用了保护变量等不规范但是没办法的写法,也不会直接关掉 IDE 的警告,而是直接用 IDE 的注释,关闭当前行/方法/类的告警。
这是我保证编码质量的一个有效手段,所以当我接手别的同事的代码,然后发现一堆包括但不限于 IDE 告警、不规范写法、垃圾设计等等的情况时,那都是非常难受的,需要花费大量时间精力去品鉴每段代码到底有没有潜在问题。。

除非我不用长期维护这些项目,短期内可以考虑捏着鼻子上,要我长期维护的话,不给我时间重构/优化的话那真的是会非常难受了。
Reficul
29 天前
可以在 CI 上配置,新修改的代码不允许 Warming ,慢慢收敛掉。
RightHand
29 天前
开源项目就全部消掉,公司内部?向现实低头,拥抱新规则
cleveryun
29 天前
直接解决掉,项目如果太大就渐进式地解决掉。
exonuclease
29 天前
反正我们这强制开着 TreatWarningsAsErrors 的 虽然还是有一堆没使用的 namespace 挂在那
cwxiaos
29 天前
管它呢,拿钱拉屎没什么不可

什么,有 bug?再拉一坨
sadyx
29 天前
我接手的代码都是一堆 error ,如何呢
washangshuang01
29 天前
看项目要求呗。我这 JS 项目可不管你这的那的,怎么舒服怎么来。什么,后面代码出问题怎么办?加个 try...catch 得了。人和程序一个能跑就行
pheyer
29 天前
不要纠结,你想为这些 warning 加班吗?英伟达软件里面的代码也是一堆屎山
spritecn
29 天前
maven xml 里全是 warning,不是这个有漏洞就是那个有漏洞 ...不敢升也升不起,因为明天还有新的
intmax2147483647
29 天前
我司就是坚决不能出现 warning😂
cw2k13as
29 天前
warning 改成 error
buffzty
29 天前
我跟你一样 不会有一个提示 很多人不写就是懒 混 菜.我项目有 lint ci,不遵守的提交不了
importmeta
29 天前
有些前端规则经常更新, 更新这些依赖的版本之后就可能出现 Warning.
ryan4290
29 天前
我问过写出来这些代码的人,他说我没看见啊……我这里没有 warning

这种情况呢,要么就是故意关掉咯,放飞自我,要么就是演员咯,把你当二逼演了再说

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

https://www.v2ex.com/t/1151594

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

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

© 2021 V2EX