组员写代码喜欢复制代码,复制完不格式化代码, 代码的写的歪七八扭的。今天说他,他改了重新提交,但是他的态度就是完全不在乎,一点都不是上心,经常犯。这是我的问题还是组员的问题,我要不要严格要求组员格式化代码,还是不管他们,只要代码能运行就行。

2019-09-24 11:01:56 +08:00
 burnbrid
14227 次点击
所在节点    程序员
129 条回复
yalin
2019-09-24 11:18:30 +08:00
提交审查工具?
whasyt
2019-09-24 11:18:54 +08:00
我看着标题,我以为你就是那个组员
tabris17
2019-09-24 11:20:33 +08:00
你可以要求做 code review 呀,格式不对打回去,不给合并。

如果没有人手就拉倒吧。做好单元测试,谁写的代码谁负责咯,出问题了扣钱
tohearts
2019-09-24 11:21:41 +08:00
code scan,代码不格式化,要以后维护的怎么看。安排去非洲吧
wc951
2019-09-24 11:24:06 +08:00
提交的时候自动格式化,静态检查
holy_sin
2019-09-24 11:26:35 +08:00
搞个 mr,格式不行不给过就好了
qshu
2019-09-24 11:28:28 +08:00
lint 整上 编辑器再 auto fix
opengps
2019-09-24 11:28:42 +08:00
我就是这类组员,只不过没不缩进那么夸张,VS 里自动缩进也就是一个快捷键的事。
至于是否上心,主要取决于是否对规范有兴趣。每个人其实不是不遵守规范,而是都形成了“自己的规范”,所以当在团队里做事的时候会不知不觉的只执行“自己的规范”,这种情况下,很多时候都宁可扣点分也懒得修正。
当然这不是不改正的理由,只是很多人的对一些“不重要”事情的观点,并不是追求 100 分,所以遇到这类问题,永远都是屡禁不止,真想解决,要么强制考核(员工容易抵触),要么培训员工的规范意识(抵触会稍低点)。
uleh
2019-09-24 11:32:25 +08:00
在 CI 上挂个 lint (偷笑)
14m3
2019-09-24 11:35:26 +08:00
可以使用 clang-format,规定使用统一格式文件(.clang-format 文件)
sanqian
2019-09-24 11:35:43 +08:00
格式化啊 一个快捷键的问题。。。
xuanbg
2019-09-24 11:36:04 +08:00
@usslss 我所见过的程序员,倒是有大半就是楼主说的这种。为啥他们对格式没有一点点的要求,因为他们写过代码就再也不看了呀。其实他们知道自己代码烂,所以就更不愿意去看了。。。

因为代码烂所以不想改,因为不想改所以代码烂。整一个蜜汁恶性循环
iPhoneXI
2019-09-24 11:37:05 +08:00
先把规范写进文档,然后 Lint 加入 CI,强制 1 推行
passerbytiny
2019-09-24 11:40:12 +08:00
明显是你的问题——不会管理,这大概也是公司的问题——不会或者不愿寻找合适的中层管理。
hoyixi
2019-09-24 11:41:35 +08:00
这种人根本不适合做程序员,和类似逻辑严密类型的工作。
usslss
2019-09-24 11:43:36 +08:00
@xuanbg ..为啥会不再看 我自己格式化备注 是为了方便自己,不然过一阵回头看好多东西根本记不住
securityCoding
2019-09-24 11:43:57 +08:00
随手格式化是职业操守啊
Shook
2019-09-24 11:44:08 +08:00
团队 review 的时候就可以进行代码风格和统一。
lihongjie0209
2019-09-24 11:45:27 +08:00
持续集成之前自动格式化一下不就好了
richzhu
2019-09-24 11:46:29 +08:00
组长有权利要求格式化,不格式化接手人或合作人不是一脸懵逼? 不要求格式化反而感觉你组长做的不到位

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

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

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

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

© 2021 V2EX