作者chehsunliu (阿勳)
看板Soft_Job
标题Re: [讨论] 写三元判断式code review被打枪
时间Wed Dec 14 16:50:27 2022
Code review 检查这些会有点太花时间，应该有更重要的东西要看。
可以用一些 Gradle plugins 卡在 CI 比较省事：
1. Checkstyle
顾名思义检查 style。
2. SpotBugs
FindBugs 的继任，这个超老牌了。
3. PMD
有超多 rule 可以用，比方说限制 code 里面不能有 System.out/err。
4. Errorprone
检查一些潜在错误，像 String split 跟你想的其实不一样、unused variables
、Format Method vararg check 等等。
5. Uber NullAway
这个是 Errorprone 的 plugin，他可以解决大部分 Java 恼人 null check 问题，
我现在新专案就直接先套这个。
6. Google Java Format
Formatter，搭配 Spotless 使用。优点是 Google，缺点是 80 chars 不能改。
7. Palantir Java Format (120 chars)
Formatter，搭配 Spotless 使用。优点是 120 chars 且 lambda 也好看，缺点
是 Palantir ，他们股价好像有点糟。
以上推荐给大家，也欢迎大家推荐好用的给我。
※ 引述《a88241050 (再回头已是百残身)》之铭言：
: 小弟写java的
: 以前常常写三元判断式
: 就比如说
: String a;
: if(con) {
: a= "aaa";
: } else {
: a="bbb";
: }
: 这样就要占掉六行
: 所以我通常都是写
: String a= con ? "aaa" : "bbb";
: 从五行变成一行
: 在我看来简洁又方便
: 但最近给一个资深前辈code review的时候
: 他说不要用三元判断式
: 因为不好阅读
: 他工作那麽久也从来不用三元判断式的
: 而且java有很多套件都可以用来取代三元判断式
: 所以用三元判断式真的不好吗？
--
※ 发信站: 批踢踢实业坊(ptt.cc), 来自: 61.216.130.235 (台湾)
※ 文章网址: https://webptt.com/cn.aspx?n=bbs/Soft_Job/M.1671007832.A.3A1.html
1F：→ Hsins: 以下开放投票：Maven or Gradle 12/14 16:51
2F：→ Hsins: (抱歉ㄌ Ant 朋朋们) 12/14 16:51
※ 编辑: chehsunliu (219.87.142.18 台湾), 12/14/2022 16:53:57
3F：→ chehsunliu: Gradle +1 12/14 16:55
4F：推 s06yji3: Maven 12/14 17:28
5F：推 jobintan: JS/TS有ESLint，不知道Java有没有相似的东西？ 12/14 17:30
6F：推 bheegrl: JAVA 就sonarlint 12/14 17:31
7F：→ Hsins: 没有偏好的话，可以用 clang-format 啦 12/14 17:31
8F：→ tw11509: Gradle 12/14 21:13
9F：推 theedge: 这篇正解 不懂前几篇在吵什麽 12/14 23:06
10F：推 purin88: grandle 12/14 23:52
11F：推 Ericz7000: 确实 12/15 03:32
12F：推 Csongs: 推这篇分享 12/15 08:47
13F：→ alan3100: 很多公司根本不管这块 所以才在review阶段吵然後被电 12/15 11:30
14F：→ alan3100: 大概接案甲方有要求才会做 12/15 11:33
15F：推 yuinami: 推 12/15 12:26
16F：推 sniper2824: 两种都用过 觉得没差 12/16 13:46
17F：推 pig0038: Maven 12/16 16:01
18F：推 lonelytea: 推 12/16 17:26
19F：推 friends29: Formatter+Linter = 舒服 01/06 06:18