Как задать логическое && в лямба выражении?

Question

[CityzenUNDEAD]

Приветствую!
Возникла сложность, которую я почему-то даже загуглить не знаю как..
Мне нужно получить данные из бд по linq запросу.
В этом запросе я пытаюсь установить фильтр. Фильтр представляет из себя диапазон из дат, максимальная и минимальная. Соответственно, мне нужно получить то что находится в этом диапазоне.

var testQuery = _repository.GetAll()
            .WhereIf(input.SortDateStart != null && input.SortDateEnd != null,
                faqs => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd)
            .WhereIf(input.SortDateStart != null && input.SortDateEnd is null,
                faqs => faqs.CreationTime >= input.SortDateStart).Count();

Проблема на этой строке

input.SortDateStart != null && input.SortDateEnd != null

В рамках ламда выражения не получается логические &&.
Дополнительные скобки в виде

((input.SortDateStart != null) && (input.SortDateEnd != null))

погоды не решили.
Буду признателен за подсказки.

Comments

[Сергей Соловьев]

а что конкретно ты хочешь и что не получается?

[#]

есть некоторые сомнения в логике..
но допустим для начала явно выделите приоритет операций скобками.. почем нет?

[#]

CityzenUNDEAD, дайте пример кода генерации данных (пусть фейковых, но подходящих по смыслу) ну и с запросом конечно

[#]

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

[#]

CityzenUNDEAD, ну или самостоятельно сделайте дебаг поэтапно:
- первая выборка, дамп
- вторая, дамп
.. я так думаю

Answer 1

[#]

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

ps количество не завешенных задач/процессов? )))
каждая .WhereIf это выборка/запрос (как удобнее)... так вот вторая идет из первой. и по моему в первой вы уже отсекаете незавершенные.
подебажде поэтапно

pps ну и да, WhereIfне совсем типичный метод. тут поддержу коллег.. опять же, может во Whereполучатся более прозрачные условия )))

ppps CityzenUNDEAD, еще смущающее место

faqs => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd

использование faqs в его определении.. сам бы ни когда не стал так делать...
... а у вас аж два раза... это рекурсия в лямбде... а с рекурсиями надо быть хорошо знакомым... я так думаю...

Comments

[Vindicar]

использование faqs в его определении.. сам бы ни когда не стал так делать...

Разве это не лямбда с одним аргументом faqs?

[#]

Vindicar, в левой части как результат, и в правой как его исходные? это уже рекурсия

[#]

Vindicar, если бы это было типа x => x > 0 но тут структурированный объект. либо научите меня ))

[#]

Vindicar, сегодня проверю в студии, может я что пропустил? ))

[Vindicar]

#, ну у меня шарп не основной язык, но я эту строку парсю так:

faqs  //аргумент лямбды
=> // разделитель аргумента и тела
// скобок нет - значит, дальше идёт вычисляемое лямбдой выражение
    faqs.CreationTime >= input.SortDateStart  //первый операнд
   && //оператор логического И
    faqs.CreationTime <= input.SortDateEnd  //второй операнд

т.е. это примерно эквивалентно такой не-лямбда записи

bool lambda_func(SomeStructure faqs)
{   //input в лямбде доступен через замыкание
    return (faqs.CreationTime >= input.SortDateStart) && (faqs.CreationTime <= input.SortDateEnd);
}

[#]

Vindicar, ну либо я что пропустил сильно.. либо это структурированный объект.. и используемый дважды... а декларации нет..(сигнатуры в смысле

[#]

Vindicar, и в любом случае это синтаксис linq... идет первая выборка, и из нее вторая... если ТС даст хотя бы фейковый пример, на пробу, пусть на 3 строки фильтрации, +3 строки , которые должны попасть в итог... погоняю в студии, что увижу = поделюсь )))

[#]

Vindicar,

faqs //аргумент лямбды

вот это место меня смущает... тут обычно должен быть приемник итога )))

[#]

Vindicar, не считая примитивов ))

[Vindicar]

#, ну на МСДН написано то же самое. о_о

Lambda expressions
In the previous example, notice that the conditional expression (num % 2 == 0) is passed as an in-line argument to the Enumerable.Where method: Where(num => num % 2 == 0). This inline expression is a lambda expression.

Но я могу что-то путать - как уже говорил, шарпом порядочно не пользовался.

[#]

Vindicar, из принципа проверю возможность структурированного приемника... если таки уже да.. ну.. отпишусь что был не во всем прав )))

[#]

Vindicar,

using NPOI.SS.Formula.Functions;
using System;
using System.Collections.Generic;
using System.Drawing;

static class Program
{
    private static void Main(string[] args)
    {
        var input = gen();
        input.print();
        faqs().print();
    }
    static void print(this string s) => Console.WriteLine(s);
    static void print(this IEnumerable<Event> en) { foreach (var e in en) $"{e.SortDateStart} {e.SortDateEnd}".print(); }
    static IEnumerable<Event> gen()
    {
        var rnd = new Random();
        var dts = DateTime.Today;
        for (var i = 0; i < 6; i++)
        {
            var r = rnd.Next();
            var t = new TimeSpan(r);
            yield return new Event { SortDateStart = dts.Add(t), SortDateEnd = dts.Subtract(t) };
        }
    }
    static IEnumerable<Event> faqs()
    {
        return IEnumerable < Event > gen() => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd;
    }
}

public class Event
{
    public DateTime SortDateStart { get; set; }
    public DateTime SortDateEnd { get; set; }
}

попробуй скомпилировать... и повторюсь - если научишь новому? буду благодарен

[#]

Vindicar, .. а там.. ну практически одни лямбды...

[#]

Vindicar, а

дата не корректно парсилась

хрень собачья.. в сравнении она и не парсится... там бинарные форматники

[#]

Vindicar,

ну у меня шарп не основной язык

но тут по пацански:
- или не лезь
- или обоснуй так, что бы все уважуху проявили

[#]

Vindicar, и если ты не в курсе IEnumerable это главный тип, который гоняется в linq... так что мой пример кода почти адекватен косяку ТС...
я один из самых первых кураторов тега... или первый?.. точно не знаю ))

[#]

Vindicar, и да.. упростим до предела

using NPOI.SS.Formula.Functions;
using System;
using System.Collections.Generic;
using System.Drawing;

static class Program
{
    private static void Main(string[] args)
    {
        var input = gen();
        input.print();
        faqs().print();
    }
    static void print(this string s) => Console.WriteLine(s);
    static void print(this IEnumerable<Event> en) { foreach (var e in en) $"{e.SortDateStart} {e.SortDateEnd}".print(); }
    static IEnumerable<Event> gen()
    {
        var rnd = new Random();
        var dts = DateTime.Today;
        for (var i = 0; i < 6; i++)
        {
            var r = rnd.Next();
            var t = new TimeSpan(r);
            yield return new Event { SortDateStart = dts.Add(t), SortDateEnd = dts.Subtract(t) };
        }
    }
    static IEnumerable<Event> faqs()
    {
        return faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd;
    }
}

public class Event
{
    public DateTime SortDateStart { get; set; }
    public DateTime SortDateEnd { get; set; }
}

[#]

Vindicar, да, сам торможу.. вот еще ляьмбдее.. только скомпилируй

using NPOI.SS.Formula.Functions;
using System;
using System.Collections.Generic;
using System.Drawing;

static class Program
{
    private static void Main(string[] args)
    {
        var input = gen();
        input.print();
        faqs().print();
    }
    static void print(this string s) => Console.WriteLine(s);
    static void print(this IEnumerable<Event> en) { foreach (var e in en) $"{e.SortDateStart} {e.SortDateEnd}".print(); }
    static IEnumerable<Event> gen()
    {
        var rnd = new Random();
        var dts = DateTime.Today;
        for (var i = 0; i < 6; i++)
        {
            var r = rnd.Next();
            var t = new TimeSpan(r);
            yield return new Event { SortDateStart = dts.Add(t), SortDateEnd = dts.Subtract(t) };
        }
    }
    static IEnumerable<Event> faqs() => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd;
}

public class Event
{
    public DateTime SortDateStart { get; set; }
    public DateTime SortDateEnd { get; set; }
}

[#]

Vindicar, язык надо не парсить.. надо на нем думать... это мое мнение

[#]

Vindicar, ты меня не видел в своих тегах? а знаешь по чему?... я знаю, что знаю не все.. и уважаю тех, кто знает другое ))

[#]

Vindicar, хочешь еще шарпа?

[Vindicar]

#, чой-то я не вкурил.
У ОПа

faqs => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd

- скобок после faqs нет.
У тебя

static IEnumerable<Event> faqs() => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd;

- скобки есть. Конечно, смысл от этого поменяется! У ОПа faqs - имя параметра, у тебя - имя метода. Как бэ не одно и то же.

У меня вариант с параметром faqs прекрасно работает как ожидалось.

namespace ConsoleApp1
{
    public class InputClass //заглушка для типа данных, экземпляром которого является input
    {
        public DateTime SortDateStart;
        public DateTime SortDateEnd;
        public InputClass(DateTime start, DateTime end) { SortDateStart = start; SortDateEnd = end; }
    }

    public class FaqsClass //заглушка для типа данных, экземпляром которого является faqs
    {
        public DateTime CreationTime;
        public FaqsClass(DateTime ctime) { CreationTime = ctime; }
    }

    internal class Program
    {
        static void Main(string[] args)
        {
            DateTime now = DateTime.Now;
            var input = new InputClass(now, now + TimeSpan.FromSeconds(5));
            // описываем лямбду-предикат
            Func<FaqsClass, bool> my_lambda = faqs => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd;
            var many_faqs_given = new FaqsClass[10];
            for (int i = 0; i < many_faqs_given.Length; i++) 
                // только 6 значений (+0...+5) попадут в диапазон, описанный input
                many_faqs_given[i] = new FaqsClass(now + TimeSpan.FromSeconds(i));
            foreach (var f in many_faqs_given.Where(my_lambda))
                Console.WriteLine("Match");  // выведет Match 6 раз
        }
    }
}

[#]

Vindicar,

чой-то я не вкурил.

ну теплее.. кода от ТС нет ))

[#]

Vindicar, ну а так то.. ну да "не первый раз на родео"..
.. сам то видишь сколько приемов использовал?...
вложенное поле чего стоит? без него ни как?
.. потом отмаз про парсинг дат?.. где его нету...

[Vindicar]

#, ну приемы приемами, а мой изначальный вопрос был "откуда утверждение про рекурсию в лямбде, если там рекурсии нет". И таки да, рекурсии там нет. На чём и разойдёмся.

[#]

Vindicar, ок. проверю в студии. а так - да, согласен, видимо что то упустил ))

[#]

Vindicar, фактически аргументом лябды является faqs.CreationTime... но не во всех конструкциях, даже компилятор не всегда согласен )))
... ты же искусственно создаешь метод...
.. а ТС вообще использует стороннюю либу...

[#]

Vindicar,

WhereIf

это не от мелких..
https://github.com/aspnetboilerplate/aspnetboilerp...
.. если хватит терпения найти, как там обрабатывается второй предикат.. сниму шляпу ))
.. образно.. по тому что не ношу их.. )))

[#]

Vindicar,

Func<FaqsClass, bool> my_lambda = faqs => faqs.CreationTime >= input.SortDateStart && faqs.CreationTime <= input.SortDateEnd;

но тут да, красавчик... но осталось малое.. узнать что в кишках https://github.com/aspnetboilerplate/aspnetboilerplate

[#]

Vindicar, ну еще мелочь.. ты же создал

//заглушка для типа данных, экземпляром которого является faqs

.. а ТС был скуп на код.. без "заглушки" эта хрень не компилируется

[#]

Vindicar, лан, мир, дружба, жвачка.. и не я начал.. ;))

Answer 2

[Nik Faraday]

WhereIf? Может просто Where использовать? Туда как раз таки можно передать лямбду. и использовать логическое &&

Answer 3

[CityzenUNDEAD]

Спасибо всем за комментарии!
Проблема была в другом, у меня просто дата не корректно парсилась)

Comments

[#]

ну хотя бы на ход мыслей подтолкнули? ;)))
а так - поздравления )))

[#]

.. хотя.. какой блин парсинг в сравнении бинарных форматов?.. упс?