Условное добавление еще одного условия в оператор LINQ

Question

Условное добавление еще одного условия в оператор LINQ

1

У меня есть что-то вроде этого:

public Assignments GetAssignmentsForProider(string provider_k, string recordType)
{
    if (recordType == "A")
    {
       var query = from ea in this.Context.Assignments
            where ea.Provider_K == provider_k
                  && ea.Active == true
                  && ea.RecordType == "A"
            select ea;
        return query.FirstOrDefault();
    }
    else if (recordType == "E")
    {
       var query = from ea in this.Context.Assignments
            where ea.Provider_K == provider_k
                  && ea.RecordType == "E"
            select ea;
        return query.FirstOrDefault();
    }
}

Две проблемы:

Он не компилируется! потому что ему также нужен оператор return по умолчанию, и я не уверен, что вернуть действительно, но в моей программе никогда не произойдет, что ни одно из этих условий не произойдет. Я всегда в первом или втором.
На самом деле все они разные, в первом мы проверяем Active == true а во втором - нет. Так есть лучший способ написать это тоже?

Falgantil 20 май 2014, в 15:00

Источник

Теги:

c#

linq-to-entities

3 ответа

1

Один из способов связать ваши условия...

public Assignments GetAssignmentsForProider(string provider_k, string recordType)
{
    var query = from ea in this.Context.Assignments
                where ea.Provider_K == provider_k
                select ea;

    if (recordType == "A")
    {
       query = from q in query
               where q.Active == true
                  && q.RecordType == "A"
               select q;
    }
    else if (recordType == "E")
    {
       query = from q in query
               where q.RecordType == "E"
               select q;    
    }

    return query;
}

Но у Bjarke Søgaard есть лучший метод фильтрации по типу записи, потому что ваш критерий параметров напрямую соответствует данным запроса... поэтому вы можете просто использовать его напрямую.

Mashton 20 май 2014, в 12:28

1

Я бы, наверное, написал это так:

public Assignments GetAssignmentsForProider(string provider_k, string recordType)
{
    var query = from ea in this.Context.Assignments
                where ea.Provider_K == provider_k &&
                      ea.RecordType == recordType
                select ea;

    if (recordType == "A")
        return query.Where(ea => ea.Active).FirstOrDefault();
    else if (recordType == "E")
        return query.FirstOrDefault();

    return null;
}

Magnus 20 май 2014, в 09:45

Ещё вопросы

Falgantil · Accepted Answer · 2014-05-20T09-47-00.000Z

Что-то вроде этого?

public Assignments GetAssignmentsForProider(string provider_k, string recordType)
{
    var query = from ea in this.Context.Assignments
        where ea.Provider_K == provider_k
                && ea.RecordType == recordType
                && (recordType == "E" ? true : ea.Active)
            select ea;
        return query.FirstOrDefault();
}

Это должно удовлетворить ваши потребности.