VPF::refactoring - Форум программистов

blackwitcher

Дата 5.9.2017, 12:56 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 149
Регистрация: 11.1.2010

Репутация: нет
Всего: нет

Код


private void singleChoiceAction1_Execute(object sender, SingleChoiceActionExecuteEventArgs e)
        {
            var selectedItem = singleChoiceAction1.SelectedItem.Id;

            if (selectedItem == 1.ToString())
            {
                _gridView.Columns["Rating"].Visible = true;
                _gridView.Columns["Price"].Visible = false;
            }
            else
            {
                _gridView.Columns["Rating"].Visible = false;
                _gridView.Columns["Price"].Visible = true;
            }
           
        }

нужна помощь по рефакторинга метода

то есть, со временем может быть добавлено другие поля кроме "Rating" и "Price", и мне не нужно будет вписывать вручную smile

Это сообщение отредактировал(а) blackwitcher - 5.9.2017, 12:58

blackwitcher

Дата 5.9.2017, 15:10 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 149
Регистрация: 11.1.2010

Репутация: нет
Всего: нет

Код


public enum Change:byte
    {
        Rating,
        Price
    }

Код


 private void GetValue()
        {
            singleChoiceAction1.Items.Clear();
            singleChoiceAction1.Items.Add(new ChoiceActionItem(Change.Rating.ToString(),Change.Rating));
            singleChoiceAction1.Items.Add(new ChoiceActionItem(Change.Price.ToString(),Change.Price));
        }

Код


private void singleChoiceAction1_Execute(object sender, SingleChoiceActionExecuteEventArgs e)
        {
            SelectedItem();
        }

        private void SelectedItem()
        {
            var selectedItem = singleChoiceAction1.SelectedItem.Id;
            if (selectedItem == nameof(Course.Rating))
            {
                ShowRating();
            }
            if (selectedItem == nameof(Course.Price)) 
            {
                ShowPrice();
            }
        }

        private void ShowPrice()
        {
            _gridView.Columns[nameof(Course.Rating)].Visible = false;
            _gridView.Columns[nameof(Course.Price)].Visible = true;
        }

        private void ShowRating()
        {
            _gridView.Columns[nameof(Course.Rating)].Visible = true;
            _gridView.Columns[nameof(Course.Price)].Visible = false;
        }

diadiavova

Дата 17.9.2017, 23:35 (ссылка)

(нет голосов)

Загрузка ...

Доктор Зло(диагност, настоящий, с лицензией и полномочиями)

Профиль
Группа: Модератор
Сообщений: 5820
Регистрация: 14.8.2008
Где: В Коньфпольте

Репутация: 31
Всего: 142

Цитата(blackwitcher @ 5.9.2017, 12:56

)

то есть, со временем может быть добавлено другие поля кроме "Rating" и "Price", и мне не нужно будет вписывать вручную

Непонятно, что это значит. Если ты изменяешь значения этих полей, то при чем тут другие поля?
По коду. Не знаю, что именно ты хочешь достичь оптимизацией, но если надо сократить, то код из первого поста можно записать одной строкой

Код


_gridView.Columns["Price"].Visible = !(_gridView.Columns["Rating"].Visible = singleChoiceAction1.SelectedItem.Id == 1.ToString());

--------------------

Хочешь получить мудрый совет - читай подписи участников форумов.
Злой доктор Щасзаболит smile

blackwitcher

Дата 18.9.2017, 10:09 (ссылка)

(нет голосов)

Загрузка ...

Шустрый

Профиль
Группа: Участник
Сообщений: 149
Регистрация: 11.1.2010

Репутация: нет
Всего: нет

Спасибо, diadiavova

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Общие вопросы по .NET и C# \| Следующая тема »