Re: [讨论] 写三元判断式code review被打枪

楼主: chehsunliu (阿勋)   2022-12-14 16:50:27
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有很多套件都可以用来取代三元判断式
: 所以用三元判断式真的不好吗?
作者: Hsins (翔)   2022-12-14 16:51:00
以下开放投票:Maven or Gradle(抱歉ㄌ Ant 朋朋们)
楼主: chehsunliu (阿勋)   2022-12-14 16:55:00
Gradle +1
作者: s06yji3 (阿南)   2022-12-14 17:28:00
Maven
作者: jobintan (Robin Artemstein)   2022-12-14 17:30:00
JS/TS有ESLint,不知道Java有没有相似的东西?
作者: bheegrl   2022-12-14 17:31:00
JAVA 就sonarlint
作者: Hsins (翔)   2022-12-14 17:31:00
没有偏好的话,可以用 clang-format 啦
作者: tw11509 (John-117)   2022-12-14 21:13:00
Gradle
作者: theedge   2022-12-14 23:06:00
这篇正解 不懂前几篇在吵什么
作者: purin88 (原来我是愤怒的乡民)   2022-12-14 23:52:00
grandle
作者: Ericz7000 (Ericz7000nolan)   2022-12-15 03:32:00
确实
作者: Csongs (西歌)   2022-12-15 08:47:00
推这篇分享
作者: alan3100 (BOSS)   2022-12-15 11:30:00
很多公司根本不管这块 所以才在review阶段吵然后被电大概接案甲方有要求才会做
作者: yuinami (yuinami)   2022-12-15 12:26:00
作者: sniper2824 (月夜)   2022-12-16 13:46:00
两种都用过 觉得没差
作者: pig0038 (颗颗)   2022-12-16 16:01:00
Maven
作者: lonelytea (霸气逼人)   2022-12-16 17:26:00
作者: friends29 (凉哥哥)   2021-01-06 06:18:00
Formatter+Linter = 舒服

Links booklink

Contact Us: admin [ a t ] ucptt.com