Не могу разобраться с логикой программы

@JustBear · Регистрация: 01.03.2013

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

Всем привет! Во вложении лежит проект, с которым возникли проблемы: надо отработать такую логику:

- Галочка в чекбоксе не стоит - выдавать сообщение Error
- Галочка в чекбоксе не стоит, в текстбоксе1 пусто и выбран радиобуттон (любой) - выдавать сообщение Error
- Галочка в чекбоксе не стоит, в текстбоксе1 что-то есть и не выбран радиобуттон (любой) - выдавать сообщение Error
- Галочка в чекбоксе не стоит, в текстбоксе1 что-то есть и выбран радиобуттон (любой) - выдавать сообщение Error

- Галочка в чекбоксе стоит, но в текстбоксе1 пусто и не выбран радиобуттон (любой) - выдавать сообщение "Поставьте точку и введите текст"
- Галочка в чекбоксе стоит, но в текстбоксе1 пусто и выбран радиобуттон (любой) - выдавать сообщение "Введите текст"
- Галочка в чекбоксе стоит, в текстбоксе1 что-то есть и не выбран радиобуттон (любой) -выдавать сообщение "Постаьте точку"
- Галочка в чекбоксе стоит, в текстбоксе1 что-то есть и выбран радиобуттон (любой) - стартовать прогрессбар

Проблема заключается в том, что радиобуттон4 зависит от текстбокса2. Если там пусто, но точка в радиобуттоне4 стоит - выдавать сообщение при попытке нажать на кнопку - "Введите число в текстбокс2". И число должно быть >=3 символов, иначе выдавать сообщение "Число меньше 100"

Вроде все просто, но я запутался в логике. Помогите, пожалуйста.

@Итори · 14.08.2013, 09:29

1)Переключатели лучше внутри панели создавать.
2)Код на кнопке... конечно тут запутаешься 4 elseif, с условиями в 2 строки.

3)В событии изменение текста(textbox1), переписать,например:

Кликните здесь для просмотра всего текста

C#

              if (textBox1.TextLength <= 2)
            {
                label1.Visible = true;
            }
            else
                label1.Visible = false;

Ах да и еще там на символ пробела проверятся, а не пустоту.

Это взгляд на код.

Теперь по поводу вопроса:
Один из переключателей всегда должен быть выбрать(по умолчанию), но тут проблема в том что вы их как флажки(checkbox) используете, или я чего-то не понимаю.

Лучше поместить проверку в метод.

Вызов:

Кликните здесь для просмотра всего текста

C#

            string s;
 
            s=check();
            if (s != "") MessageBox.Show(s);
            else timer1.Start();

Метод:

Кликните здесь для просмотра всего текста

C#

        private string check()
        {
            string s="";
            if (checkBox1.Checked)
            {
 
                if (textBox1.Text=="")
                    s="Вы не ввели значение в текстбокс1! ";
                if (textBox2.Text == "")
                    s += "Вы не ввели значение в текстбокс2! ";
                if (!radioButton1.Checked && !radioButton2.Checked &&
                !radioButton3.Checked && !radioButton4.Checked)
                    s += "Выберите переключатель! ";
                if (textBox2.TextLength < 3)
                    s += "Длина текстбокс2, должна быть не меньше 3 символов";
            }
            else
            {
                s = "Вы не поставили галочку! Поставьте и еще раз нажмите кнопку";
            }
 
            return s;
        }

@JustBear · 14.08.2013, 12:01 **[ТС]**

"Переключатели лучше внутри панели создавать." - это как?

Все конечно не плохо, но этот код не отключает лейбл, если значение ввели, например, 99, а потом стерли. Он продолжает гореть.

Сообщение от Итори

3)В событии изменение текста(textbox1), переписать,например:
Кликните здесь для просмотра всего текста
Код C#

if (textBox1.TextLength <= 2)
{
label1.Visible = true;
}
else
label1.Visible = false;

Поэтому я переписал по своему:

C#

if (textBox1.Text == "")
            {
                label1.Visible = false;
            }
            else if (textBox1.TextLength == 2)
            {
                label1.Visible = true;
 
            }
            else if (textBox1.TextLength == 1)
            {
                label1.Visible = true;
 
            }
            else
            {
                label1.Visible = false;
            }

Так все ок.

@Итори · 14.08.2013, 12:27

"Переключатели лучше внутри панели создавать." - это как?

Просто добавить на форму панель(panel)Панель элементов->контейнеры.

Все конечно не плохо, но этот код не отключает лейбл, если значение ввели, например, 99, а потом стерли. Он продолжает гореть.

true и false, местами поменять.

@JustBear · 14.08.2013, 14:50 **[ТС]**

Сообщение от Итори

Просто добавить на форму панель(panel)Панель элементов->контейнеры.

Почему лучше?

@Итори · 14.08.2013, 15:02

Панель сгруппирует переключатели, если оставить так валятся на форме, и добавить скажем 2 новых(тепло,холодно) , то они будет взаимодействовать,т.е одновременно будет включен только 1 из 6, а это не всегда нужно.

@Learx · 14.08.2013, 19:13

а в чем пробема? просто делаем метод IsErorr и в нем проверияем:

C#

if(!checkBox.Checked)
{  
    if(!string.IsNullOrWhiteSpace(textbox1.Text)) return 1;
    else 
    {
        if(radiobotton1.Checked || radiobotton2.Checked ...) return false;
    }
}
else 
{
 ....
}
return true;

для других перед возвращением занчения выводим сообщение.
если метод вернул правду - стартуем

@Итори Нарушил, 10 блоков,акка 72 / 72 / 11 Регистрация: 19.07.2013 Сообщений: 250
	14.08.2013, 12:27	4
	"Переключатели лучше внутри панели создавать." - это как? Просто добавить на форму панель(panel)Панель элементов->контейнеры. Все конечно не плохо, но этот код не отключает лейбл, если значение ввели, например, 99, а потом стерли. Он продолжает гореть. true и false, местами поменять. 0

@JustBear 7 / 7 / 0 Регистрация: 01.03.2013 Сообщений: 166
	14.08.2013, 14:50 [ТС]	5
	Сообщение от Итори Просто добавить на форму панель(panel)Панель элементов->контейнеры. Почему лучше? 0

@Итори Нарушил, 10 блоков,акка 72 / 72 / 11 Регистрация: 19.07.2013 Сообщений: 250
	14.08.2013, 15:02	6
	Панель сгруппирует переключатели, если оставить так валятся на форме, и добавить скажем 2 новых(тепло,холодно) , то они будет взаимодействовать,т.е одновременно будет включен только 1 из 6, а это не всегда нужно. 0

	14.08.2013, 19:13

Опции темы