0

пытаюсь сделать фитрацию данных в таблице, посредством нажатия на заголовок столбца. Данные в этом столбце разные, хотел сделать так - что при на заголовок , отображались сначала одни данные , потом другие и тд. Не могу додумать как это сделать! Проверка на нажалие просто обходит вторую часть цикла. Подскажите пожалуйста.

private void dataGridView1_ColumnHeaderMouseClick(object sender, DataGridViewCellMouseEventArgs e)
    {

        
        if (dataGridView1.Columns[6].HeaderText == "Статус" & e.Clicks == 1)
        {

            var statusClick = context.Inventories.Select(x => new
            {
                id = x.Inventid,
                name = x.Inventname,
                serial = x.Serialnum,
                datepov = x.Datepov,
                datenextpov = x.Datenextpov,
                filial = x.Filial.Filialname,
                status = x.Status.Statusname,
                service = x.Service.Servicename,
                yearofman = x.Yearofman
            }).AsEnumerable().Where(x => x.status == "Поверено")
          .ToList();
            dataGridView1.DataSource = statusClick;

        }   
        else if (dataGridView1.Columns[6].HeaderText == "Статус" & e.Clicks == 2) 
         {
                var statusClick = context.Inventories.Select(x => new
                {
                    id = x.Inventid,
                    name = x.Inventname,
                    serial = x.Serialnum,
                    datepov = x.Datepov,
                    datenextpov = x.Datenextpov,
                    filial = x.Filial.Filialname,
                    status = x.Status.Statusname,
                    service = x.Service.Servicename,
                    yearofman = x.Yearofman
                }).AsEnumerable().Where(x => x.status == "Не поверяется")
        .ToList();
                dataGridView1.DataSource = statusClick;

                
        }
6
  • Есть событие ColumnHeaderMouseDoubleClick learn.microsoft.com/ru-ru/dotnet/api/… Ну, либо я не понял вопроса.
    – aepot
    Commented 25 мая 2023 в 7:43
  • @aepot Да, я и обрабатываю это событие, дело в том, что он не переключается между if'ами, а выдает только первый результат
    – KuroIsekai
    Commented 25 мая 2023 в 7:46
  • Уберите AsEnumerable. Вызов Where поставьте перед Select. Commented 25 мая 2023 в 8:08
  • @AlexanderPetrov поясните пожалуйста, зачем вызов Where ставить перед Select?
    – KuroIsekai
    Commented 25 мая 2023 в 8:11
  • 2
    Сейчас у вас все данные из БД загружаются в память. И лишь после этого фильтруются. Если в таблице будут миллионы строк, то код работать будет медленно, памяти жрать много. / Фильтрацию нужно делать непосредственно в БД. Если переместить Where, то EF сгенерирует эффективный sql-код. Впрочем, если это действительно Entity Framework, а не что-то другое. Commented 25 мая 2023 в 8:15

1 ответ 1

2

Не повторяйте свой код. Используйте методы.

private void FilterStatus(string status)
{
    var filtered = context.Inventories.Select(x => new
    {
        id = x.Inventid,
        name = x.Inventname,
        serial = x.Serialnum,
        datepov = x.Datepov,
        datenextpov = x.Datenextpov,
        filial = x.Filial.Filialname,
        status = x.Status.Statusname,
        service = x.Service.Servicename,
        yearofman = x.Yearofman
    }).AsEnumerable().Where(x => x.status == status)
    .ToList();
    dataGridView1.DataSource = filtered;
}

Тогда можно написать вот такой обработчик.

private string currentStatus = "Поверено";

private void SwitchStatus()
{
    if (currentStatus == "Поверено")
        currentStatus = "Не поверяется";
    else
        currentStatus = "Поверено";
}

private void dataGridView1_ColumnHeaderMouseClick(object sender, DataGridViewCellMouseEventArgs e)
{
    if (dataGridView1.Columns[6].HeaderText == "Статус")
    {
        SwitchStatus();
        FilterStatus(currentStatus);
    }
}

Вот и вся магия.

Но в целом, у вас идет запрос к БД на каждый такой клик. А стоит ли? Быть может проще не обращаться к БД, а изначально загрузить из нее все данные, а фильтровать их через BindingSource? Подумайте над этим.


Еще обратите внимание на & в условии, изучите разницу между & и &&.

2
  • А посдкажите пожалуйста, если статусов не 2, а больше , как тогда реализовать SwitchStatus?
    – KuroIsekai
    Commented 25 мая 2023 в 10:53
  • @KuroIsekai перепишите метод SwitchStatus как вам нравится. Можно выбрать все уникальные статусы из БД запросом, и затем хранить индекс статуса в массиве статусов и покругу иего переключать.
    – aepot
    Commented 25 мая 2023 в 10:57

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.