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

请教团队合作的正确姿势

  •  
  •   test005 · 2022-04-27 23:32:34 +08:00 · 2724 次点击
    这是一个创建于 941 天前的主题,其中的信息可能已经有所发展或是发生改变。

    楼主小前端一名,带着组里 6 个人,无带队经验,现在遇到一些问题很困扰,希望能够得到帮助。

    1. 我在项目里配了 eslintgithook ,但老是有人不守规则提交代码的时候跳过验证,每次我 review 的时候,满屏幕黄色波浪线,我提过好几次了,一点回应都没有,每次都是笑呵呵就过去了。

    2. 部分人比较躺平,代码能跑就行,注释、文档几乎没有,写的代码跟实习生写的 demo 一样,为此我有时候忍不住怼过几次,但是效果不明显,更多的引起了对方的反感,开始暗暗跟我对着来。

    3. 我在团队里的态度非常低,除了遇到不合理的地方会怼几句,平时都是笑呵呵的 X 哥 X 哥的叫着。虽然我怼 X 哥怼的比较多,但是 HR 、上级、老板那边问起情况来我也是一直夸,我不觉得我对不起 X 哥,但他就是跟我对着来。

    4. 我和 X 哥之前关系其实挺好的,他是公司知名的加班劳模(虽然代码烂),但是后边团队里来了个妹子,X 哥为了追对方,自己的活不上心了,天天帮妹子改 bug 。开始的时候我也是喜闻乐见的,不会给他排太多的活,但是后边发现有点脱离掌控了。两个人干一个人的活,代码写的又敷衍,安排任务我还得把他俩安排在一起,怼的多了 X 哥也不高兴,现在问题爆发了。

    咋整呢?


    会不会有人说我是资本家的汪?应该一起躺平? 我只是一个热爱技术的小开发,不喜欢别人在我负责的项目里拉屎,其他方面我也是得过且过,你好我好大家好。

    第 1 条附言  ·  2022-05-14 10:42:13 +08:00
    感谢大家出谋划策,给大伙同步一下最新状况。

    近期刚好是 Q1 季度的绩效总结,我趁着这个机会重新提了一遍开发规范,Git 规范也相应做了一些调整,需要发起 MR 并通过 Review 才能够合并,并且明确的表示这些行为将会影响到绩效考核。

    同时约定了绩效标准参考项,Q1 暂时先给大伙都评个优秀,Q2 将会按照约定的行为标准去打分。

    目前正在向好的方向发展,感谢大伙,么么哒💋💋💋
    16 条回复    2022-04-28 17:21:43 +08:00
    yaphets666
        1
    yaphets666  
       2022-04-27 23:44:35 +08:00   ❤️ 2
    让 X 哥滚蛋啊,就算是他妈社会主义,共铲主义,无产阶级干死资本家,也得把自己活干好呀。
    ecloud
        2
    ecloud  
       2022-04-27 23:48:11 +08:00   ❤️ 1
    没有人事权的 lead ?那是你公司管理制度有严重问题。该开掉的开掉让他赶紧滚蛋
    ncepuzs
        3
    ncepuzs  
       2022-04-28 00:17:22 +08:00   ❤️ 1
    从上学起,我一直不喜欢 X 哥这类人。
    表面上看起来非常努力,平时也笑呵呵,你骂他也是笑呵呵,但是干的活就是一坨屎。
    似乎他们分不清工作关系和私人关系,团队里面这种人多了,好不了。

    对此我提不出建议,因为我都是尽量避免接触,但我觉得你至少应该如实跟上级说明。
    Pipecraft
        4
    Pipecraft  
       2022-04-28 00:29:52 +08:00   ❤️ 4
    1. CI 执行 eslint ,检查不通过的直接报错。报错的不 review 代码,禁止 merge 。
    2. 定期团队公开 review 代码,随机挑几个,大家一起看代码,坏的一起怂,好的一起学。

    开除 X 哥的话,太便宜他了,给他 N+1 吗?
    分给他喜欢的妹子两倍的活,让他俩一起干,干不完一起怂。
    X 哥的奖金一分别给,分给其他人。(前提是你有分配的权利)
    teem
        5
    teem  
       2022-04-28 00:37:41 +08:00   ❤️ 1
    「对事不对人」,搞清楚这个管理就正确了一半。
    SteveWoo
        6
    SteveWoo  
       2022-04-28 01:18:46 +08:00   ❤️ 1
    一要人事权 二要绩效权 三不要谈感情,你成为 leader 那天注定了大家跟你有隔阂
    AS4694lAS4808
        7
    AS4694lAS4808  
       2022-04-28 08:51:20 +08:00 via Android
    感觉“带着组里的”这个表述,通常是简单的临时项目组,甚至有可能 lead 和组员平级,一般这种情况有人摆烂且不听劝是无解的,自行拥有人事权做开除之类的决定不可能的。如果公司有强制代码规范,可以找领导压一压,然后互相 review 改对方的代码,不合格不 merge ,搞一搞对立。。如果没有,就请客吃饭好言相劝,无效的话也只能自己躺平了,大概率领导会觉得是 lead 事多。。
    sjhspp
        8
    sjhspp  
       2022-04-28 09:29:03 +08:00
    公司有没有正式任命你带队,或者说你有没有实权在手,如果有实权,可以先忍忍,等年终评价的时候你的作用就体现出来了,现在不急,等着一起秋后算账,如果没实权或者公司连个年终评价这种奖惩制度都没有,要么你就自己躺平跟着混个开心就行,要是求上进就跑,还有当你带队的那天起就不要想着跟大家谈感情,阶级决定大家必然跟你有隔阂
    justin2018
        9
    justin2018  
       2022-04-28 10:00:05 +08:00
    遇到这样的 代码能跑 产品能按时上线就行了 谁出问题找谁弄

    还能咋办~

    自己要管人 还要写代码 还要 review

    话说 当 leader 后工资涨了多少? 如果一点点 就躺平吧~
    xiaojie668329
        10
    xiaojie668329  
       2022-04-28 10:12:11 +08:00 via iPhone
    这不让 X 滚蛋。代码写得烂,态度不好,还喜欢瞎加班。
    zen1
        11
    zen1  
       2022-04-28 10:13:19 +08:00
    喜欢加班的一律滚蛋,破坏氛围
    janus77
        12
    janus77  
       2022-04-28 11:03:16 +08:00
    看起来你没有权力,你仔细想想你有义务吗?比如项目出问题你是第一负责人这种。如果你有义务,那这就是个大点的牛马,去找管理层要权力,要不到就跟着躺平;如果你没有义务,那你就是跟他们平级的人,别说什么带队的话了,直接躺平。
    如果你有权力,动用就行了
    duan602728596
        13
    duan602728596  
       2022-04-28 13:48:32 +08:00
    master 分支不允许 push ,只允许提 pr 。pr 先跑一堆 lint 和 hook ,不过不允许 merge 。不 code review 不允许 merge 。
    libook
        14
    libook  
       2022-04-28 14:26:19 +08:00
    管理第一要素是有实权,否则无法做管理。

    权责对等,有权利管理就得承担责任,有权利不被管理也得承担责任。
    ericgui
        15
    ericgui  
       2022-04-28 14:48:27 +08:00
    @libook 是,你定了规矩,破坏了,没有后果,那这个规矩有什么意义?你能够有权干掉不守规矩的才行,其他人才听你的。
    HAYWAEL
        16
    HAYWAEL  
       2022-04-28 17:21:43 +08:00
    不行开了就是。又没谁是不可替代的,丢给人事和上级 。如果你 感觉很多项目还要靠着他 要尽早做打算
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   1211 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 24ms · UTC 18:27 · PVG 02:27 · LAX 10:27 · JFK 13:27
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.