Überprüfen und verweigern wenn vorhanden

[Ryckmaster]

Überprüfen und verweigern wenn vorhanden

Hey leute,

habe nen script geschrieben der einen neuen administrator in die datenbank einträgt, mithilfe eines formulars.
Soweit geht es.
Ich bekomme es aber einfach nicht hin das er überprüft ob der Benutzername/Administrator schon vorhanden.
Wenn ja dann fehlermeldung und neuer versuch
wenn nein dann eintragen und gut ist.
mir ist eigentlich klar das es eine if-function sein muss, aber ich habe es bisher einfach nicht hinbekommen.
hier mal der code:

<?php
//überprüfen ob Administrator angemeldet ist
require "./session.inc.php"; 
?>
<html>
	<head><title>Administrationsbereich</title></head>
	<body>
	<?php
	
	//datenbankverbindung 

function datenbank_verbinden()
{
$db_server="localhost";
$db_name="smuv";
$db_user="root";
$db_passwort="";

$db_verbindung=mysql_connect($db_server,$db_user,$db_passwort);
if(!$db_verbindung)
die("Der Server kann nicht erreicht werden.");
if(!mysql_select_db($db_name,$db_verbindung))
die("Die Datenbank kann nicht angesprochen werden.");
return $db_verbindung;
}


     ?>
<form action="<?$_SERVER['php_self']?>" method="get">
<table>
<tr><th><b><p align=left>Neuer Administrator:</p></b></th> <th><input type="text" name="na"></th></tr><p align=right><a href=logout.php>Logout</a></p>
<tr><th><b><p align=left>Sein gewünschtes Password:</p></b></th> <th><input type="text" name="gp"></th></tr>
<tr><th><input type="submit" name="ausfuehren" value="Abschicken"></th></tr>
</table>
</form><br><br>
	<?php
	
	
//Daten überprüfen
	
function check_eintraege()
{
if(empty($_GET['na']))
{
	echo "<b>Fehler! Sie haben keinen Administratornamen angegeben.</b><br><br>";
return false;
}
if(empty($_GET['gp']))
{
echo"<b>Fehler! Sie haben kein Passwort angegeben.</b><br><br>";
return false;
}
return true;
}

//Neuen Administrator speichern

function schreib_admin($verbindungskennung)
	{
$verbindungskennung=datenbank_verbinden();
$eintrag="INSERT INTO administrator (Administrator, Kennwort) VALUES ('$_GET[na]','$_GET[gp]');";
$eintrag=mysql_query($eintrag,$verbindungskennung);
	}


// wenn alles okay, dann trage ein

function admin_speichern()
{
$verbindungskennung=datenbank_verbinden();
if(check_eintraege($verbindungskennung))
{
	schreib_admin($verbindungskennung);
	echo "<b>Der Administrator wurde hinzugefügt!</b><br>";
   echo "<b>Er kann sich nun einloggen!</b><br>";
   echo "<b>Sie können auch weitere Administrator eingeben.</b>";
}
else
{
echo "<b>Der Administrator wurde <u>nicht</u> hinzugefügt</b><br><br>";
echo "<b>Bitte versuchen sie es nochmal!!</b><br><br><br>";
}
}

//Von hier werden die Funktionen formular() und 
//  daten_merken() aufgerufen
if (isset($_GET['ausfuehren']))
	admin_speichern();
   
echo "<a href=admin_menu.php> Zurück zum Menü</a>";
?>
<body>
</html>

lg
und
thx

 

[Guardian]

Dafür wäre es sicher hilfreich zuerst die eingetragenen Admins mit einer SELECT Anweisung auszulesen und dann mit dem übergebenen Wert zu vergleichen, bevor die INSERT Anweisung an die Datenbank weiter gereicht wird.
Der Parameter Deiner Funktion: schreib_admin($verbindungskennung) ist nicht nötig, die Verbindungskennung wird als Rückgabewert von datenbank_verbinden() generiert.
Deine Funktion: check_eintraege() überprüft nur ob alle Felder ausgefüllt sind, nicht die Datenbank. Vielleicht ist das Dein Denkfehler... . Diese Funktion braucht für diesen Zweck auch nicht die Verbindungskennung... .

Grundsätzlich: trage bitte niemals ohne Prüfung GET- Variablen in eine Datenbank ein, sieh Dir mal z.B. die PHP Funktion htmlentities() an.
Ebenso hat fast jede PHP Funktion einen auswertbaren Rückgabewert, der auch benutzt werden sollte... .

dann ein frohes Neues...
bye
Guardian

 

[thomo]

Der ganze Code da oben ist (sorry) ein Haufen Müll....

Zuerst solltest du das entsprechende Feld (Namen des Admins) in der DB als UNIQUE setzen, zum anderen würde ich einfach mit einer SELECT-Abfrage prüfen, ob der Name schon in der DB existiert (bei Nichtvorhandensein liefert dir die Abfrage ein FALSE zurück -> was bedeutet, dass es diesen Admin noch nicht gibt).

Du brauchst für solche Sachen keine eigenen Funktionen zu deklarieren, alles was du brauchst liefert dir PHP und MySQL von Haus aus mit.

Gruß thomo

 

[Ryckmaster]

okay mit UNIQUE hat es geklappt, hätte ich eigentlich selber drauf kommen können.
aber nun möchte ich noch das er auch ne fehlermeldung ausgibt.
wie würde ich das denn am besten machen?

lg

hat sich erledigt danke ;-)

 

[Broken Sword]

=> Closed

Gruß
Broken Sword