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

帮助阿里巴巴遵守阿里巴巴规范(第三季)

  •  
  •   blindpirate ·
    blindpirate · 2019-03-15 07:32:29 +08:00 · 5589 次点击
    这是一个创建于 2078 天前的主题,其中的信息可能已经有所发展或是发生改变。

    前情提要: https://www.v2ex.com/t/543954

    第二季: https://www.v2ex.com/t/544369

    在前两季播出后,收到了观众朋友们的许多留言,在这里整理回复如下:

    闲的蛋疼 /你这也叫开源?

    谁能帮我向他解释一下这是不是开源?我不知道该怎么解释。如果修 code style 不算开源的话,我刚刚发现并顺手修复了一个 p3c 插件的 bug,这算开源么? https://github.com/alibaba/p3c/pull/472

    你发烧了吧 /你这是杠,不是开源 /你是杠精 /你发一次贴就行了,再发就是蹭热度了 /吃饱了撑的

    “发现问题,提交代码解决问题”对你们就这么难以理解么?到今天为止我在 GitHub 上已经提交了 370 个 PR,我就是爱给开源项目提交代码,在你眼里都是杠是不是?你以为谁都跟你一样只会动嘴当喷子?昨天我跟阿里巴巴开源负责人聊过了,人家都没意见,轮到你来唧唧歪歪?称你们一声键盘侠不过分吧?

    1

    工作量不饱和

    这些 PR 都是早晚非工作时间写的。我公司不加班,仍招人,有兴趣可以投递。如果对我的工作量有意见可以向我的主管投诉。

    刚刚向Sentinel 提交了 PR: https://github.com/alibaba/Sentinel/issues/573

    利益相关:前阿里巴巴员工,现 Gradle core team developer

    第 1 条附言  ·  2019-03-15 09:25:55 +08:00
    PR merged: https://github.com/alibaba/Sentinel/pull/574

    感谢 Sentinel 团队!
    34 条回复    2019-03-18 19:22:02 +08:00
    blindpirate
        1
    blindpirate  
    OP
       2019-03-15 07:34:01 +08:00   ❤️ 7
    @PP @VoidChen @dfjslkjdf 我已经提交了 4 个 PR 了,你们呢?你们除了高谈阔论冷嘲热讽,做了什么?
    haozhang
        2
    haozhang  
       2019-03-15 07:39:59 +08:00 via Android
    可能他们已经 block 你了,毕竟你把他们脸都扇肿了
    honeycomb
        3
    honeycomb  
       2019-03-15 08:29:04 +08:00 via Android   ❤️ 1
    @blindpirate 发现之前就已经屏蔽这三个人了
    pabupa
        4
    pabupa  
       2019-03-15 08:36:30 +08:00 via Android
    您真棒👍,没有错。
    前面是我脑袋发烧了,不了解情况,向您道歉。

    意思其实很简单,不是说您这样的 pr 不是开源,也不是说您做的不对。只是我的思维有局限吧,,,一直以为修 bug 或者有程序优化的才算……

    但是还是感觉怪怪的,就像是您在帮别人擦脸上的灰。您明白我意思吧……可能我的格局不够,没有把把别人当作我自己。您是前阿里巴巴的员工,当然没有什么问题了。

    我绝对不是什么键盘侠,,,,对您也没有什么恶意。
    xiaohuamao
        5
    xiaohuamao  
       2019-03-15 08:39:41 +08:00 via iPhone
    轮循?每天来一遍就没意思了,况且阿里又没有支付薪酬,为什么要花费自己时间精力,帮助阿里改进?
    azh7138m
        6
    azh7138m  
       2019-03-15 08:54:52 +08:00 via Android
    emmm,java 都是手动的吗?没有 format 工具吗?
    讲道理我也觉得手动改一次两次就好了,可这么多格式问题,不应该用 format 工具自动做掉吗?觉得手动做这些很不程序员。
    huanyues
        7
    huanyues  
       2019-03-15 08:56:45 +08:00 via iPhone
    我觉得挺好的。。。有句话说,圣母(而不是圣母婊)是推动人类进步的最大动力。做事总比坐在那里瞎哔哔要强。那些什么也不做只知道瞎哔哔的人,别人做点事还要阻止的人,可能就是阻止人类进步的圣母婊了。
    maxiaofeng
        8
    maxiaofeng  
       2019-03-15 09:22:14 +08:00
    做了一件你自己觉得很有意义的事, 而别人觉得没有意义的事!! 你也没必要站在道德的制高点。
    gscoder
        9
    gscoder  
       2019-03-15 09:32:54 +08:00
    支持,不懂那些酸的人。搞开源最大的好处不就是提升能力吗
    ericgui
        10
    ericgui  
       2019-03-15 10:25:27 +08:00
    第三次支持楼主
    lusi1990
        11
    lusi1990  
       2019-03-15 10:43:56 +08:00
    今天的 V 很火爆啊
    Showfom
        12
    Showfom  
       2019-03-15 10:57:44 +08:00 via iPhone
    @blindpirate 那种傻逼没必要去鸟他们的 直 B 了就完事
    shaohan0228
        13
    shaohan0228  
       2019-03-15 11:29:54 +08:00
    还是支持的,如果规范出来,无人遵守的话,规范就没有意义了, 这也能看出,阿里提出的规范在内部也并非完全推广,通过这种方式,一方面规范内部代码,另一方面,也会凸显出规范中本身不合理的地方,进一步的去更好的完善规范。
    LanAiFaZuo
        14
    LanAiFaZuo  
       2019-03-15 11:31:02 +08:00
    坐等


    帮助阿里巴巴遵守阿里巴巴规范(第 100 季)
    wzxlovesy
        15
    wzxlovesy  
       2019-03-15 11:39:36 +08:00 via Android
    没什么营养
    adminsam
        16
    adminsam  
       2019-03-15 11:44:39 +08:00
    登陆支持,确实是双赢的一件事 👍
    realPipiz
        17
    realPipiz  
       2019-03-15 11:50:21 +08:00 via iPhone
    发一个帖子持续更新就行了,别发这么多
    blindpirate
        18
    blindpirate  
    OP
       2019-03-15 11:57:37 +08:00 via iPhone
    @realPipiz 不。
    dajj
        19
    dajj  
       2019-03-15 12:39:56 +08:00
    楼主很棒
    zhix
        20
    zhix  
       2019-03-15 12:49:06 +08:00
    支持 👍,谁谁谁说中国开源生态差,现在有了先驱者又群起而攻之,大型讽刺现场。
    smeraldo
        21
    smeraldo  
       2019-03-15 13:30:49 +08:00
    @SuppressWarnings("PMD.ThreadPoolCreationRule")
    没仔细看,不过这也行啊?😂
    lyoe
        22
    lyoe  
       2019-03-15 13:36:34 +08:00
    挺好啊,支持 LZ。认真才有收获。
    yoshiyuki
        23
    yoshiyuki  
       2019-03-15 16:43:14 +08:00
    哪个公司呀
    freedomshi
        24
    freedomshi  
       2019-03-15 19:19:01 +08:00
    PP 呢?哈哈哈,还“此人不值得”,哈哈哈哈哈,笑死我了。。。
    ilgharkus
        25
    ilgharkus  
       2019-03-15 19:27:35 +08:00
    第三次支持楼主。
    ilgharkus
        26
    ilgharkus  
       2019-03-15 19:31:24 +08:00
    同时庆祝 V2 第一部连续剧开播
    blindpirate
        27
    blindpirate  
    OP
       2019-03-15 20:49:18 +08:00
    @xiaohuamao 因为我喜欢写代码呀。
    blindpirate
        28
    blindpirate  
    OP
       2019-03-15 20:52:36 +08:00
    @azh7138m 我不知道你用的什么工具,我用 IDEA,在单个文件或者项目上 右键 Analyze - Run inspections by name - 选择修复类型 - Fix 即可。
    blindpirate
        29
    blindpirate  
    OP
       2019-03-15 20:53:38 +08:00
    @smeraldo 这样做的目的是保证代码至少从此以后不会变得更差。
    smeraldo
        30
    smeraldo  
       2019-03-15 22:33:05 +08:00
    @blindpirate 但..这只是不提示了啊..
    blindpirate
        31
    blindpirate  
    OP
       2019-03-16 17:42:45 +08:00
    @smeraldo 我是这么理解的:

    首先,对于不能 /不想修复的问题,@SuppressWarnings 是常规操作。在 Sentinel 项目中,说句实话,我没有把握把这几个问题修复好(类似的注解还有 @SuppressFBWarnings,是 FindBugs 引入的,它需要额外的注解是因为 FB 扫描的是字节码,字节码中已经没有 @SuppressWarnings 信息了)。

    其次,仅仅 @SuppressWarnings 不会使代码变得更坏——它是编译期注解,对编译后的结果没有任何影响。好处是从此之后新的违背代码规范的问题能在第一时间被发现。

    最后,假如哪一天团队想要修复相关问题,全局搜索相应的 @SuppressWarnings 注解即可,无需运行构建。

    这让我想起来,gradle 在迁移 Java9 的过程中,发现有很多测试在 Java9 上不能通过。我们当时的解决方案是,引入了一个 FIX_TO_WORK_ON_JAVA9 注解,在 Java9 上直接忽略这些测试: https://github.com/gradle/gradle/blob/69e400f5f00d60684c75827ecf61d3521d333467/subprojects/internal-testing/src/main/groovy/org/gradle/util/TestPrecondition.groovy#L106。忽略这些测试并不意味着从此不管了,而只是代表这些问题我们知道了,当前限于实际无法处理。绝大多数被忽略的 Java9 上的测试已经在去年夏天被修复掉了。
    smeraldo
        32
    smeraldo  
       2019-03-17 19:00:30 +08:00
    @blindpirate 用意是好的,但有没有用那得看执行力了,搞不好以后还是没人管。。
    blindpirate
        33
    blindpirate  
    OP
       2019-03-17 20:40:42 +08:00
    @smeraldo 从目前和三个 merge 了我的 PR 的团队的接触来看,我对他们三个团队很有信心。但是从跟你的对话来看,我对你的团队没有信心。
    smeraldo
        34
    smeraldo  
       2019-03-18 19:22:02 +08:00 via Android
    @blindpirate 😂好巧,我也没有信心
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   实用小工具   ·   2721 人在线   最高记录 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 29ms · UTC 15:24 · PVG 23:24 · LAX 07:24 · JFK 10:24
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.