V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
fever105
V2EX  ›  程序员

项目组其他成员能力不够,没追求,代码糟心,怎么办?

  •  
  •   fever105 · 2018-05-14 10:34:03 +08:00 · 8862 次点击
    这是一个创建于 2414 天前的主题,其中的信息可能已经有所发展或是发生改变。

    楼主能力一般,但热爱技术,大言不惭的说愿意为自己写的每一行代码终身负责。我始终相信写的漂亮的代码就像诗一样,赏心悦目,充满美感。这个大概是我作为一个完美主义者强迫症的具体表现。

    公司最近发展迅速,不停招人,原本只有几个人的项目组突然之间热闹了起来。人多了,事情也多了。作为一个小 leader,楼主不得不抽出一部分时间进行管理,说是管理,其实就是辅助新人熟悉代码,code review,技术分享。但在这个过程中,我渐渐有一种绝望感,新的基础是真的差,如果把顶级代码比做诗歌,他们的代码也就是小学生作文。一个简单的道理,说了几遍都还是懵懵懂懂,每每看到他们一副“被水淹没,不知所措”的样子,我都深感无奈。

    作为一个强迫症患者,垃圾代码让我心碎。不过话说回来,对于那种基础差,但还有求生欲望的新人,我是愿意教的,只是多教几遍的事情。老油条才是让人恶心。说句要挨喷的话,很多人声称自己 3-5 年工作经验,但说白了不过是拿着 1-2 年的工作经验混了 3-5 年。不求甚解,应付了事,得过且过,2018 年了还用着 5 年前的技术,提醒他还不耐烦。我是真的 xxooxxx....

    有些人可能会说,你是不是不会和别人沟通,要和别人多沟通。我也反思过,也认真的在沟通,但代码水平这问题真的不是沟通就能解决的。

    以上就是我的抱怨,我知道,程序员要转管理,这是必经的阶段,但作为一个眼里揉不得沙子的人,这个过程的确痛苦。最近明显感觉心累,人也暴躁了很多。早上都懒的起床。😂。哎,叹个气,继续重构代码去了。

    66 条回复    2019-03-06 00:08:44 +08:00
    Junn
        1
    Junn  
       2018-05-14 10:54:57 +08:00   ❤️ 1
    人多起来就要有规范了,就得有绩效了。
    话说没有试用期什么的么?不是你面的人?
    ourzhang
        2
    ourzhang  
       2018-05-14 11:01:29 +08:00   ❤️ 7
    code review 的时候直接打回去。一直改,改到你认为满意为止。
    初期可能会慢点,每天开个会常发生的问题讲解。直到你满意为止。
    有难度就降低目标。没有别的办法,在 code review 之前可以用工具检查降低工作量。
    woscaizi
        3
    woscaizi  
       2018-05-14 11:03:01 +08:00   ❤️ 1
    @ourzhang #2 赞同
    lol173
        4
    lol173  
       2018-05-14 11:04:59 +08:00   ❤️ 1
    兄弟,在哪?还招人吗,2 年经验,同样追求优雅的代码,主攻 PHP,前端也还行。
    forestyuan
        5
    forestyuan  
       2018-05-14 11:05:39 +08:00
    关键还是要在招人的过程中有一定的话语权
    ltoddy
        6
    ltoddy  
       2018-05-14 11:06:37 +08:00
    l00t
        7
    l00t  
       2018-05-14 11:12:11 +08:00   ❤️ 1
    是不是钱没给够?钱没给够的时候我也是老油条……
    ycz0926
        8
    ycz0926  
       2018-05-14 11:16:59 +08:00
    哥们,能力不够,态度好就行,态度不行,你就有的受了
    aliasliyu4
        9
    aliasliyu4  
       2018-05-14 11:19:29 +08:00
    @l00t 态度得从一而终
    ik2h
        10
    ik2h  
       2018-05-14 11:19:39 +08:00
    这种心态不改变,只会越做越累,久而久之可能会什么代码都看不进去。
    qwertyssp
        11
    qwertyssp  
       2018-05-14 11:26:39 +08:00 via Android
    如果发现直属领导是这种怎么办
    nosay
        12
    nosay  
       2018-05-14 11:28:21 +08:00   ❤️ 9
    兄弟,你这种情况太常见了。哪个地方没有咸鱼啊,面向工资编程了解一下。
    反下我了解的不少人,对编程不感兴趣,通过培训班入行,工资水平差不多就行了,下了班 coding ?不存在的!
    辞退?这些人能进的公司,一般都属于小型创业公司。在一些老板眼里,他们性价比还是很高的。
    code review ?可以,但是进度还是要加快,加快..
    作为 leader,强大的心脏是少不了的。
    眼里揉不得沙子?看到咸鱼、老油条就来气?抓紧改改吧,公司都能承受的住,你一打工的置啥的气?
    实在受不了抬腿炒了公司走就是...以你现在的情况,那还不是天高任鸟飞吗?

    不说了,工头叫我去搬砖了
    zjsxwc
        13
    zjsxwc  
       2018-05-14 11:33:48 +08:00
    第一,谁招来的谁负责,猪队友谁都不喜欢
    第二,改进架构,把猪队友负责的项目内容抽离出主要代码库(甚至可以单独搞个项目让他们自己玩),保证对方的锅甩不过来。
    oneyian
        14
    oneyian  
       2018-05-14 11:35:24 +08:00
    我也遇到了,代码风格看不下去
    ianva
        15
    ianva  
       2018-05-14 11:35:58 +08:00
    公司是个商业机构,目的是赢利,为了增加产能扩大生产,多装了几台廉价的机器,对公司来说战略上是对的
    换句话说,这就是环境,环境不是给你过上快乐的编码生活准备的
    RorschachZZZ
        16
    RorschachZZZ  
       2018-05-14 12:35:52 +08:00
    强制执行,告诉他们对代码的这些要求是规定而不是规范。
    liprais
        17
    liprais  
       2018-05-14 12:41:00 +08:00   ❤️ 3
    v2ex 有一点做的非常好的地方就是:自己不能删帖
    https://www.v2ex.com/t/284211#reply131
    BigBunny
        18
    BigBunny  
       2018-05-14 15:32:01 +08:00
    轻松自如,如鱼得水
    adablue77
        19
    adablue77  
       2018-05-14 15:54:04 +08:00   ❤️ 1
    估计你在团队里的人际关系很一般是不是😁
    hoholiday
        20
    hoholiday  
       2018-05-14 16:22:17 +08:00 via Android
    @liprais 把你顶上去 哈哈
    codermagefox
        21
    codermagefox  
       2018-05-14 16:25:58 +08:00
    对不起,看到“被水淹没,不知所措”我笑了,惭愧惭愧
    anyele
        22
    anyele  
       2018-05-14 16:34:58 +08:00
    “很多人声称自己 3-5 年工作经验,但说白了不过是拿着 1-2 年的工作经验混了 3-5 年。不求甚解,应付了事,得过且过,2018 年了还用着 5 年前的技术,提醒他还不耐烦。”

    这句话我完全赞同
    3dwelcome
        23
    3dwelcome  
       2018-05-14 16:36:54 +08:00
    人其实是充满主观和偏见的,楼主你认为的好规范好代码,在别人眼里并不那么绝对。你看不惯老油条代码,说不定老油条还看不惯你的代码呢。
    h1367500190
        24
    h1367500190  
       2018-05-14 16:48:50 +08:00
    “很多人声称自己 3-5 年工作经验,但说白了不过是拿着 1-2 年的工作经验混了 3-5 年”这句在 V 站真的要挨喷,戳中很多人的痛点
    h1367500190
        25
    h1367500190  
       2018-05-14 16:57:24 +08:00
    同代码强迫症患者,但只是个小前端,也不是 leader,以前以为是少部分,现在发现很多人是这样的:
    假如他是对的,你提出看似更好的方法,他会乐意和你探讨;
    假如他是错的,你指出其中的错误,他会拼命维护
    M003
        26
    M003  
       2018-05-14 17:00:15 +08:00
    但如果有没有工作经验的,以,3~5 年经验招进来,那才是一种噩梦..........
    kaneg
        27
    kaneg  
       2018-05-14 18:48:30 +08:00 via iPhone
    如果楼主在 team 里有话语权的话可以试试用 SonarQube 来扫描代码来让成员意识到自己代码的不规范
    vitalis
        28
    vitalis  
       2018-05-14 19:08:54 +08:00
    招聘环节出了问题。
    sagaxu
        29
    sagaxu  
       2018-05-14 19:38:05 +08:00 via Android
    很多公司声称自己要招 3-5 年工作经验的人,但说白了不过是拿着 1-2 年的待遇糊弄 3-5 年的人
    ww2000e
        30
    ww2000e  
       2018-05-14 19:45:09 +08:00 via Android
    我的结局是最后我离职得哄着新来的和我交接,他们不说行了,经理不放人
    Jsonz
        31
    Jsonz  
       2018-05-14 21:34:46 +08:00
    同感 哎
    之前还因为这事和老大有点闹脾气
    吐个槽 不会 ajax 的前端
    wr410
        32
    wr410  
       2018-05-14 21:45:18 +08:00
    1、不知道所谓的技术落后是指“已经有 autowired 了你还要去 getbean ”,还是说“已经有 hashmap 了你还要用二维数组”?
    2、是否提供了开发规范?如果没有,只要人家写的代码再烂,那能和期望结果相符,那便没有可指责的地方。
    3、也许人家的代码比你所谓的顶级代码更可行?或者说你有提供顶级代码去和人家交换意见?再或者说有了解一下人家为什么会坚持他的看法?

    所以说白了,还是沟通的方式问题。
    大多做开发的人都喜欢以自己为中心,总觉得人家是错的,所以你觉得他是错的,他也觉得你是错的,就这么简单。
    pexcn
        33
    pexcn  
       2018-05-14 21:50:58 +08:00
    同完美主义患者
    guoyu4126
        34
    guoyu4126  
       2018-05-14 22:22:16 +08:00
    人才需要培养,试用期就是简单培养阶段,
    可以进阶就留, 不可进阶,就别浪费大家时间了
    mafeifan
        35
    mafeifan  
       2018-05-14 22:26:21 +08:00
    敢问当初新人是谁招进来的。
    对于拖慢整个团队的人早点处理得好。
    所以当时面试的时候多考察解决问题的能力。
    CFO
        36
    CFO  
       2018-05-14 22:29:08 +08:00 via Android
    有 review 已经很幸运了 不合团队规范的让他一直改 改到满意为止
    技术面不广的鼓励去学 不愿意学 甚至学不会的直接开掉吧 对双方都好
    Mingsaul
        37
    Mingsaul  
       2018-05-14 23:47:21 +08:00 via Android
    楼主要不要实习生?-自学编程的那种
    18076362432
        38
    18076362432  
       2018-05-14 23:51:50 +08:00 via iPhone
    楼主是广州的吗?有 iOS 开发岗位吗?
    yhzwy
        39
    yhzwy  
       2018-05-15 00:13:08 +08:00
    五年前的技术指得是什么?
    soothsayer
        40
    soothsayer  
       2018-05-15 00:55:45 +08:00
    我很庆幸之前遇到那么多优秀的同事,潜移默化的影响了我对代码的追求和看法, 只能说严格控制招聘真的很重要。
    ZSeptember
        41
    ZSeptember  
       2018-05-15 01:28:02 +08:00 via Android
    同样的问题。看他们的代码都想骂人。。
    lsmgeb89
        42
    lsmgeb89  
       2018-05-15 01:39:19 +08:00
    leader 还不好办?

    这种直接开掉,招一批靠谱的不就完了?
    orafy
        43
    orafy  
       2018-05-15 01:39:52 +08:00 via iPhone
    有个问题,这种优秀的员工留不住,都去往更好的平台了。
    hhacker
        44
    hhacker  
       2018-05-15 09:02:13 +08:00
    code review 的时候打回去最直接。。。。
    lepig
        45
    lepig  
       2018-05-15 09:04:23 +08:00
    楼主 刚入门的时候 就开始写诗了吗
    jorneyr
        46
    jorneyr  
       2018-05-15 09:31:10 +08:00
    看清,看轻
    Shynoob
        47
    Shynoob  
       2018-05-15 10:00:28 +08:00
    2018 年了还用着 5 年前的技术

    这句话有歧义吧
    tbag781623489
        48
    tbag781623489  
       2018-05-15 10:02:10 +08:00 via iPhone
    @liprais 同英砖,学的 py 和 js 因为动力是面向 av 编程
    vimutt
        49
    vimutt  
       2018-05-15 10:02:53 +08:00
    是领导的话就开除他们,不是领导的话有两个选择,一、弄死他们 二、做好自己的工作,不去苛责他们。
    话说,他们还有没娶媳妇没对象的 你要是能一并解决了 ,代码风格不是问题。
    ycz0926
        50
    ycz0926  
       2018-05-15 10:03:54 +08:00
    木桶理论了解下
    k5716170
        51
    k5716170  
       2018-05-15 10:16:16 +08:00
    实话实说 我周围 也都是这样的人 有时候真的感觉他们选错了行业
    andrewDDC
        52
    andrewDDC  
       2018-05-15 10:37:45 +08:00
    人员流动,总能遇到合适的人
    chenyu8674
        53
    chenyu8674  
       2018-05-15 10:41:26 +08:00
    优雅的代码和团队协作本身就是互斥的
    错的不是 LZ,而是这行的大环境
    同为代码洁癖症患者常常有辞职做独立开发的念头
    ryuzaki113
        54
    ryuzaki113  
       2018-05-15 10:43:47 +08:00
    菜还不让人说,不如直接踹了
    ihaveadrame
        55
    ihaveadrame  
       2018-05-15 11:04:20 +08:00
    @ourzhang 我们现在就是这么做的
    Kraken
        56
    Kraken  
       2018-05-15 11:19:44 +08:00
    @ourzhang 请问如何做 codereview 我们现在每周两次 codereview 是集体开会的那种 极其耗时间,一开会我就想睡觉,请问你们是如何做 codereview 的 ? 我们现在用 git 管理 每个功能一个分支 感觉贼鸡儿乱
    evlos
        57
    evlos  
       2018-05-15 11:23:36 +08:00 via iPhone
    每个月的精神损失费没给够呀
    loy6491
        58
    loy6491  
       2018-05-15 11:36:01 +08:00
    @h1367500190 这句怎么会被喷,这是共识吧?
    bookit
        59
    bookit  
       2018-05-15 12:25:27 +08:00
    给他们加薪,例如每个月先加一万
    3dwelcome
        60
    3dwelcome  
       2018-05-15 13:20:57 +08:00
    @bookit 可是对于本身就对代码没兴趣的人,给再多钱也没用啊。我同事是码农,但对写代码完全无爱,家里电脑都没有的那种,有也只炒炒股,不写程序,给再多钱也没啥用。
    vivili
        61
    vivili  
       2018-05-15 13:33:03 +08:00
    有一种叫做,你拿着别人双份的薪水 要求别人和你做一样的事
    wolfie
        62
    wolfie  
       2018-05-15 13:57:50 +08:00
    看自己过去一年以前的代码都会感觉到有差的地方吧
    tinyuu
        63
    tinyuu  
       2018-05-15 14:22:26 +08:00
    那就都辞掉 ,都找 会写诗的, 不会徒手写的都不要。
    ourzhang
        64
    ourzhang  
       2018-05-15 17:05:54 +08:00
    @Kraken 是专门的人在做这个,程序写完之后先过 checkstyle,规则是自己公司定义的。
    上传之后要过工具检查。全都过了之后才是 code review。做这个的人权限很大,不符合开发规约,写法问题,开发风格不统一,就连多一个空格都会上 Redmine,你想在项目中添加一个包都要提申请的。。

    那时候很痛苦,前期很慢,对应 Redmine 的问题基本都是加班修正,后期慢慢适应了开发效率也没差别了。现在想起来对自己开发的帮助还是很大的,至今也保留下来了很多好习惯,那个项目做完之后维护起来很简单,注释很多很全,风格都是一样的,定位问题之后基本不用找就能知道去哪个文件哪个方法修改。

    作为一个程序员我是不太喜欢这种开发的,虽然后期维护的时候很幸福。。。
    Kraken
        65
    Kraken  
       2018-05-15 23:03:15 +08:00
    @ourzhang 懂了 谢谢大佬 光是听起来就觉得很幸福啊
    maninfog
        66
    maninfog  
       2019-03-06 00:08:44 +08:00 via iPhone
    @liprais 感觉没啥毛病啊,楼主这贴也说了能力一般,但是对代码很注重
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1091 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 27ms · UTC 23:33 · PVG 07:33 · LAX 15:33 · JFK 18:33
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.