如何说服团队成员统一规范?

2018-04-18 09:24:31 +08:00
 c466934322
本来以为成员都会默认按照原来代码的规范来写代码,但是后来发现是我想的太简单了。有人使用驼峰动词+名词结合的方式,有人使用名词的方式。git 有人使用中文,有人使用中文拼音。和他们说这个问题后,人家来了一句:我以前就是这样写代码的,没有啥问题。背后还说:这家伙老是挑我的刺
6606 次点击
所在节点    程序员
60 条回复
fengbica
2018-04-18 10:17:29 +08:00
那就不合代码
dikT
2018-04-18 10:19:30 +08:00
python 变量都支持中文了, git commit 中文咋地了
zr8657
2018-04-18 10:21:33 +08:00
@c466934322 大部分情况下让管理又不给明确的权力,就是扯淡。大家都是同级谁搭理你?我吃过这样的亏,我身边也正在发生这样的事,都是泪。
imnpc
2018-04-18 10:24:47 +08:00
git 一般建议 英文+中文 如果针对国际化的项目 可以只英文
whosesmile
2018-04-18 10:25:37 +08:00
构建工程的时候加上强制的语法验证啊??这有什么好纠结的?构建失败他还能去改构建规则不成...
locktionc
2018-04-18 10:28:38 +08:00
先说代码命名问题,例如 Python 有 pylint,可以检查变量命名方式。你用的那个语言应该也有这种东西。

再说 Git,Git 中英文我觉得问题不大。你可以做一个 Git commit 的模板,他们提交的时候可以设置自动使用这个模板。

建议上持续集成,提交代码到 Git 以后,Jenkins 自动拉取代码,自动检查代码规范,不合格不让进。
NonClockworkChen
2018-04-18 10:31:19 +08:00
你如果想让拼音的人写英语,这是不可能的- -
Chyroc
2018-04-18 10:34:26 +08:00
@whosesmile 人家不舒服真的改了怎么办,哈哈哈
c466934322
2018-04-18 10:34:32 +08:00
@LeoNG 上面还是很重视的,关键是上面不懂技术。
@zr8657 我现在有一点名义上的权利,总部下发文件承认了我存在的意义
@nfer php 开发的
@dikT 大佬级的,萌新瑟瑟发抖

@yanjinbin 发的意义是探寻如何补救,不能说出了问题就坐视不理。现在就是在找补救方案
@flyz 是的,平级确实很尴尬。
goodryb
2018-04-18 11:20:19 +08:00
首先,你要制定一个规范,最好是邀请所有组员参与讨论制定,让大家都没有异议
其次,说服老板,讲清楚好处,让规范变成制度,写进每个人的 KPI 里面
第三,建立有效的审查机制,代码 review,自动规范检查
onion83
2018-04-18 11:33:37 +08:00
最佳实践是:用技术去约束规范

每个人背景不一样,甚至对文字的理解都不一样,你的文字规范、口头劝说对别人只有指导意义,而没有强制意义。

- 代码层面:在代码提交阶段,服务器对代码做前置检查,不通过就不能提交。( Mess Detector 了解一下)
包括且不限于:语法检查、文件编码约束、禁止 tab、函数 /方法命名检测,未使用函数、变量检查,自动化测试等

- 数据层面:分离 DAO 层,无法直连数据库,只能通过中间件、API 取数据

- 产品层面:少写代码,能自动生成尽量自动生成。高度抽象思考现有业务,能做后台做后台,通过配置驱动业务
c466934322
2018-04-18 11:49:05 +08:00
@goodryb
@onion83
@locktionc
@whosesmile
@imnpc
@nfer
@nfer
@imnpc
@nfer
@xianyu0
@eric1202
@hjw45611
@loshine1992
@ai277014717

感谢各位的解惑,我想我有了解决方案,确实不能单对单说:单对单有种不一样的地方,以后还是开会讨论,然后用技术去实现讨论的结果,强行约束所有人
6diyipi
2018-04-18 12:00:38 +08:00
抓住每一个人打一顿,问他服不服。
tomczhen
2018-04-18 12:01:09 +08:00
技术上:

1. 用脚本解决习惯和偏好问题(空格,tab )
2. CI Pipeline + lint
3. 根据实际情况添加自动检查机制
4. 根据 CI 生成业绩指标数据


管理上:

1. 明确权力
2. 干掉不听话的,如果干不掉就边缘化

其他:

做好失败之后找下家的准备
woodface2233
2018-04-18 12:08:58 +08:00
这是强制不需要说服
SuperMild
2018-04-18 12:10:53 +08:00
看来你们老板对管理也不是很懂,光口头上支持你没用的,要给你实权,有实权才能谈如何恩威并施,没有实权都懒得鸟你,不反过来怼你就已经很客气了。
chiu
2018-04-18 12:15:37 +08:00
用 svn,权限限制,这样你就看不到别人不规范的代码了
SuperMild
2018-04-18 12:16:50 +08:00
另外,老板如果想在公司内培养管理者,一般最好放出消息,画个大饼,然后暗示明示两三个候选人,对他们表示重视,让他们去竞争,在这个过程中慢慢地对其中一人显示出偏好,让全部员工看到这个过程,这样他的威信才能建立起来。

但如果本来就有一个技术和管理都优秀的人才,就不用这么麻烦,他自己就有足够的魅力去管人。
swulling
2018-04-18 13:07:35 +08:00
@c466934322 你有权利拒绝代码入库,罚款,扣除绩效么,没有权利没有义务

向你的 Leader 要授权
yanjinbin
2018-04-18 13:23:03 +08:00
管好你自己吧。。。要提高团队代码质量从招聘就可以开始了。。呵呵

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/447690

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX