संभावित रूप से अपरिभाषित चरों को संभालने के दौरान PHP में संदर्भ असाइनमेंट का उपयोग करना बुरा व्यवहार है?

Question

मैं संभावित रूप से अपरिभाषित चर से निपटने के लिए शॉर्टकट के रूप में संदर्भ असाइनमेंट का उपयोग करने के बारे में सोच रहा हूं।

दूसरे शब्दों में, के बजाय:

$foo = &$this->blah['something']['else']; 
if (!is_null($foo) && ...){ 
    //do something with $foo 
} 

, सरल लगता है सही:

$foo = isset($this->blah['something']['else']) ? $this->blah['something']['else'] : null; 
if (!is_null($foo) && ...){ 
    //do something with $foo 
} 

मैं यह कर सकता है? जिस तरह से PHP संदर्भ द्वारा असाइनमेंट को संभालता है, मुझे इस बारे में चिंता करने की ज़रूरत नहीं है-> ब्लाह ['कुछ'] ['else'] परिभाषित किया जा रहा है, क्योंकि यदि यह अस्तित्व में नहीं है तो यह स्वचालित रूप से बनाया जाएगा और सेट हो जाएगा नल के लिए।

क्या यह रणनीति इस पर भरोसा है?

Answer 1

तुम सिर्फ अगर एक चर सेट किया गया है और रिक्त नहीं परीक्षण करना चाहते हैं, तो आप बेहतर परीक्षण:

if (! empty($this->blah['something']['else'])) {...} 

इस तरह आप भी है कि नहीं रह सकता एक मूल्य के लिए दो संदर्भ बनाने से बचें।

मेरी राय में, हर बार जब आप एक ही मूल्य के लिए एक नया संदर्भ बनाते हैं, तो आपका कोड एक नज़र में समझना मुश्किल हो जाता है।

आप शून्य के रूप में गैर खाली की जरूरत है, तो आप बेहतर एक वैश्विक समारोह इस प्रकार बना:

function eempty(& $var) { 
    return empty($var) && $var !== 0; 
} 

Answer 2

मुझे व्यक्तिगत रूप से लगता है कि यह अधिक पारंपरिक हो सकता है, लेकिन इस तरह के एक निर्दिष्ट मूल्य को असाइन करना मेरे लिए थोड़ा अनावश्यक लगता है। मैं बस अपनी स्थिति में अधिक सामान्य रूप से इस्तेमाल किए जाने वाले टर्नरी ऑपरेटर के साथ रहूंगा। अक्सर यह समझने के लिए बहुत स्पष्ट है। बेशक, आपको जितना संभव हो सके अपने व्यक्तिगत अर्थशास्त्र का पालन करना चाहिए।