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

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

  •  
  •   blindpirate · 252 天前 · 3639 次点击
    这是一个创建于 252 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

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

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

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

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


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