Как реализовать проверку входных данных при нажатии кнопки?

Question

[Евгений Семашко]

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

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

Пытался закостылить через булеву переменную, но костыль не прошёл

Сам проект

Проект делаю на EF Core с использованием MS Sql

P.s. Код слишком оказался громоздким и его сюда не пускает, ибо слишком много символов.

Comments

[#]

Код слишком оказался громоздким и его сюда не пускает, ибо слишком много символов.

можете выложит на github. а еще лучше на Azure DevIps (гугл и Visual Studio в помощь. безлимит для личных проектов)

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

Answer 1

[#]

Хочу реализовать проверку корректности всех входных данных в поля регистрации и после нажатия на кнопку Registration

ход мыслей понятен, типичен для новичка. но не лучший
поясню:
для проверки корректности вводимых данных в любых полях, общепринято делать валидацию (по ссылке частный случай. все это есть и в WinForms, и в WPF, и в UWP, и в вебинтерфейсе) после ввода, или даже по мере ввода.
код не смотрел вообще, по причине изначально неудачного подхода к проблеме проверки корректности upd пользовательского ввода данных. для этого процесса есть устоявшийся термин - валидация.
upd ну и устоявшиеся же приемы, которые, в случае C#, компания Microsoft, любезно предоставляет нам, что называется из коробки, для большинства основных сценариев разработки десктоп приложений

Comments

[Евгений Семашко]

Но будет ли это не пускать на новую форму если валидированное поле некорректно заполнено? Или надо для этого ещё что-то додумывать?

[Евгений Семашко]

Ибо все поля у меня имееют проверку на ввод, нужно только при нажатии кнопки проверять заполнены ли они корректно или это лучше все вопхнуть в метод и возвращать и метода true/false если что-то некорректно?

[#]

Евгений Семашко, с ходу - оптимально делать целевую кнопку (Registration? да?) недоступной, пока все поля не будут корректны. но не готов на ходу подсказать как скоординировать..

.. вертится в голове типа - каждая итерация проверки (событие валидирования) ставит по итогу свой bool флаг у меня все хорошо/плохо, и дергает специально выточенный метод обновления статуса доступности кнопки, который, в свою очередь, ставит Enalble как f1 && f2 && f3 && .. всех флагов
.. хотя метод по ходу сводится к одной строке... но лучше отдельный метод, что бы не копипастить
.. это все в уме.. но я не вижу проблем, и вроде не должно быть много кода

[Евгений Семашко]

#, Получается надо заново переписывать валидацию для каждой модели? Но есть 1 трабл, с keypress'om, как проверять тогда, что вводятся не числа в поля с именем допустим? Если валидация представляет собой объект CancelEventArgs e, а для проверки на ввод только букв использую KeyPressEventArgs e. Если вопрос не ясен, то переформулирую

[#]

Евгений Семашко,

Но есть 1 трабл, с keypress'om

а не надо вообще пользоваться keypress'om для этой цели. события валидации изначально созданы для проверки корректности значения. keypress скорее для создания хоткеев в приложении, или еще чего такого. в сети полно примеров валидации на их основе. но это отсебятина авторов

Получается надо заново переписывать валидацию для каждой модели?

не надо. элементарно переносим в правильный метод. не знаю какой у вас там код, но весьма вероятно что все еще и упростить можно

[Евгений Семашко]

#, хорошо, только тогда как вытянуть из метода Validating булеву переменную? Создавать новое событие Validated и добавлять туда булеву переменную? Или в Validating добавить булеву переменную и ей руководствоваться?

Просто не до конца понимаю, как дальше с Validating работать когда все проверки есть

[#]

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

если все еще не понятно - в личку (платно, но не дорого. данный вопрос вообще на уровне примитива)

[Евгений Семашко]

#, всё, разобрался. Оказалось реально примитивно. У меня просто привычка идти окольными путями)

[#]

Евгений Семашко, ну так что тогда отметку ответ не поставить? )))

Answer 2

[Евгений Семашко]

Вот решение для одного из боксов

public bool IsNameValid(string name)
        {
            bool valid = false;
            Regex check = new Regex(@"^([A-Z][a-z-A-z]+)$");
            valid = check.IsMatch(name);

            if (valid == true)
            {
                return valid;
            }
            else
            {
                return valid;
            }
        }

        private void nameBox_TextChanged(object sender, EventArgs e)
        {
            TextChange(sender, e);
        }

        private void nameBox_Validating(object sender, CancelEventArgs e)
        {
            if (!IsNameValid(nameBox.Text))
            {
                inputincorrect.SetError(nameBox, "Invalid name");
            }
            else
            {
                inputincorrect.SetError(nameBox, null);
            }
        }

Сама проверка в кнопке

if (IsNameValid(nameBox.Text) == true && // ещё проверки)
           {
                // какой-то код
            }

Comments

[#]

все не так. упрощаем по шагам

[#]

сначала чикаем

public bool IsNameValid(string name)
        {
            bool valid = false;
            Regex check = new Regex(@"^([A-Z][a-z-A-z]+)$");
            valid = check.IsMatch(name);

            if (valid == true)
            {
                return valid;
            }
            else
            {
                return valid;
            }
        }

[#]

до состояния (обновлено)

public bool IsNameValid(string name) => (new Regex(@"^([A-Z][a-z-A-z]+)$")).IsMatch(name);

[#]

вот этот артефакт тут не понятно зачем? и вообще не понятно зачем? мне кажется это что то совершенно лишнее

private void nameBox_TextChanged(object sender, EventArgs e)
        {
            TextChange(sender, e);
        }

[#]

ни какая проверка в кнопке не нужна.. см в следующем каменте

if (IsNameValid(nameBox.Text) == true && // ещё проверки)
           {
                // какой-то код
            }

[#]

(тоже обновим до полного модерна)

private void nameBox_Validating(object sender, CancelEventArgs e)
    =>  имяКномпки.Enable &= IsNameValid(nameBox.Text);

а еще лучше

private void nameBox_Validating(object sender, CancelEventArgs e)
   =>  имяКномпки.Enable &= (new Regex(@"^([A-Z][a-z-A-z]+)$")).IsMatch(name);

тогда и функцию IsNameValid тоже смело выкидываем

[#]

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

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

[#]

.. хотя нет, не позволяет.. сори. лучше флаги )))
тогда примерно так

private bool f1 = false; // для nameBox1
private bool f2 = false; // для другойBox2
private bool f3 = false; // для другойBox3
// сколько там надо
private void nameBox1_Validating(object sender, CancelEventArgs e)
   =>  имяКномпки.Enable = (f1 = (new Regex(@"^([A-Z][a-z-A-z]+)$")).IsMatch(name)) && f2 && f3;
// ... && f100; если их сто.. как то так
// при своей валидации, каждый бокс обновляет свой собственный флаг
// и обновляет статус целевой кнопки
// как тут показано, это возможно даже одной строкой, если собственная валидация достаточно проста

[Евгений Семашко]

#, это свой метод, который делает 1ю букву сразу заглавной. Чуть с неймингом обосрался

[#]

Евгений Семашко, nameBox_TextChanged?
а зачем? по смыслу TextChanged опять таки для случая, когда по изменению текста надо какую то логику отработать
все это может слегка путать, и тут есть избыточность. но есть документация
https://docs.microsoft.com/ru-ru/dotnet/framework/...
и есть отладчик - ставите точку прерывания, и выясняете, что валидация происходит даже чаще чем TextChanged. их точно не надо использовать вместе
(правка в таком простом случае нет смысла. хотя иногда логика может быть такой, что предпочтительно увязать с разными событиями по смыслу. но в данном случае, у вас всего лишь исправление для приближения успеха валидации).
но в принципе можно опереться на TextChanged вместо валидации, если очень захочется. и все проделать в одном методе

на примере Validating

private bool f1 = false; // для nameBox1
private bool f2 = false; // для другойBox2
private bool f3 = false; // для другойBox3
// сколько там надо
private void nameBox1_Validating(object sender, CancelEventArgs e)
{
      прямо тут правим первую букву на заглавную
      // дальше проверяем валидность
      // и обновляем доступность кнопки логина
      имяКномпки.Enable = (f1 = (new Regex(@"^([A-Z][a-z-A-z]+)$")).IsMatch(name)) && f2 && f3;
}

[Евгений Семашко]

#, кстати на счет Enable, почему в кнопке есть свойство только Enabled, что делает кнопку inactive если все проходит.

Получается это надо делать через if? Если значения валидны, то ставить enabled = true?

Просто не до конца понятно как сделать кнопку активной обратно

[#]

Евгений Семашко, вообще не понимаю вопроса..

почему в кнопке есть свойство только Enabled, что делает кнопку inactive если все проходит.

1 - так решили разработчики
2 - из практики, я нахожу что это очень удобно. и ни чего больше и не надо. это очень простая и эффективная возможность управлять доступностью любого контрола

Получается это надо делать через if? Если значения валидны, то ставить enabled = true?

да

Просто не до конца понятно как сделать кнопку активной обратно

????
когда все флаги true она автоматом станет true проще не куда

[Евгений Семашко]

#, окей, большое спасибо, что возился со мной все это время)

[#]

Евгений Семашко, надеюсь что понял )))

[Евгений Семашко]

#, как доберусь до ноута, то апдейтну ответ. А так в принципе все ясно)

[#]

Евгений Семашко, да в общем то тут ни кому не нужен твой улучшенный ответ, да и вообще ни какой. и даже тебе

ценность может иметь только твой переход на новый уровень понимания. и эта ценность только для тебя самого. демонстрировать ее не обязательно

[Евгений Семашко]

#, хорошо, подчеркну для себя)