|| (или) оператор в Linq с C#

Прочитайте эту документацию , в которой объясняется, как linq и c # могут испытывать разъединение.

Поскольку ожидается, что выражения Linq будут сокращены до что-то отличное от простых методов, вы можете обнаружить, что этот код ломается, если позже он используется в некотором контексте, отличном от Linq to Objects.

Тем не менее,

String.IsNullOrEmpty(fromname) || 
(   !String.IsNullOrEmpty(fromname) && 
    msg.FromName.ToLower().Contains(fromname.ToLower())
)

неправильно сформирован, поскольку он действительно должен быть

String.IsNullOrEmpty(fromname) || 
msg.FromName.ToLower().Contains(fromname.ToLower())

, что делает его красивым и понятным что вы полагаетесь на msg и msg.FromName, чтобы они также были ненулевыми.

Чтобы упростить вашу жизнь в c #, вы можете добавить следующий метод расширения строки

public static class ExtensionMethods
{
    public static bool Contains(
        this string self, string value, StringComparison comparison)
    {
        return self.IndexOf(value, comparison) >= 0;
    }

    public static bool ContainsOrNull(
        this string self, string value, StringComparison comparison)
    {
        if (value == null)
            return false;
        return self.IndexOf(value, comparison) >= 0;
    }
}

Затем используйте:

var messages = (from msg in dc.MessageItems
where  msg.FromName.ContainsOrNull(
    fromname, StringComparison.InvariantCultureIgnoreCase)
select msg);

Однако это не проблема. Проблема заключается в том, что аспекты системы Linq to SQL пытаются использовать значение fromname для создания запроса , который отправляется на сервер.

Поскольку fromname является переменной, механизм трансляции отключается и выполняет то, что от него требуется (создавая строчное представление fromname , даже если оно пустое, что вызывает исключение) ).

в этом случае вы можете либо сделать то, что вы уже обнаружили: сохранить запрос как есть, но убедитесь, что вы всегда можете создать ненулевое значение fromname с желаемым поведением, даже если оно пустое.

Возможно, лучше было бы:

IEnumerable<MessageItem> results;
if (string.IsNullOrEmpty(fromname))
{ 
    results = from msg in dc.MessageItems 
    select msg;    
}
else
{
    results = from msg in dc.MessageItems 
    where msg.FromName.ToLower().Contains(fromname) 
    select msg;    
}

Это не так здорово, поскольку запрос содержал другие ограничения и, таким образом, вызывал больше дублирования, но для простого запроса фактически должен привести к более читаемому / поддерживаемому коду. Это неприятно, если вы полагаетесь на анонимные типы, но, надеюсь, это не проблема для вас.

в этом случае вы можете либо сделать то, что вы уже обнаружили: сохранить запрос как есть, но убедитесь, что вы всегда можете создать ненулевое значение fromname с желаемым поведением, даже если оно пустое.

Возможно, лучше будет :

IEnumerable<MessageItem> results;
if (string.IsNullOrEmpty(fromname))
{ 
    results = from msg in dc.MessageItems 
    select msg;    
}
else
{
    results = from msg in dc.MessageItems 
    where msg.FromName.ToLower().Contains(fromname) 
    select msg;    
}

Это не так здорово, поскольку запрос содержал другие ограничения и, следовательно, требовал большего дублирования, но для простого запроса фактически должен получиться более читаемый / поддерживаемый код. Это неприятно, если вы полагаетесь на анонимные типы, но, надеюсь, это не проблема для вас.

в этом случае вы можете либо сделать то, что вы уже обнаружили: сохранить запрос как есть, но убедитесь, что вы всегда можете создать ненулевое значение fromname с желаемым поведением, даже если оно пустое.

Возможно, лучше будет :

IEnumerable<MessageItem> results;
if (string.IsNullOrEmpty(fromname))
{ 
    results = from msg in dc.MessageItems 
    select msg;    
}
else
{
    results = from msg in dc.MessageItems 
    where msg.FromName.ToLower().Contains(fromname) 
    select msg;    
}

Это не так здорово, поскольку запрос содержал другие ограничения и, следовательно, требовал большего дублирования, но для простого запроса фактически должен получиться более читаемый / поддерживаемый код. Это неприятно, если вы полагаетесь на анонимные типы, но, надеюсь, это не проблема для вас.

IEnumerable<MessageItem> results;
if (string.IsNullOrEmpty(fromname))
{ 
    results = from msg in dc.MessageItems 
    select msg;    
}
else
{
    results = from msg in dc.MessageItems 
    where msg.FromName.ToLower().Contains(fromname) 
    select msg;    
}

Это не так здорово, поскольку запрос содержал другие ограничения и, следовательно, требовал большего дублирования, но для простого запроса на самом деле должен получиться более читаемый / поддерживаемый код. Это неприятно, если вы полагаетесь на анонимные типы, но, надеюсь, это не проблема для вас.

IEnumerable<MessageItem> results;
if (string.IsNullOrEmpty(fromname))
{ 
    results = from msg in dc.MessageItems 
    select msg;    
}
else
{
    results = from msg in dc.MessageItems 
    where msg.FromName.ToLower().Contains(fromname) 
    select msg;    
}

Это не так здорово, поскольку запрос содержал другие ограничения и, следовательно, требовал большего дублирования, но для простого запроса на самом деле должен получиться более читаемый / поддерживаемый код. Это неприятно, если вы полагаетесь на анонимные типы, но, надеюсь, это не проблема для вас.