if-statement में सशर्त ऑपरेटर?

Question

मैं जावा में निम्नलिखित if -statement लिखा है:

if(methodName.equals("set" + this.name) || 
    isBoolean() ? methodName.equals("is" + this.name) : 
        methodName.equals("get" + this.name)) { 
    ... 
} 

इस if में इस तरह के भाव लिखने के लिए, हालत से राज्य को अलग करने के एक अच्छा अभ्यास है? और क्या यह अभिव्यक्ति सरल हो सकती है?

Answer 1

मैं

if (methodName.equals("set" + this.name) 
|| methodName.equals((isBoolean() ? "is" : "get") + this.name)) { 
    ... 
} 

Answer 2

करने के लिए इसे बदल जाएगा यह अच्छा अभ्यास है? यह अच्छा है अगर यह पढ़ने में आसान बनाता है। यह पढ़ना आसान बनाता है अगर (1) यह करता है और (2) उस व्यक्ति की तरह जो इसे भ्रमित कर लेती है उसे पढ़ नहीं पाएगी। इसे पढ़ने वाला कौन है?

Answer 3

निम्नलिखित काम की तरह कुछ नहीं होगा?

if (methodName.equals("set" + this.name) 
    || methodName.equals("get" + this.name) 
    || (isBoolean() && methodName.equals("is" + this.name))) { 
    ... 
} 

यह तरीका है जिसमें आप त्रिगुट ऑपरेटर का इस्तेमाल किया और निश्चित रूप से समझने में अधिक आसान की तुलना में अधिक पठनीय है। यह भी लाभ है कि यह isBoolean विधि के लिए एक अनावश्यक विधि कॉल से बच सकता है (इसमें या तो 1, 2 या 4 विधि कॉल हैं जबकि आपके पास हमेशा 1 या 3 है; प्रदर्शन लाभ/हानि शायद नोटिस करने में बहुत कम है)।

इसके अलावा वहाँ एक समान प्रश्न यहाँ शीर्षक "Is this a reasonable use of the ternary operator?" एक उपयोगकर्ता the following to say था है:

त्रिगुट ऑपरेटर वापसी एक मूल्य के लिए है।

आईएमओ, इसे राज्य को म्यूटेट नहीं करना चाहिए, और वापसी मूल्य का उपयोग किया जाना चाहिए।

अन्य मामले में, यदि बयानों का उपयोग करें। यदि कथन कोड ब्लॉक्स निष्पादित करने के लिए हैं।

ध्यान दें कि मैं पठनीयता के लिए '& &' वाले अभिव्यक्ति के आसपास कोष्ठक शामिल थे। वे आवश्यक नहीं हैं क्योंकि x && y का मूल्यांकन m || n से पहले किया गया है।

चाहे आप इसका उपयोग करना चुनते हैं, आप पर निर्भर है, लेकिन मैं इसे पठनीयता के पक्ष में टालना चाहता हूं।

Answer 4

मैं निकाले कुछ कार्यों के साथ

if (methodName.equals(setterForThis()) 
    || methodName.equals(getterForThis())) { 
    ... 
} 

लिए इसे बदलने के लिए इच्छुक होगा:

private String setterForThis() { 
    return "set" + this.name; 
} 

private String getterForThis() { 
    return (isBoolean() ? "is" : "get") + this.name; 
} 

यह अब निश्चित रूप से है, लेकिन मैं वैसे भी गोल्फ में वास्तव में नहीं हूँ।