Diskuze: Loggin
Je tam moc chyb?
Petr Nymsa:2.10.2013 10:51
Bohužel tam je mnoho chyb. Předpokládám že nevíš co dělá klíčové slovo abstract ??
Celý návrh je špatně. Do začátku poradím. Snaž se to udělat co nejjednodušeji a postupně přijdeš na to jak to vylepšit. Uděláš si třídu Uživatel, kteýr bude mít jméno, heslo a bool přihlášen. Tuto třídu udělej podle návrhu správně -> použij vlastnosti apod.
Až budeš mít tohle, postni nám to sem a poradíme dál 
Myslím, že jsem konečně pochopil to, co se mi tu snažíte vysvětlit
již od začátku... Zbytečně jsem si komplikoval život. Před chvílí jsem
došel ze školy a za 5 minut jsem napsal nový kód. Teď se mlátím do hlavy,
jak jsem mohl být tak slepý 
:[ Mrkněte na to:
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}
/// <summary>
/// Nový Form2
/// </summary>
private void NewForm2()
{
this.Hide();
Form2 newForm2 = new Form2();
newForm2.Show();
}
private void loggin_Click(object sender, EventArgs e)
{
if (user.Checked)
{
User newUser = new User(accountName.Text, accountPassword.Text);
newUser.Loggin();
if (newUser.logged)
NewForm2();
}
else if (admin.Checked)
{
User newAdmin = new Admin(accountName.Text, accountPassword.Text);
newAdmin.Loggin();
if (newAdmin.logged)
NewForm2();
}
}
private void exit_Click(object sender, EventArgs e)
{
Application.Exit();
}
}
// USER
public class User
{
/// <summary>
/// Jméno účtu
/// </summary>
public string name { get; private set; }
/// <summary>
/// Heslo účtu
/// </summary>
public string password { get; private set; }
/// <summary>
/// Je uživatel přihlášen?
/// </summary>
public bool logged;
public User(string name, string password)
{
this.name = name.ToUpper();
this.password = password.ToUpper();
}
public virtual void Loggin()
{
logged = true;
MessageBox.Show(String.Format("Ahoj uživateli {0}. Tvé heslo je: {1}.", name, password));
}
}
// ADMIN
public class Admin : User
{
public Admin(string name, string password):base(name, password)
{
}
public override void Loggin()
{
if ((name == "ADMIN") && (password == "HESLO"))
{
logged = true;
MessageBox.Show(String.Format("Ahoj admine {0}. Tvé heslo je: {1}.", name, password));
}
else
{
logged = false;
MessageBox.Show("Zadal jste špatné přihlašovací údaje nebo nejste admin!");
}
}
}
Jakub Lásko[Saarix]:3.10.2013 17:54
Zirko má pravdu eště bych si tam poupravil to, že když je newUser
prázdný, tak se vytvoří instance Uživatele/Admina, ale opakovat to vždy je
zbytečné. Ale eště radši bych to dal někam k registraci a zde ověřoval z
listu registrovaných
Opravdu jsem si teď ověřil, že každé vytvoření nové instance
zbytečně plní operační paměť. Tak snad jsem to vyřešil
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}
private void NewForm2()
{
this.Hide();
Form2 newForm2 = new Form2();
newForm2.Show();
}
private void loggin_Click(object sender, EventArgs e)
{
if ((user.Checked) && (accountName.Text.ToUpper() == "USER") && (accountPassword.Text.ToUpper() == "HESLO"))
{
User newUser = new User(accountName.Text, accountPassword.Text);
newUser.Loggin();
if (newUser.logged)
NewForm2();
}
/////////////////////
else if ((admin.Checked) && (accountName.Text.ToUpper() == "ADMIN") && (accountPassword.Text.ToUpper() == "HESLO"))
{
User newAdmin = new Admin(accountName.Text, accountPassword.Text);
newAdmin.Loggin();
if (newAdmin.logged)
NewForm2();
}
else
MessageBox.Show("Zadal jste špatné přihlašovací údaje!");
}
private void exit_Click(object sender, EventArgs e)
{
Application.Exit();
}
}
// USER
public class User
{
public string name { get; private set; }
public string password { get; private set; }
public string account;
public bool logged;
public User(string name, string password)
{
this.name = name.ToUpper();
this.password = password.ToUpper();
}
public virtual void Loggin()
{
account = "USER";
logged = true;
}
}
// ADMIN
public class Admin : User
{
public Admin(string name, string password):base(name, password)
{
}
public override void Loggin()
{
account = "ADMIN";
logged = true;
}
}
Zobrazeno 29 zpráv z 29.