Как упростить код с lambda-операциями?

Question

[Роман Кофф]

В зависимости от некоего условия нужно составить lambda expression. Как сделать проще, чтобы код был изящнее.
В примере, в зависимости от наличия значения в UserId используются разные лябда-выражения для фильтра запроса:

public CatalogModel GetListCatalogModels(int? userId)
{
	Expression<Func<Catalog, bool>> filter1 = x => x.IsPublished;
	Expression<Func<Catalog, bool>> filter2 = x => x.UserPtr == userId && x.IsPublished;
	Expression<Func<Catalog, bool>> filter;
	if (userId == null)
		filter = filter1;
	else
		filter = filter2;
	...

Не очень силен в лямбда-синтаксисте, но полагаю, что это может быть изящнее...

Answer 1

[Дмитрий Макаров]

Expression<Func<Catalog, bool>> filter = x => (userId==null || x.UserPtr == userId) && x.IsPublished;

Comments

[Роман Кофф]

Спасибо за варианты, рабочее решение, задачу решает.

Но для общей ситуации без привязки к моему примеру — есть ли возможность селекции делегатов в одну строку? Типа такого: filter = userId.hasValue? filter1 : filter2;, но чтобы использовались не ранее объявленные делегаты, а непосредственно лямбда-выражения. Выражения могут быть абстрактно разными, без возможности объединения двух логических выражений в одно.

[Дмитрий Макаров]

Возможно что-то вроде этого:

Expression<Func<Catalog, bool>> filter = userId==null? 
                                                        (Expression<Func<Catalog, bool>>)x => x.IsPublished:
                                                        (Expression<Func<Catalog, bool>>)x => x.UserPtr == userId && x.IsPublished;

но как-то не очень красиво.
Я бы предпочел что-нибудь типа:

Expression<Func<Catalog, bool>> filter = x=> userId==null? x.IsPublished: x.UserPtr == userId && x.IsPublished;
<code>

Answer 2

[Дмитрий Ковальский]

1)

правильно не 
if (userId == null)
    filter = filter1;
а
if (userId.hasValue)
    filter = filter1;

2)
filter = userId.hasValue? filter1 : filter2;
3)

filter = x => userId.hasValue?x.UserPtr == userId.Value && x.IsPublished : x.IsPublished

4)

filter = x =>  x.IsPublished && (userId.hasValue?x.UserPtr == userId.Value : true)

какой нибудь способ да сработает

Answer 3

[Артём]

Есть очень хороший пример, который когда-то сподвиг меня посмотреть в сторону expressions. m.habrahabr.ru/post/83169
Все что вам требуется, можно узнать там. А дальше msdn, так как по данной теме информации мало.

Comments

[Роман Кофф]

Довольно интересная статья, спасибо...