PTT推薦

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

看板Soft_Job標題Re: [討論] 寫三元判斷式code review被打槍作者
chehsunliu
(阿勳)
時間推噓11 推:11 噓:0 →:7

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 留言評論
※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 61.216.130.235 (臺灣)
PTT 網址

Hsins12/14 16:51以下開放投票:Maven or Gradle

Hsins12/14 16:51(抱歉ㄌ Ant 朋朋們)

※ 編輯: chehsunliu (219.87.142.18 臺灣), 12/14/2022 16:53:57

chehsunliu12/14 16:55Gradle +1

s06yji312/14 17:28Maven

jobintan12/14 17:30JS/TS有ESLint,不知道Java有沒有相似的東西?

bheegrl12/14 17:31JAVA 就sonarlint

Hsins12/14 17:31沒有偏好的話,可以用 clang-format 啦

tw1150912/14 21:13Gradle

theedge12/14 23:06這篇正解 不懂前幾篇在吵什麼

purin8812/14 23:52grandle

Ericz700012/15 03:32確實

Csongs12/15 08:47推這篇分享

alan310012/15 11:30很多公司根本不管這塊 所以才在review階段吵然後被電

alan310012/15 11:33大概接案甲方有要求才會做

yuinami12/15 12:26

sniper282412/16 13:46兩種都用過 覺得沒差

pig003812/16 16:01Maven

lonelytea12/16 17:26