बेस्ट तरीके प्रश्नों

Question

इससे पहले कि आप/वोट करने के लिए करीब इस सवाल पर ध्यान न दें, मैं इस एक वैध प्रश्न पर विचार पूछने के लिए क्योंकि कोड स्पष्टता चर्चा का एक महत्वपूर्ण विषय है, यह पोषणीय कोड लिखने के लिए आवश्यक है और मैं उन लोगों के उत्तरों की बहुत सराहना करता हूं जो इससे पहले आए हैं।

मैं हाल ही में इस समस्या हुई, LINQ प्रश्नों घोंसले की बड़ी राशि की वजह से बहुत बुरा असली जल्दी मिल सकता है।

नीचे है कि मैं के साथ

कोई स्वरूपण (समान अपेक्षाकृत गैर जटिल क्वेरी के लिए)

var allInventory = system.InventorySources.Select(src => new { Inventory = src.Value.GetInventory(product.OriginalProductId, true), Region = src.Value.Region }).GroupBy(i => i.Region, i => i.Inventory); 

ऊंचा प्रारूपण

आए हैं स्वरूपण में मतभेद के कुछ उदाहरण हैं

var allInventory = system.InventorySources 
    .Select(src => 
     new { 
      Inventory = src.Value.GetInventory(product.OriginalProductId, true), 
      Region = src.Value.Region }) 
       .GroupBy(
        i => i.Region, 
        i => i.Inventory); 

ब्लॉक स्वरूपण

var allInventory = system.InventorySources 
    .Select(
     src => new 
     { 
      Inventory = src.Value.GetInventory(product.OriginalProductId, true), 
      Region = src.Value.Region 
     }) 
     .GroupBy(
      i => i.Region, 
      i => i.Inventory 
     ); 

सूची

var allInventory = system.InventorySources 
    .Select(src => new { Inventory = src.Value.GetInventory(product.OriginalProductId, true), Region = src.Value.Region }) 
    .GroupBy(i => i.Region, i => i.Inventory); 

का प्रारूपण मैं इतना है कि यह पठनीयता & समझ अधिकतम और स्वच्छ और पेशेवर लग रहा है LINQ स्वरूपण के लिए एक मानक के साथ आने के लिए चाहते हैं। अब तक मैं फैसला नहीं कर सकता इसलिए मैं यहां पेशेवरों को प्रश्न बदल देता हूं।

Answer 1

मैं ब्लॉक स्वरूपण पर बस गया हूं। इसने थोड़ी देर के लिए "बर्बाद जगह" की भावना को परेशान किया लेकिन अंत में सभी को लगा कि यह अधिक लोगों द्वारा अधिक पठनीय था। चूंकि हम पहले से ही नई लाइनों पर ब्रेसिज़ लगा रहे थे, यह बाकी कोड के साथ बेहतर फिट बैठता है। व्याख्या के लिए भी कम जगह है। हम सार्वजनिक स्टोर में एक सीएस फ़ाइल रखते हैं जिसमें उदाहरण स्वरूपण होते हैं ... जब कोई लिनक के अद्वितीय हंक के साथ आता है तो हम इसे फ़ाइल में जोड़ते हैं ... वास्तव में नए लोगों की मदद करता है।

Answer 2

इसका बहुत ही व्यक्तिपरक है।

मैं ब्लॉक स्वरूपण विधि का उपयोग करता हूं।

मैं स्टाइलकॉप के खिलाफ कोड भी जांचता हूं और यह सुनिश्चित करता हूं कि यह कोई स्टाइलकॉप चेतावनी नहीं देता है।

Answer 3

मेरे लिए, यह मुझे पूछे जाने वाले प्रश्न की लंबाई पर निर्भर करता है। बुनियादी चयन या सरल जॉइन जैसे छोटे सरल बयान के लिए मैं लिस्ट फॉर्मेटिंग के साथ जाऊंगा क्योंकि यह मेरे कोड को लाइनों के भार के बिना बिना पढ़ने और पढ़ने में आसान बनाता है।

यदि मेरे पास एक जटिल या बड़ा लिनक कथन है तो मैं ब्लॉक स्वरूपण के साथ जाता हूं ताकि यह अन्य लोगों को पढ़ने और अनुसरण करने का प्रयास कर सके जो मैं करने की कोशिश कर रहा था।

मुझे नहीं लगता कि यह अलग-अलग वक्तव्यों के लिए अलग-अलग स्वरूपण करने का बुरा अभ्यास है जब तक कि आप इसके साथ कैसे संपर्क करते हैं।

Answer 4

मेरे प्रारूपण:

var allInventory = system.InventorySources 
    .Select(src => new 
    { 
    Inventory = src.Value.GetInventory(product.OriginalProductId, true), 
    Region = src.Value.Region 
    }) 
    .GroupBy(
    i => i.Region, 
    i => i.Inventory 
); 

नोट्स:

• उद्घाटन तरीकों पर कोष्ठक एक नई लाइन के योग्य नहीं हैं।
• समापन माता-पिता उस लाइन के इंडेंटिंग से मेल खाते हैं जिसमें शुरुआती माता-पिता शामिल हैं।
• src => नया चयन के समान लाइन पर रहता है, क्योंकि यह सिर्फ एक नई लाइन के योग्य नहीं है।
• अज्ञात प्रकार हमेशा ब्लॉक उपचार प्राप्त करता है, जैसे कि यह किसी क्वेरी के बाहर उपयोग किया जाता था (लेकिन समापन माता-पिता एक नई रेखा के योग्य नहीं है)।
• दो पैरामीटर GroupBy अधिभार को आम तौर पर नहीं कहा जाता है। यद्यपि यह एक पंक्ति पर आसानी से फिट हो सकता है, यह स्पष्ट करने के लिए एक अतिरिक्त रेखा का उपयोग करें कि कुछ असामान्य हो रहा है।