[C#] [MySql] Eintrag vorhanden

[Darkhomer]

Guten Tag,

Ich wollte mich mal in C# Probieren.

bekomme es einfach nur nicht hin eine Abfrage zu machen ob der jeweilige username schon vergeben ist mein Code zur abfrage lautet:

 Spoiler

if (OpenConnection() == true)
{
string query = "SELECT * FROM benutzer WHERE username = "+ tbox_benutzer +"";

MySqlCommand cmd = new MySqlCommand(query, conn);
MySqlDataReader rdr = cmd.ExecuteReader();

if (rdr.HasRows == true)
{
MessageBox.Show("Benutzer Vorhanden!");
}
else
{
MessageBox.Show("Benutzername FREI");
}
conn.Close();
}

könnte mir da mal jemand etwas unter die Arme greifen?
Danke im vorraus

MfG

[Serraniel]

#moved

[florian0]

string query = "SELECT * FROM benutzer WHERE username = "+ tbox_benutzer +"";

Resultiert in einem QueryString der so aussieht:

SELECT * FROM benutzer WHERE username = florian0

Um Texte zu vergleichen brauchst du aber " oder '

SELECT * FROM benutzer WHERE username = "florian0"

SELECT * FROM benutzer WHERE username = 'florian0'

[Serraniel]

Gehe des Weiteren davon aus das

tbox_benutzer

ne Textbox auf deiner GUI ist, oder? Dann musst du auch

tbox_benutzer.Text

verwenden, damit du den Inhalt der Textbox in das Query einbaust.

[warfley]

Eventuell solltest du auch darüber nachdenken seinen querry zu parametrisieren, dann kümmert sich der Treiber selbst um die Formatierung (mal ganz davon abgesehen das es sicherer ist)
Schau mal hier:

[Darkhomer]

Originally Posted by Serraniel

Gehe des Weiteren davon aus das

tbox_benutzer

ne Textbox auf deiner GUI ist, oder? Dann musst du auch

tbox_benutzer.Text

verwenden, damit du den Inhalt der Textbox in das Query einbaust.

Danke. das war der fehler xD

kann mal Passieren

Danke aber auch allen weiteren.
MfG

so Funktioniert es. fals es noch jemand übersieht xD

 Spoiler

if (OpenConnection() == true)
{
string query = "SELECT * FROM benutzer WHERE username = '"+ tbox_benutzer.Text +"'";

MySqlCommand cmd = new MySqlCommand(query, conn);
MySqlDataReader rdr = cmd.ExecuteReader();

if (rdr.HasRows == true)
{
MessageBox.Show("Benutzer Vorhanden!");
}
else
{
MessageBox.Show("Benutzername FREI");
}
conn.Close();
}

[mrapc]

Was bereits gesagt wurde

if (OpenConnection() == true)
{
string query = "SELECT * FROM benutzer WHERE username = [COLOR="Red"]'"+ tbox_benutzer.Text +"'"[/COLOR];

MySqlCommand cmd = new MySqlCommand(query, conn);
MySqlDataReader rdr = cmd.ExecuteReader();

if (rdr.HasRows == true)
{
MessageBox.Show("Benutzer Vorhanden!");
}
else
{
MessageBox.Show("Benutzername FREI");
}
conn.Close();
}

Ist schlecht


mach lieber:
cmd.CommandText = "SELECT * FROM benutzer WHERE username = ?username";
cmd.Parameters.AddWithValue("?username", tbox_benutzer.Text);

[Xio.]

Originally Posted by mrapc

Was bereits gesagt wurde

if (OpenConnection() == true)
{
string query = "SELECT * FROM benutzer WHERE username = [COLOR="Red"]'"+ tbox_benutzer.Text +"'"[/COLOR];

MySqlCommand cmd = new MySqlCommand(query, conn);
MySqlDataReader rdr = cmd.ExecuteReader();

if (rdr.HasRows == true)
{
MessageBox.Show("Benutzer Vorhanden!");
}
else
{
MessageBox.Show("Benutzername FREI");
}
conn.Close();
}

Ist schlecht


mach lieber:
cmd.CommandText = "SELECT * FROM benutzer WHERE username = ?username";
cmd.Parameters.AddWithValue("?username", tbox_benutzer.Text);

mein benutzername ist aber DROP DATABASE;

Fuer den OP: