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

看板Soft_Job (軟體人)作者 (阿勳)時間1年前 (2022/12/14 16:50), 1年前編輯推噓11(1107)
留言18則, 15人參與, 1年前最新討論串3/15 (看更多)
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://www.ptt.cc/bbs/Soft_Job/M.1671007832.A.3A1.html

12/14 16:51, 1年前 , 1F
以下開放投票:Maven or Gradle
12/14 16:51, 1F

12/14 16:51, 1年前 , 2F
(抱歉ㄌ Ant 朋朋們)
12/14 16:51, 2F
※ 編輯: chehsunliu (219.87.142.18 臺灣), 12/14/2022 16:53:57

12/14 16:55, 1年前 , 3F
Gradle +1
12/14 16:55, 3F

12/14 17:28, 1年前 , 4F
Maven
12/14 17:28, 4F

12/14 17:30, 1年前 , 5F
JS/TS有ESLint,不知道Java有沒有相似的東西?
12/14 17:30, 5F

12/14 17:31, 1年前 , 6F
JAVA 就sonarlint
12/14 17:31, 6F

12/14 17:31, 1年前 , 7F
沒有偏好的話,可以用 clang-format 啦
12/14 17:31, 7F

12/14 21:13, 1年前 , 8F
Gradle
12/14 21:13, 8F

12/14 23:06, 1年前 , 9F
這篇正解 不懂前幾篇在吵什麼
12/14 23:06, 9F

12/14 23:52, 1年前 , 10F
grandle
12/14 23:52, 10F

12/15 03:32, 1年前 , 11F
確實
12/15 03:32, 11F

12/15 08:47, 1年前 , 12F
推這篇分享
12/15 08:47, 12F

12/15 11:30, 1年前 , 13F
很多公司根本不管這塊 所以才在review階段吵然後被電
12/15 11:30, 13F

12/15 11:33, 1年前 , 14F
大概接案甲方有要求才會做
12/15 11:33, 14F

12/15 12:26, 1年前 , 15F
12/15 12:26, 15F

12/16 13:46, 1年前 , 16F
兩種都用過 覺得沒差
12/16 13:46, 16F

12/16 16:01, 1年前 , 17F
Maven
12/16 16:01, 17F

12/16 17:26, 1年前 , 18F
12/16 17:26, 18F
文章代碼(AID): #1ZcOvOEX (Soft_Job)
討論串 (同標題文章)
文章代碼(AID): #1ZcOvOEX (Soft_Job)