यहां तक ​​कि "IsNullOrEmpty" चेक भी "IENumerable की संभावित एकाधिक गणना" चेतावनियां

Question

पहले से ही question on SO about "possible multiple enumerations" है, लेकिन यह प्रश्न अधिक विशिष्ट है।

निम्न विधि है, जो इनपुट के रूप में एक IEnumerable<string> लेता है और उसके तत्वों में से प्रत्येक के खिलाफ दिए गए विधि कार्यान्वित करने पर विचार करें:

public static bool SomeMethod(IEnumerable<string> enumerable) 
{ 
    if (enumerable.IsNullOrEmpty()) 
    { 
     // throw exception. 
    } 
    else 
    { 
     return (enumerable.All(SomeBooleanMethod)); 
    } 
} 

उपरोक्त कोड में, IsNullOrEmpty सिर्फ एक विस्तार विधि है जो

चलाता है

return (!ReferenceEquals(enumerable, null) || enumerable.Any()); 

समस्या यह है कि ReSharper मुझे "आईनेमेरेबल के संभावित एकाधिक अंकन" के बारे में चेतावनी दे रहा है, और मुझे वास्तव में यह नहीं पता कि यह वास्तव में एक समस्या हो सकती है या नहीं।

मैं चेतावनी का अर्थ समझता हूं, लेकिन अगर आपको वास्तव में शून्यता या खालीता के मामले में अपवाद की जांच करने और अपवाद की आवश्यकता है तो आप वास्तव में क्या कर सकते हैं?

Answer 1

इसका मतलब है कि आप (आंशिक रूप से) एक से अधिक बार IENumerable पर पुनरावृत्त कर रहे हैं: पहले Any() पर कॉल करने के लिए (जिसे कम से कम एक पुनरावृत्ति शुरू करने की आवश्यकता है यह देखने के लिए कि कोई संख्या किसी तत्व को लौटाती है), और दूसरी बार All (जो शुरुआत से ही पुनरावृत्त होता है)।

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

Answer 2

@tdammers की पहचान के रूप में, "एकाधिक गणना" को संदर्भित किया गया है जो Any और All द्वारा आवश्यक दो गणनाएं हैं। चूंकि आप एक खाली अनुक्रम को अस्वीकार करना चाहते हैं, सबसे अच्छा मैं इसके साथ आ सकता हूं:

public static bool SomeMethod(IEnumerable<string> enumerable) 
{ 
    if (enumerable == null) 
     throw new ArgumentNullException(); 

    // Manually perform an All, keeping track of if there are any elements 
    bool anyElements = false; 

    bool result = true; 

    foreach (string item in enumerable) 
    { 
     anyElements = true; 
     result = result && SomeBooleanMethod(item); 

     // Can short-circuit here 
     if (!result) 
      break; 
    } 

    if (!anyElements) 
     throw new ArgumentException(); // Empty sequence is invalid argument 

    return result; 
}