Re: [討論] 寫三元判斷式code review被打槍

軟工

17120

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 ,他們股價好像有點糟。

以上推薦給大家,也歡迎大家推薦好用的給我。

: 小弟寫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://www.ptt.cc/bbs/Soft_Job/M.1671007832.A.3A1.html
Hsins1樓以下開放投票:Maven or Gradle 12/14 16:51
Hsins2樓(抱歉ㄌ Ant 朋朋們) 12/14 16:51
本人3樓Gradle +1 12/14 16:55
s06yji34樓Maven 12/14 17:28
jobintan5樓JS/TS有ESLint,不知道Java有沒有相似的東西? 12/14 17:30
bheegrl6樓JAVA 就sonarlint 12/14 17:31
Hsins7樓沒有偏好的話,可以用 clang-format 啦 12/14 17:31
tw115098樓Gradle 12/14 21:13
theedge9樓這篇正解 不懂前幾篇在吵什麼 12/14 23:06
purin8810樓grandle 12/14 23:52
Ericz700011樓確實 12/15 03:32
Csongs12樓推這篇分享 12/15 08:47
alan310013樓很多公司根本不管這塊 所以才在review階段吵然後被電 12/15 11:30
alan310014樓大概接案甲方有要求才會做 12/15 11:33
yuinami15樓 12/15 12:26
sniper282416樓兩種都用過 覺得沒差 12/16 13:46
pig003817樓Maven 12/16 16:01
lonelytea18樓 12/16 17:26
friends2919樓Formatter+Linter = 舒服 01/06 06:18