为什么公司的人写代码都无视 IDE 的警告提示?

2022-08-18 17:43:54 +08:00
 magese

现在 IDE 的代码提示已经很完善了,为啥大家都不关注警告的?

前前后后也待过好几家大大小小的公司了,没见过一个人写的代码是 0 Warning 0 Error的。

打开一个类全是黄黄的不难受的吗?

强迫症已经要死了……

16473 次点击
所在节点    程序员
147 条回复
iamqk
2022-08-19 13:29:49 +08:00
我们这个项目 1623 的警告
icyalala
2022-08-19 13:35:21 +08:00
对各种 warning 我反正很不爽,所以在公司我有时间肯定会去改。
但是现在剩下的大部分 warning 都是提示 deprecated 声明或用法,改得麻烦还要过测试。还有的是其他团队代码带来的警告直接没法改,时间长了也只能放着了。

至于我自己的开源项目,CI 里什么 -Werror -Wall -Wextra 之类的全都加上去,希望别人用得时候至少不会在 warning 上糟心。
fyxtc
2022-08-19 14:59:12 +08:00
有意义,但意义不大,矫枉过正,改警告的时间去写新功能或者摸鱼任何一个都比这个有价值
asmoker
2022-08-19 15:49:19 +08:00
说不定人家的 IDE 已经忽略了这些 ⚠️ 看不到呢 [/doge]
nicegp
2022-08-19 17:01:39 +08:00
开发第一原则:能跑就不动
kkbblzq
2022-08-19 17:15:46 +08:00
我自己的代码会尽量处理,别人的管不着,当然没办法一定清零,部分 warning 只是建议项,有的改起来成本挺高的。
loryyang
2022-08-19 17:17:20 +08:00
warning 就是 warning ,如果觉得没问题,就没必要处理。
我在内部要求是:要看 warning ,但不要求全部处理。但是明显有问题的 warning 没发现,事后会被打板子
pkwenda
2022-08-19 17:26:59 +08:00
@magese 你在说什么
pkwenda
2022-08-19 17:27:46 +08:00
FrankHB
2022-08-19 18:49:33 +08:00
@icyalala 极不建议默认 -Werror ,即便是自己控制的项目。要是刚好有用户升级了新版编译器多检查出了没在你这里报的 warning 然后变成 error 就糟心了。要是源码发行版用这种默认策略很可能整个系统动不动升级失败。
349865361
2022-08-19 19:15:29 +08:00
type Template struct {
templates *template.Template
}

func (t *Template) Render(w io.Writer, name string, data interface{}, c echo.Context) error {
return t.templates.ExecuteTemplate(w, name, data)
}


报黄色 c 参数没有使用 但是这是通过其他库的接口方法 c 参数不传会报错 传了不用会警告变黄 我也木法
x86
2022-08-19 19:27:14 +08:00
不出点错我怎么摸鱼修改
cosmosz
2022-08-19 19:35:19 +08:00
CI 里加上没加 lint 或者 format 的要求, 能过 CI 就算过。
或者 IDE 没 config 好, 没读 linter 的 configuration , 用了初设。
fstar
2022-08-19 19:49:31 +08:00
逆风而行的靓丽风景线
akira
2022-08-19 19:53:48 +08:00
项目明天上线,目前还有 32 个 warning ,你是改还是不改
aigonna
2022-08-19 22:43:03 +08:00
反正能跑就行了
icyalala
2022-08-20 00:55:32 +08:00
@FrankHB 正常 Release 当然不会加,严格的检查只在开发阶段 CI 里用参数启用,至少能给那些瞎提 PR 的人看
mingl0280
2022-08-20 10:19:26 +08:00
我们这的话,给客户发的示例代码一定是 0 err 0 warn 的,自己用的软件大部分是 0 err 0 warn 的,少数软件有一些以前的旧硬件的操作必然有 warning 也没办法……
mingl0280
2022-08-20 10:20:46 +08:00
不过这个是编译器的 0 err 0 warn ,不是 lint 的,lint 不要求 0 warning ( C++项目要按 Lint 做,能把你坑死,特指某些傻逼的不让用全局变量 /不让用裸指针的 Lint 规则。)
Aixtuz
2022-08-20 13:15:30 +08:00
1. 有些我不擅长的,只是拿来用用,不敢动;
2. 有些就算擅长的,又不是我写的,懒得管;
3. 有些确实我写的,又不是不能用,催的紧;

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

https://tanronggui.xyz/t/873803

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

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

© 2021 V2EX