Function liefert immer false @ tutorials.de: Tutorials, Forum & Hilfe

Divo1984 · 04.02.12 22:34

Ich habe nun das nächste Problem, ich habe meine Klasse erweitert um eine Funktion UserExist die liefert mir immer false zurück

PHP-Code:


<?php
require 'classes/class_DB.php';

class userManagement extends DB
{
    public $dbh;
    public $userExist=false;
    
    
    public function __construct(&$dbh)
    {
        parent::__construct();
        if(is_object($dbh) && ($dbh instanceof PDO))
            $this->dbh=$dbh;

    }
    public function UserExist($user)
    {
        try
        {
            $sql="SELECT u_id 
                    FROM user
                   WHERE u_uname='$user'";
            $result=$this->dbh->prepare($sql);
            $result=$this->dbh->query($sql);
            $tmp=$result->fetchAll(PDO::FETCH_ASSOC);
            $ta=trim($tmp);
            if(!empty($ta))
            {
                $this->userExist=false;
            }        
        }
        catch (PDOException $e)
        {
            echo $e->getMessage();
        }
    }    
    public function add($daten)
    {
        //Daten eintragen        
        if($this->userExist!==true)
        {
            #return false;
                print"Dieser Username ist bereits vergeben!";
        }
        else 
        {
            try 
            {
                $sql="INSERT INTO user(u_uname, 
                                       u_password, 
                                       u_email,
                                       u_vname, 
                                       u_nname, 
                                       u_plz, 
                                       u_ort, 
                                       u_strasse, 
                                       u_hnr, 
                                       u_geb_tag, 
                                       u_geb_mon, 
                                       u_geb_jahr, 
                                       u_code)
                                VALUES(:user,
                                       :password,
                                       :email,
                                       :vname,
                                       :nname,
                                       :plz,
                                       :ort,
                                       :strasse,
                                       :hnr,
                                       :geb_tag,
                                       :geb_mon,
                                       :geb_jahr,
                                       :code)";
                
                $result=$this->dbh->prepare($sql);
                $result->bindParam(':user', $daten['uname'], PDO::PARAM_STR,36);
                $result->bindParam(':password', $daten['pw'], PDO::PARAM_STR,32);
                $result->bindParam(':email', $daten['email'], PDO::PARAM_STR,60);
                $result->bindParam(':vname', $daten['vname'], PDO::PARAM_STR,40);
                $result->bindParam(':nname', $daten['nname'], PDO::PARAM_STR,32);
                $result->bindParam(':plz', $daten['plz'], PDO::PARAM_STR,6);
                $result->bindParam(':ort', $daten['ort'], PDO::PARAM_STR,32);
                $result->bindParam(':strasse', $daten['strasse'], PDO::PARAM_STR,32);
                $result->bindParam(':hnr', $daten['hnr'], PDO::PARAM_STR,7);
                $result->bindParam(':geb_tag', $daten['geb_tag'], PDO::PARAM_STR,2);
                $result->bindParam(':geb_mon', $daten['geb_mon'], PDO::PARAM_STR,2);
                $result->bindParam(':geb_jahr', $daten['geb_jahr'], PDO::PARAM_STR,4);
                $result->bindParam(':code', $daten['code'], PDO::PARAM_STR,32);
                $result->execute();
                if($result->rowCount()==1)
                {
                    print"Datensatz wurde eingetragen";
                    return true;
                }
                else
                    return false;            
                
            } 
            catch (PDOException $e) 
            {
                echo $e->getMessage();
            }
        }
    }
    
}
?>

Ich weiß nur nicht warum, mir diese IMMER false liefert

***einfach nur crack*** · 04.02.12 22:48

Punkt 1: Warum führst du hintereinander prepare und query aus? Punkt 2: Ich glaube, dass dir fetchAll mit dem Parameter FETCH_ASSOC ein Array zurückgibt. Darauf kannst du nicht die Funktion trim anwenden.

Raisch · 04.02.12 22:56

Zitat von Divo1984

Ich habe nun das nächste Problem, ich habe meine Klasse erweitert um eine Funktion UserExist die liefert mir immer false zurück

PHP-Code:


<?php
require 'classes/class_DB.php';

class userManagement extends DB
{
    public $dbh;
    public $userExist=false;
    
    
    public function __construct(&$dbh)
    {
        parent::__construct();
        if(is_object($dbh) && ($dbh instanceof PDO))
            $this->dbh=$dbh;

    }
    public function UserExist($user)
    {
        try
        {
            $sql="SELECT u_id 
                    FROM user
                   WHERE u_uname='$user'";
            $result=$this->dbh->prepare($sql);
            $result=$this->dbh->query($sql);
            $tmp=$result->fetchAll(PDO::FETCH_ASSOC);
            $ta=trim($tmp);
            if(!empty($ta))
            {
                $this->userExist=false;
            }        
        }
        catch (PDOException $e)
        {
            echo $e->getMessage();
        }
    }    
    public function add($daten)
    {
        //Daten eintragen        
        if($this->userExist!==true)
        {
            #return false;
                print"Dieser Username ist bereits vergeben!";
        }
        else 
        {
            try 
            {
                $sql="INSERT INTO user(u_uname, 
                                       u_password, 
                                       u_email,
                                       u_vname, 
                                       u_nname, 
                                       u_plz, 
                                       u_ort, 
                                       u_strasse, 
                                       u_hnr, 
                                       u_geb_tag, 
                                       u_geb_mon, 
                                       u_geb_jahr, 
                                       u_code)
                                VALUES(:user,
                                       :password,
                                       :email,
                                       :vname,
                                       :nname,
                                       :plz,
                                       :ort,
                                       :strasse,
                                       :hnr,
                                       :geb_tag,
                                       :geb_mon,
                                       :geb_jahr,
                                       :code)";
                
                $result=$this->dbh->prepare($sql);
                $result->bindParam(':user', $daten['uname'], PDO::PARAM_STR,36);
                $result->bindParam(':password', $daten['pw'], PDO::PARAM_STR,32);
                $result->bindParam(':email', $daten['email'], PDO::PARAM_STR,60);
                $result->bindParam(':vname', $daten['vname'], PDO::PARAM_STR,40);
                $result->bindParam(':nname', $daten['nname'], PDO::PARAM_STR,32);
                $result->bindParam(':plz', $daten['plz'], PDO::PARAM_STR,6);
                $result->bindParam(':ort', $daten['ort'], PDO::PARAM_STR,32);
                $result->bindParam(':strasse', $daten['strasse'], PDO::PARAM_STR,32);
                $result->bindParam(':hnr', $daten['hnr'], PDO::PARAM_STR,7);
                $result->bindParam(':geb_tag', $daten['geb_tag'], PDO::PARAM_STR,2);
                $result->bindParam(':geb_mon', $daten['geb_mon'], PDO::PARAM_STR,2);
                $result->bindParam(':geb_jahr', $daten['geb_jahr'], PDO::PARAM_STR,4);
                $result->bindParam(':code', $daten['code'], PDO::PARAM_STR,32);
                $result->execute();
                if($result->rowCount()==1)
                {
                    print"Datensatz wurde eingetragen";
                    return true;
                }
                else
                    return false;            
                
            } 
            catch (PDOException $e) 
            {
                echo $e->getMessage();
            }
        }
    }
    
}
?>

Ich weiß nur nicht warum, mir diese IMMER false liefert

Salve,

also dieses:

Code PHP:

1	$result=$this->dbh->prepare($sql);

kannst Du komplett weglassen da Du eh mit query arbeitest.

Und dieses:

Code PHP:

            if(!empty($ta)) 
            { 
                $this->userExist=false; 
            }

setzt userExist immer auf false, wenn $ta NICHT leer ist.

Desweiteren kannst Du diesen Block:

Code PHP:

class userManagement extends DB 
{ 
    public $dbh; 
    public $userExist=false; 
     
     
    public function __construct(&$dbh) 
    { 
        parent::__construct(); 
        if(is_object($dbh) && ($dbh instanceof PDO)) 
            $this->dbh=$dbh; 
 
    }

in diesen ändern:

Code PHP:

class userManagement extends DB 
{ 
    public $userExist=false; 
     
    public function __construct() 
    { 
        parent::__construct();
    }

da Du $this->dbh in der Elternklasse setzt (falls Du das nicht geändert hast)!

Gruß

//EDIT

Zitat von einfach nur crack

Punkt 1: Warum führst du hintereinander prepare und query aus? Punkt 2: Ich glaube, dass dir fetchAll mit dem Parameter FETCH_ASSOC ein Array zurückgibt. Darauf kannst du nicht die Funktion trim anwenden.

Falsch!

trim() kann auch auf Arrays angewendet werden.

***einfach nur crack*** · 04.02.12 23:03

Ich glaube eher nicht, dass trim auch auf Arrays angewendet werden kann:

Code :

1	string trim ( string $str [, string $charlist ] )

Alex_T · 04.02.12 23:06

Grüße,

@einfach nur crack:
Das Manual bietet da ne Möglichkeit: Manual Trim
Unter dem Punkt "Beispiel #2 Trimmen von Array-Werten mittels trim()".

Divo1984 · 04.02.12 23:08

Zitat von Raisch

Salve,

also dieses:

Code PHP:

1	$result=$this->dbh->prepare($sql);

kannst Du komplett weglassen da Du eh mit query arbeitest.

oki is raus

Zitat von Raisch

Und dieses:

Code PHP:

            if(!empty($ta)) 
            { 
                $this->userExist=false; 
            }

setzt userExist immer auf false, wenn $ta NICHT leer ist.

Ob ich das nun auf false oder true setze spielt doch keine Rolle, denn wenn die Datenbank leer sollte da true kommen, denn diese Abfrage sollte 0 ausgeben - klar ich könnte das auch umformulieren in true nur gehts ja deshalb trotzdem nicht

Zitat von Raisch

Desweiteren kannst Du diesen Block:

Code PHP:

class userManagement extends DB 
{ 
    public $dbh; 
    public $userExist=false; 
     
     
    public function __construct(&$dbh) 
    { 
        parent::__construct(); 
        if(is_object($dbh) && ($dbh instanceof PDO)) 
            $this->dbh=$dbh; 
 
    }

in diesen ändern:

Code PHP:

class userManagement extends DB 
{ 
    public $userExist=false; 
     
    public function __construct() 
    { 
        parent::__construct();
    }

da Du $this->dbh in der Elternklasse setzt! (falls Du das nicht geändert hast)!

Nein habe ich nicht geändert, aber wenn ich das so mache, wie du beschrieben hast, bekomme ich ja nen Notice, das will ich net

Nun weiß ich aber leider immernoch net warum das IMMER false liefert

***einfach nur crack*** · 04.02.12 23:10

@Alex_T: Mir ist durchaus bewusst, dass man die Funktion trim als Callback-Funktion auf ein Array anwenden kann, aber man kann der Funktion trim kein Array als ersten Parameter übergeben. Das ist etwas ganz anderes!

***einfach nur crack*** · 04.02.12 23:12

@Divo1984: Doch, wenn du die Wertzuweisung auf true änderst, dann ist dein Problem behoben – theoretisch.

Divo1984 · 04.02.12 23:14

Selbst wenn ich auf trim verzichte bringt das nichts

PHP-Code:


<?php

require 'classes/class_DB.php';



class userManagement extends DB

{

    public $dbh;

    public $userExist=false;

    

    

    public function __construct(&$dbh)

    {

        parent::__construct();

        if(is_object($dbh) && ($dbh instanceof PDO))

            $this->dbh=$dbh;



    }

    public function UserExist($user)

    {

        try

        {

            $sql="SELECT u_id 

                    FROM user

                   WHERE u_uname='$user'";

            #$result=$this->dbh->prepare($sql);

            #$result->bindParam(':uname', $user, PDO::PARAM_STR,36);

            #$result->execute();

            $result=$this->dbh->query($sql);

            $tmp=$result->fetchAll(PDO::FETCH_ASSOC);

            #$ta=trim();

            if($tmp['u_id']=="")

            {

                $this->userExist=true;

            }        

        }

        catch (PDOException $e)

        {

            echo $e->getMessage();

        }

    }    

    public function add($daten)

    {

        //Daten eintragen        

        if($this->userExist!==true)

        {

            #return false;

                print"Dieser Username ist bereits vergeben!";

        }

        else 

        {

            try 

            {

                $sql="INSERT INTO user(u_uname, 

                                       u_password, 

                                       u_email,

                                       u_vname, 

                                       u_nname, 

                                       u_plz, 

                                       u_ort, 

                                       u_strasse, 

                                       u_hnr, 

                                       u_geb_tag, 

                                       u_geb_mon, 

                                       u_geb_jahr, 

                                       u_code)

                                VALUES(:user,

                                       :password,

                                       :email,

                                       :vname,

                                       :nname,

                                       :plz,

                                       :ort,

                                       :strasse,

                                       :hnr,

                                       :geb_tag,

                                       :geb_mon,

                                       :geb_jahr,

                                       :code)";

                

                $result=$this->dbh->prepare($sql);

                $result->bindParam(':user', $daten['uname'], PDO::PARAM_STR,36);

                $result->bindParam(':password', $daten['pw'], PDO::PARAM_STR,32);

                $result->bindParam(':email', $daten['email'], PDO::PARAM_STR,60);

                $result->bindParam(':vname', $daten['vname'], PDO::PARAM_STR,40);

                $result->bindParam(':nname', $daten['nname'], PDO::PARAM_STR,32);

                $result->bindParam(':plz', $daten['plz'], PDO::PARAM_STR,6);

                $result->bindParam(':ort', $daten['ort'], PDO::PARAM_STR,32);

                $result->bindParam(':strasse', $daten['strasse'], PDO::PARAM_STR,32);

                $result->bindParam(':hnr', $daten['hnr'], PDO::PARAM_STR,7);

                $result->bindParam(':geb_tag', $daten['geb_tag'], PDO::PARAM_STR,2);

                $result->bindParam(':geb_mon', $daten['geb_mon'], PDO::PARAM_STR,2);

                $result->bindParam(':geb_jahr', $daten['geb_jahr'], PDO::PARAM_STR,4);

                $result->bindParam(':code', $daten['code'], PDO::PARAM_STR,32);

                $result->execute();

                if($result->rowCount()==1)

                {

                    print"Datensatz wurde eingetragen";

                    return true;

                }

                else

                    return false;            

                

            } 

            catch (PDOException $e) 

            {

                echo $e->getMessage();

            }

        }

    }

    

}

?>

***einfach nur crack*** · 04.02.12 23:22

Dann probier doch einfach mal aus, was dir var_dump von $tmp ausgibt. Einmal im Fall, wo du weißt, dass der Nutzer existiert, und einmal, wo du weißt, dass er nicht existiert. Und dann natürlich hier posten.

Alex_T · 04.02.12 23:23

PHP-Code:


if($tmp['u_id']=="") 

{ 

   $this->userExist=true; 

}

also das ist jetzt Quatsch. Wenn die u_id, die der Nutzer bestimmt hat, leer ist, dann soll der Nutzer existieren?!.

Wohl dann eher so:

PHP-Code:


if($tmp['u_id']!="") 

{ 

   $this->userExist=true; 

}

Aber mal davon abgesehen ist da ja viel arbeiten mit der Glaskugel - mich würde mal der var_dump der '$tmp' interessieren...

Divo1984 · 04.02.12 23:46

Ich weiß, blöde Frage aber wo muss ich das einfügen damit auch was ausgegeben wird?

***einfach nur crack*** · 04.02.12 23:50

Erstmal: facepalm. Ansonsten: schreib es direkt nach deinem fetchAll.

Raisch · 05.02.12 00:06

Hallo Divo1984,

Zitat von Divo1984

Nein habe ich nicht geändert, aber wenn ich das so mache, wie du beschrieben hast, bekomme ich ja nen Notice, das will ich net

Nun weiß ich aber leider immernoch net warum das IMMER false liefert

poste mal bitte beide Klassen und die Stelle wo Du das Objekt von userManagement erstellst.

Zitat von einfach nur crack

@Alex_T: Mir ist durchaus bewusst, dass man die Funktion trim als Callback-Funktion auf ein Array anwenden kann, aber man kann der Funktion trim kein Array als ersten Parameter übergeben. Das ist etwas ganz anderes!

Du hast vollkommen Recht.

Gruß

//EDIT
Teste es mal bitte mit dieser Methode:

Code PHP:

    public function UserExist( $user )
    {
        try
        {
            $sql = "SELECT 1 FROM user WHERE u_uname = '$user'";
 
            $result = $this->dbh->query( $sql );
 
            if( $result->fetch( PDO::FETCH_ASSOC ) === false )
            {
                $this->userExist = false;
            }
            else
            {
                $this->userExist = true;
            }
        }
        catch ( PDOException $e )
        {
            echo $e->getMessage();
        }
    }

Achso, mMn sollten Objektmethoden nicht mit einem Großbuchstaben anfangen.

Divo1984 · 05.02.12 15:47

Danke sehr, das geht

tut orials.de	tut verbinden	tut dir helfen	tut kooperieren
©2000-2012 tutorials.de Alle Rechte vorbehalten	tut informieren » Impressum » Nutzungsregeln » Netiquette » Team » Redaktion	» Tutorials » Videotutorials » Showroom » Events & Contests » Chat » RSS-Feed » Blogs	» der-webdesigner.net » DesignerInAction » Design Nation » Directorforum » Flashhilfe » PHP-Resource » PHPwelt » PSD-Tutorials

Themen-Optionen

Anzeige

Ähnliche Themen

ResultSet.next gibt immer false.

ASP.NET MVC Request.IsAjaxRequest() ist immer false

mysql_query liefert immer false

C++ Net - Ein If auf ein String immer false

is_int() immer false?

[PHP][Snippet] Array zu XML konvertieren

PHP: LDAP / Active Directory abfragen

E-Mail:

Abonnieren Abbestellen