想吐槽一下经理的code review

xchuang916 发表于 2021-04-27 23:55:30
size_t一点毛病没有 这个不用review CI都过不了

我司在示例中就用
For int i

而且，single statement loop, braces are optional （我个人不习惯这个）

davzhao93 发表于 2021-4-29 01:49
不敢苟同。
以前我司有个大佬，做ppt根本不理会前后slides风格一样。
为此，下面有些人还laugh at them, a ...

well我说的这个人不只是公司大佬，还在外面的社区有很高的reputation，他活儿好大概是公认的。我感觉跟lz的情况比较类似，但是我不懂码。

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

????? 您这是自黑？？？

我是楼主，在小厂工作，谢谢大家的评论，发现自对经理的考虑有偏差，也大致理解了大厂的code review和coding style的 concern，楼主是比较菜

我声明一下，a+b+c是我简化描述的，实际上是有复杂的公式表示a, b,  c，而不是三个variable。

用int是parse configuration 明确知道不会越界的情况下。

感谢大多数的人建议！很实用，感谢大家中肯的建议，我以后会更努力的！

对于少部分的喷子，我也不想怼回去了，祝你们一生平安🤪

想法是好的，只是执行方法值得商榷。有这工夫写个好点儿的doc不就行了，有新问题随时update code doc

油冬冬 发表于 2021-04-28 05:34:41
不是有required reviewers and optional reviewers吗？是不是所有required reviewers approve后就可以merge了？

我们code review用的stash，好像没有required/optional之类的feature（或者我们没有用）

这个不归经理管，但你经理说得没错。type check，naming convention这些都是编程语言大忌。不是能写个dfs就算完事的。不归听起来他应该把这些事情交给你的peers来review

感觉如果刚入行有人跟我说这些，我会觉得很开心

匿名者 发表于 2021-4-29 02:40
我是楼主，在小厂工作，谢谢大家的评论，发现自对经理的考虑有偏差，也大致理解了大厂的code review和codin ...

那你有没有去google一下，int和size_t的区别啊？以及有没有可能因为这个区别，造成code crash？