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

如果你作为 leader,怎么委婉的跟下属说他的代码有一些小的毛病?

  •  3
     
  •   xiaojunjor · 2017-12-12 12:43:46 +08:00 · 10359 次点击
    这是一个创建于 2289 天前的主题,其中的信息可能已经有所发展或是发生改变。

    某一个模块,业务是实现了的,但是有的地方不是很完美

    比如效率低,但是因为模块涉及到的数据量很小,所以不仔细看代码的话,发现不了

    比如重复定义变量,方法变量名不规范等等

    就是这些我感觉,跟“工作能力”没有太大的关系,而是跟“工作是否用心”有关系的事,怎么去讲,能让对方比较好接受一些,并且知道以后类似的这些东西需要“用点心”去写,而不是单纯的“实现业务”就可以了?

    109 条回复    2017-12-13 14:55:00 +08:00
    1  2  
    T110E5
        1
    T110E5  
       2017-12-12 12:48:55 +08:00 via Android
    大家一起搞个 review
    liangdu
        2
    liangdu  
       2017-12-12 12:49:27 +08:00   ❤️ 20
    老大,我知道了,下次会注意写好代码的。下周五我想请假去面试。
    doublleft
        3
    doublleft  
       2017-12-12 12:57:10 +08:00   ❤️ 1
    委婉?不存在的,我一般都会说这里有颗鼻屎你擦一下
    kiwi95
        4
    kiwi95  
       2017-12-12 13:03:18 +08:00   ❤️ 1
    你是老大,又不是他是老大。。。老大都还这么谨慎的吗
    tmac
        5
    tmac  
       2017-12-12 13:04:46 +08:00 via Android
    每天下午下班前固定 code review 半小时。这些都是对大家有帮助的。直说也无妨。
    maemual
        6
    maemual  
       2017-12-12 13:04:57 +08:00
    code review 的时候会明确说
    zjuster
        7
    zjuster  
       2017-12-12 13:07:38 +08:00
    你是他老大,不需要委婉...
    Kmzl
        8
    Kmzl  
       2017-12-12 13:08:39 +08:00 via Android
    我希望我老大直说
    SourceMan
        9
    SourceMan  
       2017-12-12 13:09:49 +08:00
    不说他怎么进步??
    notreami
        10
    notreami  
       2017-12-12 13:10:23 +08:00
    直接吐槽,不服来辩。。
    php01
        11
    php01  
       2017-12-12 13:11:26 +08:00   ❤️ 2
    让他解释下这段代码的逻辑,从他所说的逻辑中提问,比如问“这段逻辑中会不会出现某种情况,如果出现了某种情况的话怎么处理”
    Phariel
        12
    Phariel  
       2017-12-12 13:23:16 +08:00
    有什么问题直说无妨 大家都是成年人
    discrete
        13
    discrete  
       2017-12-12 13:26:35 +08:00
    项目里用 Git 的话用 Codacy 一类的自动检查代码质量,「重复定义变量」这种简单的问题还是能查出来的。这样就不用你明说了。
    sunny352787
        14
    sunny352787  
       2017-12-12 13:35:23 +08:00
    委婉?干嘛要委婉?做的不好直说啊,少给他发工资的时候他会不会找你啊? review 的时候直接说你这里写的不对,我这边的小崽子们我要求换行都要按我说的来,否则不允许上传,“委婉”这玩意太影响效率了。
    Gonejack
        15
    Gonejack  
       2017-12-12 13:35:47 +08:00 via iPhone
    要有很明确的点,不要简单说这里不好却又没有好的理由。

    要尝试做正确示范,不然很容易站着说话不腰疼或者没留意到别人踩的坑。
    Lothar
        16
    Lothar  
       2017-12-12 13:38:25 +08:00   ❤️ 1
    用 Github / Gitlab 之类的话,直接在合分支的 PR / MR 里加 Code Review 评论嘛~ 没啥需要委婉的,程序员之间代码上的事情,直接开喷就行。
    qq976739120
        17
    qq976739120  
       2017-12-12 13:41:08 +08:00   ❤️ 2
    有你这样的老大真幸福
    Anshi
        18
    Anshi  
       2017-12-12 13:55:55 +08:00
    我希望我老大直说;(有你这样的老大真幸福,还招人吗)
    akagi
        19
    akagi  
       2017-12-12 13:57:11 +08:00
    编译级别错误 > 自动化检查 > Code Review > 正面一对一

    变量名不规范通常是经验和能力范畴,我自己也还有点儿这个毛病。重点是看到本质原因,并且给出解决方案;如果做不到,对事不对人、语气轻一些,一般也是会感谢指出的人的。
    Anshi
        20
    Anshi  
       2017-12-12 13:59:20 +08:00
    比如效率低,但是因为模块涉及到的数据量很小,所以不仔细看代码的话,发现不了

    比如重复定义变量,方法变量名不规范等等
    ————

    以上这些是我初入职场完全不在意的问题,我觉得就我自己写,自己看,就自己知道就好了。虽然我知道这么写不对,但是懒得改了。这种心态。会这么干估计我不不爱写这份代码,这份业务。

    ...后来用了相关规范工具,类似的错误会标黄色,或者红色,波浪线等等...我受不了。。就改了。。。
    zhouquanbest
        21
    zhouquanbest  
       2017-12-12 14:00:17 +08:00 via Android   ❤️ 5
    我一般很直白,直接拉过来问:
    傻屌,你这写的什么玩意
    openSUSE
        22
    openSUSE  
       2017-12-12 14:00:47 +08:00
    搞事的话,#1 @T110E5 的做法可取。
    直接说,别绕弯子,绕弯子容易产生误解。

    Q:“如效率低,但是因为模块涉及到的数据量很小,所以不仔细看代码的话,发现不了”
    A:直接说要改,效率要提高 balabala ……

    Q:“比如重复定义变量,方法变量名不规范等等”
    A:让他们把开发规范文档再细看一遍。

    Q:“就是这些我感觉,跟“工作能力”没有太大的关系,而是跟“工作是否用心”有关系的事,怎么去讲,能让对方比较好接受一些,并且知道以后类似的这些东西需要“用点心”去写,而不是单纯的“实现业务”就可以了?”
    A:方便告诉我贵司的地址吗?你的团队缺人不,我能加入吗?我能帮你传达心声。
    Catyuki
        23
    Catyuki  
       2017-12-12 14:00:48 +08:00
    ....好吧 或许我改检讨..每次低级问题都压不住火气直接喷了 如果是重大问题反而耐心去沟通解决
    sbw
        24
    sbw  
       2017-12-12 14:01:08 +08:00
    review 里直接回复
    很多开源项目贡献代码的时候对格式、命名还有 commit msg 都有很高要求的
    经常因为一个空格,一个大小写就扣分
    iFlicker
        25
    iFlicker  
       2017-12-12 14:04:49 +08:00
    为什么不直接说呢。。
    Felldeadbird
        26
    Felldeadbird  
       2017-12-12 14:15:01 +08:00   ❤️ 1
    有什么好委婉的。直接说出来:
    小李啊,你这段代码写得不够优雅啊。你这样的话,没有抽象出来,后面维护的人会难写接口的。你改一下。
    fasling
        27
    fasling  
       2017-12-12 14:17:17 +08:00
    代码上的事情直接说,没什么委婉不委婉的。
    zhoucan007
        28
    zhoucan007  
       2017-12-12 14:18:46 +08:00
    之前当过老师,有一个活动叫评课,跟程序员的 Code Review 很像啦。我们评价一个人是有套路的,如果有什么缺陷,会这么说:XX 的这个做的非常的好,那个做的也非常的好,如果在 XXXXXXX 方面稍微加强一下,那就太完美了。

    自己体会。
    forestyuan
        29
    forestyuan  
       2017-12-12 14:20:13 +08:00
    “比如效率低,但是因为模块涉及到的数据量很小,所以不仔细看代码的话,发现不了”

    既然数据量很小,效率低点怕什么?貌似楼主有点吹毛求疵了
    linuxchild
        30
    linuxchild  
       2017-12-12 14:26:30 +08:00
    直接说就好,是我我不会介意的
    chuhemiao
        31
    chuhemiao  
       2017-12-12 14:30:11 +08:00
    有这样的老大何愁不进步,还招人吗
    hitmanx
        32
    hitmanx  
       2017-12-12 14:30:50 +08:00
    增加个 code review 的环节,有啥问题大家就直说咯,对事不对人。其实团队的代码要求提高了,对于团队中的每个人来说都是有帮助的。
    algery
        33
    algery  
       2017-12-12 14:32:17 +08:00
    我们是直接怼的 哈哈哈
    DiverRD
        34
    DiverRD  
       2017-12-12 14:37:38 +08:00
    我多希望我们老大会跟我说,你这块代码不够好 ,应该这样写。balabala.......
    nekoyaki
        35
    nekoyaki  
       2017-12-12 14:41:49 +08:00
    说一个情况,我不知道贵司有没有绩效考核或者每天汇报之类的,如果有的话,可能也会阻止员工对代码的追求。
    像我在小公司的时候,每天什么时候发现代码里有不合理的地方都会试图进行优化甚至重构,但在大公司这个自我改良动力就明显减弱了,因为其中许多都没法算到我的绩效里,也影响每天的汇报。
    xomix
        36
    xomix  
       2017-12-12 14:42:52 +08:00   ❤️ 3
    某一个模块,业务是实现了的,但是有的地方不是很完美
    模块需求文档里面怎么写的?有什么标准为完美吗?你一开始不说的话都指望自觉又希望大家越快越好,大家把握不住这个度出现什么不完美都正常。

    比如效率低,但是因为模块涉及到的数据量很小,所以不仔细看代码的话,发现不了
    没有效率需求,又是小数据量,低效率快速开发是一种解决方案。

    比如重复定义变量,方法变量名不规范等等
    这是一个编程习惯,你需要从需求上就说清楚。

    就是这些我感觉,跟“工作能力”没有太大的关系,而是跟“工作是否用心”有关系的事,怎么去讲,能让对方比较好接受一些,并且知道以后类似的这些东西需要“用点心”去写,而不是单纯的“实现业务”就可以了?
    那你布置业务的时候怎么不会“用点心”的布置下去,让同事更清楚你的需求。

    一到公司给一个模块让你随便写,写完了再这儿那儿的毛病一挑,这是立威吗?
    wangdu2012
        37
    wangdu2012  
       2017-12-12 14:44:43 +08:00 via iPhone
    不需要委婉直接说
    wampyl
        38
    wampyl  
       2017-12-12 14:44:47 +08:00
    我想要个这个的老大.jpg 。毕业刚加入的公司老大没啥经验,他都不重视这个,很想在自己的团队加入 code review。
    saulshao
        39
    saulshao  
       2017-12-12 14:56:59 +08:00
    我觉得首先要表扬人家做的好的方面,然后再指出不足,如果只是小的问题,其实作为 leader,大可不管。
    如果 leader 觉得严重影响了产品 /代码质量,那就不是小的问题
    Khlieb
        40
    Khlieb  
       2017-12-12 14:59:59 +08:00 via Android
    大家一块 debug
    zenxds
        41
    zenxds  
       2017-12-12 15:17:46 +08:00
    code review 直接说
    Moorj
        42
    Moorj  
       2017-12-12 15:33:17 +08:00
    不需要委婉+1

    我之前给我的女售后和女运营讲解一些要点的时候,得跪着说,不然她总以为我找茬,是故意欺负她

    “是是是,你说的对,你是领导,都听你的”

    就很气
    sm0king
        43
    sm0king  
       2017-12-12 15:45:41 +08:00
    有什么委婉的,直接说呀
    mengzhuo
        44
    mengzhuo  
       2017-12-12 15:46:48 +08:00
    直接说,或者 review 的时候说
    Mikey
        45
    Mikey  
       2017-12-12 15:53:35 +08:00
    委婉?那还是 leader ?
    magic3584
        46
    magic3584  
       2017-12-12 16:03:30 +08:00
    我的主管会直接给我指明,我觉得挺好,能学到东西
    wingoo
        47
    wingoo  
       2017-12-12 16:06:01 +08:00
    都是直接说的
    一般会在 merge 的时候看下
    yongzhong
        48
    yongzhong  
       2017-12-12 16:31:24 +08:00
    @xomix 赞同你的看法
    loryyang
        49
    loryyang  
       2017-12-12 16:32:23 +08:00
    看到直说,对事不对人呀。你既然是 leader,本身就有这个权力和义务来协助提升小弟的能力
    你如果不说,就是你的失职了
    fengwei23
        50
    fengwei23  
       2017-12-12 16:36:52 +08:00
    直说比较好,或者进行 code review,我是希望老大说出来的
    chztv
        51
    chztv  
       2017-12-12 16:41:07 +08:00
    作为 leader,不需要委婉,这是你的义务。
    uleh
        52
    uleh  
       2017-12-12 16:42:41 +08:00
    这还需要“委婉”么。。
    熟一点的一般会说:这什么玩意儿,重写!
    不太熟的会说:这里有 xx 问题,你改下
    hustlike
        53
    hustlike  
       2017-12-12 16:55:20 +08:00
    pull request 可以加 comments
    13036101641
        54
    13036101641  
       2017-12-12 16:55:49 +08:00
    有可能你不说,他不知道,现在说了,以后他做 TL 了,恰好遇见一个这样子的下属,他也会再去教下去
    GuangTsang
        55
    GuangTsang  
       2017-12-12 17:04:41 +08:00
    带着你的解决方案去和他谈,用事实证明你的观点,你的方案比他的更有效率
    sunsmooth
        56
    sunsmooth  
       2017-12-12 17:13:52 +08:00
    要让他能感觉到你是在帮他,而不是故意在找茬。
    codermagefox
        57
    codermagefox  
       2017-12-12 17:17:21 +08:00
    哇,楼主,我巴不得被带我的大佬喷.
    被喷,说明有进步的空间,说明代码确实烂.
    只要不涉及父母,人身攻击都没问题啊(没错我就是抖 M
    事实上被喷了以后确实长记性快.
    nickel123666
        58
    nickel123666  
       2017-12-12 17:24:20 +08:00
    居然还要委婉,我的 leader 在指点我代码的时候都是说“你也好意思写这种代码哦”;
    “这里!怎么办吧,要不要改?”
    “为什么这里 xxxx ”;
    哇,感觉被骂得好爽。。
    xxbutoo
        59
    xxbutoo  
       2017-12-12 17:27:13 +08:00
    真为了你的小弟好 直接喷他
    最好周围没人的时候
    sensui7
        60
    sensui7  
       2017-12-12 17:29:42 +08:00
    直接讲哪里可以有问题
    Damon4V
        61
    Damon4V  
       2017-12-12 17:32:31 +08:00
    还招人吗?遇到这样的老大真得是幸福啊
    wupher
        62
    wupher  
       2017-12-12 17:43:31 +08:00
    不懂,我一般就是直说,也建议直说。

    code style 的话,最好设立规范,然后用自动代码检查。

    如果是算法不合理,没考虑到当数据很大会有较大耗时或性能消耗。这就是与工作能力相关了。一般来说,能提供职业技术能力的建议,程序员一般还是会乐于接受的。(来不来得及马上改可能存在排期问题)

    如果确实是态度不好,那应该反省一下招聘了。招聘和试用阶段为什么没有发现这种情况。

    如果是 N 年老员工,可能就要看看彼此利益干系,实在不行就只能曲线处理,关键任务不交给他。
    cosmosz
        63
    cosmosz  
       2017-12-12 17:44:23 +08:00
    把 coding standard 弄出来,按照 coding standard 改变量名。 对效率的要求在 non-functional requirement 中提出来。 如果这些都有,但还是“没用心”,就直接当面说;如果这些没有,就是团队管理的问题。
    Katherina
        64
    Katherina  
       2017-12-12 17:47:47 +08:00
    希望老大直接说,只要说的中肯有帮助

    命名规则、代码规范什么的最好有一个文档的吧~如果哪里逻辑复杂又没加注释、或者命名让人根本看不出这个是干嘛的,后面的人维护起来也很头疼,肯定要点出来的。而且这都属于习惯的问题,没什么接受不了

    如果是具体到逻辑、效率上的,我还要感谢老大帮我上了一课好嘛~
    当然有些模块划分或者功能实现上我和老大有点各持己见,偶尔会有争论,其实最后还是学习到了
    dozeboy
        65
    dozeboy  
       2017-12-12 17:49:20 +08:00
    如果你是我老大,我希望能直接跟我说,没必要顾及太多,虽然有可能尴尬,但我更希望得到提高
    Takahashi
        66
    Takahashi  
       2017-12-12 17:49:34 +08:00
    想要有这样的老大
    vagranth
        67
    vagranth  
       2017-12-12 17:51:23 +08:00
    别人我不知道,反正如果人家指出我的代码里有问题,我一般都是先听明白,再确认。
    如果真的有问题我还感觉蛮高兴的,因为这样下次就不会再犯了。
    yuhr123
        68
    yuhr123  
       2017-12-12 18:04:51 +08:00
    私底下明说就是了,目的是为了让程序更好,如果连优化建议都听不进去这人心胸就太小了。太委婉反倒耽误事。
    fhefh
        69
    fhefh  
       2017-12-12 18:23:10 +08:00
    我希望老大直接跟我说 这样我自己也会提升
    fhefh
        70
    fhefh  
       2017-12-12 18:24:12 +08:00
    @Anshi 相关规范工具是哪些 求个列表~ 目前用的是 IDE 自带的
    akagi
        71
    akagi  
       2017-12-12 18:39:24 +08:00
    @fhefh 路过,这个貌似要看写啥,C++有 cppcheck, js 有 eslint, 用 lint 做关键字一般可以搜到些~
    fhefh
        72
    fhefh  
       2017-12-12 18:53:23 +08:00
    @akagi 3Q 现在在用在 嘿嘿~
    m999
        73
    m999  
       2017-12-12 19:26:53 +08:00 via Android
    下属身体很壮么
    vdvvdd
        74
    vdvvdd  
       2017-12-12 19:33:26 +08:00
    这还需要委婉
    forkon
        75
    forkon  
       2017-12-12 19:44:09 +08:00
    楼主怎么定义委婉……有内情?
    scratbai
        76
    scratbai  
       2017-12-12 19:44:37 +08:00
    只是个代码问题,我跟我老板说,都不用特别注意的。
    srx1982
        77
    srx1982  
       2017-12-12 19:57:15 +08:00
    为什么要委婉,直接说啊
    sampeng
        78
    sampeng  
       2017-12-12 20:13:06 +08:00
    亲,你这坨代码写的真好,就是有那么一丢丢的问题。
    楼主是要这样么。。。
    毛线啊,leader 的职责就是支出代码问题啊。。
    sampeng
        79
    sampeng  
       2017-12-12 20:17:55 +08:00
    其实吧,都说直接说好。但是真轮到自己头上指不定怎么炸毛呢。。
    lz 这个话题,我也经常碰到来着,但是我看着不爽还不让我提,你来当 leader 啊。。。
    cchange
        80
    cchange  
       2017-12-12 20:35:44 +08:00 via iPhone
    记得 SVN 的时候 貌似有个 SVN blamed
    不过我没用过 SVN 所以……
    wdlth
        81
    wdlth  
       2017-12-12 21:35:47 +08:00
    你们不规定编码规范,不用静态分析么?
    l00t
        82
    l00t  
       2017-12-12 22:04:50 +08:00
    有需求,或者有要求,你提前说。写完再唧唧歪歪的还没个实锤,就算是领导,底下人也未必服的。编程上很多问题不过是个意见分歧和个人偏好而已。当然如果你三番五次强调过了对方还是再犯,那确实是工作态度有问题。
    RaymondYip
        83
    RaymondYip  
       2017-12-12 23:43:19 +08:00
    之前说过几次指出来都以为我在骂人,,后面不说了 直接自己看到顺手就改掉舒服多了,就是自己累点
    romisanic
        84
    romisanic  
       2017-12-13 00:30:58 +08:00
    为什么要委婉?
    没有由头的话,现在设立一个规则或者要求,让大家遵循,说明的时候就拿这部分代码举例子。
    有问题说出来才好进步。
    说了不改,那就该换人了。。。
    takato
        85
    takato  
       2017-12-13 00:45:28 +08:00
    完全可以找一份和他问题相似的代码,找个非正规场合单独给他看看,不要指出问题。这时候如果对方够聪明,应该能够自己发现一些问题。
    如果对方没意识到再考虑评审时候说吧。

    个人意见
    jsfaint
        86
    jsfaint  
       2017-12-13 07:38:21 +08:00 via Android
    大家都在说楼主委婉的问题,我换个思路
    如果代码风格问题,用 lint 工具扫了让他改或者参考团队代码风格
    如果是性能问题,不上 profiling 就说有性能问题,这样不严谨啊
    missdeer
        87
    missdeer  
       2017-12-13 08:37:01 +08:00
    你是老大你说了算,委婉干什么

    一劳永逸的方法是制订编码规范,落实 code review
    Adamla
        88
    Adamla  
       2017-12-13 09:24:25 +08:00
    @T110E5 E5 大哥好
    ii4Rookie
        89
    ii4Rookie  
       2017-12-13 09:52:14 +08:00
    @zhouquanbest 哥,不是你教我这样写的吗?
    zjddp
        90
    zjddp  
       2017-12-13 10:08:34 +08:00
    渴望有这样的老大
    zhangfeiwudi
        91
    zhangfeiwudi  
       2017-12-13 10:23:30 +08:00
    直接就说了,不用委婉啊
    8355
        92
    8355  
       2017-12-13 10:25:36 +08:00
    我觉得先看看是能力不够还是能力够随意乱写
    如果能力不够虚心接受那你多教教
    诚心乱写就骂人啊..
    skylancer
        93
    skylancer  
       2017-12-13 10:26:41 +08:00
    组织一场 code review 不是最好的办法么..?
    scnace
        94
    scnace  
       2017-12-13 10:30:40 +08:00 via Android
    那么反过来呢 要怎么委婉地说(
    hundred
        95
    hundred  
       2017-12-13 10:36:38 +08:00
    有什么好委婉的,发工资的时候委婉发了 8 折么
    myself659
        96
    myself659  
       2017-12-13 10:46:41 +08:00
    代码规范有的话,直接上规范
    没有的话通过代码 review 论对错与优劣
    T110E5
        97
    T110E5  
       2017-12-13 11:03:10 +08:00
    @Adamla 是 T29 小兄弟?还是 M103 呀,,,,2333
    meathill
        98
    meathill  
       2017-12-13 11:08:04 +08:00
    不用委婉,直说即可。

    代码规范用 lint 工具,必须通过才能提交。

    定期组织 Code Review,新人的代码必须 Code Review 才能合并到 master。
    kkzxak47
        99
    kkzxak47  
       2017-12-13 11:10:23 +08:00 via Android
    直接叫过来指出来啊
    m939594960
        100
    m939594960  
       2017-12-13 11:22:26 +08:00
    我现在特想问一个问题,我作为员工,怎么指出 leader 的代码有一些小问题呢
    1  2  
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   5335 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 33ms · UTC 07:03 · PVG 15:03 · LAX 00:03 · JFK 03:03
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.