Вопрос по архитектуре программы

@MerrinZ · Регистрация: 23.09.2016

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

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

@ViterAlex · 08.06.2017, 08:11

Сначала ознакомиться с Архитектура ПО в WinForms (FAQ & HowTo)

Если принимается ввод от пользователя, то всё нужно проверять (события Validating/Validated). Я предпочитаю деактивировать контролы, если вызываемое ими действие требует введённые данные, а они ещё не введены.
Убрать абсолютные пути из кода.

Использовать using или прямой вызов Dispose() для объектов, реализующих IDisposable. В частности, там где ты создаёшь каптчу, объект графикс у тебя висит. Уборщик его, конечно, уберёт после использования, но это небрежность. Вот так было бы лучше:

C#

//область для рисования
using (Graphics g = Graphics.FromImage((Image) result)) {
    g.Clear(Color.Gray);
    string textCaptchaSymbol = "";
    string alphabet = "1234567890QWERTYUIOPASDFGHJKLZXCVBNM";
    for (int i = 0; i < 5; ++i)
        textCaptchaSymbol += alphabet[rnd.Next(alphabet.Length)];
    g.DrawString(textCaptchaSymbol,
        new Font("Arial", 15), colors[rnd.Next(colors.Length)], new PointF(Xpos, Ypos));
    //помехи и линии
    g.DrawLine(Pens.Black,
        new Point(0, 0), new Point(Width - 1, Height - 1));
    g.DrawLine(Pens.Black, new Point(0, Height - 1), new Point(Width - 1, 0));
}

Пароли нельзя хранить явно. Для сравнения паролей проще всего использовать хэши и сравнивать их.

Если в форме что-то создаётся, то она об этом должна сообщать тому, кто её вызывал

C#

private void button1_Click(object sender, EventArgs e)
{
    RegisterName RegName = new RegisterName();
    RegName.Closed += (_sender, _e) =>
    {
        Show();
        RegName.Dispose();
    };
    RegName.Show();
}

Тут вызывается форма регистрации нового имени. ОК. А как понять, чем завершился процесс? Зарегистрировался или нет?

@MerrinZ · 09.06.2017, 19:41 **[ТС]**

постоянно выводит сообщение о несовпадении, в чем ошибка?

C#

        private void RegIsEnd_Click(object sender, EventArgs e)
        {
            bool userRegParolRes = false, captchaRegRes = false;
 
            if (userParolAlpha.Text == userParolVita.Text)
                userRegParolRes = true;
            else MessageBox.Show("пароли не совпадают");
            if (captcha.Text == textCaptchaSymbol)
                captchaRegRes = true;
            else MessageBox.Show("капча не совпадает");
            if (User.checkNick(userName.Text, userMail.Text, Data.ReadList(Data.dirPath+Data.fileInitPath)) && userRegParolRes && captchaRegRes)
            {
                MessageBox.Show("регистрация завершена");
                joinInUserList(userName.Text, userMail.Text, userMail.Text);
                //calcByo.Show();
                //Hide();
            }
            else MessageBox.Show("пользователь с таким именем или почтой уже существует");
        }

Добавлено через 21 час 5 минут
помогите, может в дизайнере ошибка?

	09.06.2017, 19:41

Вопрос по архитектуре программы

Решение