ReSharper चेतावनी - संशोधित बंद

Question

तक पहुंच मैं निम्नलिखित कोड है:

string acctStatus = account.AccountStatus.ToString(); 
if (!SettableStatuses().Any(status => status == acctStatus)) 
    acctStatus = ACCOUNTSTATUS.Pending.ToString(); 

ध्यान दें कि account.AccountStatus प्रकार ACCOUNTSTATUS का एक enum है। दूसरी पंक्ति पर, रेशेपर मुझे एक्टस्टैटस के लिए "संशोधित बंद करने की पहुंच" चेतावनी दे रहा है।

string acctStatus = realAccount.AccountStatus.ToString(); 
string s = acctStatus; 
if (!SettableStatuses().Any(status => status == s)) 
    acctStatus = ACCOUNTSTATUS.Pending.ToString(); 

क्यों अच्छा है या क्या मैं मूल रूप से किया था बेहतर है: जब मैं सिफारिश संचालन करते हैं, स्थानीय चर को कॉपी, यह निम्न करने के लिए कोड को संशोधित करता है?

string[] acctStatus = {realAccount.AccountStatus.ToString()}; 
if (!SettableStatuses().Any(status => status == acctStatus[0])) 
    acctStatus[0] = ACCOUNTSTATUS.Pending.ToString(); 

यह मेरे लिए पूरी तरह निराला लगता है:

संपादित

यह भी सरणी में लपेटें स्थानीय चर, जो पैदा करता है की सिफारिश की।

Answer 1

चेतावनी का कारण यह है कि एक लूप के अंदर आप बदल रहे एक चर का उपयोग कर सकते हैं। हालांकि, इस गैर-लूप संदर्भ में "फिक्स" वास्तव में आपके लिए कुछ भी नहीं कर रहा है।

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

आप नहीं चाहते क्या का एक उदाहरण:

string acctStatus 

foreach(...) 
{ 
    acctStatus = account.AccountStatus[...].ToString(); 
    if (!SettableStatuses().Any(status => status == acctStatus)) 
     acctStatus = ACCOUNTSTATUS.Pending.ToString(); 
} 

समस्या यह है कि बंद acctStatus के लिए एक संदर्भ हड़पने जाएगा, लेकिन प्रत्येक पाश यात्रा है कि मूल्य बदल जाएगा। (पाश के लिए)

foreach(...) 
{ 
    string acctStatus = account.AccountStatus[...].ToString(); 
    if (!SettableStatuses().Any(status => status == acctStatus)) 
     acctStatus = ACCOUNTSTATUS.Pending.ToString(); 
} 

चर के संदर्भ के रूप में, पाश है एक नया उदाहरण हर बार बनाया जाएगा क्योंकि हम स्थानीय संदर्भ के अंदर चर स्थानांतरित कर दिया: में मामले में यह बेहतर होगा।

सिफारिश उस कोड के Resharper के विश्लेषण में एक बग की तरह लगता है। हालांकि, कई मामलों में यह एक वैध चिंता है (जैसे पहला उदाहरण, जहां बंद होने पर इसके कब्जे के बावजूद संदर्भ बदल रहा है)।

अंगूठे का मेरा नियम है, जब संदेह में स्थानीय बनाते हैं।

 menu.MenuItems.Clear(); 
     HistoryItem[] crumbs = policyTree.Crumbs.GetCrumbs(nodeType); 

     for (int i = crumbs.Length - 1; i > -1; i--) //Run through items backwards. 
     { 
      HistoryItem crumb = crumbs[i]; 
      NodeType type = nodeType; //Local to capture type. 
      MenuItem menuItem = new MenuItem(crumb.MenuText); 
      menuItem.Click += (s, e) => NavigateToRecord(crumb.ItemGuid, type); 
      menu.MenuItems.Add(menuItem); 
     } 

ध्यान दें कि मैं NodeType प्रकार स्थानीय कब्जा, ध्यान दें nodeType, और HistoryItem crumb.ItemGuid, टुकड़ों [i] .ItemGuid नहीं:

यहाँ एक वास्तविक दुनिया उदाहरण मैं ने काट लिया था है। यह सुनिश्चित करता है कि मेरे बंद होने वाले आइटमों में संदर्भ नहीं होंगे जो बदलेगा।

स्थानीय लोगों का उपयोग करने से पहले, घटनाएं मौजूदा मूल्यों के साथ ट्रिगर होंगी, न कि कैप्चर किए गए मूल्यों की अपेक्षा।