LINQ के अंदर मान निर्दिष्ट करना?

Question

मैं निम्नलिखित प्रश्न हैं:

drivers.Select(d => { d.id = 0; d.updated = DateTime.Now; return d; }).ToList(); 

ड्राइवरों एक सूची है जो विभिन्न पहचान-पत्र और अद्यतन मूल्यों के साथ में आता है, इसलिए मैं चयन में मूल्यों में परिवर्तन कर रहा हूँ, लेकिन यह करने के लिए उचित तरीका है। मैं पहले से ही पता है कि मैं ड्राइवरों के लिए ड्राइवरों फिर नियत नहीं कर रहा हूँ क्योंकि Resharper इसके बारे में शिकायत है, तो मुझे लगता है कि यह बेहतर होगा अगर यह था:

drivers = drivers.Select(d => { d.id = 0; d.updated = DateTime.Now; return d; }).ToList(); 

लेकिन यह अभी भी जिस तरह से किसी में प्रत्येक तत्व के लिए नए मान निर्दिष्ट करना चाहिए ड्राइवर सूची?

Answer 1

हालांकि यह निर्दोष दिखता है, विशेष रूप से ToList कॉल के साथ संयोजन में जो कोड तुरंत निष्पादित करता है, मैं निश्चित रूप से किसी प्रश्न के हिस्से के रूप में कुछ भी संशोधित करने से दूर रहूंगा: चाल इतनी असामान्य है कि यह आपके प्रोग्राम के पाठकों को यात्रा करेगी , यहां तक ​​कि अनुभवी वाले, खासकर यदि उन्होंने पहले कभी नहीं देखा।

foreach छोरों के साथ गलत कुछ भी नहीं है - सच है कि आप LINQ के साथ यह कर सकते हैं कि इसका मतलब यह नहीं है कि आप चाहिए यह कर रहा।

Answer 2

यह कभी नहीं करें। एक प्रश्न क्वेरी होना चाहिए; यह गैर-विनाशकारी डेटा स्रोत के प्रश्न पूछना चाहिए। यदि आप साइड इफेक्ट का कारण बनना चाहते हैं तो foreach लूप का उपयोग करें; यही वह है जो इसके लिए है। नौकरी के लिए सही उपकरण का उपयोग करें।

Answer 3

ठीक है, मैं खुद जवाब दूंगा।

Xaisoft, Linq प्रश्न, यह लैम्ब्डा अभिव्यक्ति या क्वेरी अभिव्यक्ति हो, सूची को उत्परिवर्तित करने के लिए उपयोग नहीं किया जाना चाहिए। इसलिए आपके Select

drivers = drivers.Select(d => { d.id = 0; d.updated = DateTime.Now; return d; }).ToList(); 

खराब शैली है। यह मानक/मानक नहीं है, मानक नहीं है, और Linq दर्शन के खिलाफ। अंतिम परिणाम को प्राप्त करने का एक और गरीब शैली है:

drivers.Any(d => { d.id = 0; d.updated = DateTime.Now; return false; }); 

लेकिन वह नहीं कहना ForEachList<T> पर अनुचित है। यह आपके जैसे मामलों में उपयोग पाता है, लेकिन Linq क्वेरी के साथ उत्परिवर्तन मिश्रण नहीं करता है। मैं कुछ लिखना पसंद करता हूं:

drivers.ForEach(d => d.updated = DateTime.Now); 

इसकी सुरुचिपूर्ण और समझदार। चूंकि यह Linq से निपटता नहीं है, यह भी भ्रमित नहीं है। मुझे लैम्बडा के अंदर कई कथन (आपके मामले में) के लिए उस वाक्यविन्यास को पसंद नहीं है। चीजें जटिल होने पर यह थोड़ा कम पठनीय और डीबग करने के लिए कठिन होता है। आपके मामले में मैं सीधे foreach लूप पसंद करता हूं।

foreach (var d in drivers) 
{ 
    d.id = 0; 
    d.updated = DateTime.Now; 
} 

व्यक्तिगत तौर पर मैं ForEachIEnumerable<T>as a terminating call to Linq expression पर की तरह (यानी, काम एक प्रश्न लेकिन एक निष्पादन होना नहीं है)।