जटिल व्यवसाय "आईएफ" तर्क को सरल कैसे करें?

Question

जटिल व्यापार तर्क को संभालने के अच्छे तरीके क्या हैं कि पहली नज़र से बयान में कई घोंसले की आवश्यकता होती है?

उदाहरण:

डिस्काउंट कूपन। हो सकता है:

1 क) मूल्य छूट
1b) का प्रतिशत छूट

2 ए) सामान्य छूट
2b) प्रगतिशील छूट

3a) आवश्यक पहुँच कूपन
3 बी) का उपयोग कूपन

की आवश्यकता नहीं है

4 ए) केवल उस ग्राहक को लागू किया गया जो पहले से ही
4b से पहले खरीदा गया है) किसी भी ग्राहक को लागू

5a) केवल उन्हीं देशों (पहला, दूसरा, से ग्राहकों के लिए लागू होता है ...)

कोड की आवश्यकता है यही कारण है कि और भी जटिल तो यह:

if (discount.isPercentage) { 
    if (discount.isNormal) { 
     if (discount.requiresAccessCoupon) { 
     } else { 
     } 
    } else if (discount.isProgressive) { 
     if (discount.requiresAccessCoupon) { 
     } else { 
     } 
    } 
} else if (discount.isValue) { 
    if (discount.isNormal) { 
     if (discount.requiresAccessCoupon) { 
     } else { 
     } 
    } else if (discount.isProgressive) { 
     if (discount.requiresAccessCoupon) { 
     } else { 
     } 
    } 
} else if (discount.isXXX) { 
    if (discount.isNormal) { 
    } else if (discount.isProgressive) { 
    } 
} 

भले ही आप आईएफएस की जगह स्विच करने के लिए/मामला यह है अभी भी बहुत जटिल है। इसे पठनीय, रखरखाव करने योग्य, अधिक परीक्षण करने योग्य और समझने में आसान बनाने के तरीके क्या हैं?

Answer 1

मैं एक सामान्य राज्य मशीन है कि चीजों की तुलना करने के लिए की सूची पर फ़ीड लिखेंगे।

Answer 2

Specification pattern जो भी आप खोज रहे हैं हो सकता है।

सारांश:

कंप्यूटर प्रोग्रामिंग में, विनिर्देश पैटर्न, एक विशेष सॉफ्टवेयर डिज़ाइन पैटर्न है जिससे व्यापार तर्क एक साथ व्यापार तर्क चेनिंग बूलियन तर्क का उपयोग करके पुनर्संयोजन जा सकता है।

Answer 3

कर की वस्तु उन्मुख जिस तरह से यह एक आम इंटरफेस को लागू करने के लिए कई छूट वर्गों है:

dicsount.apply(order) 

का निर्धारण करने के क्रम छूट वर्गों के भीतर छूट के लिए उत्तीर्ण के लिए तर्क रखो।

Answer 4

मेरा पहला विचार यह है कि यह टेस्टेबल नहीं है, जो मुझे परीक्षण करने योग्य बनाने के लिए एक समाधान के लिए प्रेरित करता है।

if (discount.isPercentage) { 
    callFunctionOne(...); 
} else if (discount.isValue) { 
    callFunctionThree(...); 
} else if (discount.isXXX) { 
    callFunctionTwo(...); 
} 

फिर यदि आप एक अलग कॉल हो तो प्रत्येक नेस्टेड हो सकता है। इस तरह आप उन्हें अलग-अलग परीक्षण कर सकते हैं और जब आप बड़े समूह का परीक्षण करते हैं तो आप जानते हैं कि प्रत्येक व्यक्ति काम करता है।

Answer 5

किसी विशेष मामले की जांच करने वाली विधियां बनाएं।

bool IsValueNormalAndRequiresCoopon (डिस्काउंट छूट) {...}

bool IsValueNormalAndRequiresCoupon (डिस्काउंट छूट) {...}

आदि

एक बार जब आप कर रहे हैं कि यह देखने के लिए आसान हो जाता है शुरू जहां आप कर सकते हैं विकल्पों के बीच बाहर सार आम तर्क। फिर आप वहां से जा सकते हैं।

जटिल निर्णयों के लिए मैं अक्सर एक ऐसे वर्ग के साथ समाप्त होता हूं जो संभावित राज्यों को संभालता है।

Answer 6

guard clauses का उपयोग करने से कुछ मदद मिल सकती है।

Answer 7

अच्छा सवाल। "सशर्त जटिलता" एक कोड गंध है। Polymorphism आपका मित्र है।

सशर्त तर्क, अपनी प्रारंभिक अवस्था में निर्दोष है जब यह समझते हैं और कोड का एक कुछ लाइनों के भीतर समाहित करने के लिए आसान है। दुर्भाग्यवश, यह शायद ही कभी उम्र बढ़ता है। आप कई नई सुविधाओं को लागू करते हैं और अचानक आपका सशर्त तर्क जटिल और विशाल हो जाता है। [यहोशू Kerevsky: पैटर्न को Refactoring]

सरल चीजें आप नेस्टेड से बचने के लिए अगर ब्लॉक Guard Clauses उपयोग करने के लिए सीखने के लिए है क्या कर सकते हैं में से एक।

double getPayAmount() { 
if (_isDead) return deadAmount(); 
if (_isSeparated) return separatedAmount(); 
if (_isRetired) return retiredAmount(); 
return normalPayAmount(); 
}; 

दूसरी बात मैं पाया है बहुत अच्छी तरह से चीजों को सरल है, और जो अपने कोड आत्म दस्तावेज़ीकृत कर बनाता है, Consolidating conditionals है।

double disabilityAmount() { 
    if (isNotEligableForDisability()) return 0; 
    // compute the disability amount 

अन्य मूल्यवान refactoring सशर्त भाव के साथ जुड़े तकनीक Decompose Conditional, Replace Conditional with Visitor, और Reverse Conditional शामिल हैं।

Answer 8

एफडब्ल्यूआईडब्ल्यू, मैंने Hamcrest का उपयोग इस तरह की चीज के लिए बहुत सफलतापूर्वक किया है। मेरा मानना ​​है कि आप कह सकते हैं कि यह विशिष्टता पैटर्न लागू करता है, @ अरनिस ने बात की।

Answer 9

आप वास्तव में Misko Hevery

Google प्रौद्योगिकी वार्ता 20 नवंबर तक

Clean Code Talks - Inheritance, Polymorphism, & Testing
देखना चाहिए, 2008

अमूर्त

अपने कोड बयान करता है, तो से भरा है? कथन स्विच करें? क्या आपके पास विभिन्न स्थानों पर एक ही स्विच स्टेटमेंट है? जब आप परिवर्तन करते हैं तो क्या आप खुद को एक ही बदलाव कर पाते हैं यदि कई स्थानों पर स्विच किया जाता है? क्या आपने कभी एक भूल लिया था?

यह बात उन सशर्तों को हटाने के लिए ऑब्जेक्ट ओरिएंटेड तकनीकों का उपयोग करने के दृष्टिकोणों पर चर्चा करेगी। परिणाम क्लीनर, कड़ा, बेहतर डिज़ाइन किया गया कोड है जो परीक्षण, समझने और बनाए रखने में आसान है।