MYSQL Usernamen Kontrolle

Defkil · 4. April 2012

Hey, mysql ist nicht meine Staerke, also eine Frage, ich habe hier ein Register:

PHP

<?php
$getplayerinfos = mysql_query("SELECT * FROM players WHERE name='$playername'");
$infos = mysql_fetch_array($getplayerinfos);
if($_POST['name']){
$name = $_POST['name'];
$email = $_POST['email'];
$pw = $_POST['password'];
$pwwdh = $_POST['passwordwdh'];
$pass = md5($pw);
$signupdate = date("d.n.Y");
$startmoney = 0;
$header = "Content-Type: text/html; charset=iso-8859-1";
$subject = $settings['title'];
$gamelink = $settings['url']."/game.php?site=rcah&uid=".$name;
$message= "<b>Hi und herzlich willkommen bei unserem Spiel!</b><br>Wir freuen uns, dich begrüssen zu dürfen und wünschen dir viel Glück und viel Spass beim Spielen.<br>
<br>Dein persönlicher Diebspiel-Link ist: $gamelink<br><br>Mit ihm kannst du auf \"Opfersuche\" gehen. Nährere Infos dazu findest du auf unserer Seite.<br>
<br><br>Bis denn,<br>Dein Game-Administrator";
if($pwwdh != $pw)
die("Passwörter stimmen nicht überein. <a href=\"game.php?site=signup\">Zurück</a>");
if(!$email)
die("Du musst eine gültige Email-Adresse angeben. <a href=\"game.php?site=signup\">Zurück</a>");
if(!$name)
die("Du musst einen Namen angeben. <a href=\"game.php?site=signup\">Zurück</a>");
mysql_query("INSERT INTO `players` (`name`, `email`, `password`, `signupdate`, `money`, `game-link`)
VALUES ( '$name', '$email', '$pass', '$signupdate', '$startmoney', '$gamelink')");
echo "Du wurdest erfolgreich registriert. Eine Email mit deinem persönlichen Diebspiel-Link ist unterwegs.";
mail($email, $subject, $message, $header);
}else{
?>
<form method="post" action="game.php?site=signup">
<table border="0">
<tr>
<td colspan="2" class="table-head">Anmeldung</td>
</tr>
<tr>
<td>Name:</td>
<td><input type="text" name="name" size="20"></td>
</tr>
<tr>
<td>Email*:</td>
<td><input type="text" name="email" size="20"></td>
</tr>
<tr>
<td>Passwort:</td>
<td><input type="password" name="password" size="20"></td>
</tr>
<tr>
<td>Passwort (Wdh.): </td>
<td><input type="password" name="passwordwdh" size="20"></td>
</tr>
<tr>
<td colspan="2"><input type="submit" value="Anmelden"></td>
</tr>
</table>
<p>* Dein Diebspiel-Link wird an diese Adresse versendet. </p>
</form>
<? } ?>

Alles anzeigen

und in der Datenbank wird es unter Players gespeichert, wenn man sich mit den Gleichen Namen nochmal registriert, wird es ein mal angenommen und zweitens man kan sich mit beiden PW einloggen, wie kann ich das verhindern?

MFG

Tolive · 4. April 2012

Indem du vorher eine Abfrage machst ob der Name bereits existiert. Die Abfrage könnte etwa so aussehen:

SQL

$sql = "SELECT count(name) FROM players WHERE name='$name'"

Dann halt noch schauen ob die Anzahl größer 0 ist:

PHP

if ($erg > 0)

Wenn ja gibt es den Benutzer bereits ...

UK-Host · 5. April 2012

Erstmal solltest du lernen den Code richtig einzurücken. So lese ich mir die Zeilen nicht durch Danach helfe ich dir gerne

Defkil · 5. April 2012

Habe es jetzt geschaft,

PHP

$checkuser = mysql_query("SELECT name FROM players WHERE name='$name'");
$name_exist = mysql_num_rows($checkuser);
if($name_exist > 0){
echo "Entschuldigung, der Nickname ist schon besetzt. Bitte benutze einen anderen Nicknamen.<a href=\"game.php?site=signup\">Zurück</a>";
unset($name);
exit();
}

Alles anzeigen

Hat gefehlt

UK-Host · 5. April 2012

Warum nimmst du größer/kleiner Zeichen?

PHP

$name_exist = mysql_num_rows(mysql_query("SELECT name FROM players WHERE name='$name'"));
if($name_exist == 0)
{
echo "Entschuldigung, der Nickname ist schon besetzt. Bitte benutze einen anderen Nicknamen.<a href=\"game.php?site=signup\">Zurück</a>";
unset($name);
exit();
}

Alles anzeigen

Zudem, warum löschst du die Variable name?

Buchstabensalat · 5. April 2012

Zu "Warum verwendest du größer/kleiner Zeichen"
Vielleicht, weil bei ==0 der name ja gerade NICHT existiert?

Gut man könnte ==0 verwenden um den Player zu registrieren, und im else-Teil dann n Fehler zu werfen, aber das ist ja jedem selbst überlassen

EDIT:
ich würd vielleicht das exit(); weg lassen. Weil so wird die Seite ja nicht weiter ausgegeben. Die meisten Browser versuchen zwar die Seite auch ohne schließende HTML-Tags möglichst richtig zu rendern, aber schön ist trotzdem anders

UK-Host · 5. April 2012

Dann nimmst du !=

Willkommen! Melden Sie sich an oder registrieren Sie sich.

MYSQL Usernamen Kontrolle

Teilen