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

[6167]

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

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

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

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


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

[westoy]

几个指定的功能? 有没有可能他们给每个应聘者都提了几个特定的功能.........

[leo108]

1. 在有版本控制的情况下不是很懂在文件头添加作者信息有什么用
2. 注释掉跑不通的代码，中间状态的代码在提交之前应该删掉，没有人会愿意去读一个半成品的代码

如果楼主工作年限不长的话这些确实都不是什么大问题，只能说这家公司的筛选条件有点诡异

[sky96111]

有没有一种可能，公司在白嫖这些应聘的

[swulling]

多个条件换行是好习惯

可能这个公司是以代码行数算绩效，怕你过去占便宜吧

[wenjingxiaohuo]

个人感觉能力是可以的，但是可能有更好地候选人，只是随便找一个理由来拒绝你

[6167]

@leo108 之前公司的框架就这么写的，所以这个习惯就留下来了，注释掉的代码确实不应该上传，不过我的考虑是以后再完善一下把这部分加回来

[isRealLeven]

我不喜欢在工程里注释代码，直接删掉最好

[satoru]

> 至于换行，一些 sql 查询语句的条件比如说 filter()以及 order_by()这种的我确实习惯每个条件都分行写

这点没问题，这样以后修改 diff 会比较好看。

[cppc]

交上去的就是你的作品，不是草稿。你这样显得随意了。不过这种小事我觉得提醒一下，改正就行了

[chuanqirenwu]

用 black 格式化一下嘛，刚好可以筛掉不用 black 的公司。另外文件头加作者信息注释，遗留不用的代码确实不是好习惯。

[lozzow]

是不是有树的留言板😉

[wdwwtzy]

@leo108 加 1 ，很不喜欢文件头作者时间等，毫无用处

[ElmerZhang]

楼主是不理解为什么因为「代码注释没清理，多个换行，代码比较粗心大意，代码质量要求不高」这个理由挂掉你？
其实这种情况很常见。比如我现在招人，要招的是平时和我一起工作的人，除了技术能力有要求，对做事风格也是有要求的，不然性格不合天天吵架还怎么一起工作。

[treecoder]

是不是不限层级的留言板？我也做了，前后端，react+flask ，也挂了，比你还惨，原因都没说

[treecoder]

slasify ，我面的好像是这家公司，你的是不是？

[luffy]

这个面试要求的是一些 common sense. 楼主面挂掉无可厚非。

关于代码风格这件事，难道楼主不清楚什么叫 行业标准 嘛？

每种语言，都会有一些 best practice, 一些标准化的。

如果想了解 python 标准化的 style, 请参考 python 官网介绍。

[6167]

@treecoder compass 题目是无穷评论树

[wxiao333]

@treecoder
@6167
我也是这家这道题，笔试挂，在我的一再要求下，给我回了原因，说我的无限层级性能不行
麻蛋我自己测试数据 50 几层，后端响应不到 100ms ，不知道为什么说我性能不行。

个人感觉，在候选人已经完成了题目所有要求的前提下，不给面试机会真的是太恶心了
做他这道题，我加班熬了几个晚上，最后给了我这个结果。

[aaaaaaaaa]

@lozzow
@treecoder
@6167
我也是这个，啥情况啊，我看他这岗位挂了十个多月了都没招到人，不会是骗子吧

[aaaaaaaaa]

@wxiao333
@6167
虽然说是 compass 中国团队，但应该类似外包，签合同是新加坡人力资源公司