Daten aus MySQL Datenbank auslesen

[anonymous-f4h279]

Hallo ihr Coder,

ich versuche derzeit ein simples Login System zu coden.
Hier erstmal der Code:

private void button1_Click(object sender, EventArgs e)
        {
            string conn = "Server=localhost;Port=3306;Database=users;Uid=root;password=;";
            MySqlConnection connection = new MySqlConnection(conn);
            MySqlCommand command = connection.CreateCommand();
            command.CommandText = "select * from usertable";
            connection.Open();
            MySqlDataReader Reader = command.ExecuteReader();
            while (Reader.Read())
            {
            //reader soll lesen und daten aus textboxen mit der datenbank vergleichen
            }
        }

Nun muss der Reader auslesen und Daten aus Textboxen mit der Datenbank vergleichen.
Da weiß ich nicht mehr weiter ;<

#edit
Eventuell hilft es jemanden von euch mal in Problemen.
Ich hab mal ein wenig Brainstorming gemacht und bin auf die Lösung gekommen

private void button1_Click(object sender, EventArgs e)
        {
            string conn = "Server=localhost;Port=3306;Database=users;Uid=root;password=;";
            MySqlConnection connection = new MySqlConnection(conn);
            MySqlCommand command = connection.CreateCommand();
            command.CommandText = "select * from usertable";
            connection.Open();
            MySqlDataReader Reader = command.ExecuteReader();
            while (Reader.Read())
            {
                String login = Reader["username"].ToString();
                String pass = Reader["password"].ToString();
                if (textBox1.Text.Equals(login) && textBox2.Text.Equals(pass))
                {
                    Form2 Form = new Form2();
                    Form.Show();
                    this.Hide();
                }
                else
                {
                    MessageBox.Show("Incorrect username or password!", "Error", MessageBoxButtons.OK, MessageBoxIcon.Error);
                }
            }
        }

    }

Es gibt da aber noch was auszubessern!
Ihr solltet wenn ihr das verwendet, die Verbindung zur Datenbank über einen anderen Button herstellen, denn sonst gibts die MessageBox auch beim erfolgreichen login.


Grüße,Drewfire

[fatalaty]

Guten Morgen,

du solltest lieber das Query von vorne rein so anpassen, dass im result nur der benutzer steht, der sich auch einloggen will. Also:

command.CommandText = "select * from usertable WHERE username = " + login;

Wenn du mehr mitglieder hast muss er sonst im schlechtesten Fall alle usernamen abgleichen.

Zudem rate ich dir, das passwort als hash in der Datenbank zu speichern und folglich dann auch nur die hashes zu vergleichen. Sollte mal jemand zugriff zu deiner Datenbank erhalten hat der Angreifer dann wenigstens keine Passwörter im Klartext.

Gruß

[anonymous-f4h279]

Danke, wie schreibe(insert) ich in die Datenbank hinein? Das wäre dann fürs registrieren.

[fatalaty]

Originally Posted by Drewfire

Danke, wie schreibe(insert) ich in die Datenbank hinein? Das wäre dann fürs registrieren.

Im allgemeinen oder den Hash?

[anonymous-f4h279]

Im allgemeinen erstmal als Text.
Es gibt die Tabelle usertable und 3 Spalten darin.
ID(primary key),username,password.

ID als Integer und die andren als Text.

[#SoNiice]

Dazu kann ich dir diese Seite ans Herz legen:

Wenn du genauere Hilfe zu 'ner Queryerstellung mit Textboxen etc. brauchst, melde dich einfach bei mir, schaue leider nicht dauerhaft in die Threads hier, Skype wäre auch okay.

//edit:

Das wäre dann dein Query, wenn es aus Textboxen ausgelesen wird:

string conn = "Server=localhost;Port=3306;Database=users;Uid=root;password=;";
    MySqlConnection myConnection = new MySqlConnection(conn);
    string myInsertQuery = "INSERT INTO usertable (username, password) 
                            Values('" + textBoxUSER + "', '" + textBoxPW + "')";
    MySqlCommand myCommand = new MySqlCommand(myInsertQuery);
    myCommand.Connection = myConnection;
    myConnection.Open();
    myCommand.ExecuteNonQuery();
    myCommand.Connection.Close();

[anonymous-f4h279]

Ein Danke an Walter, der mich darauf hingewiesen hat, dass ich Auto Increment für die ID Spalte festlegen muss

[Lawliet]

Allgemein solltest du dein Projekt nicht so aufbauen, dass deine SQL Connection direkt in dem OnClick Event steht.

Ein möglicher aufbau ist:
Datalayer<-->Repository<-->Entities

Der Datalayer bastelt dir das Query zusammen und ballert es auf die Datenbank:


Beispielmethode

var sqlParameters = new SqlParameter[] {
                new SqlParameter( "@id", SqlDbType.Int ) {
                    Value = id
                }
            };
            const string selectQuery = "SELECT * FROM tblXXXXX WHERE id = @id";
            var connection = CreateSqlConnection();

            var command = CreateCommand(selectQuery, connection, sqlParameters);


            var reader = command.ExecuteReader();
            TaetigkeitEntity result = null;
            if (reader.Read())
            {
                result = MapXXXXX(reader);
            }
            connection.Close();
            return result;

CreateCommand

 private static SqlCommand CreateCommand(string query, SqlConnection connection, SqlParameter[] sqlParameters = null)
        {
            var command = connection.CreateCommand();
            command.Connection = connection;
            command.CommandType = CommandType.Text;
            command.CommandText = query;
            if ( sqlParameters != null ) {
                command.Parameters.AddRange(sqlParameters);
            }
            return command;
        }

CreateSQLConnection

private static SqlConnection CreateSqlConnection()
        {
            var connectionInfo = ConfigurationManager.ConnectionStrings["TheOnlyOneConnection"]; //sollte in der App.config gespeichert sein
            var connection = new SqlConnection(connectionInfo.ConnectionString);
            connection.Open();
            return connection;
        }

MAP Methode

private XXXXXEntity MapPerson(SqlDataReader reader)
        {
            var personEntity = new PersonEntity
                             {
                                 Vorname = reader["vorname"].ToString(),
                                 Nachname = reader["nachname"].ToString(),
                                 Ausbildungsstart = (DateTime) reader["ausbildungsstart"],
                                 PasswordHash = reader["md5passwort"].ToString(),
                                 Nummer = (int) reader["id"]
                             };
            return personEntity;
        }

Im Repository steht dann nurnoch:

public XXXXXEntity Get(string nachname)
        {
            var result = _datalayer.GetPerson(nachname);
            return result;
        }

Und das XXXXXEnitity hat eben die Properties die du haben willst:

public class XXXXXEntity
    {
        public string Vorname { get; set; }
        public string Nachname { get; set; }
        public int Nummer { get; set; }
        public DateTime Ausbildungsstart { get; set; }
        public string PasswordHash { get; set; }
    }

Im Code kannst du dann in deinem OnClick Event so auf die Properties zugreifen:

_XXXXXRepository.Get(stringNachname).Nachname/Vorname/Nummer/usw

Genauso kann du eben auch Updaten / Create / Insert ausführen.

Der Vorteil?
Du musst nicht bei jedem Event den ganzen Code reinhacken, du kannst einfach im Datalayer die Quelle ändern (natürlich kannst XML, andere DB, INI), ...

[CryLike]

Ich würde erstmal deine mysql daten aus dem programm code nehmen.
Einmal decompilen und jeder hat zugriff auf deine DB.

[anonymous-f4h279]

Weil da auch so viel drauf ist und hier meine IP steht
Es sollte nun langsam jeder geblickt haben, dass es nur zum üben ne lokale DB ohne nix ist.

[kissein]

Kleine Anmerkung zum #edit vom OP

Es ist "ok" die Mysql Verbindung nicht zu beenden, wenn es nicht:
-zuviele idle connections gibt und dadurch das mysql connection limit erreicht wird.

-idle verbindung gibt die über Stunden nichts tut, da dass Mysql Timeoutlimit bei 8 Stunden liegt. Benutzt man aber die Connection danach erneut, erhält man “MySQL server has gone away”...

[#SoNiice]

Drew, ich glaube es wurde eher gemeint, dass du allgemein die MySQL Daten nicht im Programm haben sollst.