V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
6167
V2EX  ›  求职

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

  •  1
     
  •   6167 · 2022-05-30 21:16:02 +08:00 · 4351 次点击
    这是一个创建于 907 天前的主题,其中的信息可能已经有所发展或是发生改变。
    机试题目是用 flask 写一个简易的后端,包含授权认证和几个指定功能,我做完了后端这部分的接口,写完了测试,提了 pr ,写好了部署文档,可以说完美符合题目要求。

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

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

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


    顺便有谁能提供一个 PR 的要领这类的,这东西到底怎么样才算格式化的 flask 代码,怎么样才叫严谨的 PR ?
    28 条回复    2022-06-18 00:13:20 +08:00
    westoy
        1
    westoy  
       2022-05-30 21:30:09 +08:00   ❤️ 1
    几个指定的功能? 有没有可能他们给每个应聘者都提了几个特定的功能.........
    leo108
        2
    leo108  
       2022-05-30 21:30:25 +08:00   ❤️ 1
    1. 在有版本控制的情况下不是很懂在文件头添加作者信息有什么用
    2. 注释掉跑不通的代码,中间状态的代码在提交之前应该删掉,没有人会愿意去读一个半成品的代码

    如果楼主工作年限不长的话这些确实都不是什么大问题,只能说这家公司的筛选条件有点诡异
    sky96111
        3
    sky96111  
       2022-05-30 21:34:40 +08:00 via Android
    有没有一种可能,公司在白嫖这些应聘的
    swulling
        4
    swulling  
       2022-05-30 21:35:26 +08:00 via iPad
    多个条件换行是好习惯

    可能这个公司是以代码行数算绩效,怕你过去占便宜吧
    wenjingxiaohuo
        5
    wenjingxiaohuo  
       2022-05-30 21:36:46 +08:00
    个人感觉能力是可以的,但是可能有更好地候选人,只是随便找一个理由来拒绝你
    6167
        6
    6167  
    OP
       2022-05-30 21:44:28 +08:00
    @leo108 之前公司的框架就这么写的,所以这个习惯就留下来了,注释掉的代码确实不应该上传,不过我的考虑是以后再完善一下把这部分加回来
    isRealLeven
        7
    isRealLeven  
       2022-05-30 22:25:23 +08:00
    我不喜欢在工程里注释代码,直接删掉最好
    satoru
        8
    satoru  
       2022-05-31 06:58:36 +08:00   ❤️ 1
    > 至于换行,一些 sql 查询语句的条件比如说 filter()以及 order_by()这种的我确实习惯每个条件都分行写

    这点没问题,这样以后修改 diff 会比较好看。
    cppc
        9
    cppc  
       2022-05-31 08:46:55 +08:00 via iPhone
    交上去的就是你的作品,不是草稿。你这样显得随意了。不过这种小事我觉得提醒一下,改正就行了
    chuanqirenwu
        10
    chuanqirenwu  
       2022-05-31 09:10:08 +08:00
    用 black 格式化一下嘛,刚好可以筛掉不用 black 的公司。另外文件头加作者信息注释,遗留不用的代码确实不是好习惯。
    lozzow
        11
    lozzow  
       2022-05-31 09:21:52 +08:00 via Android   ❤️ 1
    是不是有树的留言板😉
    wdwwtzy
        12
    wdwwtzy  
       2022-05-31 09:22:15 +08:00 via iPhone
    @leo108 加 1 ,很不喜欢文件头作者时间等,毫无用处
    ElmerZhang
        13
    ElmerZhang  
       2022-05-31 09:53:16 +08:00
    楼主是不理解为什么因为「代码注释没清理,多个换行,代码比较粗心大意,代码质量要求不高」这个理由挂掉你?
    其实这种情况很常见。比如我现在招人,要招的是平时和我一起工作的人,除了技术能力有要求,对做事风格也是有要求的,不然性格不合天天吵架还怎么一起工作。
    treecoder
        14
    treecoder  
       2022-05-31 10:15:29 +08:00   ❤️ 2
    是不是不限层级的留言板?我也做了,前后端,react+flask ,也挂了,比你还惨,原因都没说
    treecoder
        15
    treecoder  
       2022-05-31 10:17:57 +08:00
    slasify ,我面的好像是这家公司,你的是不是?
    luffy
        16
    luffy  
       2022-05-31 10:39:39 +08:00
    这个面试要求的是一些 common sense. 楼主面挂掉无可厚非。

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

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

    如果想了解 python 标准化的 style, 请参考 python 官网介绍。
    6167
        17
    6167  
    OP
       2022-05-31 14:23:17 +08:00
    @treecoder compass 题目是无穷评论树
    wxiao333
        18
    wxiao333  
       2022-05-31 15:44:17 +08:00   ❤️ 2
    @treecoder
    @6167
    我也是这家这道题,笔试挂,在我的一再要求下,给我回了原因,说我的无限层级性能不行
    麻蛋我自己测试数据 50 几层,后端响应不到 100ms ,不知道为什么说我性能不行。

    个人感觉,在候选人已经完成了题目所有要求的前提下,不给面试机会真的是太恶心了
    做他这道题,我加班熬了几个晚上,最后给了我这个结果。
    aaaaaaaaa
        19
    aaaaaaaaa  
       2022-05-31 15:58:54 +08:00 via iPhone
    @lozzow
    @treecoder
    @6167
    我也是这个,啥情况啊,我看他这岗位挂了十个多月了都没招到人,不会是骗子吧
    aaaaaaaaa
        20
    aaaaaaaaa  
       2022-05-31 16:05:39 +08:00 via iPhone
    @wxiao333
    @6167
    虽然说是 compass 中国团队,但应该类似外包,签合同是新加坡人力资源公司
    gouflv
        21
    gouflv  
       2022-05-31 16:05:47 +08:00 via iPhone
    代码发出来啊
    treecoder
        22
    treecoder  
       2022-05-31 20:46:16 +08:00   ❤️ 1
    @aaaaaaaaa 实际上应该是 slasify 这个外包公司在给 compass 招人,就是外包,这个岗位年前就在招,只是当时我没有做题,五一过后开始找 remote job 的时候有个 slasify 的哥们联系了我,我才发现这个岗还在招,就做了题,难受,花了好几天呢
    whyzp2019
        23
    whyzp2019  
       2022-06-01 09:37:07 +08:00
    我也觉得只要他不想要,连呼吸都是错的
    iamgodot
        24
    iamgodot  
       2022-06-01 10:25:14 +08:00
    @treecoder @wxiao333 估计对方想看闭包表的实现吧,但其实邻接表 cte 完全可以满足需求了。个人对这种挂羊头卖狗肉的笔试需求有一些反感。为了缩减招聘成本无可厚非,但是很浪费面试者的时间。
    cszchen
        25
    cszchen  
       2022-06-01 10:26:06 +08:00 via iPhone
    楼上好多说楼主不应该怎么怎么样的,如果给了规范楼主没按规范来,那肯定是他的问题。
    但是没看到楼主说给规范,关注的重点难道不是功能实现吗。
    nazhenhuiyi294
        26
    nazhenhuiyi294  
       2022-06-03 06:49:57 +08:00
    compass 我面试过前端,给了一个只有我目前 50% 不到的薪资范围,把我整笑了。
    CarolFxl
        27
    CarolFxl  
       2022-06-18 00:11:31 +08:00
    @nazhenhuiyi294 我也面过他们公司的前端,不过我最后面试没过,不知道他们压薪资这么厉害的?
    CarolFxl
        28
    CarolFxl  
       2022-06-18 00:13:20 +08:00
    我也有一次挂掉的理由居然是:过渡封装!我只是把我能想到的多写了一些,然后抽离了组件,更加丰富了功能,结果告诉我过渡封装了,我真的是服气了,花了我两个晚上写的。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2864 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 36ms · UTC 07:54 · PVG 15:54 · LAX 23:54 · JFK 02:54
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.