अगर (बूलियन == गलत) बनाम अगर (बूलियन!)

Question

संभव डुप्लिकेट:
Is it bad to explicitly compare against boolean constants e.g. if (b == false) in Java?

this NotePadProvider sample code में, मैंने देखा है कि लेखक प्रपत्र चुना है:

if (values.containsKey(NoteColumns.CREATED_DATE) == false) { 
     values.put(NoteColumns.CREATED_DATE, now); 
    } 

ओवर:

if (!values.containsKey(NoteColumns.CREATED_DATE)) { 
     values.put(NoteColumns.CREATED_DATE, now); 
    } 

क्या अधिक तार्किक पर पहले फॉर्म का लाभ है?

Answer 1

"पठनीयता" के अलावा, नहीं। वे कार्यात्मक रूप से समकक्ष हैं।

Answer 2

यह एक शैली विकल्प है। यह कम से कम कोड के प्रदर्शन को प्रभावित नहीं करता है, यह सिर्फ पाठक के लिए इसे अधिक verbose बनाता है।

Answer 3

नहीं, मैं किसी भी लाभ नहीं दिख रहा है ("पठनीयता" उद्धरण में क्योंकि मैं == false नफरत और ! और अधिक पठनीय पाते हैं। लेकिन दूसरों को नहीं है।)। दूसरा एक और अधिक कठोर है।

बीटीडब्ल्यू: दूसरी शैली जेडीके स्रोत के हर कोने में पाई जाती है।

Answer 4

-1 विकल्प बहुत स्पष्ट है, लेकिन फिर 2 एक काफी सुरुचिपूर्ण है - यहाँ अपने कार्यक्षमता ....

होने से coding style के बारे में अधिक। .. कोई अपराध नहीं, यह सिर्फ मेरा विचार है ..

Answer 5

अधिकतर उत्तरदायित्व। अन्य कोड पढ़ते समय, के रूप में पढ़ने के लिए यह अधिक सहज ज्ञान युक्त हैपढ़ने के बजाय इसमें कुंजी है(values.containsKey(NoteColumns.CREATED_DATE) == false)।

Answer 6

पहला फॉर्म, जब एपीआई के साथ उपयोग किया जाता है जो Boolean देता है और बूलियन.एफएएलएसई के मुकाबले तुलना करता है, तो कभी भी NullPointerException फेंक नहीं देगा।

दूसरा रूप है, जब java.util.Map इंटरफेस के साथ प्रयोग किया जाता है, भी एक NullPointerException क्योंकि यह एक boolean और नहीं एक Boolean रिटर्न फेंक कभी नहीं होगा।

यदि आप लगातार कोडिंग मुहावरे के बारे में चिंतित नहीं हैं, तो आप अपनी पसंद के एक को चुन सकते हैं, और इस ठोस मामले में यह वास्तव में कोई फर्क नहीं पड़ता। यदि आप लगातार कोडिंग के बारे में परवाह करते हैं, तो विचार करें कि आप Boolean की जांच करते समय क्या करना चाहते हैं जो NULL हो सकता है।

Answer 7

नोट: ConcurrentMap के साथ आप अधिक कुशल

values.putIfAbsent(NoteColumns.CREATED_DATE, now); 

उपयोग कर सकते हैं मैं कम वर्बोज़ समाधान पसंद करते हैं और IsTrue या IsFalse या उनके जैसे तरीकों से बचें।