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

你们在公司写代码有没有什么编程风格规范?一般都是怎么实施?

  •  
  •   jdhao ·
    jdhao · 2019-06-20 19:13:07 +08:00 via Android · 6225 次点击
    这是一个创建于 1967 天前的主题,其中的信息可能已经有所发展或是发生改变。

    如题,譬如 Python 有 PEP8,谷歌的规范等。你们在公司写代码会不会遵循编程规范,还是大体上确保代码没问题就行,不是严格遵守这些规范?

    如果遵守这些规范的话是怎么保证大家都遵守统一规范的,一般都采用什么办法

    57 条回复    2019-06-23 10:20:51 +08:00
    zyqzyq08
        1
    zyqzyq08  
       2019-06-20 19:16:15 +08:00 via Android
    pycharm 直接 refomat code
    moyupoi
        2
    moyupoi  
       2019-06-20 19:19:00 +08:00 via iPhone   ❤️ 1
    前端可以加 eslint,然后增加 git 提交检查,如果有不规范直接自动禁止提交,非常管用
    jdhao
        3
    jdhao  
    OP
       2019-06-20 19:19:44 +08:00 via Android
    @zyqzyq08 pycharm format 出来的虽然看上去遵守规范了,但是有时候比较丑,不美观
    jdhao
        4
    jdhao  
    OP
       2019-06-20 19:20:57 +08:00 via Android
    @moyupoi 是 commit hook 之类的嘛,是不是有 warning 就提交不了
    moyupoi
        5
    moyupoi  
       2019-06-20 19:34:03 +08:00 via iPhone
    恩,但可以强制提交,加个-n 绕过 eslint 检查,但下一个人如果普通提交会被上一个人强制提交的代码卡住,然后这个人就被打死的
    moyupoi
        6
    moyupoi  
       2019-06-20 19:37:34 +08:00 via iPhone
    这种方式不适合那种水平很菜,代码已经写了好几万行没人维护的项目,适用于水平差不多的人,提高代码质量,相互约束,还有一种场景就是制那些水平很烂却牛逼哼哼的代码“大神”
    russian
        7
    russian  
       2019-06-20 20:15:02 +08:00
    没有 code review 吗? code review 的时候可以给他提意见
    jdhao
        8
    jdhao  
    OP
       2019-06-20 20:46:16 +08:00 via Android
    @russian 没有,都是自己写自己的,能跑就行
    russian
        9
    russian  
       2019-06-20 20:48:33 +08:00
    @jdhao 那至少开会把?有的人代码写得很烂,比如一个函数 200 行,肯定所有人都能知道。这种容易出 bug 或者不好测试的,肯定会说起来。说起来的时候让他自己改
    jdhao
        10
    jdhao  
    OP
       2019-06-20 20:56:19 +08:00 via Android
    @russian 我们做算法的,目前对代码要求不高,能跑就行,没有任何 review 过程
    russian
        11
    russian  
       2019-06-20 21:54:36 +08:00
    @jdhao 我发现你没看明白我上边那句话。我觉得你们的团队要么很业余,要么你不是学 cs 的
    jdhao
        12
    jdhao  
    OP
       2019-06-20 22:48:50 +08:00 via Android
    @russian 我知道你的意思,我们目前没有 code review 之类的,算法主要是输出训练好的模型,保证模型能得到需要的结果
    hyrepo
        13
    hyrepo  
       2019-06-20 23:11:13 +08:00
    Java 有静态检查插件,比如 checkstyle,风格不对会挂 build
    russian
        14
    russian  
       2019-06-20 23:50:08 +08:00
    @jdhao 不。。。你不知道。。。
    qiaobeier
        15
    qiaobeier  
       2019-06-20 23:55:59 +08:00
    eslint config
    luozic
        16
    luozic  
       2019-06-21 07:26:45 +08:00 via iPhone
    没有 code review 并且有基本规范和定期分享。也么有门槛监督,所有的规范都会变成空中的飞机。
    Jonz
        17
    Jonz  
       2019-06-21 08:21:25 +08:00
    我们 Java 团队最近的新项目就是按阿里的编码规范来要求,IDEA 装 P3C 插件扫描。。
    kerassss
        18
    kerassss  
       2019-06-21 08:31:40 +08:00 via Android
    @moyupoi 这东西真的有用嘛,每次感觉检查跑半天,影响前端同学开发效率的说。
    xsiong
        19
    xsiong  
       2019-06-21 08:38:39 +08:00
    阿里的 java 手册
    jowan
        20
    jowan  
       2019-06-21 08:51:54 +08:00
    PSR
    avenger
        21
    avenger  
       2019-06-21 09:39:27 +08:00 via iPhone
    @moyupoi #2 git hook 不是在本地吗?自己禁止了怎么办?
    l00t
        22
    l00t  
       2019-06-21 09:52:16 +08:00
    代码风格在编程规范里是最不重要的东西,没必要花太多心思。
    axbx
        23
    axbx  
       2019-06-21 10:02:35 +08:00
    提交之前会跑一遍 sonar。
    kaedea
        24
    kaedea  
       2019-06-21 10:06:19 +08:00 via Android
    .editor onfig
    hexingb
        25
    hexingb  
       2019-06-21 10:13:31 +08:00
    @russian 200 行代码还差劲吗?看我这里几千行的。自从接手这个烂坑,我心里骂了多少 fuck,绝对比代码字母数多。

    很多人、很多公司不考虑这个,总是拿追进度做借口。实际上,规定一套很简单的规范就能够很好的统一代码风格。比如空格 tab,比如 dos/unix 换行。
    yvescheung
        26
    yvescheung  
       2019-06-21 10:13:40 +08:00
    c++遵循谷歌规范,不规范编译无法通过
    zhuweiyou
        27
    zhuweiyou  
       2019-06-21 10:22:17 +08:00
    搬砖而已,只要能完成功能就行了
    GoLand
        28
    GoLand  
       2019-06-21 10:25:13 +08:00   ❤️ 1
    加 pre-commit,把代码风格检查放进去。过不了就 commit 不了,ci 里也会过一下 diff,代码风格不对 ci 都过不了,别说 Merge 了。
    no1xsyzy
        29
    no1xsyzy  
       2019-06-21 10:29:31 +08:00
    @russian 算法是 cs 不是 se 啊……
    做算法的不算程序员的,算科学家或者技术工,中间其实没有工程师的位置。
    jdhao
        30
    jdhao  
    OP
       2019-06-21 10:39:07 +08:00 via Android
    @no1xsyzy 做算法感觉有点尴尬,和做前端后端,做 APP 不一样,一般都是在训练模型,代码质量好像没太多人 care …
    no1xsyzy
        31
    no1xsyzy  
       2019-06-21 10:50:04 +08:00
    @jdhao (其实 APP 也算前端,只是现在前端常特指网页前端)
    是的,甚至理论上不需要写代码,只需要写伪代码。不过写个代码能够很明显地看到:它能跑,能出需要的结果,所以逻辑没错。
    至于代码质量?算法写完的代码是需要后端重新写一遍的。算法写的代码可能是 Python, Matlab, R 甚至是 Fortran 的,但是后端的是 Java 或者 Golang。让算法维护代码?经济效益就低了,老板是不干的(如果老板能懂这点的话)。但后端可能只是看得来 Python 等,稍微写一点写不了很复杂的,那不如重新写一遍。
    所以说为什么 Python 的一些地方非常垃圾但算法常用,因为其实就是当个能跑的伪代码用。
    neverfelly
        32
    neverfelly  
       2019-06-21 11:09:45 +08:00
    @no1xsyzy 做算法也算程序员啊...只要是 cs 的,代码就应该以规范化来写,传统 linux 科学家哪个不是写得一手好代码,就说最近火热的机器学习这一块,xgboost,lightgdm 也是按照规范在运作...纯粹训练模型算是偏数学科研工作者了
    no1xsyzy
        33
    no1xsyzy  
       2019-06-21 11:18:29 +08:00
    @neverfelly 以前的程序员还搞硬件呢,不也是逐渐分开了?随着社会发展,让职位分开是大势所趋啊。
    而且 Linux 科学家是指哪些啊…… Unix 就是工程师的事了吧。
    dilu
        34
    dilu  
       2019-06-21 11:22:46 +08:00 via Android
    有,git 钩子检查,不符合规范直接扣绩效
    neverfelly
        35
    neverfelly  
       2019-06-21 11:25:06 +08:00
    @no1xsyzy 可能表达有点错误,linux 科学家指:是 linux 教徒科研工作者...虽然目前来说确实就训练模型来说,对代码的抽象性和理论推导要求比实际代码能力来要强一点,所以并不会要求代码写得很漂亮,但这也确实正在成为所诟病的一点,诸如脚本调参侠之类的戏称。个人感觉 cs 最重要的还是对代码如同艺术以及对数学如同神明一样古典美感结合最好
    no1xsyzy
        36
    no1xsyzy  
       2019-06-21 12:40:29 +08:00
    @neverfelly 我倾向于算法这块构造属于算法的新的美感,就好像数学的美感在程序员里就可能是丑陋的(单字母公有自由变量)。
    jdhao
        37
    jdhao  
    OP
       2019-06-21 13:01:05 +08:00
    @no1xsyzy 这个就是代码之美,写的代码能够被其他人容易读懂,所以变量命令要有意义,不能像搞数学的弄一些单变量的名字,也是软件工程的要求吧,清楚的代码更加容易维护。
    ayasakinagi
        38
    ayasakinagi  
       2019-06-21 13:12:46 +08:00   ❤️ 1
    python 是 pep8, js 是 esline, 代码通过 git review 提交到 gerrit 上面, 通过 jenkins 检查格式.
    loading
        39
    loading  
       2019-06-21 13:25:39 +08:00 via Android
    结对编程,只要另外一个能看懂,注释够,编译能过就行,←_←
    intsilence
        40
    intsilence  
       2019-06-21 13:59:52 +08:00
    CI 里检查
    janxin
        41
    janxin  
       2019-06-21 14:02:54 +08:00
    工具检查
    russian
        42
    russian  
       2019-06-21 15:33:10 +08:00
    @hexingb 200 行的我感觉恶心,懒得说。如果有人写函数超过一千行我真的要和他好好谈谈,真的。绝对不能这么妥协。
    brust
        43
    brust  
       2019-06-21 15:40:58 +08:00
    java 没有规范 但是我按阿里的 java 手册
    Ritr
        44
    Ritr  
       2019-06-21 15:57:07 +08:00
    我们公司没什么规范,能跑就行!
    我个人的代码风格是增强约束性,变量名起全名,function 行数在 50 行以内,减少公共变量等等
    WispZhan
        45
    WispZhan  
       2019-06-21 16:06:43 +08:00
    xxx-lint
    checkstyle
    sonarqube

    最重要的 code review,不符合风格的,打回去
    no1xsyzy
        46
    no1xsyzy  
       2019-06-21 18:28:15 +08:00
    @jdhao 类似地,我认为算法工作最后得出的美感也会和程序员(软件工程)的不一样。
    这不是在说个体差异,而是工作内容、工作需求、工作环境决定的。
    至于最后会变成什么样我也不知道,就算这时候有谁预言中了也不过是瞎猫(略
    gxm44
        47
    gxm44  
       2019-06-21 18:34:28 +08:00
    @jdhao 配合 black 吧
    kljsandjb
        48
    kljsandjb  
       2019-06-21 18:36:33 +08:00 via iPhone
    大括号换行?
    jdhao
        49
    jdhao  
    OP
       2019-06-21 19:02:58 +08:00 via Android
    @gxm44 目前用的是 pylint 和 flake8
    ben1024
        50
    ben1024  
       2019-06-21 19:05:59 +08:00
    PSR
    PHPCS
    xiangliudev
        51
    xiangliudev  
       2019-06-21 19:47:45 +08:00
    我司 java 是用 checkstyle
    FireCat
        52
    FireCat  
       2019-06-21 20:27:37 +08:00   ❤️ 1
    问题一:在网上搜下使用语言的比较有名的一个或几个规范。综合生成适合自己现在公司项目的。
    问题二:拿着规范进行培训、使用代码检查工具等多管齐下
    w292614191
        53
    w292614191  
       2019-06-22 08:46:13 +08:00
    @Jonz
    @brust
    @xsiong
    我也装了 P3C,我感觉代码写不动了,时时刻刻都有问题,需要纠正写法,有时候特殊原因没办法按照完全规范,很尴尬。
    Jonz
        54
    Jonz  
       2019-06-22 10:03:38 +08:00
    @w292614191 只能看具体是什么问题了,如果一直是没办法按规范来的就把那个类型的错误先关掉提示呗。刚刚加班想偷懒拷了一堆重复代码,也是一堆波浪线,后面想了想就还是抽出来公用方法了。感觉就是可以时刻提醒自己去按规范来
    brust
        55
    brust  
       2019-06-22 12:29:35 +08:00
    @w292614191 #53
    尽量遵循吧
    阿里的 java 手册和 google java 手册 有点冲突
    也不是一定要完全遵循
    Earl
        56
    Earl  
       2019-06-22 13:39:42 +08:00
    规范肯定有,而且我们是要求强制执行的。
    合入前必须通过静态检查,不然不能合入。
    l4ever
        57
    l4ever  
       2019-06-23 10:20:51 +08:00 via iPhone
    @l00t 反对,团队协作里面,代码规范很重要。
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   3108 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 13:08 · PVG 21:08 · LAX 05:08 · JFK 08:08
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.