क्या इस जावा कोड को लिखने के लिए कोई और संक्षिप्त तरीका है?

Question

यह foo जो lookup द्वारा वापस किया गया है null हो सकता है।

है कारण है कि मैं पहले null अगर foo प्रत्यावर्तित करके null मूल्य पर foo.getFooStr() बुला से बचने के लिए कोशिश कर रहा हूँ कि null है।

लेकिन क्या यह लिखने के लिए एक बेहतर (अधिक संक्षेप) तरीका है?

public static String getFooStr(String input) 
{ 
    Foo foo = lookup(input); 
    if(foo==null) 
    { 
     return null; 
    } 
    return foo.getFooStr(); 
} 

Answer 1

क्यों दिया गया lookup कि उचित foo स्ट्रिंग रिटर्न नहीं है?

Answer 2

आप दो सवाल है: वहाँ एक बेहतर तरह से कोड लिखने के लिए है, और वहाँ एक अधिक संक्षिप्त तरह से कोड लिखने के लिए है।

के बारे में अधिक संक्षिप्त, इस काम कर सकता था:

public static String getFooStr(String input) { 
    Foo foo = lookup(input);   
    return foo == null ? null : foo.getFooStr(); 
} 

बेहतर के बारे में: मैं किसी भी दिन संक्षिप्तता से अधिक पठनीयता महत्व देते हैं और एक व्यापक अंतर से। आपका मूल कोड मुझे ठीक लग रहा है। क्या मायने रखता है जो आपके लिए अच्छा लग रहा है, और अब से 3 महीनों को समझने और डीबग करना आपके लिए आसान है। मैंने सुना है कि किसी ने इसे सर्वश्रेष्ठ कहा है - अपना कोड लिखें ताकि दूसरों द्वारा आसानी से समझ सकें, और इससे भी महत्वपूर्ण बात यह है कि आपके भविष्य के स्वयं से।

Answer 3

मैं जावा में नहीं हूं, लेकिन मुझे साफ कोड पसंद है ... स्रोत कोड मनुष्यों के लिए पढ़ने और समझने में आसान होना चाहिए - मशीन परवाह नहीं है कि यह कैसा दिखता है लेकिन आपके सहयोगी क्या करते हैं। अधिक संक्षिप्त कोड आमतौर पर समझने के लिए एक पल या दो लंबा होता है (कभी-कभी मात्रा और जटिलता के आधार पर बहुत अधिक लंबी अवधि)। कोड को समझने योग्य रखें और यह रखरखाव योग्य होगा (भले ही यह थोड़ा और वर्बोज़ हो)!

Answer 4

Groovy does it nicer।

return lookup(input)?.fooStr

or even just:

lookup(input)?.fooStr

Answer 5

जावा 7, कुछ बिंदु पर यह योजना बनाई गई थी कि आप बस लिख सकते के लिए:

public static String getFooStr(String input) 
{ 
    Foo foo = lookup(input); 
    return foo?.getFooStr(); 
} 

लेकिन जब तक इस सुविधा का व्यापक रूप से है ज्ञात है, आपको ?: ऑपरेटर से चिपके रहना होगा।

Answer 6

मुझे किसी भी कोड में कहीं भी एकाधिक रिटर्न पसंद नहीं है। मैं बस से मछली के @Hovercraft पूर्ण मेरी राय में अच्छा लेकिन कम पढ़ी जा सकती है, लेकिन अभी भी संस्करण करने का एक आम तरीका

public static String getFooStr(String input) 
{ 
    Foo foo = lookup(input); 
    String fooString; 
    if(foo!=null) 
    { 
     fooString = foo.getFooStr(); 
    } 
    return fooString; 
} 

इसके अलावा करने के लिए इसे बदल जाएगा।