最近写了个远程工作的机试，被以“代码注释没清理，多个换行，代码比较粗心大意，代码质量要求不高”的理由挂了。。。

[6167]

机试题目是用 flask 写一个简易的后端，包含授权认证和几个指定功能，我做完了后端这部分的接口，写完了测试，提了 pr ，写好了部署文档，可以说完美符合题目要求。

然后今天就收到了如上反馈，我还特地问了一下 hr ，回复说代码功能基本满足需求

回看了一下代码，只有拦截器那里注释掉了一些跑不通的代码，个别逻辑上注释了几句，然后每个文件都有个注释头，类似于 author 、time 、version 等这类的信息，这还是我特意加的。至于换行，一些 sql 查询语句的条件比如说 filter()以及 order_by()这种的我确实习惯每个条件都分行写，代码逻辑中不同的部分习惯加个空行方便阅读，其他的就没了，pycharm 又没黄，这么写完全没问题吧。

我就纳闷了，代码风格难道不是随着学习和工作慢慢变化的吗？一家公司一家代码风格没什么毛病吧，。


顺便有谁能提供一个 PR 的要领这类的，这东西到底怎么样才算格式化的 flask 代码，怎么样才叫严谨的 PR ？

[gouflv]

代码发出来啊

[treecoder]

@aaaaaaaaa 实际上应该是 slasify 这个外包公司在给 compass 招人，就是外包，这个岗位年前就在招，只是当时我没有做题，五一过后开始找 remote job 的时候有个 slasify 的哥们联系了我，我才发现这个岗还在招，就做了题，难受，花了好几天呢

[whyzp2019]

我也觉得只要他不想要，连呼吸都是错的

[iamgodot]

@treecoder @wxiao333 估计对方想看闭包表的实现吧，但其实邻接表 cte 完全可以满足需求了。个人对这种挂羊头卖狗肉的笔试需求有一些反感。为了缩减招聘成本无可厚非，但是很浪费面试者的时间。

[cszchen]

楼上好多说楼主不应该怎么怎么样的，如果给了规范楼主没按规范来，那肯定是他的问题。
但是没看到楼主说给规范，关注的重点难道不是功能实现吗。

[nazhenhuiyi294]

compass 我面试过前端，给了一个只有我目前 50% 不到的薪资范围，把我整笑了。

[CarolFxl]

@nazhenhuiyi294 我也面过他们公司的前端，不过我最后面试没过，不知道他们压薪资这么厉害的？

[CarolFxl]

我也有一次挂掉的理由居然是：过渡封装！我只是把我能想到的多写了一些，然后抽离了组件，更加丰富了功能，结果告诉我过渡封装了，我真的是服气了，花了我两个晚上写的。