想吐槽一下经理的code review

ymsfd007 发表于 2021-04-28 08:27:02
有规矩管得严都是好事，但要有理有据，不能独断专行或者主观臆断。我刚进组还是**的时候就是这个原则，但凡是无凭无据的主观反馈，一律怼回去，对方理亏，或者公说公有理 婆说婆有理，下次就不乱挑刺了，只说客观

嗯嗯，我们小厂，没有具体的coding style guidance，都是经理说了算，不太了其他地方怎么弄得，谢谢分享！

001596 发表于 2021-04-28 13:33:44
那你有没有去google一下，int和size_t的区别啊？以及有没有可能因为这个区别，造成code crash？

我本来也知道的int 和size_t 区别，刚才又特意Google一下😂，看看是不是理解的正确

面试问题 发表于 2021-04-28 10:50:16
你看看有26个downvote就知道你的想法不对 你提出的那些都是最基本的coding style啊 根本不是扣细节 你以后慢慢coding style变好的不就不会收到这么多comment了吗？cl

嗯是的，是的有道理，现在理解了，谢谢指出来！

backtouzhaogong 发表于 2021-04-27 18:53:28
如果真心想统一这些语法风格细节，应该花点时间写个linter或交给ide去解决。code review不是干这个的。

我在小厂哈，了解了，谢谢你的分享！

首先 好公司能做管理的（说的少一点）70%都是比你厉害、有经验、成熟的人，等你真的上到一定层次你会理解我说的.（我相信已经和我坐过这个位置的人都会有“过同感”）
然后 你要把代码写完还是把代码写好这是两回事！！！ 就好像python和C++一样，为啥C++比python繁琐这么多？你也可以写python呀，定义就那几种不用管。

最后 如果经理真的仅仅是追求个人感觉的，那么你可以整理写好报告并在报告中提出为什么、解决方法，如果这种方式得不到解决那么你可以换工作，你的个人工作环境也很重要，前提真的是对方错了！！！！而不是你自嗨（你用好的眼光看待我的回复，你看到的就是我对你的善意）

“还比如map<int,int> , vector<double>不能直接写，必须用typedef成 IntVector”，谁能帮忙解释一下，什么原理？是C写多了的后遗症吗？

UaDNhsTf7E 发表于 2021-04-28 15:23:19
首先 好公司能做管理的（说的少一点）70%都是比你厉害、有经验、成熟的人，等你真的上到一定层次你会理解我说的.（我相信已经和我坐过这个位置的人都会有“过同感”）
然后 你要把代码写完还是把代码写好这

嗯嗯，谢谢你的建议！很有帮助！

一开始以为是经理想要故意跳出一些问题，现在感觉有些东西，确实按着经理的说的，是从team code统一性，可读性考虑的，我之前想的有些欠妥了

感谢你的分析和建议！

我现在这个厂也是个信用卡公司小厂，各种不正规，哎我和公司就互相坑了两年，现在要去大厂了，不知道我这个坏习惯到新公司丢脸要丢成啥样...

tonycy0 发表于 2021-4-29 06:25
“还比如map , vector不能直接写，必须用typedef成 IntVector”，谁能帮忙解释一下，什么原理？是C写多了的 ...

是不是实际情况map和vector里的类型不是int这种，是更长名字的class或者嵌套了更多层别的container。如果不给个alias担心可读性不好

匿名用户 发表于 2021-04-28 14:31:56
我承认我菜，需要努力了，我也诚恳感谢大家点评。

世界上有的人就像你这样不会好好说话，就让人道歉，有意思吗？我觉得不是我理解有偏差，是你理解的问题吧，一副高高在上的样子。你会看好好看我的发的内容吗？我

谦虚点，你就别怼，瞧瞧你这幅气急败坏的模样，可怜