स्ट्रिंग तुलना में अक्षर की सही स्थिति क्या है?

Question

मैं

if (localName.equals("TaxName")) { 

है, लेकिन PMD कहते

Position literals first in String comparisons 

Answer 1

"TaxName".equals(localName) बेहतर है के रूप में यदि localName अशक्त आप एक नल पॉइंटर एक्सेप्शन नहीं मिलेगा है।

Answer 2

मैं पहले की स्थिति शाब्दिक पसंद करते हैं, अर्थात:

if ("TaxName".equals(localName)) { ... 

इस तरह आप के बजाय NullPointerException होने का अशक्त के मामले के लिए एक सही तुलना करते हैं,।

Answer 3

पीएमडी आपको क्यों बता रहा है क्यों यह चेतावनी उत्पन्न करता है। PMD वेबसाइट पर the rules documentation से:

स्ट्रिंग तुलना में पहली

स्थिति शाब्दिक - इस तरह से स्ट्रिंग अशक्त आप एक NullPointerException नहीं मिलेगा है अगर, यह सिर्फ झूठी आरंभ कर देंगे।

Answer 4

व्यक्तिगत रूप से, यह मुझे समझ में नहीं आता है। यदि कोड एक NullPointerException पकड़ता है, तो यह काम किया गया है कि आपको बाद में नहीं करना होगा। यदि स्थानीय नाम शून्य हो जाता है, और इससे बाद में कोई समस्या आती है, तो पता लगाना कठिन होगा। कंपाइलर को खुश करने के लिए कोड न बदलें। यदि आपका कोड NullPointerException फेंकता है, तो यह बाद में आपको डीबगिंग सहेजा गया है।

Answer 5

कि चेतावनी से बचने के लिए एक सरल समाधान की जांच nullpointers से पहले, जो हर वस्तु हम प्रबंधन, में सिफारिश की है न केवल यह बहुत ही मामले में:

if (localName!=null && localName.equals("TaxName")) { 
    ... 
}