Pomocy jestem początkujący w tym więc popełniam błędy

Question

Nie mam pojęcia czy robię to źle.

napisz część programu, która będzie odpowiedzialna za to że po trzech błędnych próbach wyrzuci informacje - błędne hasło, i zamknie program wprowadzenia hasła - wyłączy po 30 sekundach

string haslo = txtHaslo.Text;

            if (string.IsNullOrWhiteSpace(haslo))
            {
                MessageBox.Show("Hasło nie może być puste!");
            }

            else if (haslo.Length < 8)
            {
                MessageBox.Show("Hasło jest za krótkie, musi posiadac co najmnniej 8 znaków!");
            }

            else if (haslo.Contains("?") || haslo.Contains("-") || haslo.Contains("!"))
            {
                MessageBox.Show("Hasło zawiera niedozwolone znaki");
            }

            else
            {
                MessageBox.Show("Podane hasło jest prawidłowe");

string haslo = txtHaslo.Text;

            if (string.IsNullOrWhiteSpace(haslo))
            {
                MessageBox.Show("Hasło nie może być puste!");
            }

            else if (haslo.Length < 8)
            {
                MessageBox.Show("Hasło jest za krótkie, musi posiadac co najmnniej 8 znaków!");
            }

            else if (haslo.Contains("?") || haslo.Contains("-") || haslo.Contains("!"))
            {
                MessageBox.Show("Hasło zawiera niedozwolone znaki");
            }

            else
            {
                MessageBox.Show("Podane hasło jest prawidłowe");

Dobra zrobiłem a jeszcze jedno pytanko jak dodać takie coś żeby wyłączało się po 30 sekundach jak 3 razy wpiszemy źle hasło.

Answer 1

Sformatuj kod do bloków kodu, łatwiej wtedy odczytać i pomóc
 

if (podaneHaslo>Haslodoodgadniecia)

wiesz dokładnie co siedzi pod wskazanymi zmiennymi ? Debugowałeś to? Warunek może być niepoprawny. 
Zresztą to są stringi które rzutujesz na int więc raczej bym tego nie robił

Answer 2

żeby wyłączało się po 30 sekundach jak 3 razy wpiszemy źle hasło.

Możesz użyć System.Windows.Forms Timer np.:

using System;
using System.Linq;
using System.Windows.Forms;

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        private const int INCORRECT_PASSWORD_TIME_TO_CLOSE_APPLIOCATION = 30; // 30 sekund
        private const int INCORRECT_PASSWORD_MAXIMUM_ATTEMPTS = 3;
        private int incorrect_password_attempts_counter = 0;

        private const int PASSWORD_MIN_LENGTH = 8;
        private const string PASSWORD_FORBIDDEN_CHARACTERS = "?-!";

        private const string CORRECT_PASSWORD = "lorem123";

        public Form1()
        {
            InitializeComponent();
        }

        private void textBox1_KeyDown(object sender, KeyEventArgs e)
        {
            if (e.KeyData == Keys.Enter) checkPassword();
        }

        private void button1_Click(object sender, EventArgs e)
        {
            checkPassword();
        }

        private void checkPassword()
        {
            string password = textBox1.Text.ToString();
            if (! String.IsNullOrWhiteSpace(password))
            {
                if (password.Length >= PASSWORD_MIN_LENGTH)
                {
                    if (! PASSWORD_FORBIDDEN_CHARACTERS.Any(password.Contains))
                    {
                        if (CORRECT_PASSWORD == password)
                        {
                            MessageBox.Show("Podane hasło jest prawidłowe");
                        }
                        else
                        {
                            incorrect_password_attempts_counter++;
                            if (incorrect_password_attempts_counter == INCORRECT_PASSWORD_MAXIMUM_ATTEMPTS)
                            {
                                textBox1.Enabled = false;
                                button1.Enabled = false;                               
                                string message = "Podano " + incorrect_password_attempts_counter + " razy nieprawidłowe hasło\n"
                                               + "Za " + INCORRECT_PASSWORD_TIME_TO_CLOSE_APPLIOCATION + " s. program zostanie zamknięty.";
                                MessageBox.Show(message);
                                Timer IncorrectPasswordApplicationClose = new Timer {
                                    Interval = INCORRECT_PASSWORD_TIME_TO_CLOSE_APPLIOCATION * 1000
                                };
                                IncorrectPasswordApplicationClose.Enabled = true;
                                IncorrectPasswordApplicationClose.Tick += new EventHandler(delegate (Object o, EventArgs a) {
                                    Application.Exit();
                                });
                            }
                        }
                    }
                    else
                    {
                        MessageBox.Show("Hasło zawiera niedozwolone znaki\n" + PASSWORD_FORBIDDEN_CHARACTERS);
                    }
                }
                else 
                {
                    MessageBox.Show("Hasło jest za krótkie,\n musi posiadac co najmnniej 8 znaków!");
                }
            }
            else
            {
                MessageBox.Show("Hasło nie może być puste!");
            }

            textBox1.Focus();
        }
    }
}

 

Comments

Ale po co tak skomplikowanie :D Coś takiego nie zadziała?

        private async void button1_Click(object sender, EventArgs e)
        {
            await checkPassword();
        }
 
        private async void checkPassword()
            // ...
                    MessageBox.Show("Podano N razy nieprawidłowe hasło");
                    await Task.Delay(TIME_TO_CLOSE);
                    Application.Exit();

 

---

Pewnie, że można krócej    ( System.Threading.Timer )

new System.Threading.Timer(s => Application.Exit(), null, (INCORRECT_PASSWORD_TIME_TO_CLOSE_APPLIOCATION * 1000), 0);

 

private void checkPassword()
        {
            string password = textBox1.Text.ToString();
            if (! String.IsNullOrWhiteSpace(password))
            {
                if (password.Length >= PASSWORD_MIN_LENGTH)
                {
                    if (! PASSWORD_FORBIDDEN_CHARACTERS.Any(password.Contains))
                    {
                        if (CORRECT_PASSWORD == password)
                        {
                            MessageBox.Show("Podane hasło jest prawidłowe");
                        }
                        else
                        {
                            incorrect_password_attempts_counter++;
                            if (incorrect_password_attempts_counter == INCORRECT_PASSWORD_MAXIMUM_ATTEMPTS)
                            {
                                textBox1.Enabled = false;
                                button1.Enabled = false;                               
                                string message = "Podano " + incorrect_password_attempts_counter + " razy nieprawidłowe hasło\n"
                                               + "Za " + INCORRECT_PASSWORD_TIME_TO_CLOSE_APPLIOCATION + " s. program zostanie zamknięty.";
                                MessageBox.Show(message);
                                new System.Threading.Timer(s => Application.Exit(), null, (INCORRECT_PASSWORD_TIME_TO_CLOSE_APPLIOCATION * 1000), 0);
                            }
                        }
                    }
                    else
                    {
                        MessageBox.Show("Hasło zawiera niedozwolone znaki\n" + PASSWORD_FORBIDDEN_CHARACTERS);
                    }
                }
                else
                {
                    MessageBox.Show("Hasło jest za krótkie,\n musi posiadac co najmnniej 8 znaków!");
                }
            }
            else
            {
                MessageBox.Show("Hasło nie może być puste!");
            }
 
            textBox1.Focus();
        }

 

 

[ Timer to close the application ]