请问大家对于代码风格比较差的 PR 是如何合并的

[sqfphoenix]

目前有一个代码风格比较差（当然是我认为，能跑就行的老哥没必要看下去了）的 PR ，涉及到的代码行数有好几百行，新增的方法名称还都是模棱两可的名称，没有明确语义。现在人已经看麻了，根本看不懂改了啥，也没办法描述 PR 有啥问题，对于这种 PR ，大家是直接打回吗

[CodeM]

风格好的代码大几百行估计也没几个人愿意每次看完就是了（

[ss098]

编写 Review 意见打回呀

[sqfphoenix]

@ss098 很多代码写的不太好，导致没法给出很明确的 Review ，这种情况是让作者先把代码优化一下吗

[anonymousar]

@sqfphoenix 如果是你负责 review 那我觉得还是应该一行一行仔细 review 给出明确建议的。
比如名称你觉得模棱两可 那就应该给出你认可的名称。

[sqfphoenix]

@anonymousar 感谢，我现在是一行一行看的，但是疑问比较多，感觉可能是看多了不够耐心，我还是沉下心来继续看吧

[ss098]

@sqfphoenix 是的，可以写逻辑不够清晰之类的。

[HuskyYellow]

肯定打回, 但是最好说下哪里需要改进,如果着急上线, 那就先合并,以后提优化的板子.

[ragnaroks]

功能没有问题就先合并，然后在其基础上格式化

[bfdh]

@anonymousar #4 其实这个很难。有些代码真的是差到根本无法 review 。每一行好像都没问题，但加在一起就全是问题，逻辑不对、条理不清，要给建议那也只能建议推倒重来。遇到这种代码，有时候真的是无从下手。

[sqfphoenix]

@bfdh 我现在就是这个情况哈哈，已经提了十几个点，先解决可读性和基本逻辑上的问题，再往下看具体功能实现