首先你要清楚 什麼是逆天程式碼 這個順序並不影響 只是習慣不好 其次 是否呼叫現有工具類方法是看個人愛好 如果不存在連續空串 那麼也並不出錯 如果你認為這樣兩個判斷也一定要掉方法 那也太死板了 jdk中大量的不推薦寫法 不也是寫了 程度保證正常執行即可 if只是不推薦複雜判斷 兩個也較複雜嗎

看工作經驗吧，如果是新人，還可以理解，如果是工作幾年的，那估計是沒救了。話說回來，除了新人，我還真沒見過有誰會寫出來上面的程式碼。

應該有錯，改成先判斷是否null再盤點是否空字串。

if(str==null ||str=="")

不過java不支援這樣判斷那就不行，一般可以除錯看出來的。估計他不會除錯程式碼吧。其他語言都可以這樣寫，java一定要用函式才能判斷，那就用函式，那也是沒有辦法的事。但不是什麼逆天程式碼吧

這麼寫沒問題的，不要想太多，||只有false才會執行後面的條件，換言之如果場景中“”和null的情況多不多 多多少，就算null的情況多，那麼這樣的效能問題也是忽略不計的，工具類先判斷null那是因為防止空指標異常 s！=null &&s.length>0

我覺得應該把你開除。

寫程式碼沒有毛病啊！如果連正常程式碼但跟你想要的不一樣都不能允許存在，那麼可想你的技術應該也是很有限。如果你是基礎開發人員還好，如果你是負責人，難麼對公司發展和公司人才培養是很不利的

要知道，實現一個功能有很多方式，即使判空也有很多方式，程式碼要包容，編碼符合規範就可以了！

這個程式碼沒什麼問題，要知道xml裡面就是這樣判斷的，至於順序，這個不會異常問題不大，至於封裝的方法裡面也是這些東西，有啥可逆天的，在這大驚小怪

這個，看看是不是新人吧，首先順序問題，判斷null應該在前，其次，字串判斷是否相等用equals，這應該是常識，如果剛入門，多帶帶人家吧

我一直以為寫UNIX的那才是逆天程式碼，沒想到java裡這麼一句就能逆天？好幸福！趕緊開除他吧讓他火～

悄悄問一句：JAVA啥時候有指標了？C語言老貓子還真沒見識過。

題主有點極端了，如果這個人就是一個小白，老妖認為他考慮事情還是挺全面的，最起碼空和null兩種情況都想到了。從這一點看來，這個人是值得培養的。至於你說的StringUtils之類的工具，只不過是一個簡化的校驗工具而已，會用而不知其理，那對自己成長沒多大幫助。

當然，若這個人是個java老鳥，那就有點故意之嫌了，這種情況下，如果此種現象經常出現，老妖認為你做為一個管理者從保證專案質量角度而言就需要找他聊聊了。

不懂程式碼。單純從管理角度來講，你假如從多方面確認員工犯錯，在屢教不改的情況下開除是很有必要的。但是從你釋出的問題中我感覺你好像在與員工因一條程式碼在抬槓？這個情況下你的情緒支配你的決定，你最該做的是冷靜而不是上網求助。