मैं कई कोडसेट चेक और दोहराव वाले PHP कोड को कैसे कम कर सकता हूं, लेकिन परिवर्तनीय पीढ़ी के तर्क?

Question

ग्रीटिंग्स। मैं एक कोड सेगमेंट को कम करने के लिए संघर्ष कर रहा हूं जो काफी लंबा दिखता है, जिससे मुझे इसकी अनिवार्यता के बारे में असुविधाजनक छोड़ दिया जाता है। यह फ़ॉर्म भरने के लिए उपयोग किए जाने वाले सत्र एरेज़ की एक बड़ी संख्या उत्पन्न करने के लिए एक फ़ंक्शन है, और यह मुझे तर्क सरणी में कुछ मानों के अस्तित्व की पुष्टि कर रहा है, जिसमें प्रत्येक व्यक्ति अनुरोधित सरणी उत्पन्न करने के लिए मामलों के साथ है। यहाँ जाता है:

function prepOptional($formData) { 

    $baseInfo=getBaseInfo(); 

    $_SESSION['fooData'] = (isset($formData['cbFoo']) ? prepBaseForm($baseInfo, 'foo', 
      'Option foo') : ''); 

    $_SESSION['opt1Data'] = (isset($formData['cbOpt1']) ? prepBaseForm($baseInfo, 'opt1', 
      'Option 1') : ''); 

    $_SESSION['barData'] = (isset($formData['cbBar']) ? prepBaseForm(prepOtherArray($formData), 'bar', 
      'Option bar', 'Optional argument') : ''); 

} 

समारोह तर्क के रूप में एक formdata सरणी स्वीकार करता है, और निहित मूल्यों यह सत्र में इसी सरणियों उत्पन्न करता है पर निर्भर करता है; यह मिलान करने वाले ध्वज के अस्तित्व की जांच के बाद ही होता है, इसलिए जारीकर्ता और टर्नरी ऑपरेटर।

बुलाया गया फ़ंक्शन, प्रीबेसबेसफॉर्म, पहले तर्क के लिए, एक फ़ाइल नाम और एक दोस्ताना नाम से संबंधित दूसरे और तीसरे तर्कों को जोड़ने से अलग करता है, जो एक सरणी है; यह एक वैकल्पिक चौथा पैरामीटर भी स्वीकार करता है, जो एक भरे हुए फॉर्म को जोड़ने के लिए एक फ़ाइल को इंगित करता है।

यदि मान सरणी में मौजूद नहीं है, तो पीढ़ी के आदेश को बरकरार रखने के लिए कोड जानबूझकर संबंधित सत्र चर को "'साफ़ करता है (इस अंत में: क्या ऐसा करने का एक बेहतर तरीका है, इस पर विचार करना वे परिदृश्य हैं जिनमें दिए गए पीढ़ी के आदेश को तोड़ा जाएगा?)।

वास्तविक कोड में इस प्रारूप के साथ लगभग बीस पीढ़ी के ब्लॉक हैं, और यह काफी लंबा हो रहा है; एक ऑप्टिमाइज़ेशन मैंने सोचा था कि दिए गए क्षेत्रों के माध्यम से लूप करना था और पीढ़ी की प्रक्रिया को सामान्य बनाना था, लेकिन जैसा कि आप देख सकते हैं, कई बार तर्क अलग-अलग होते हैं, या prepBaseForm फ़ंक्शन के इनपुट इनपुट को एक और तरीका उत्पन्न करना होता है। इस रिफैक्टरिंग से निपटने के तरीके पर कोई विचार? तुम्हारी प्रतिक्रिया के लिए अग्रिम धन्यवाद।

Answer 1

एक विकल्प डिफ़ॉल्ट प्रदान करना होगा, और सामान्य स्थितिओं का उपयोग करके मूल्यों को बदलते समय इसे चलाएं। फिर आप सबकुछ एक साधारण लूप में डालकर और साफ कर सकते हैं .:

function prepOptional($formData) { 
    $baseInfo=getBaseInfo(); 

    $options = array(
     'foo' => 'Option foo', 
     'opt1' => 'Option Opt1', 
     'bar' => 'Option Bar', 
    ); 
    foreach ($options as $name => $text) { 
     //Add defaults for formData and Session arrays) 
     $formData += array('cb' . ucfirst($name) => ''); 
     $_SESSION += array($name . 'Data' => ''); 

     if ($formData['cb' . ucfirst($name)]) { 
      $_SESSION[$name . 'Data'] = prepBaseForm(
       $baseInfo, 
       $name, 
       $text 
      ); 
     } 
    } 
}