सुन्दरता जावा

Question

में NullPointerException परहेज

इस लाइन पर विचार करें:

if (object.getAttribute("someAttr").equals("true")) { // .... 

जाहिर है इस लाइन एक संभावित बग है, गुण null हो सकता है और हम एक NullPointerException मिल जाएगा।

पहले विकल्प:

if ("true".equals(object.getAttribute("someAttr"))) { // .... 

दूसरा विकल्प:

String attr = object.getAttribute("someAttr"); 
if (attr != null) { 
    if (attr.equals("true")) { // .... 

पहला विकल्प को पढ़ने के लिए अजीब लेकिन अधिक संक्षिप्त है तो हम दो विकल्पों में से एक करने के लिए इसे refactor करने के लिए की जरूरत है , जबकि दूसरा इरादा स्पष्ट है, लेकिन verbose।

पठनीयता के मामले में आप कौन सा विकल्प पसंद करते हैं?

Answer 1

मैं हमेशा

if ("true".equals(object.getAttribute("someAttr"))) { // .... 

का उपयोग किया है, क्योंकि हालांकि यह एक छोटे से अधिक यह बहुत कम वर्बोज़ है पढ़ने के लिए और मुझे लगता है कि यह काफी पठनीय है, तो आप इसे करने के लिए बहुत आसानी से

Answer 2

मुझे पसंद आदत हो मुश्किल है विकल्प 1 और मैं तर्क दूंगा कि यह पर्याप्त पठनीय है।

विकल्प 3 बीटीडब्ल्यू एक getAttribute विधि पेश करना होगा जो पैरामीटर के रूप में डिफ़ॉल्ट मान लेता है।

Answer 3

Util.isEmpty(string) - रिटर्न string == null || string.trim().isEmpty() Util.notNull(string) रिटर्न "" यदि string == null, स्ट्रिंग अन्यथा। Util.isNotEmpty(string) रिटर्न! Util.isEmpty(string)

और हमारे पास एक सम्मेलन है कि तारों के लिए, Util.isEmpty(string) अर्थात् सत्य और Util.isNotEmpty(string) अर्थात् झूठा मतलब है।

Answer 4

दूसरा विकल्प में, आप शॉर्ट-सर्किट && का लाभ ले सकते:

String attr = object.getAttribute("someAttr"); 
if (attr != null && attr.equals("true")) { // .... 

Answer 5

हमेशा की तरह, कम कोड की आकांक्षा यह देखते हुए कि दोनों functionaly बराबर हैं। विशेष रूप से इस तरह के मामले में जहां पठनीयता का त्याग नहीं किया जाता है।

Answer 6

ऐसी कुछ स्थितियां हैं जहां संक्षिप्त दृष्टिकोण शुरू करने में गलत लगता है लेकिन प्रभावी ढंग से बेवकूफ बन जाता है। यह उनमें से एक है; दूसरा कुछ ऐसा है:

String line; 
while ((line = bufferedReader.readLine()) != null) { 
    // Use line 
} 

किसी स्थिति में साइड इफेक्ट्स? अकल्पनीय! जब आप विशेष पैटर्न को पहचानते हैं, तो यह विकल्प के मुकाबले मूल रूप से अच्छा है।

यह पैटर्न समान है - यह जावा में इतना आम है कि मैं किसी भी उचित अनुभवी डेवलपर को इसकी पहचान करने की अपेक्षा करता हूं। परिणाम सुखद संक्षेप में है। (असल में, मैं कभी-कभी सी # कोड देखता हूं जो एक ही मुहावरे का उपयोग करता है, अनावश्यक रूप से - समानता ऑपरेटर सी # में तारों के साथ ठीक काम करता है।)

नीचे पंक्ति: पहले संस्करण का उपयोग करें, और इससे परिचित हो जाएं।

Answer 7

यह एक बहुत अच्छा सवाल है। मैं आमतौर पर सुंदर नहीं का उपयोग करें:

if (object.getAttribute("someAttr") != null && object.getAttribute("someAttr").equals("true")) { // .... 

(और मैं इसे अब और का उपयोग नहीं होगा)

Answer 8

मैं एक जवाब है,

List<Map<String, Object>> group = jjDatabase.separateRow(db.Select("SELECT * FROM access_user_group WHERE user_id=1 ;")); 

वहाँ, मेरे डेटाबेस में 'access_user_group' कॉलम के रूप में "group_c80" नहीं है तो में प्राप्त (0) .Get ("group_c80") नल पॉइंटर एक्सेप्शन समझौते। लेकिन मैं इसे नीचे दिए गए कोड के माध्यम से नियंत्रित:

for (int j = 1; j < 100; j++) { 
        String rulId="0";//defult value,to privent null pointer exeption in group_c 
        try { 
         rulId = group.get(0).get("group_c" + j)).toString(); 
        } catch (Exception ex) { 
         ServerLog.Print("Handeled error in database for " + "group_c" + (j < 10 ? "0" + j : j) +"This error handeled and mot efect in program"); 
         rulId = "0"; 
        }} 

Answer 9

यहाँ मेरी दृष्टिकोण है, हालांकि एक PropertyUtil वर्ग की जरूरत है, लेकिन इसके लिए केवल एक बार लिखा:

/** 
* Generic method to encapsulate type casting and preventing nullPointers. 
* 
* @param <T>   The Type expected from the result value. 
* @param o   The object to cast. 
* @param typedDefault The default value, should be of Type T. 
* 
* @return Type casted o, of default. 
*/ 
public static <T> T getOrDefault (Object o, T typedDefault) { 
    if (null == o) { 
     return typedDefault; 
    } 
    return (T) o; 
} 

क्लाइंट कोड कर सकते हैं:

PropertyUtil.getOrDefault(obj.getAttribute("someAttr"), "").equals("true"); 

या, सूची के लिए:

PropertyUtil.getOrDefault(
    genericObjectMap.get(MY_LIST_KEY), Collections.EMPTY_LIST 
).contains(element); 

,210

या सूची की एक उपभोक्ता के लिए, कि अस्वीकार होगा वस्तु:

consumeOnlyList(
    PropertyUtil.getOrDefault(
     enericObjectMap.get(MY_LIST_KEY), Collections.EMPTY_LIST 
    ) 
) 

डिफ़ॉल्ट अशक्त वस्तु पैटर्न https://en.wikipedia.org/wiki/Null_Object_pattern