首页   注册   登录
 blindpirate 最近的时间轴更新

blindpirate

V2EX 第 162843 号会员,加入于 2016-03-11 09:21:48 +08:00
今日活跃度排名 5647
blindpirate 最近回复了
按照楼主的描述,项目肯定不是标准的 Maven/Gradle 项目。有抱怨的时间,不如尝试把项目变成 IDE 无关的。
我觉得答案显而易见啊。。。简历质量不高让人没有面试的欲望啊。。
数学系啊。。。。
@smeraldo 从目前和三个 merge 了我的 PR 的团队的接触来看,我对他们三个团队很有信心。但是从跟你的对话来看,我对你的团队没有信心。
@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 上的测试已经在去年夏天被修复掉了。
10 天前
回复了 blindpirate 创建的主题 Java 帮助阿里巴巴遵守阿里巴巴规范(第三季)
@smeraldo 这样做的目的是保证代码至少从此以后不会变得更差。
10 天前
回复了 blindpirate 创建的主题 Java 帮助阿里巴巴遵守阿里巴巴规范(第三季)
@azh7138m 我不知道你用的什么工具,我用 IDEA,在单个文件或者项目上 右键 Analyze - Run inspections by name - 选择修复类型 - Fix 即可。
10 天前
回复了 blindpirate 创建的主题 Java 帮助阿里巴巴遵守阿里巴巴规范(第三季)
@xiaohuamao 因为我喜欢写代码呀。
10 天前
回复了 blindpirate 创建的主题 Java 帮助阿里巴巴遵守阿里巴巴规范(第三季)
@realPipiz 不。
10 天前
回复了 blindpirate 创建的主题 Java 帮助阿里巴巴遵守阿里巴巴规范(第三季)
@PP @VoidChen @dfjslkjdf 我已经提交了 4 个 PR 了,你们呢?你们除了高谈阔论冷嘲热讽,做了什么?
关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   2739 人在线   最高记录 4385   ·  
创意工作者们的社区
World is powered by solitude
VERSION: 3.9.8.3 · 9ms · UTC 13:34 · PVG 21:34 · LAX 06:34 · JFK 09:34
♥ Do have faith in what you're doing.
沪ICP备16043287号-1