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

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

  •  1
     
  •   6167 · 76 天前 · 3322 次点击
    这是一个创建于 76 天前的主题,其中的信息可能已经有所发展或是发生改变。
    机试题目是用 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  
       76 天前   ❤️ 1
    几个指定的功能? 有没有可能他们给每个应聘者都提了几个特定的功能.........
    leo108
        2
    leo108  
       76 天前   ❤️ 1
    1. 在有版本控制的情况下不是很懂在文件头添加作者信息有什么用
    2. 注释掉跑不通的代码,中间状态的代码在提交之前应该删掉,没有人会愿意去读一个半成品的代码

    如果楼主工作年限不长的话这些确实都不是什么大问题,只能说这家公司的筛选条件有点诡异
    sky96111
        3
    sky96111  
       76 天前 via Android
    有没有一种可能,公司在白嫖这些应聘的
    swulling
        4
    swulling  
       76 天前 via iPad
    多个条件换行是好习惯

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

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

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

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

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

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