想吐槽一下经理的code review

想吐槽一下经理的code review, 总提一些吹毛求疵的问题, code 明明有很多种写法，必须要限制你的写法和对variable 的命名, 组是用C++

第一次code review 说，for loop 不能用 int i = 0; 必须用 size_t i = 9 ，让全改过来。 还比如map<int,int> , vector<double>不能直接写，必须用typedef成 IntVector. 再比如 一个document 的算法最后要把三个数加一起, 不能写 sum = 0; sum += a; sum += b; sum += c; 必须写成 sum += a + b + c, 说这么写跟document 算法是一样的 更make sense;

再比如code 名字也必须按照他的要求，比如不能用subProductID, 必须用modelID;  再比如header file 两个variable中间空了三行，让删除多余的两行。

跟这种manager，有时候真的心累。

如果真心想统一这些语法风格细节，应该花点时间写个linter或交给ide去解决。code review不是干这个的。

别的不说，int vs size_t那个，你认错吧。如果用int，越界了你怎么办？production code直接crash吗？

这也叫写严格？？？？？？？？？？？我真是满头问号 多了两行空行不删除留着过年？？？？？变量名取成更make sense的名字也心累？？？？？？？？？？？信男愿一生吃肉不用遇到不注重代码质量的队友

你们大厂难道code review都这么严格么？

你们有细致的coding doc规定必须怎么写吗？没有的话，照着大多数人的写法应该就可以了。经理不花时间想怎么拉项目，扩大组的影响力，天天抠细节，不是好经理啊。

mandmwantapig 发表于 2021-04-27 18:32:23
你们大厂难道code review都这么严格么？

不是大厂哈哈

匿名用户 发表于 2021-04-27 18:43:40
你们有细致的coding doc规定必须怎么写吗？没有的话，照着大多数人的写法应该就可以了。经理不花时间想怎么拉项目，扩大组的影响力，天天抠细节，不是好经理啊。

没有coding doc规定怎么写的哈哈，感觉经理是想code按照他的思路写，确实经理没怎么想着扩大影响力，感觉经理比较想着安于现状，确实看见过怎么拉项目之类的，反而想着推项目，有的改太费劲，就让别的组负责，或者拉长project 时长

同意楼上的看法, 感觉你的经理太关注细节, 但又没有采用更有效率的方法来达到他的目的: 我觉得他想要追求的东西可能就是整个codebase风格统一易读, 这本身也算合理, 比如说一个码工读别人code的时候一般也不希望一会儿七言一会儿散文一会儿又是十四行诗. 但达到这类目的比较有效率的做法要么是靠组里都遵循某个coding style guide, 要么就是靠一些工具(linter之类)来做; 完全靠人工review来口口相传的做法相对比较低效了些. Google挺早就有不少相关的guide和工具再外加人工coding style review来帮助新人在这方面融入, 不过也许需要这类大厂才有精力来搞这个?

组内如果对格式有要求 应该写一个模版 大家每个人都遵守

危！ zszszszs

作为新手，我也常常遇到这个问题。没有办法，他们说一个地方，那就改一个地方呗。改多了就习惯了。