[Olden]

Hallo alle zusammen,

ich habe das Login Tutorial von Ben für mich etwas modifiziert.
([PHP] Ein sessionbasiertes Loginsystem)

Leider funktioniert das Skript nicht, seitdem ich die Datenbankabfrage im Konstruktor mit drin habe. Kann den Fehler einfach nicht finden. Die Dateien habe ich mal im Anhang mitgeschickt. Vileicht kann mir einer von euch weiter helfen. Bin für jeden Hilfe dankbar.

Mir würde auch ein fertiges Freeware Skript weiter helfen, falls jemand eins empfehlen kann(habe schon gegoogelt).

Dennis

[Ben]

"Funktioniert nicht" ist mir zu vage. Schreib doch einfach kurz hier rein, was du geändert hast.

[Olden]

Hallo Ben,

ich habe vom Inhalt selbst fast nichts verändert. Habe nur alles in Methoden gepackt um dieses System Objektorientiert zu haben. Dies hat auch am Anfang funktioniert, bis "wie ich schon geschrieben habe" die Datenbankabfrage mit in den Konstruktor gepackt habe.

Sobald ich mich dann Anmelde, müsste ich entweder in den Passwortgeschützten Bereich weitergeleitet werden, wenn das Passwort richtig ist und wenn nicht, dann halt in den nicht passwortgeschützten Bereich. Aber es passiert halt nichts vom beiden. Nach dem Absenden der daten, öffnet sich ein neues leeres Fenster ohne Fehlermeldung, aber auch ohne anderen Text(der mir sagt, dass ich passowrtgeschütztem Bereich bin).

Dennis

[Ben]

Hi,
also wenn ich das hier

Zitat von Olden

Dies hat auch am Anfang funktioniert, bis "wie ich schon geschrieben habe" die Datenbankabfrage mit in den Konstruktor gepackt habe.

richt verstehe, so hast du etwas in dieser Art

PHP-Code:

public function __construct() {
  $sql = '..';
  mysql_query($sql) or exit(mysql_error());
} 


Korrekt?
Hast du da auch vorher die Verbindung zur Datenbank hergestellt? Bzw. wie sieht denn der Aufruf der Methoden aus?

Ich muss mich leider wiederholen .. die Angaben sind mir noch zu vage, da ich nicht weiß, wie deine Klasse bzw. der Aufruf der Methoden nun aussieht.
Klar, ich könnte in der Datei nachschauen, aber da ich nicht an meinem eigenen PC sitze ist das für mich aktuell nicht machbar.

[DasMööp]

evtl da der Construct() kein return liefert ? poste doch mal den Code ..

[Ben]

Code hängt im Archiv an (erster Post).

[Olden]

Also hier ist der Code der nicht funktioniert. Davor hatte ich die DB Verbindung und Abfrage in einer separaten Methode. Da hat es super geklappt. Nur finde ich diese Lösung besser(Logischer und einfacher zu nutzen).

PHP-Code:

<?php
class Login {
    
    public function _construct($table, $user_row, $user, $pass_row, $pass){

          if( !$this->connection = mysql_connect( "localhost", "root", "") ) {
               die( 'Verbindung zum Datenbankserver konnte nicht hergestellt werden.' );
          }

          if( !mysql_select_db("projekt", $this->connection ) ) {
              die ( 'Die Datenbank projekt kann nicht verwendet werden. <br />
                       MySQL-Error: <br />' . mysql_error() );
          }

        $sql = "SELECT
                           *
                      FROM
                           {$table}
                      WHERE
                           ".$user_row." = '" . trim($user) . "'
                      AND
                           ".$pass_row." = '" . md5(trim($pass)) . "'";

        $res = mysql_query($sql) or die( 'Error[SELECT|User]: <br /><pre>' . $sql . '</pre><br />MySQL-Error: ' . mysql_error() );
        mysql_close($this->connection);
        
        if( mysql_num_rows($res) != 1 ) {
              header( 'Location: http://localhost/projekt/loginfehler.html');
              exit();
        }else {
             session_start();
             $_SESSION['loggedIn'] = true;

             header( 'Location: http://localhost/projekt/geheim.php');
             exit();
          }
    }
    
    public static function check_login(){
        session_start();

        if( !$_SESSION['loggedIn'] ) {
            header( 'Location: http://localhost/projekt/loginformular.html' );
            exit();
        }

    }
    
    public static function logout(){
        session_start();
        session_destroy();
        header( 'Location: http://localhost/projekt/loginformular.html' );
         exit();
    }

}

Danke euch für die schnelle Antworten.

Dennis

[Ben]

Mach mal aus _construct __construct() .. zwei Unterstriche!

[DasMööp]

Hmm, bin wohl Blind ... na dran aber nicht nah genug.
Werd mich mal wieder um meine Excepti kümmern

[Olden]

Hallo,

ich danke euch für die Hilfe. Jetzt gehts! Es lag am zweiten Unterstrich, wie Ben gesagt hat...

Dennis