शर्तों में असाइनमेंट क्यों खराब हैं?

Question

मैं PHP 6.5 के लिए नेटबीन का उपयोग कर रहा हूं।

if (($row = $db->get_row($sql))) { 
     return $row->folder; 
    } else { 
     return FALSE; 
    } 

Netbeans मुझसे कहता है कि मैं अगर बयान में कार्य नहीं का उपयोग करना चाहिए:

मेरी कोड में मैं अक्सर आदेश निम्न प्रकार का उपयोग करें।

क्यों? , हालांकि मैं एक वातावरण बहुत चालाक है कि के बारे में चेतावनी देने के लिए, यह भी बहुत चालाक "के लिए ओह होने की अपेक्षा करेंगे

if(a = b) 
    //logic error 

:

Answer 1

वे खराब नहीं हैं, लेकिन वे खतरनाक गलतियों का कारण बन सकते हैं।

भाषाओं, जहां एक काम एक अभिव्यक्ति है की तरह ग में, (उदाहरण के लिए एक के लिए समर्थन करने के लिए = b = c = 1;) एक सामान्य त्रुटि है:

if (a = 1) { .. } 

लेकिन आप

करना चाहते थे

if (a == 1) { .. } 

कुछ डेवलपर्स टाइप करने के लिए अगर एक '=' भुला दिया जाता है

if (1 == a) { .. } 

एक त्रुटि बनाने के लिए सीख लिया है। लेकिन मुझे लगता है कि यह पठनीयता में सुधार नहीं करता है। यदि आप लिखना

if (a = 1) { .. } 

जो मुझे लगता है कि एक बेहतर समाधान है

हालांकि आधुनिक compilers, एक चेतावनी दे। उस स्थिति में आपको यह जांचने के लिए मजबूर होना पड़ता है कि क्या वास्तव में इसका मतलब था।

Answer 2

यह शायद मदद से आप खतरनाक टाइपो से बचने की कोशिश कर रहा है उस मामले के बारे में चिंता मत करो "शर्तें।

Answer 3

मैं उन्हें हर समय का उपयोग करें, छोरों के साथ (यकीन नहीं क्यों कि एक फर्क होता है), जैसे:

$counter = 0; 
while($getWhateverDataObj = mysql_fetch_object($sqlResult)) { 
    $getWhateverObj->firstName[$counter] = $getWhateverDataObj->firstName; 
    $getWhateverObj->lastName[$counter] = $getWhateverDataObj->lastName; 
    $counter++; 
} 

और यह ठीक काम करता है।

Answer 4

भाषाओं कि allways कार्य पर कोई मान में यह नहीं है बुरा (मुझे लगता है कि यह कार्यात्मक भाषाओं में बहुत आम है), लेकिन (जबकि मैं इस टाइप किया के रूप में दूसरों allready कहा है) यह आम तौर पर आप या कोई बाद से परहेज किया जाना चाहिए और तुलना के लिए इसे गलती हो सकती है। कंपाइलर को आमतौर पर इसके बारे में चेतावनी दी जानी चाहिए, लेकिन अगर आप सुनिश्चित हैं कि आप क्या कर रहे हैं तो इसे अनदेखा किया जा सकता है ...

Answer 5

कंडीशनर में अक्सर शॉर्ट सर्किट ऑपरेटर शामिल होते हैं।

if (a=func(x) && b=func(y)) 
{ 
    // do this 
} 

यह तुरंत स्पष्ट नहीं हो सकता है, लेकिन अगर पहले >0 लौटे दूसरा काम ही घटित होता है, और अगर func(y) अन्य दुष्प्रभाव है कि आप उम्मीद कर रहे थे था, वे या तो नहीं होगा: तो, यह उदाहरण दिया ।

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

साथ ही, भविष्य देखरेख आप निम्नलिखित इरादा लगता है हो सकता है:

if (a==func(x) && b==func(y)) ... 

अगर वे "ठीक" अपने कोड, वे वास्तव में इसे तोड़ने।

Answer 6

यदि कोड लूप स्थिति में $ पंक्ति मान निर्दिष्ट नहीं करता है तो यह कैसा दिखता है, यह मुझे लगता है कि यह बहुत जटिल होगा ... हालांकि कुछ रखरखाव के लिए पढ़ने के लिए अच्छा नहीं है, नहीं? अच्छी तरह से आप इसे पसंद

$next = mysql_fetch_assoc($result) 
do{ 
... 
... 
... 

$next = mysql_fetch_assoc($result) or break; 
}while ($next)