这是一个创建于 849 天前的主题，其中的信息可能已经有所发展或是发生改变。
目前有一个代码风格比较差（当然是我认为，能跑就行的老哥没必要看下去了）的 PR ，涉及到的代码行数有好几百行，新增的方法名称还都是模棱两可的名称，没有明确语义。现在人已经看麻了，根本看不懂改了啥，也没办法描述 PR 有啥问题，对于这种 PR ，大家是直接打回吗
 |
1
CodeM 2022-08-08 15:54:52 +08:00
风格好的代码大几百行估计也没几个人愿意每次看完就是了（
|
 |
2
ss098 2022-08-08 15:56:35 +08:00
编写 Review 意见打回呀
|
 |
3
sqfphoenix OP @ss098 很多代码写的不太好，导致没法给出很明确的 Review ，这种情况是让作者先把代码优化一下吗
|
 |
4
anonymousar 2022-08-08 16:08:14 +08:00
@sqfphoenix 如果是你负责 review 那我觉得还是应该一行一行仔细 review 给出明确建议的。
比如名称你觉得模棱两可 那就应该给出你认可的名称。 |
|
5
sqfphoenix OP @anonymousar 感谢，我现在是一行一行看的，但是疑问比较多，感觉可能是看多了不够耐心，我还是沉下心来继续看吧
|
|
6
ss098 2022-08-08 16:23:11 +08:00
@sqfphoenix 是的，可以写逻辑不够清晰之类的。
|
 |
7
HuskyYellow 2022-08-08 18:06:31 +08:00
肯定打回, 但是最好说下哪里需要改进,如果着急上线, 那就先合并,以后提优化的板子.
|
 |
8
ragnaroks 2022-08-09 09:12:10 +08:00
功能没有问题就先合并，然后在其基础上格式化
|
 |
9
bfdh 2022-08-09 09:52:17 +08:00
@anonymousar #4 其实这个很难。有些代码真的是差到根本无法 review 。每一行好像都没问题，但加在一起就全是问题，逻辑不对、条理不清，要给建议那也只能建议推倒重来。遇到这种代码，有时候真的是无从下手。
|
|
10
sqfphoenix OP @bfdh 我现在就是这个情况哈哈，已经提了十几个点，先解决可读性和基本逻辑上的问题，再往下看具体功能实现
|
Select Language