इस स्थिति में link का उपयोग कर रहा है overkill

Question

मैं फिर से कारक के बारे में सोच रहा हूं, लेकिन मुझे नहीं पता कि अंतिम परिणाम केवल ओवरकिल है या नहीं। वर्तमान में मैं

IList<myobjecttypebase> m_items; 

    public int GetIncrementTotal() 
    { 
    int incrementTot; 
    foreach(myobjecttypebase x in m_items) 
    { incrementTot += x.GetIncrement(); } 
    } 

है यह overkill और/या डाली यहाँ एक महत्वपूर्ण ओवरहेड होगा foreach

m_items.ToList().ForEach(x => incrementTot += x.GetIncrement()); 

के लिए LINQ का उपयोग करने के कम कुशल होगा?

Answer 1

ToList विधि LINQ के साथ प्रयोग किया एक विस्तार तरीका है, लेकिन foreach विधि सिर्फ एक तरीका है सूची वर्ग में।

यहां प्रमुख ओवरहेड टूलीस्ट विधि का आह्वान है, जो संग्रह से एक नई सूची बनाता है। फॉरएच में थोड़ी सी ओवरहेड भी है क्योंकि इसे प्रत्येक आइटम के लिए एक प्रतिनिधि को कॉल करना है।

आप LINQ तरीकों का उपयोग करना चाहते हैं, सकल विधि अधिक उपयुक्त लगता है:

public int GetIncrementTotal() { 
    return m_items.Aggregate(0, (t, i) => t + i.GetIncrement()); 
} 

या सम:

public int GetIncrementTotal() { 
    return m_items.Sum(i => i.GetIncrement()); 
} 

या तो, अपने मूल संस्करण पर एक मामूली भूमि के ऊपर है, इसलिए यदि आप चाहते हैं सबसे कुशल, बस एक साधारण पाश के लिए छड़ी।

Answer 2

ओवरहेड संग्रह को दो बार फिर से करने में होगा।

m_items.ToList() // First iteration to add to the list all the items 
    .ForEach(x => incrementTot += x.GetIncrement()); /* Second iteration to 
                 perform logic */ 

ToList सबसे LINQ बयान के रूप में आलसी तरीके से यात्रा प्रदर्शन नहीं करता, तो इसके यह संग्रह से अधिक कोड पुनरावृति दो बार बाध्य करेगा।

सामान्य रूप से LINQ फॉर्म पढ़ने के लिए अच्छा लगता है, लेकिन यदि आप प्रदर्शन के बारे में चिंता कर रहे हैं तो आप इससे बेहतर बचें।

Answer 3

क्यों SUM ऑपरेटर का उपयोग सीधे IList पर नहीं करें?

यह Func प्रतिनिधियों का उपयोग कर कई भार के है:

m_items.Sum(x => x.GetIncrement()); 

Answer 4

यह इस तरह से मत करो।
ToList() यहां एक नई सूची आवंटित और पॉप्युलेट करेगा, आईएमएचओ आवश्यक नहीं है। हालांकि, यह किसी भी ऑब्जेक्ट पर पुनरावृत्ति करने के लिए फॉरएच एक्सटेंशन विधि लिखने के लिए एक छोटा अभ्यास है, लेकिन मैं इसके खिलाफ सलाह दूंगा (एरिक लिपर्ट foreach V's ForEach द्वारा इस आलेख को देखें)।

मैं foreach के साथ जाना होगा।

पीएस आप incrementTot आरंभ करने के लिए (क्षमा करें, मैं अपने आप को मदद नहीं कर सकता है)

Answer 5

जरूरत Oded उल्लेख उपयोग योग

incrementTot = m_items.ToList().Sum(x => x.GetIncrement()); 

Answer 6

ForEach LINQ का हिस्सा नहीं है के रूप में - यह List<T> एपीआई का हिस्सा है और के बाद से किया गया है। नेट 2.0।

कारण यह LINQ का हिस्सा नहीं है क्योंकि यह स्वाभाविक रूप से साइड-इफेक्टिंग विधि है ... और LINQ साइड इफेक्ट्स को प्रोत्साहित नहीं करता है।

सही समाधान योग का प्रयोग किया जाता है यहाँ है, लेकिन आप पहली बार एक सूची बनाने की जरूरत नहीं है:

return m_items.Sum(x => x.GetIncrement());