最近写了个远程工作的机试，被以“代码注释没清理，多个换行，代码比较粗心大意，代码质量要求不高”的理由挂了。。。
6167 · 2022-05-30 21:16:02 +08:00 · 4381 次点击这是一个创建于 930 天前的主题，其中的信息可能已经有所发展或是发生改变。
机试题目是用 flask 写一个简易的后端，包含授权认证和几个指定功能，我做完了后端这部分的接口，写完了测试，提了 pr ，写好了部署文档，可以说完美符合题目要求。
然后今天就收到了如上反馈，我还特地问了一下 hr ，回复说代码功能基本满足需求
回看了一下代码，只有拦截器那里注释掉了一些跑不通的代码，个别逻辑上注释了几句，然后每个文件都有个注释头，类似于 author 、time 、version 等这类的信息，这还是我特意加的。至于换行，一些 sql 查询语句的条件比如说 filter()以及 order_by()这种的我确实习惯每个条件都分行写，代码逻辑中不同的部分习惯加个空行方便阅读，其他的就没了，pycharm 又没黄，这么写完全没问题吧。
我就纳闷了，代码风格难道不是随着学习和工作慢慢变化的吗？一家公司一家代码风格没什么毛病吧，。
顺便有谁能提供一个 PR 的要领这类的，这东西到底怎么样才算格式化的 flask 代码，怎么样才叫严谨的 PR ？
然后今天就收到了如上反馈，我还特地问了一下 hr ，回复说代码功能基本满足需求
回看了一下代码，只有拦截器那里注释掉了一些跑不通的代码，个别逻辑上注释了几句，然后每个文件都有个注释头，类似于 author 、time 、version 等这类的信息，这还是我特意加的。至于换行，一些 sql 查询语句的条件比如说 filter()以及 order_by()这种的我确实习惯每个条件都分行写，代码逻辑中不同的部分习惯加个空行方便阅读，其他的就没了，pycharm 又没黄，这么写完全没问题吧。
我就纳闷了，代码风格难道不是随着学习和工作慢慢变化的吗？一家公司一家代码风格没什么毛病吧，。
顺便有谁能提供一个 PR 的要领这类的，这东西到底怎么样才算格式化的 flask 代码，怎么样才叫严谨的 PR ？
 |
1
westoy 2022-05-30 21:30:09 +08:00  1
几个指定的功能? 有没有可能他们给每个应聘者都提了几个特定的功能.........
|
 |
2
leo108 2022-05-30 21:30:25 +08:00 1
1. 在有版本控制的情况下不是很懂在文件头添加作者信息有什么用
2. 注释掉跑不通的代码，中间状态的代码在提交之前应该删掉，没有人会愿意去读一个半成品的代码 如果楼主工作年限不长的话这些确实都不是什么大问题，只能说这家公司的筛选条件有点诡异 |
 |
3
sky96111 2022-05-30 21:34:40 +08:00 via Android
有没有一种可能，公司在白嫖这些应聘的
|
 |
4
swulling 2022-05-30 21:35:26 +08:00 via iPad
多个条件换行是好习惯
可能这个公司是以代码行数算绩效，怕你过去占便宜吧 |
 |
5
wenjingxiaohuo 2022-05-30 21:36:46 +08:00
个人感觉能力是可以的，但是可能有更好地候选人，只是随便找一个理由来拒绝你
|
 |
6
6167 OP @leo108 之前公司的框架就这么写的，所以这个习惯就留下来了，注释掉的代码确实不应该上传，不过我的考虑是以后再完善一下把这部分加回来
|
 |
7
isRealLeven 2022-05-30 22:25:23 +08:00
我不喜欢在工程里注释代码，直接删掉最好
|
 |
8
satoru 2022-05-31 06:58:36 +08:00 1
> 至于换行，一些 sql 查询语句的条件比如说 filter()以及 order_by()这种的我确实习惯每个条件都分行写
这点没问题，这样以后修改 diff 会比较好看。 |
 |
9
cppc 2022-05-31 08:46:55 +08:00 via iPhone
交上去的就是你的作品，不是草稿。你这样显得随意了。不过这种小事我觉得提醒一下，改正就行了
|
 |
10
chuanqirenwu 2022-05-31 09:10:08 +08:00
用 black 格式化一下嘛，刚好可以筛掉不用 black 的公司。另外文件头加作者信息注释，遗留不用的代码确实不是好习惯。
|
 |
11
lozzow 2022-05-31 09:21:52 +08:00 via Android 1
是不是有树的留言板😉
|
 |
13
ElmerZhang 2022-05-31 09:53:16 +08:00
楼主是不理解为什么因为「代码注释没清理，多个换行，代码比较粗心大意，代码质量要求不高」这个理由挂掉你？
其实这种情况很常见。比如我现在招人，要招的是平时和我一起工作的人，除了技术能力有要求，对做事风格也是有要求的，不然性格不合天天吵架还怎么一起工作。 |
 |
14
treecoder 2022-05-31 10:15:29 +08:00 2
是不是不限层级的留言板？我也做了，前后端，react+flask ，也挂了，比你还惨，原因都没说
|
|
15
treecoder 2022-05-31 10:17:57 +08:00
slasify ，我面的好像是这家公司，你的是不是？
|
 |
16
luffy 2022-05-31 10:39:39 +08:00
这个面试要求的是一些 common sense. 楼主面挂掉无可厚非。
关于代码风格这件事，难道楼主不清楚什么叫 行业标准 嘛？ 每种语言，都会有一些 best practice, 一些标准化的。 如果想了解 python 标准化的 style, 请参考 python 官网介绍。 |
 |
18
wxiao333 2022-05-31 15:44:17 +08:00 2
|
 |
19
aaaaaaaaa 2022-05-31 15:58:54 +08:00 via iPhone
|
|
20
aaaaaaaaa 2022-05-31 16:05:39 +08:00 via iPhone
|
 |
21
gouflv 2022-05-31 16:05:47 +08:00 via iPhone
代码发出来啊
|
|
22
treecoder 2022-05-31 20:46:16 +08:00 1
@aaaaaaaaa 实际上应该是 slasify 这个外包公司在给 compass 招人，就是外包，这个岗位年前就在招，只是当时我没有做题，五一过后开始找 remote job 的时候有个 slasify 的哥们联系了我，我才发现这个岗还在招，就做了题，难受，花了好几天呢
|
 |
23
whyzp2019 2022-06-01 09:37:07 +08:00
我也觉得只要他不想要，连呼吸都是错的
|
 |
24
iamgodot 2022-06-01 10:25:14 +08:00
|
 |
25
cszchen 2022-06-01 10:26:06 +08:00 via iPhone
楼上好多说楼主不应该怎么怎么样的，如果给了规范楼主没按规范来，那肯定是他的问题。
但是没看到楼主说给规范，关注的重点难道不是功能实现吗。 |
 |
26
nazhenhuiyi294 2022-06-03 06:49:57 +08:00
compass 我面试过前端，给了一个只有我目前 50% 不到的薪资范围，把我整笑了。
|
 |
27
CarolFxl 2022-06-18 00:11:31 +08:00
@nazhenhuiyi294 我也面过他们公司的前端，不过我最后面试没过，不知道他们压薪资这么厉害的？
|
|
28
CarolFxl 2022-06-18 00:13:20 +08:00
我也有一次挂掉的理由居然是：过渡封装！我只是把我能想到的多写了一些，然后抽离了组件，更加丰富了功能，结果告诉我过渡封装了，我真的是服气了，花了我两个晚上写的。
|
Select Language