@ $ array ['pros_missing_key'] विरोधी-पैटर्न है?

Question

क्या एक PHP सरणी से संभावित रूप से अनुपलब्ध मान निकालने पर @ का उपयोग करना ठीक है? उदाहरण:

$value = @$array['possibly_missing_key']; 

अभिप्रेत व्यवहार:

if (isset($array['possibly_missing_key'])) { 
    $value = $array['possibly_missing_key']; 
} else { 
    $value = null; 
} 

मैं जानना चाहता हूँ उपयोग पैटर्न फैलने से पहले।

Answer 1

@ ऑपरेटर त्रुटि संदेशों को दबाता है, और इसका उपयोग संभावित अन्य त्रुटियों और अप्रत्याशित व्यवहार के लिए आपका कोड सेट करता है जो ट्रैक करने के लिए कठिन होता है। इस प्रकार यह निश्चित रूप से एक antipattern है।

इस प्रकार, मैं दूसरी बिट को बहुत पसंद करूंगा। यह यह बहुत स्पष्ट

• कि यह सरणी में मौजूद नहीं हो सकता है, और
• क्या डिफ़ॉल्ट मान अगर यह मौजूद नहीं है

यह अधिक संक्षिप्त आप त्रिगुट उपयोग कर सकते हैं बनाने के लिए बनाता है सशर्त ऑपरेटर ?:, जैसा कि Mark Baker's answer में देखा गया है। थोड़ा कम कोड और अधिक प्रतीकों लेकिन अर्थ अच्छी तरह से पहचाना जाता है।

Answer 2

या

$value = (isset($array['possibly_missing_key'])) ? $array['possibly_missing_key']: null; 

Answer 3

चेतावनी की उपेक्षा कर निश्चित रूप से एक antipattern है; तो हाँ, यह एक विरोधी पैटर्न है (और मैं गारंटी दे सकता हूं कि यदि आप चेतावनियों को दबाने के लिए सीखते हैं, तो उनमें से एक वापस आ जाएगा और आपको बदतर में कटौती करेगा, अगर बदतर नहीं है)।

इसके अलावा, जबकि दूसरा संस्करण अधिक वर्बोज़ है, यह अनियमित वैरिएबल को एक ज्ञात स्थिति देता है (या समस्या को संभालने के लिए इस्तेमाल किया जा सकता है, यदि परिवर्तनीय भरना है)।

Answer 4

तीसरा विकल्प:

$value = (isset($array['key']) ? $array['key'] : null); 

मैं जानता हूँ कि यह सीधे सवाल का जवाब नहीं है; मैं इसे एक टिप्पणी के रूप में रखता, सिवाय इसके कि इसे वास्तव में प्रारूपित करने की आवश्यकता होती है।

यहां विचार यह है कि यदि आप एक और-लाइन के बजाय एक-लाइनर का उपयोग करके अपना कोड छोटा करने की कोशिश कर रहे हैं, तो आप अभी भी एक टर्नरी ऑपरेटर का उपयोग करके एक संक्षिप्त एक-लाइनर में प्राप्त कर सकते हैं, आपको दोनों दुनिया के सर्वश्रेष्ठ प्रदान करते हैं।

Answer 5

कोड का दूसरा ब्लॉक (या मार्क बेकर का विकल्प जो वास्तव में वही काम करेगा) बेहतर है। मैं PHP के बारे में पूरी तरह से निश्चित नहीं हूं, लेकिन कई अन्य प्रोग्रामिंग भाषाओं में, केवल एक चर को अनदेखा करने के लिए लगभग निश्चित रूप से एक त्रुटि फेंक देगी। कम से कम दूसरे ब्लॉक के साथ आप वैरिएबल को कुछ मान या स्मृति स्थान पर प्रारंभ कर रहे हैं।

यदि आप किसी फ़ंक्शन को अंत-उत्पाद में अपेक्षित त्रुटि को फेंकने की अपेक्षा करते हैं तो त्रुटि दमन का अधिक सामान्य उपयोग किया जाना चाहिए (हालांकि, अधिकांश समय यह मामला नहीं होगा)।

शुभकामनाएं!
डेनिस एम

Answer 6

दरअसल isset विविधता विरोधी पैटर्न है।यदि आप "त्रुटि" को दबाने के इरादे से isset($var)?$var:NULL का उपयोग करते हैं, तो आपने त्रुटियों को दबाने के लिए उचित वाक्यविन्यास का उपयोग करने पर कुछ भी हासिल नहीं किया है। इसका एक ही परिणाम है, फिर भी कम पठनीय है।

लोग "स्वच्छता" के कारण इसके लिए बहस कर रहे हैं और क्योंकि जारी करने का उपयोग एक माइक्रो अनुकूलन है। @ से बचें और सिंटैक्टिक नमक प्रतिस्थापन के रूप में जारी करने का उपयोग केवल कार्गो पंथ प्रोग्रामिंग है।