नियंत्रकों में linq क्वेरी रखने के लिए यह अच्छा अभ्यास है?

Question

मैं एमवीसी पैटर्न से बहुत परिचित नहीं हूं। क्या आप मुझे बता सकते हैं कि निम्नलिखित तीन नियंत्रक कार्रवाइयों में से कौन सा बेहतर है? धन्यवाद :)

(1) कार्रवाई में क्वेरी है:

public ActionResult List() 
{ 
    var query = repository.Query().Where(it => it.IsHandled).OrderBy(it => it.Id); 
    // ... 
} 

(2) सेवा में क्वेरी है:

public ActionResult List() 
{ 
    var items = service.GetHandledItemsOrderById(); 
    // ... 
} 

(3) आदेश कार्रवाई में से है :

public ActionResult List() 
{ 
    var items = service.GetHandledItems().OrderBy(it => it.Id); 
    // ... 
} 

अगर हम (1) चुनते हैं, तो हमारे पास नियंत्रक में बहुत अधिक व्यवसाय तर्क है?

यदि हम (2) चुनते हैं, तो GetXXXByYYY() जैसी कई सेवा विधियां हो सकती हैं।

अगर हम चुनें (3), हम क्यों संपुटित Where(it => it.IsHandled) नहीं बल्कि
OrderBy(it => it.Id।

कोई विचार?

Answer 1

यह निर्भर करता है। :)

मेरी राय:

मैं अपने सेवा ढीला रखने के लिए, डुप्लिकेट कोड को कम करना चाहते हैं। मैं पाइप और फिल्टर का प्रशंसक भी हूं।

यहां मैं क्या करूंगा (और क्या करें)।

सेवा

public ICollection<Item> GetHandledItems<TKey>(OrderingOptions<Item, TKey> orderingOptions) 
{ 
    return repository 
     .Query() 
     .WhereHandled() 
     .WithOrdering(orderingOptions) 
     .ToList();  
} 

ItemFilters.cs

public static IQueryable<Item> WhereHandled(this IQueryable<Item> source) 
{ 
    return source.Where(it => it.IsHandled); 
} 

public static IOrderedQueryable<T> WithOrdering<T, TKey>(
    this IQueryable<T> source, 
    OrderingOptions<T, TKey> orderingOptions) 
{ 
    return orderingOptions.SortDescending 
     ? source.OrderByDescending(orderingOptions.OrderingKey) :              
     source.OrderBy(orderingOptions.OrderingKey); 
} 

OrderingOptions.cs

public class OrderingOptions<T,TKey> 
{ 
    public OrderingOptions(Expression<Func<T,TKey>> orderingKey, bool sortDescending = false) 
    { 
     OrderingKey = orderingKey; 
     SortDescending = sortDescending; 
    } 

    public Expression<Func<T,TKey>> OrderingKey { get; private set; } 
    public bool SortDescending { get; private set; } 
} 

इस तरह, यो यू नियंत्रक में आदेश निर्दिष्ट कर सकते हैं: ऊपर और 3 विकल्पों के बीच

var items = service.GetHandledItems(new OrderingOptions(it => it.Id)); 

अंतर:

• से ऊपर नियंत्रक पर लौटने से पहले अनुक्रम materializes। विकल्प 3 नहीं है, जो खतरनाक है (आप एमवीसी पैटर्न को देखने और तोड़ने के लिए लौटने वाली क्वेरी को समाप्त कर सकते हैं)।
• जेनेरिक "ऑर्डरिंग" पीओसीओ, कहीं भी इस्तेमाल किया जा सकता है और आपके प्रश्न डी-आर-वाई रखता है।
• सेवा गूंगा बन जाती है, और बस रिपोजिटरी और नियंत्रक (जो इसे करना चाहिए, आईएमओ) के बीच एक कमजोर हो जाता है। तर्क (उदाहरण फ़िल्टर) एक स्थान पर सारणीबद्ध।

HTH

Answer 2

मुझे यकीन है कि राय अलग-अलग हो सकती हैं, लेकिन मैंने सेवा में उतना ही व्यवसाय तर्क रखने की कोशिश की है जितनी आप कर सकते हैं। 3 मेरी पसंद होगी। 1 के साथ, आप पहले ही इस मुद्दे को देख चुके हैं। 2 के साथ, आप एक सेवा में प्रदर्शन प्राथमिकता पेश कर रहे हैं। 3 के साथ, आप आवश्यक वरीयताओं को प्रदर्शित करते हैं। यदि आप अपनी व्यावसायिक परत में एक और इंटरफ़ेस पेश करना चाहते हैं, तो आपको संभावित रूप से, अनावश्यक कोड पुनरावृत्तियों की आवश्यकता होती है 2.