Linq最有效的方法

Question

我想知道下面是否有更有效的方法來完成我需要做的事情：

    return HttpContext.Current.User.IsInRole("Admin") 
    ? dbContext.Prog_Search1(searchField, searchString, searchOper).Where(a => a.Id == 2)      
    : dbContext.Prog_Search1(searchField, searchString, searchOper).Where(a => a.Id == 1)

我想知道是否只有一種聰明的使用方式

     dbContext.Search1(searchField, searchString, searchOper)

與我正在使用的2，然后有條件地執行Where子句？

Answer 1

是的，聽起來您想要這樣的東西：

int targetId = HttpContext.Current.User.IsInRole("Admin") ? 2 : 1;
return dbContext.Prog_Search1(searchField, searchString, searchOper)
                .Where(a => a.Id == targetId);

請注意，這並沒有真正改變效率 ，但確實會影響可讀性。

Answer 2

我想你想要這樣的東西：

var id = HttpContext.Current.User.IsInRole("Admin") ? 2 : 1
return dbContext.Prog_Search1(searchField, searchString, searchOper)
                .Where(a => a.Id == id);

Answer 3

或者：

var srch = dbContext.Prog_Search1(searchField, searchString, searchOper);
return HttpContext.Current.User.IsInRole("Admin") 
    ? srch.Where(a => a.Id == 2)      
    : srch.Where(a => a.Id == 1);

在您的特定實例中，它的整潔性不只是預先設置i=1 or 2 ，而是通常應該在兩個不同的Where命令之間的差異更大時更加靈活。