目前有一个代码风格比较差（当然是我认为，能跑就行的老哥没必要看下去了）的 PR ，涉及到的代码行数有好几百行，新增的方法名称还都是模棱两可的名称，没有明确语义。现在人已经看麻了，根本看不懂改了啥，也没办法描述 PR 有啥问题，对于这种 PR ，大家是直接打回吗
This topic created in 1557 days ago, the information mentioned may be changed or developed.
 |
CodeMAug 8, 2022
1
风格好的代码大几百行估计也没几个人愿意每次看完就是了（
|
 |
ss098Aug 8, 2022
2
编写 Review 意见打回呀
|
 |
3
@ss098 很多代码写的不太好，导致没法给出很明确的 Review ，这种情况是让作者先把代码优化一下吗
|
 |
anonymousarAug 8, 2022
4
@sqfphoenix 如果是你负责 review 那我觉得还是应该一行一行仔细 review 给出明确建议的。
比如名称你觉得模棱两可 那就应该给出你认可的名称。 |
|
5
@anonymousar 感谢，我现在是一行一行看的，但是疑问比较多，感觉可能是看多了不够耐心，我还是沉下心来继续看吧
|
|
ss098Aug 8, 2022
6
@sqfphoenix 是的，可以写逻辑不够清晰之类的。
|
 |
ZhaokunZhangAug 8, 2022
7
肯定打回, 但是最好说下哪里需要改进,如果着急上线, 那就先合并,以后提优化的板子.
|
 |
ragnaroksAug 9, 2022
8
功能没有问题就先合并，然后在其基础上格式化
|
 |
bfdhAug 9, 2022
9
@anonymousar #4 其实这个很难。有些代码真的是差到根本无法 review 。每一行好像都没问题，但加在一起就全是问题，逻辑不对、条理不清，要给建议那也只能建议推倒重来。遇到这种代码，有时候真的是无从下手。
|
|
10
@bfdh 我现在就是这个情况哈哈，已经提了十几个点，先解决可读性和基本逻辑上的问题，再往下看具体功能实现
|
Select Language