OOP Account Klasse

10.03.2010, 15:53

PHP-Code:

  <?php
class Account {
    private $nick;
    private $pwd;
    private $isLogin;
    
    public function __construct(  ) 
    {
        init();
    }
    
    public function init(  ) 
    {
        $this->nick    = "";
        $this->pwd     = "";
        $this->isLogin = 0;
    }

    public function SetLoginVars( $eingNick, $eingPwd ) 
    {
        $this->nick = $eingNick;
        $this->pwd  = $eingPwd;
    }
    
    public function Checklogin(  )
    {
        if( ($this->nick == "test") && ($this->pwd == "123") )
        {
            return true;
            $this->isLogin = 1;
        }
        else
        {
            return false;
        }    
    }
    
    public function GetLoginStatus() 
    {
        return $this->isLogin;
    }
}

$login = new Account();
$login->SetLoginVars("test", "123");
$login->CheckLogin();

if( $login->GetLoginStatus() == 1 ) 
{
    echo "Erfolgreich eingeloggt!";
}
else
{
    echo "Fehlgeschlagen!";
}
?>

Was sagt Ihr dazu? Ist das so richtig? Habt Ihr Verbesserungsvorschläge?
Hab ich das einigermaßen Verstanden? Hab jetzt keine Datenbank, sondern Hardcoding benutzt.

php2go · 10.03.2010, 16:01

Hallo

PHP-Code:

  return true;
$this->isLogin = 1;

Wenn ich mich nicht irre, wird hier die zweite Zeile nicht mehr ausgeführt! Besser: die Zeilen vertauschen!

Gruß, php2go

10.03.2010, 16:06

PHP-Code:

  <?php
class Account {
    private $nick;
    private $pwd;
    private $isLogin;
    
    public function __construct(  ) 
    {
        init();
    }
    
    public function init(  ) 
    {
        $this->nick    = "";
        $this->pwd     = "";
        $this->isLogin = 0;
    }

    public function SetLoginVars( $eingNick, $eingPwd ) 
    {
        $this->nick = $eingNick;
        $this->pwd  = $eingPwd;
    }
    
    public function Checklogin(  )
    {
        if( ($this->nick == "test") && ($this->pwd == "123") )
        {
            SetLoginStatus(1);
            return true;
        }
        else
        {
            SetLoginStatus(0);
            return false;
        }    
    }
    
    public function GetLoginStatus(  ) 
    {
        return $this->isLogin;
    }
    
    public function SetLoginStatus ( $Modus ) 
    {
        switch($Modus)
        case 0:
            $this->isLogin = 0;
            break;
        case 1:
            $this->isLogin = 1;
            break;
    }
}

$login = new Account();
$login->SetLoginVars("test", "123");
$login->CheckLogin();

if( $login->GetLoginStatus() == 1 ) 
{
    echo "Erfolgreich eingeloggt!";
}
else
{
    echo "Fehlgeschlagen!";
}
?>

Habe die Klasse um eine SET Methode erweitert. Wie findet Ihr das nun so?

ChrisvA · 10.03.2010, 16:08

Also 2 Dinge hätte ich anders gemacht:
Zum einen würde ich statt $this->isLogin= 0 false und true benutzen.
Andererseits hätte ich Checklogin() am Ende von SetLogin() aufrufen. Das kommt doch sowieso immer vor.

BlackScorp · 10.03.2010, 16:12

ich hätte das so gemacht:

PHP-Code:

  public function SetLoginVars( $eingNick, $eingPwd ) 

    {

        $this->nick = $eingNick;

        $this->pwd  = $eingPwd;

return $this->CheckLogin();

    }

private function Checklogin(){

.... //return true oder false;

} 

 
.....

if( $login->SetLoginVars($nick,$pw) ) 

{

    echo "Erfolgreich eingeloggt!";

}

else

{

    echo "Fehlgeschlagen!";

}

10.03.2010, 16:19

Aber prinzipell ist das so korrekt, was ich gemacht habe?

Diese Klasse würde ich dann in meine Login.php laden und die Sachen ausführen?!

BlackScorp · 10.03.2010, 16:22

im prizip ja, also der untere teil(instanzieren der klasse usw.) würde dann in login.php reinkommen und dann würstest du die methoden , je nach ereignis, ausführen.

PHP-Code:

  if(isset($_POST["submit"])){

$account = new Account();

if($account->loginAccount($_POST["username"],$_POSt["pw"])){

echo "Login erfolgreich";

}else{

echo "Nicht erfolgreich";

}

}

so in etwa

10.03.2010, 16:24

Geil, dann habe ich es verstanden. BlackScorp?! Ich liebe dich!

Danke für deine Codebeispiele!!

Und in der Klasse habe ich ja bei der Methode Checklogin() Hardcodings benutzt. Da würde ich dann einfach eine MYSQL Abfrage mit einer Where Bedingung einfügen?! Ja klar odeR?

BlackScorp · 10.03.2010, 16:29

wobei hier hätte ich was zu meckern:

PHP-Code:

  
 public function SetLoginStatus ( $Modus ) 
    {
        switch($Modus)
        case 0:
            $this->isLogin = 0;
            break;
        case 1:
            $this->isLogin = 1;
            break;
    }

1. wieso public? wenn du nur intern, login status setzt
2. wieso $modus und switch? wenn du gleich $this->isLogin = $modus setzen kannst.
3. wozu überhaupt die Methode wenn du auch gleich weiter oben im code $this->isLogin = 0 oder 1 einsetzen kannst

MFG

mYkon · 10.03.2010, 16:31

PHP-Code:

  public function Checklogin(  )
    {
        if( ($this->nick == "test") && ($this->pwd == "123") )
        {
            SetLoginStatus(1);
            return true;
        }
        else
        {
            SetLoginStatus(0);
            return false;
        }    
    }

Was macht die globale Funktion/en SetLoginStatus(0) hier? ^^

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
[Erledigt] Nur einer bestimmten Klasse Zugriff andere Klasse erlauben	Turamisi	Software-Design	2	20.02.2010 20:31
100% OOP wirklich sinnvoll ?	Pehapeh	PHP-Fortgeschrittene	31	04.12.2009 11:19
Eure Tipps/Hinweise/Empfehlungen zum Thema Modellierung und OOP	ChillSn	Off-Topic Diskussionen	1	30.09.2009 15:19
Tutorial: PHP/MySQL und OOP	Chriz	Tutorials	11	19.08.2009 23:51
Klasse für Objekteigenschaft erzeugen?	Asipak	PHP-Fortgeschrittene	25	30.05.2009 19:45
OOP - Klasse spiegelt Datenbanktabelle wieder	grunz	PHP Tipps 2009	7	30.03.2009 20:59
[Erledigt] Eine klasse einbinden	newWorldOrder	PHP Tipps 2009	2	23.02.2009 19:32
Methode einer anderen Klasse aufrufen	Luka	PHP-Fortgeschrittene	15	09.11.2008 14:19
[Erledigt] MySQL Klasse	methanol	PHP Tipps 2008	23	23.10.2008 09:29
Klasse aus externer Klasse aufrufen	kostja	PHP Tipps 2008	8	07.08.2008 14:13
Variable aus Klasse herausbekommen	GSJLink	PHP Tipps 2008	7	16.02.2008 22:25
String-Parser Klasse - was muss rein?	Matze	PHP Tipps 2007	2	08.04.2007 22:14
mehr als eine Klasse einbinden	Alpha Centauri	PHP-Fortgeschrittene	4	13.04.2006 20:56
Instanz einer Klasse in einer anderen Klasse verwenden	Buhmann	PHP-Fortgeschrittene	7	28.10.2005 23:12

10.03.2010, 15:53
Jens P. Gast Beiträge: n/a	OOP Account Klasse PHP-Code: <?php class Account { private $nick; private $pwd; private $isLogin; public function __construct( ) { init(); } public function init( ) { $this->nick = ""; $this->pwd = ""; $this->isLogin = 0; } public function SetLoginVars( $eingNick, $eingPwd ) { $this->nick = $eingNick; $this->pwd = $eingPwd; } public function Checklogin( ) { if( ($this->nick == "test") && ($this->pwd == "123") ) { return true; $this->isLogin = 1; } else { return false; } } public function GetLoginStatus() { return $this->isLogin; } } $login = new Account(); $login->SetLoginVars("test", "123"); $login->CheckLogin(); if( $login->GetLoginStatus() == 1 ) { echo "Erfolgreich eingeloggt!"; } else { echo "Fehlgeschlagen!"; } ?> Was sagt Ihr dazu? Ist das so richtig? Habt Ihr Verbesserungsvorschläge? Hab ich das einigermaßen Verstanden? Hab jetzt keine Datenbank, sondern Hardcoding benutzt. Geändert von Jens P. (10.03.2010 um 16:08 Uhr).


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

10.03.2010, 16:01
php2go Erfahrener Benutzer Registriert seit: 09.01.2010 Beiträge: 185 PHP-Kenntnisse: Fortgeschritten	Hallo PHP-Code: `return true; $this->isLogin = 1;` Wenn ich mich nicht irre, wird hier die zweite Zeile nicht mehr ausgeführt! Besser: die Zeilen vertauschen! Gruß, php2go

10.03.2010, 16:06
Jens P. Gast Beiträge: n/a	PHP-Code: <?php class Account { private $nick; private $pwd; private $isLogin; public function __construct( ) { init(); } public function init( ) { $this->nick = ""; $this->pwd = ""; $this->isLogin = 0; } public function SetLoginVars( $eingNick, $eingPwd ) { $this->nick = $eingNick; $this->pwd = $eingPwd; } public function Checklogin( ) { if( ($this->nick == "test") && ($this->pwd == "123") ) { SetLoginStatus(1); return true; } else { SetLoginStatus(0); return false; } } public function GetLoginStatus( ) { return $this->isLogin; } public function SetLoginStatus ( $Modus ) { switch($Modus) case 0: $this->isLogin = 0; break; case 1: $this->isLogin = 1; break; } } $login = new Account(); $login->SetLoginVars("test", "123"); $login->CheckLogin(); if( $login->GetLoginStatus() == 1 ) { echo "Erfolgreich eingeloggt!"; } else { echo "Fehlgeschlagen!"; } ?> Habe die Klasse um eine SET Methode erweitert. Wie findet Ihr das nun so? Geändert von Jens P. (10.03.2010 um 16:09 Uhr).

10.03.2010, 16:08
ChrisvA Erfahrener Benutzer Registriert seit: 10.01.2010 Beiträge: 386 PHP-Kenntnisse: Fortgeschritten	Also 2 Dinge hätte ich anders gemacht: Zum einen würde ich statt $this->isLogin= 0 false und true benutzen. Andererseits hätte ich Checklogin() am Ende von SetLogin() aufrufen. Das kommt doch sowieso immer vor.

10.03.2010, 16:12
BlackScorp Erfahrener Benutzer Registriert seit: 25.09.2009 Beiträge: 2.114 PHP-Kenntnisse: Fortgeschritten	ich hätte das so gemacht: PHP-Code: `public function SetLoginVars( $eingNick, $eingPwd ) { $this->nick = $eingNick; $this->pwd = $eingPwd; return $this->CheckLogin(); } private function Checklogin(){ .... //return true oder false; } ..... if( $login->SetLoginVars($nick,$pw) ) { echo "Erfolgreich eingeloggt!"; } else { echo "Fehlgeschlagen!"; }` __________________ Mein kleines Projekt -Cruel Online- 5 von 3 Leuten können kein Bruchrechnen

10.03.2010, 16:19
Jens P. Gast Beiträge: n/a	Aber prinzipell ist das so korrekt, was ich gemacht habe? Diese Klasse würde ich dann in meine Login.php laden und die Sachen ausführen?!

10.03.2010, 16:22
BlackScorp Erfahrener Benutzer Registriert seit: 25.09.2009 Beiträge: 2.114 PHP-Kenntnisse: Fortgeschritten	im prizip ja, also der untere teil(instanzieren der klasse usw.) würde dann in login.php reinkommen und dann würstest du die methoden , je nach ereignis, ausführen. PHP-Code: `if(isset($_POST["submit"])){ $account = new Account(); if($account->loginAccount($_POST["username"],$_POSt["pw"])){ echo "Login erfolgreich"; }else{ echo "Nicht erfolgreich"; } }` so in etwa __________________ Mein kleines Projekt -Cruel Online- 5 von 3 Leuten können kein Bruchrechnen

10.03.2010, 16:24
Jens P. Gast Beiträge: n/a	Geil, dann habe ich es verstanden. BlackScorp?! Ich liebe dich! Danke für deine Codebeispiele!! Und in der Klasse habe ich ja bei der Methode Checklogin() Hardcodings benutzt. Da würde ich dann einfach eine MYSQL Abfrage mit einer Where Bedingung einfügen?! Ja klar odeR?

10.03.2010, 16:29
BlackScorp Erfahrener Benutzer Registriert seit: 25.09.2009 Beiträge: 2.114 PHP-Kenntnisse: Fortgeschritten	wobei hier hätte ich was zu meckern: PHP-Code: `public function SetLoginStatus ( $Modus ) { switch($Modus) case 0: $this->isLogin = 0; break; case 1: $this->isLogin = 1; break; }` 1. wieso public? wenn du nur intern, login status setzt 2. wieso $modus und switch? wenn du gleich $this->isLogin = $modus setzen kannst. 3. wozu überhaupt die Methode wenn du auch gleich weiter oben im code $this->isLogin = 0 oder 1 einsetzen kannst MFG __________________ Mein kleines Projekt -Cruel Online- 5 von 3 Leuten können kein Bruchrechnen

10.03.2010, 16:31
mYkon Erfahrener Benutzer Registriert seit: 29.10.2009 Beiträge: 113 PHP-Kenntnisse: Fortgeschritten	PHP-Code: `public function Checklogin( ) { if( ($this->nick == "test") && ($this->pwd == "123") ) { SetLoginStatus(1); return true; } else { SetLoginStatus(0); return false; } }` Was macht die globale Funktion/en SetLoginStatus(0) hier? ^^