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

26 天前
 ciaoSora

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

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

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

6116 次点击
所在节点    职场话题
72 条回复
hefish
26 天前
摒弃学院派的陋习,拥抱工程派的崭新规矩。
yb2313
26 天前
到文件首行增加注释关闭警告, 或者直接把这个项目的警告关闭
kokerkov
26 天前
我寻思你随便打开个网站,按 F12 然后转到 console ,不都一大堆 warning 吗
jzhm
26 天前
我待过的几个公司,都是要消掉全部 warning ,可能数通行业比较严格?
klo424
26 天前
我都是直接关掉全部 warning ,因为我也忍不了,但是无奈别人会写,只能屏蔽了。
ciaoSora
26 天前
@jzhm 羡慕你们啊,我命令行编译时看到几十几百个 warning 头都要炸了
ciaoSora
26 天前
@klo424 我不敢关,怕自己的代码也出问题,只能硬着头皮看编辑器里一堆 warning 提示
yuanmomo
26 天前
不太可能,我在的公司已经很小心了,但是还是不能避免,有的东西就是这样的
klo424
26 天前
没办法啊,就算自己辛苦一点,把别人的 warning 也都改好,还是会有人写出新的 warning 。唯一的解决办法就是公司层面强制要求无 warning 。
klo424
26 天前
你无法要求别人跟你一样,就只能去适应或者找一个能满足你要求的公司了。
deplives
26 天前
还是太闲
dfkjgklfdjg
26 天前
很多老项目打开确实就是会出现一堆的 `warning` 。很多告警其实只是某一些依赖已经停止维护或者 API 即将废弃的提醒。
在没有明确需要升级的场景下,你去处理这些 warning ,升级依赖和改用 API 可能就会造成生产事故,到时候是谁背锅担责呢?

最简单的一个坑,比如说 JS 项目中的 `==` 在开启 lint 之后可能就会让改成 `===` 那么你到底是改还是不改呢?还是去改 lint 配置文件?需要你去了解很多犄角旮旯的历史包袱才能明确,这是需要投入时间成本的。
可能我们拿到一个老的项目,把所有的 warning 和 error 都解决确实会非常的“爽”,但是这个 “爽” 并没有实际价值。工作中都是大部分人和项目项目能跑就行了。不是刚毕业的人了是需要改变自己代码洁癖和强迫症的。

如果可以预知这个接盘的项目以后会成为你主要的工作内容。那么你可以向上级提出你的想法和计划,做成你的 KPI 。然后去安排去花时间处理这些需要改进的部分。
但是不管怎么样,项目团队不会因为你一个人的坚持,就不是草台班子了。
nickfox5880
26 天前
老板都不痛苦 你为啥觉得痛苦 代码又不是你的
echoechoin
26 天前
我直接一个-Werror 让你编不过
wangtian2020
26 天前
typescript 的所有 warning 都在我的控制中,不是 error 只是我还没提起干劲修的 type 而已
Subfire
26 天前
老哥咱们一样, 别人的那就只能当看不见了, 自己写的尽量 0 warning
P945
26 天前
一开始强迫症真的受不了,后来能跑就行,除非自己一手撸的代码,否则千万别没事找事干。
proxychains
26 天前
warning is not error
jzhm
26 天前
@ciaoSora #6 我们代码要是有 warning 提交上去会被通报
hwdq0012
26 天前
用 c++写 react native ,ts 和 c++ 都有很多 warning ,直接关掉不看

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

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

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

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

© 2021 V2EX