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

V2EX = way to explore

V2EX 是一个关于分享和探索的地方

现在注册

已注册用户请登录

这是一个创建于 943 天前的主题，其中的信息可能已经有所发展或是发生改变。

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

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

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

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

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

代码

注释

flask

机试

28 条回复 • 2022-06-18 00:13:20 +08:00

westoy

2022-05-30 21:30:09 +08:00

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

leo108

2022-05-30 21:30:25 +08:00

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

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

sky96111

2022-05-30 21:34:40 +08:00 via Android

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

swulling

2022-05-30 21:35:26 +08:00 via iPad

多个条件换行是好习惯

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

wenjingxiaohuo

2022-05-30 21:36:46 +08:00

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

6167

2022-05-30 21:44:28 +08:00

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

isRealLeven

2022-05-30 22:25:23 +08:00

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

satoru

2022-05-31 06:58:36 +08:00

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

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

cppc

2022-05-31 08:46:55 +08:00 via iPhone

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

chuanqirenwu

2022-05-31 09:10:08 +08:00

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

lozzow

2022-05-31 09:21:52 +08:00 via Android

是不是有树的留言板😉

wdwwtzy

2022-05-31 09:22:15 +08:00 via iPhone

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

ElmerZhang

2022-05-31 09:53:16 +08:00

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

treecoder

2022-05-31 10:15:29 +08:00

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

treecoder

2022-05-31 10:17:57 +08:00

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

luffy

2022-05-31 10:39:39 +08:00

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

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

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

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

6167

2022-05-31 14:23:17 +08:00

@treecoder compass 题目是无穷评论树

wxiao333

2022-05-31 15:44:17 +08:00

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

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