Не получается перевести из if - else в switch case

@fumigatus · Регистрация: 13.02.2014

Студворк — интернет-сервис помощи студентам

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
private void label1_Click(object sender, EventArgs e)
        {
            if (radioButton1.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Red;
            }
            if (radioButton2.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Orange;
            }
            if (radioButton3.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Yellow;
            }
            if (radioButton4.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Purple;
            }
            if (radioButton5.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Blue;
            }
            if (radioButton6.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Cyan;
            }
            if (radioButton7.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Green;
            }
            if (radioButton8.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Pink;
            }
            if (radioButton9.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Brown;
            }
            if (radioButton10.Checked)
            {
                label1.Text = textBox1.Text;
                label1.ForeColor = Color.Black;
            }
            else
            {
                label3.Text = "Выберите цвет текста!!!";
                label3.ForeColor = Color.Red;
            }
        }
    }
}

@volvo · 07.03.2014, 23:50

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
using System.Linq;
// ...
            RadioButton checkedButton = this.Controls.OfType<RadioButton>()
                .FirstOrDefault(r => r.Checked);
            switch(checkedButton.Name)
            {
                case "radioButton1":
                    label1.Text = textBox1.Text;
                    label1.ForeColor = Color.Red;
                    break;
                case "radioButton2":
                    label1.Text = textBox1.Text;
                    label1.ForeColor = Color.Orange;
                    break;
                // и так далее ...
                default:
                    label3.Text = "Выберите цвет текста!!!";
                    label3.ForeColor = Color.Red;
                    break;
            }

@ViterAlex · 07.03.2014, 23:54

Тут, в принципе всё правильно, в switch-case не переделаешь, но сам факт, что приходится писать такую "бороду" говорит, что что-то не так. А не так вот что: выбирать цвет с помощью переключателя - это моветон. Используй комбобокс с DropDownStyle = ComboBoxStyle.DropDownList. Заполни его нужными цветами и по выбранному элементу назначай цвет.

Добавлено через 1 минуту
UI, не в упрёк, но вот только linq здесь не хватало.

@volvo · 08.03.2014, 00:33

Сообщение от ViterAlex

но вот только linq здесь не хватало.

Если кто-то не умеет им пользоваться - это сугубо его проблемы. Я стараюсь использовать любые средства, которые мне предоставляет библиотека/фреймворк, если это необходимо, а не использовать донельзя кастрированный набор средств ("Linq - нельзя, ибо мы еще не проходили, циклы тоже нельзя, до них еще не дошли..." Да мне все равно, кто что проходил, я под третьеклассников не подстраиваюсь).

Как еще быстро определить, какая из радиокнопок включена? Неужели же делать вручную цикл по всем контролам?

@ViterAlex · 08.03.2014, 01:44

Сообщение от UI

Как еще быстро определить, какая из радиокнопок включена?

Не использовать их в данном случае. Конкретно в данном случае вообще нужно отказаться от радиокнопок, поскольку это неправильно с точки зрения интерфейса. Т.е. использование linq здесь вынужденная мера, потому что изначально сделано неправильно.

Сообщение от UI

Я стараюсь использовать любые средства

Поступаю аналогично

@zna926 · 08.03.2014, 01:55

Это цитата: "Да мне все равно, кто что проходил, я под третьеклассников не подстраиваюсь)."
Я, конечно, и на первоклассника не тяну, но все же осмелюсь предложить свой код

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
Forms Code
{
  public int n;
  
  if(radioButton1.Checked)
    n=0;
   if(radioButton2.Checked)
    n=1;
    .....................
 
    switch (n)
    {
      case 0:
       label1.Text = textBox1.Text;
        label1.ForeColor = Color.Red;
      break;
      case 1:
       label1.Text = textBox1.Text;
        label1.ForeColor = Color.Green;
      break;
       ............................
 
    }
 
}

@Ev_Hyper · 08.03.2014, 03:31

fumigatus, А почему нельзя вынести:

C#
1
label1.Text = textBox1.Text;

за все кроме 1 if?

И почему не создать 1 массив цветов, и в зависимости от индекса задавать соответствующий цвет текста.

@fumigatus · 08.03.2014, 17:35 **[ТС]**

Если бы я знал как в C# создавать массив, то я бы создал. Я пока что плохо знаю этот язык, но делаю контрольную работу, времени сейчас нет на изучение. Главное сделать, чтобы работало, а каким будет код меня сейчас не особо интересует. Кстати я хотел еще сделать проверку на то, пустая ли строка или нет. Если пуста, то тоже отдельно выводить сообщение о том, что нужно ввести слово. Но я не нашел функцию, которая отвечает за проверку.

@Worf · 08.03.2014, 17:42

Если используешь RadioButton, то не проще бы было при нажатии на определённый RadioButton в глобальную int переменную, например, записывать число(номер RadioButton-а, например) и потом, в зависимости от числа, производить определённое действие?

@Ev_Hyper · 09.03.2014, 10:32

fumigatus, может лучше напишите условие задания полностью?

Сообщение от fumigatus

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

C#
1
2
3
4
5
           
 if (String.IsNullOrEmpty(textBox3.Text))
            {
                MessageBox.Show("Не оставляйте поле пустым!");
            }

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .

@ViterAlex 8951 / 4863 / 1886 Регистрация: 11.02.2013 Сообщений: 10,246
	07.03.2014, 23:54
	Тут, в принципе всё правильно, в switch-case не переделаешь, но сам факт, что приходится писать такую "бороду" говорит, что что-то не так. А не так вот что: выбирать цвет с помощью переключателя - это моветон. Используй комбобокс с DropDownStyle = ComboBoxStyle.DropDownList. Заполни его нужными цветами и по выбранному элементу назначай цвет. Добавлено через 1 минуту UI, не в упрёк, но вот только linq здесь не хватало. 1

@fumigatus 0 / 0 / 0 Регистрация: 13.02.2014 Сообщений: 20
	08.03.2014, 17:35 [ТС]
	Если бы я знал как в C# создавать массив, то я бы создал. Я пока что плохо знаю этот язык, но делаю контрольную работу, времени сейчас нет на изучение. Главное сделать, чтобы работало, а каким будет код меня сейчас не особо интересует. Кстати я хотел еще сделать проверку на то, пустая ли строка или нет. Если пуста, то тоже отдельно выводить сообщение о том, что нужно ввести слово. Но я не нашел функцию, которая отвечает за проверку. 0

@Worf 44 / 44 / 8 Регистрация: 06.02.2014 Сообщений: 233
	08.03.2014, 17:42
	Если используешь RadioButton, то не проще бы было при нажатии на определённый RadioButton в глобальную int переменную, например, записывать число(номер RadioButton-а, например) и потом, в зависимости от числа, производить определённое действие? 0