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

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

  •  
  •   blindpirate · 9 天前 · 3095 次点击

    前情提要: 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 条附言  ·  9 天前
    PR merged: https://github.com/alibaba/Sentinel/pull/574

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

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

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

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


    帮助阿里巴巴遵守阿里巴巴规范(第 100 季)
        15
    wzxlovesy   9 天前 via Android
    没什么营养
        16
    adminsam   9 天前
    登陆支持,确实是双赢的一件事 👍
        17
    realPipiz   9 天前 via iPhone
    发一个帖子持续更新就行了,别发这么多
        18
    blindpirate   9 天前 via iPhone
    @realPipiz 不。
        19
    dajj   9 天前
    楼主很棒
        20
    zhix   9 天前
    支持 👍,谁谁谁说中国开源生态差,现在有了先驱者又群起而攻之,大型讽刺现场。
        21
    smeraldo   9 天前
    @SuppressWarnings("PMD.ThreadPoolCreationRule")
    没仔细看,不过这也行啊?😂
        22
    lyoe   9 天前
    挺好啊,支持 LZ。认真才有收获。
        23
    yoshiyuki   9 天前
    哪个公司呀
        24
    freedomshi   9 天前
    PP 呢?哈哈哈,还“此人不值得”,哈哈哈哈哈,笑死我了。。。
        25
    ilgharkus   9 天前
    第三次支持楼主。
        26
    ilgharkus   9 天前
    同时庆祝 V2 第一部连续剧开播
        27
    blindpirate   8 天前
    @xiaohuamao 因为我喜欢写代码呀。
        28
    blindpirate   8 天前
    @azh7138m 我不知道你用的什么工具,我用 IDEA,在单个文件或者项目上 右键 Analyze - Run inspections by name - 选择修复类型 - Fix 即可。
        29
    blindpirate   8 天前
    @smeraldo 这样做的目的是保证代码至少从此以后不会变得更差。
        30
    smeraldo   8 天前
    @blindpirate 但..这只是不提示了啊..
        31
    blindpirate   8 天前
    @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 上的测试已经在去年夏天被修复掉了。
        32
    smeraldo   7 天前
    @blindpirate 用意是好的,但有没有用那得看执行力了,搞不好以后还是没人管。。
        33
    blindpirate   6 天前
    @smeraldo 从目前和三个 merge 了我的 PR 的团队的接触来看,我对他们三个团队很有信心。但是从跟你的对话来看,我对你的团队没有信心。
        34
    smeraldo   6 天前 via Android
    @blindpirate 😂好巧,我也没有信心
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2287 人在线   最高记录 4385   ·  
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.3 · 22ms · UTC 11:39 · PVG 19:39 · LAX 04:39 · JFK 07:39
    ♥ Do have faith in what you're doing.
    沪ICP备16043287号-1