Login + Mysql

pinky · 22.02.2010, 17:05

Hey
Hoffe ihr könnt mir bei meinen Problem helfen.

Habe mir in Delphi ein Login system zusammengebastel wo man sich registrieren und Einloggen kann. Das Registrieren klapt wunderbar nur der Login klappt nicht.
Könntet ihr mir mein Php Script mal anschauen bitte und sagen ob da Fehler drin sind?
lg

Hier mein Php Code

PHP-Code:

  <?php

$sqlhost = "xxxxxxxxxxxxxxxxxxxx";

// Username und Passwort zum einloggen in den Datenbankserver

$sqluser = "xxxxxxxxxxx";

$sqlpasswort = "xxxxxxxxxxx";

// Name der Datenbank

$sqldb = "xxxxxxxxxx";

 
$conn = mysql_connect($sqlhost, $sqluser, $sqlpasswort);

mysql_select_db($sqldb, $conn);

 
header("Content-Type: text/plain");

if (isset($_REQUEST['username'])) $username = mysql_real_escape_string($_REQUEST['username']); else $username = "";

if (isset($_REQUEST['passwort'])) $passwort = mysql_real_escape_string($_REQUEST['passwort']); else $passwort = "";

if (($username != "") AND ($passwort != "")){

    $result = mysql_query("SELECT * FROM scriptmaker_users WHERE username='".$username."' AND passwort='".md5($passwort)."';") OR die("MySQL-Fehler: ".mysql_error());

    $c = 0;

    while ($row = mysql_fetch_array($result)){

        if ($row["passwort"] != md5($passwort)){

            echo("Fehler beim Login");

            exit;

        } else {

            $c++;

            if ( (int) $row["aktiviert"] == 1){

                echo("Erfolgreich Eingelogt");

            }    else if ( (int) $row["aktiviert"] == 0){

                echo("Noch Nicht Aktiviert");

            } else {

                echo "Sollte nicht auftreten :D";

            }

        }

    }

    if ($c == 0){

        echo "Login nicht möglich. Username und/oder Passwort falsch";

    }        

}

echo "\r\n";

print_r($_POST);

print_r($_GET);

print_r($_REQUEST);

?>

P.S Daten von Db ausgeixt

nikosch · 22.02.2010, 17:11

Zitat:

';

Bitte keine Semikolons für mysql_query.

Zitat:

SELECT *

Bitte nicht wahllos selektieren, sondern konkrete Feldnamen verwenden!

Zitat:

while ($row = mysql_fetch_array($result)){

Unnütz. Benutze if und prüfe zusätzlich die Menge der Ergebnisse. Es sollte nur ein gültiges Wertpaar (oder eben keinen Datensatz) geben.

Zitat:

if ($row["passwort"] != md5($passwort)){

Unnütz. Dieser Fall kann lt. Deiner Query nicht eintreffen.

Zitat:

else if ( (int) $row["aktiviert"] == 0){
echo("Noch Nicht Aktiviert");
} else {
echo "Sollte nicht auftreten

";
}

Warum machst Du nicht alles ausser 1 zum else-Fall?

Und was zum Geier heißt:
Zitat:

Login klappt nicht.

pinky · 22.02.2010, 17:18

Hey
leider kenn ich mich nicht so aus in Php und wegen enen Script gleich
des ganze Php lernen dazu fehlt mir leider zeit.
Ich brauch des Php Script für ein Login system wo cih mir mit Delphi erstellt habe.
wen ich in delphi username und pw eingib sollte es in der DB prüfen ob es vorhanden ist und wen ja ob er aktiviert ist wen ja sollte login klappen wen nicht aktiviert ist sollte es nicht klappen.
Könntest du mir den Code pls anpassen?
lg

jGeee · 22.02.2010, 17:39

dir fehlt die zeit, und anderen nicht?! - würde jeder programmierer alles verschenken, könnten die agenturen dicht machen

nikosch · 22.02.2010, 17:51

Ist ja wohl ein Witz. Da hat ja meine Antwort oben schon mehr Zeit gekostet, als Du selbst bereit bist zu investieren. Dass Du auf Fragen nicht mal eingehst und Deine grausige Grammatik erwähnen wir mal erst gar nicht.

[MOD: verschoben]

php-guru · 22.02.2010, 20:57

hey,
ich würde dir empfehlen ein fertiges script zu nehmen, ich hab da eines gemacht, bei dem auch auf den internen seiten grptüft wird ob du überhaupt eingeloggt bist,
bei Ineresse bitte ne PN an mich

nikosch · 22.02.2010, 21:01

Wie immer darf nicht in einen Topf geworfen werden:

- Registrierung (einmalig für einen User)
- Authentifizierung (einmalig pro Session)
- Autorisierung (bei jedem Request auf geschützte Bereiche)

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
MYSQL Login und Navigation	dh1sbg	PHP Tipps 2008	1	28.12.2008 11:49
GUI-Tool kann sich nicht mit MySql verbinden	KWitt72	PHP Tipps 2008	3	21.12.2008 16:56
Mysql LoginScriptzu MySqli LoginScript	lithium	PHP Tipps 2008	27	17.11.2008 19:48
Login Script mit if-Abfrage für Rechte (in vbdg mit mySQL)	haubna	PHP Tipps 2008	9	18.08.2008 14:03
[Erledigt] MySQL - ERROR 1044 bei erstellen einer Datenbank	_youngenterpriser_	Datenbanken	2	05.02.2008 17:56
PHP / MySQL \| Login Script Problem		PHP Tipps 2006	15	17.02.2006 12:26
Kein Zugriff über ODBC mit der IP-Adresse auf MySql DB		Datenbanken	4	09.02.2006 11:04
[Erledigt] Mysql 4.1.x unter php 4.3.9		Datenbanken	3	15.11.2005 13:49
Login mithilfe von MySql	Brease	PHP Tipps 2007	20	09.11.2005 15:07
Suche Tipps für Persormance-Steigerung (Geld für Nützliches)		Beitragsarchiv	18	16.08.2005 10:57
MYSQL läuft nur wenn /tmp auf 777		Datenbanken	5	06.07.2005 08:38
mysql root passwort vergessen		Datenbanken	1	29.05.2005 11:33
MYSQL root login und logging Problem		Datenbanken	2	16.03.2005 08:41
PHP mysql problem beim login.	c01001	PHP Tipps 2004	8	05.08.2004 15:57
php login mit mysql	c01001	PHP Tipps 2004	2	04.08.2004 21:30

22.02.2010, 17:05
pinky Neuer Benutzer Registriert seit: 22.02.2010 Beiträge: 2 PHP-Kenntnisse: Anfänger	Login + Mysql Hey Hoffe ihr könnt mir bei meinen Problem helfen. Habe mir in Delphi ein Login system zusammengebastel wo man sich registrieren und Einloggen kann. Das Registrieren klapt wunderbar nur der Login klappt nicht. Könntet ihr mir mein Php Script mal anschauen bitte und sagen ob da Fehler drin sind? lg Hier mein Php Code PHP-Code: <?php $sqlhost = "xxxxxxxxxxxxxxxxxxxx"; // Username und Passwort zum einloggen in den Datenbankserver $sqluser = "xxxxxxxxxxx"; $sqlpasswort = "xxxxxxxxxxx"; // Name der Datenbank $sqldb = "xxxxxxxxxx"; $conn = mysql_connect($sqlhost, $sqluser, $sqlpasswort); mysql_select_db($sqldb, $conn); header("Content-Type: text/plain"); if (isset($_REQUEST['username'])) $username = mysql_real_escape_string($_REQUEST['username']); else $username = ""; if (isset($_REQUEST['passwort'])) $passwort = mysql_real_escape_string($_REQUEST['passwort']); else $passwort = ""; if (($username != "") AND ($passwort != "")){ $result = mysql_query("SELECT * FROM scriptmaker_users WHERE username='".$username."' AND passwort='".md5($passwort)."';") OR die("MySQL-Fehler: ".mysql_error()); $c = 0; while ($row = mysql_fetch_array($result)){ if ($row["passwort"] != md5($passwort)){ echo("Fehler beim Login"); exit; } else { $c++; if ( (int) $row["aktiviert"] == 1){ echo("Erfolgreich Eingelogt"); } else if ( (int) $row["aktiviert"] == 0){ echo("Noch Nicht Aktiviert"); } else { echo "Sollte nicht auftreten :D"; } } } if ($c == 0){ echo "Login nicht möglich. Username und/oder Passwort falsch"; } } echo "\r\n"; print_r($_POST); print_r($_GET); print_r($_REQUEST); ?> P.S Daten von Db ausgeixt


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

22.02.2010, 17:18
pinky Neuer Benutzer Registriert seit: 22.02.2010 Beiträge: 2 PHP-Kenntnisse: Anfänger	Hey leider kenn ich mich nicht so aus in Php und wegen enen Script gleich des ganze Php lernen dazu fehlt mir leider zeit. Ich brauch des Php Script für ein Login system wo cih mir mit Delphi erstellt habe. wen ich in delphi username und pw eingib sollte es in der DB prüfen ob es vorhanden ist und wen ja ob er aktiviert ist wen ja sollte login klappen wen nicht aktiviert ist sollte es nicht klappen. Könntest du mir den Code pls anpassen? lg

22.02.2010, 17:39
jGeee Erfahrener Benutzer Registriert seit: 09.03.2009 Beiträge: 145 PHP-Kenntnisse: Fortgeschritten	dir fehlt die zeit, und anderen nicht?! - würde jeder programmierer alles verschenken, könnten die agenturen dicht machen

22.02.2010, 17:51
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.994 PHP-Kenntnisse: Fortgeschritten	Ist ja wohl ein Witz. Da hat ja meine Antwort oben schon mehr Zeit gekostet, als Du selbst bereit bist zu investieren. Dass Du auf Fragen nicht mal eingehst und Deine grausige Grammatik erwähnen wir mal erst gar nicht. [MOD: verschoben] __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --

22.02.2010, 20:57
php-guru Benutzer Registriert seit: 30.12.2009 Beiträge: 69 PHP-Kenntnisse: Fortgeschritten	hey, ich würde dir empfehlen ein fertiges script zu nehmen, ich hab da eines gemacht, bei dem auch auf den internen seiten grptüft wird ob du überhaupt eingeloggt bist, bei Ineresse bitte ne PN an mich

22.02.2010, 21:01
nikosch moderatives Dielektrikum Registriert seit: 21.05.2008 Beiträge: 35.994 PHP-Kenntnisse: Fortgeschritten	Wie immer darf nicht in einen Topf geworfen werden: - Registrierung (einmalig für einen User) - Authentifizierung (einmalig pro Session) - Autorisierung (bei jedem Request auf geschützte Bereiche) __________________ -- One pixel is still too big. Please make it smaller. ASAP. Initiative Mittelstand. Die wichtigste Gestaltungsregel im Screendesign ist Pi mal Daumen des Arbeitgebers. --