Как исправить мой LINQ запрос?

Question

[Михаил]

Здравствуйте, делаю интернет магазин и пытаюсь реализовать функцию "похожие товары", тоесть мы заходим в определенную категорию товаров, заходим в вьюшку с детальным описаниям товара и внизу мы предлагаем похожие за категорией товары, за идею было взято использовать LINQ запросы с рандомным выводом записей по категории из таблицы БД, и выводить их на вьюшку. Смотрите вот один из моих запросов:

public PartialViewResult Test2()
        {

            ListViewModel model = new ListViewModel
            {
                Furnitures = repository.Furnitures
                .Where(x =>   x.Category.Equals("Кухни"))
                .OrderBy(r => Guid.NewGuid()).Take(1)              
        };
            return PartialView(model);
            
        }

Тут всё хорошо и работает , выводит рандомную запись из таблицы по категории "Кухни" и отображается на странице. Но такой запрос я не могу использовать потому что на всех категориях у меня будет отображаться только товары из категории "Кухни". Вопрос: Как выводить рандомные товары из таблицы в зависимости от той категории на которой мы находимся?? Например, мы зашли в категорию "кровати" и нам вывело рандомный список товаров из кроватей, зашли в раздел Шкафы, нам вывело список рандомных товаров из категории шкафов. Я попробовал переделать данный метод на такой:

public PartialViewResult Test2(string category)
        {

            ListViewModel model = new ListViewModel
            {
                Furnitures = repository.Furnitures
                .Where(x => category == null ||  x.Category == category)
                .OrderBy(r => Guid.NewGuid()).Take(1),

            CurrentCategory = category
            };
            return PartialView(model);
            
        }

Но он выводит со всех категорий данные, можно как то переделать запрос?? Спасибо!

Answer 1

[MrDywar Pichugin]

Сделайте просто как в первом примере.
.Where(x.Category.Equals(category))
Если category будет null или пустой строкой то и категории такой не будет или там товары не попадающие под категорию.

У вас отображались все категории потому что срабатывало условие category == null. В отладчике точно проверьте что переменная задана.

Comments

[Михаил]

MrDywar Pichugin увы если написать так как вы написали вообще ничего не выводит

[Михаил]

MrDywar Pichugin .Where(x.Category.Equals(category)) если вот так

[Михаил]

прошел отладчиком то category = null , что в таком случае делать? это вьюшка неправильная ?

[MrDywar Pichugin]

M-Misha-M: Прежде чем решать проблему с тем что делать когда category == null, нужно решить проблему что бы category вообще передавался аргументом, на 100% быть уверенным что первая часть работает. И после этого переходить ко второй части, работе с аргументов, что делать когда он null или не null.

У вас возвращаются все категории потому что category == null и это условие возвращает ИСТИНА. Это означает что подходит любой item из репозитория Furnitures.

То что код .Where(x.Category.Equals(category)) ничего не возвращает означает что:
1) У вас category всегда равен null.
2) У вас нет категории значение которой равно null, поэтому выборка не имеет элементов.

Нужно:
1) Сделать рабочим передачу category.
2) Изменить условие допустим на .Where( category == null ? true : x.Category.Equals(category))

[Михаил]

MrDywar Pichugin: да ваше условие тоже выводит рандом их всех категорий, а как сделать параметр рабочим, у меня вьюшка просто такая вот @{Html.RenderAction("Test2", "Furniture");} я так понял из-за этого не пашет, я пробовал через html.partial но дело в том что во вьюшках модели разные , можете подсказать что нужно сделать?

[MrDywar Pichugin]

M-Misha-M: Вам нужно полностью прочитать руководство по MVC, понять как работает MVC контроллер, VIEW, Model.
Рекомендую - metanit.com/sharp/mvc5

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

Когда вся схема будет в голове, вы сможете отследить нужные вам этапы и точно найти место где что то пошло не так.

[Михаил]

MrDywar Pichugin: спасибо, моя проблема оказалась в том, что в html.RenderAction у меня не передавалась модель @{Html.RenderAction("Test2", new { category = Model.Category });} выправило мою проблему