Klassen und globale Variablen / sql error reporting [lösung]

wolf · 19. April 2014

Wieso bekomme ich nen Fehler? :0

PHP

$SYS['hallo']['geht'] = "funktioniert";
class test{
    function systeminfos(){
        print_r($SYS);
    }
}
$ausführen = new test;
$ausführen->systeminfos();
// gibt aus >> Notice: Undefined variable: SYS in /var/www/datei.php on line 203

Btw, wie bekomme ich es hin, das das Array auch in den Funktionen verfügbar ist.

cottton · 19. April 2014

PHP

$SYS['hallo']['geht'] = "funktioniert";
class test{
    function systeminfos(){
        global $SYS;
        print_r($SYS);
    }
}
$ausführen = new test;
$ausführen->systeminfos();

durch global wird es verfuegbar. hab aber vergessen, ob es auch aenderungen nach draussen uebernimmt
musste mal testen

edit:
oder so (wie ich es nutze)

PHP

class test{
    private $SYS;




    __construct(&$SYS)
    {
        $this->SYS =& $SYS;
    }
    function systeminfos(){
        print_r($this->SYS);
    }
}

Alles anzeigen

durch das erste & in den parametern wird SYS als referenz von aussen "aufgenommen".
bei der zuweisung in $this->SYS =& wird wieder eine referenz zum eingehenden aufgebaut, wodurch es auf den ggleichen inhalt wie aussen zeig.
heisst:
wenn du drin was aenderst, ist es draussen auch geaendert.
und ungekehrt.

wolf · 19. April 2014

Ähhm, mein Verständniss ist gebremst
aber danke

Parse error: syntax error, unexpected '__construct' (T_STRING), expecting function (T_FUNCTION)

cottton · 20. April 2014

Ja mist. Muss
Function __construct
Heissen
Sorry

The Scout · 20. April 2014

Hmm, ich glaub, ich habs verstanden:
die Datei: eine Straße
Variable: ein Haus
Referenz: die Hausnummer

wolf: Was du versuchst ist quasi, die Leute zwischen den verschiedenen Häusern in der Straße umziehen zu lassen. cottton tauscht einfach die Hausnummern aus
Keine garantie...

cottton · 20. April 2014

Ich hatte das mal hier versucht zu deuten; Referenz in Klassen
Auf php.net wird von bezeichner gesprochen.
Schwer zu erklaeren.

wolf · 20. April 2014

Ich bin zu blöd

Warning: Missing argument 1 for test::__construct(), called in .. on line 246 and defined in .. on line 238

cottton · 21. April 2014

sorry, wollte das eigtl noch nachschieben, habs dann aber vergessen =)

PHP

class Test
{
    private $SYS;




    function __construct(&$SYS)
    {
        $this->SYS =& $SYS;
    }
    public function change()
    {
        $this->SYS['test'] = 'innnen zugewiesen';
    }
    public function del()
    {
        $this->SYS['test'] = null;
    }
    public function systeminfos()
    {
        print_r($this->SYS['test'].'<br>');
    }
}




$SYS['test'] = 'aussen zugewiesen';




$obj = new Test($SYS);
$obj->systeminfos(); // aussen zugewiesen
$obj->change();
$obj->systeminfos(); // innnen zugewiesen
echo $SYS['test']; // innnen zugewiesen
$obj->del();
echo '<br>'.gettype($SYS['test']); // NULL

Alles anzeigen

beim erstellen eines Objekts von Test (also bei $x = new Test()) wird __construct ausgeführt.
Daher $obj = new Test($SYS)
Mit der Variante - der Referenz - kannst Du innen den Wert ändern und außen.
Löschen auch. Einfach mal ausprobieren.

wolf · 21. April 2014

Ich bekomme nur iwi was ich drinnen veränder nicht mehr Raus .. Zeile 200 & 211 sind interessant

PHP: 178

class usermag{
    var $username;
    function __construct(&$SYS){
        //außen + innen verknüpfen
        $this->SYS =& $SYS;
    }
    function logIn($pw){
        try {
            $dbconnect = new PDO('mysql:host='.$this->SYS['conf']['db']['server'].';dbname='.$this->SYS['conf']['db']['database'], $this->SYS['conf']['db']['user'], $this->SYS['conf']['db']['password']);
            // IP geblockt?
            $sql = $dbconnect->query("SELECT * FROM `blockedip` WHERE `ip` = '".$_SESSION['user']['ip']."';");
            $sql = $sql->rowCount();
            if ($sql >= "1" || $_SESSION['user']['blocked']) {
                //IP geblockt
                echo "<p>Blocked</p>";
            }
            echo $this->username." logged in.";
            echo "Password is:".$pw;               
        }
        catch(PDOException $e) {
            $this->SYS['conf']['db']['satus'] = 1;
            $this->SYS['conf']['db']['message'] = $this->SYS['conf']['db']['message']." || ".$e->getMessage();
            echo "<br>-------------<br>".$this->SYS['conf']['db']['message']."<br>-------------<br>";//gibt die außen + innen aus
        }#catch
    }
}
if (isset($SYS['conf']['db']['satus']) && $SYS['conf']['db']['satus'] == 1) {
        // wenn mails an den admin aktiviert sind 
        if ($SYS['conf']['db']['mail'] >= 1) {
            mail($SYS['conf']['admin']['mail'] , 'SQL FAIL at '.$_SERVER["SERVER_NAME"].'');
        }
        // wenn error reporting über E_STRICT
        if ($SYS['conf']['security']['errors'] >= 8) {
            echo $SYS['conf']['db']['message'];//gibt nur außen aus
        }
}

Alles anzeigen

außerdem bekomme ich bei dem hier

PHP: 187

// IP geblockt?
                $sql['ipblock'] = $dbconnect->query("SELECT * FROM `blockedip` WHERE `ip` = '".$_SESSION['user']['ip']."';");
                $sql['ipblock']['rows'] = $sql['ipblock']->rowCount();
                if ($sql['ipblock']['rows'] >= "1" || $_SESSION['user']['blocked']) {

Fatal error: Cannot use object of type PDOStatement as array in /var/www/USER/usermanagement.php on line 189
Kann man das lösen?

cottton · 21. April 2014

Muss ja beides (aussen und innen) anzeigen, denn Du verkettest ja Drinnen:

PHP

$this->SYS['conf']['db']['message'] /*die msg ist*/ = 
    $this->SYS['conf']['db']['message'] /*die msg*/   .   /*und*/"  || ".$e->getMessage();

hier mal n test ohne der Verkettung. Dafür als array:

PHP

<?php
class usermag
{
    var $username;
    function __construct(&$SYS){
        //außen + innen verknüpfen
        $this->SYS =& $SYS;
    }
    function logIn($pw){
        try {
            # absichlicher fehler:
            $dbconnect = new PDO('failme');




/*            $dbconnect = new PDO('mysql:host='.$this->SYS['conf']['db']['server'].';dbname='.$this->SYS['conf']['db']['database'], $this->SYS['conf']['db']['user'], $this->SYS['conf']['db']['password']);
            // IP geblockt?
            $sql = $dbconnect->query("SELECT * FROM `blockedip` WHERE `ip` = '".$_SESSION['user']['ip']."';");
            $sql = $sql->rowCount();
            if ($sql >= "1" || $_SESSION['user']['blocked']) {
                //IP geblockt
                echo "<p>Blocked</p>";
            }
            echo $this->username." logged in.";
            echo "Password is:".$pw;*/
        }
        catch(PDOException $e) {
            $this->SYS['conf']['db']['satus'] = 1;
            $this->SYS['conf']['db']['message'][] = $e->getMessage();
            #echo "<br>-------------<br>".$this->SYS['conf']['db']['message']."<br>-------------<br>";//gibt die außen + innen aus
        }#catch
    }
}




$SYS['conf']['db']['satus'] = 'aussen';
$SYS['conf']['db']['message'][] = 'aussen';




/*DEBUG*/ test($SYS,'$SYS ganz am anfang');







$obj = new usermag($SYS);
$obj->logIn('wieso muss hier n pw rein?');




/*DEBUG*/ test($SYS,'$SYS nachdem von innen was geadded wurde');




 
 
function test($val,$title=''){
    $br = '<br />';
    echo '<pre># <b>DEBUG</b>: <u>'.$title.'</u>'.$br;
    echo '<i>type:</i> '.gettype($val).$br.'<i>val:</i> ';
    if(is_bool($val)){
        echo ($val === true)?'true':'false';
    }elseif(is_null($val)){
        echo 'NULL';
    }elseif(!is_numeric($val)and empty($val)){
        echo 'empty';
    }elseif(is_resource($val)){
        echo get_resource_type($val);
    }else{
        print_r($val);
    }
    echo $br.'#/DEBUG</pre>';
} 
?>

Alles anzeigen

zum 2. Fehler:

PHP

$sql['ipblock']['rows']

Daran liegts.
$dbconnect ist sicherlich ein Objekt von PDO.
Daher:

PHP

$sql['ipblock'] = $dbconnect->query("SELECT * FROM `blockedip` WHERE `ip` = '".$_SESSION['user']['ip']."';");
// PDO::query — Executes an SQL statement, returning a result set as a PDOStatement object

Also $sql['ipblock'] ist jetzt ein obkect. Du kannst also keine Array-Ebene reinquetschen =)
Besser ne temp-var:

PHP

$query = $dbconnect->query("SELECT * FROM `blockedip` WHERE `ip` = '".$_SESSION['user']['ip']."';");

wolf · 21. April 2014

hmm fürs sql wollt ich das so cool umgehen
aber dein array ebenen kommentar hats gebracht

PHP: 187

// IP geblockt?
                $sql['ipblock']['sql'] = $dbconnect->query("SELECT * FROM `blockedip` WHERE `ip` = '".$_SESSION['user']['ip']."';");
                $sql['ipblock']['rows'] = $sql['ipblock']['sql']->rowCount();
                if ($sql['ipblock']['rows'] >= "1" || $_SESSION['user']['blocked']) {

cottton · 21. April 2014

Das geht auch =)
BTW: Du nutzt die PDO, dann aber keine Daten direkt in die Query!

PHP

// auch wenn es die SESSION ist - beim nächsten mal vergisst du iwas und nutzt POST ect in der query.
$_SESSION['user']['ip'] = "' AND `ip` = 'nicht vorhanden; --'";
$dbconnect->query("SELECT * FROM `blockedip` WHERE `ip` = '".$_SESSION['user']['ip']."';");

Nutze besser ->prepare und ->execute
http://www.php.net/manual/de/pdo.query.php

PHP

$dbconnect = new PDO('mysql:host=host','name','pw',array(PDO::ATTR_PERSISTENT => true)); # persitent muss nicht sein
# ...
$query = "SELECT * FROM `blockedip` WHERE `ip` = :ip;";
$stmt = $dbconnect->prepare($query);
$stmt->execute(
    array(
        ':ip' => $_SESSION['user']['ip'] 
        # BTW: es geht auch OHNE : (doppelpunkt) in den parametern 
        # 'ip' => $_SESSION['user']['ip'] 
        # das macht es einfach mit arrays dynamisch zu arbeiten
    )
);
while($row = $stmt->fetch(PDO::FETCH_ASSOC)){
    # ...
}

Alles anzeigen

wolf · 21. April 2014

vielen dank für den Tipp mit dem Array
bloß dass es immernoch eingesperrt ist

aber, was bewirkt denn ATTR_PERSISTENT?

cottton · 21. April 2014

http://www.php.net/manual/de/pdo.connections.php
Bei "Beispiel #4 Persistente Verbindungen"

Zitat

Persistente Verbindungen werden nicht am Ende des Skripts geschlossen, sondern werden gecacht und wiederverwendet, ...

Dadurch fällt das connecten bei jedem erneuten aufrufen der Seite weg.

/
Bei mir ging es mit Deinem Bsp oO?

wolf · 22. April 2014

Bei mir auch mit deinem Beispiel oO

hier dann nichtmehr
aber was mich viel mehr interresiert.. wieso bekomme ich
Fatal error: Call to undefined function sqlInfo() in .. on line 243

PHP: 182

class usermag{
    // fillable vars
    var $username;
    var $password;

    function __construct(&$SYS){
        //außen + innen verknüpfen
        $this->SYS =& $SYS;
        //filtern
        $this->username = trim($this->username);
        $this->password = trim($this->password);
        $_SESSION['user']['username'] = $this->username;
        //evtl Verbindung erneuern
        try {
            $this->SYS['conf']['db']['conect'] = new PDO('mysql:host='.$SYS['conf']['db']['server'].';dbname='.$SYS['conf']['db']['database'], $SYS['conf']['db']['user'], $SYS['conf']['db']['password'],array(PDO::ATTR_PERSISTENT => true));
        }
        catch(PDOException $e) {
            $SYS['conf']['db']['status'] = 1;
            $SYS['conf']['db']['message'][] = $e->getMessage();
        }#catch
    }
    function checklogIn(){
        try {
            // IP geblockt?
            $SQL['ipblock']['query'] = "SELECT * FROM `blockedip` WHERE `ip` = :ip;";
            $SQL['ipblock']['stmt'] = $this->SYS['conf']['db']['conect']->prepare($SQL['ipblock']['query']);
            $SQL['ipblock']['stmt']->execute(
                array(
                    ':ip' => $_SESSION['user']['ip']
                )
            );
            $SQL['ipblock']['rows'] = $SQL['ipblock']['stmt']->rowCount();
            echo $SQL['ipblock']['rows'];
            while($SQL['ipblock']['fetch'] = $SQL['ipblock']['stmt']->fetch(PDO::FETCH_ASSOC)){
                    $SQL['ipblock']['rows'] ++;
                    $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['id'] = $SQL['ipblock']['fetch']['id'];
                    $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['ip'] = $SQL['ipblock']['fetch']['ip'];
                    $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['time'] = $SQL['ipblock']['fetch']['time'];
                    $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['until'] = $SQL['ipblock']['fetch']['until'];
                    $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['reason'] = $SQL['ipblock']['fetch']['reason'];
            }
            if ($SQL['ipblock']['rows'] >= 1) {
                //IP geblockt
                foreach ($SQL['ipblock']['exec'] as $SQL['ipblock']['single']) {
                    //wenn der Block vorbei ist
                    if (date("Y-m-d H:i:s") > date($SQL['ipblock']['single']['until'])) {
                        //alles reseten
                        $_SESSION['user']['block']['blocked'] = false;
                        $_SESSION['user']['block']['time'] = NULL;
                        $_SESSION['user']['block']['until'] = NULL;
                        $_SESSION['user']['block']['id'] = NULL;
                        $_SESSION['user']['block']['message'] = NULL;
                        $SQL['ipbldel']['query'] = "DELETE * FROM `blockedip` WHERE `id` = :id;";
                        $SQL['ipbldel']['stmt'] = $this->SYS['conf']['db']['conect']->prepare($SQL['ipbldel']['query']);
                        $SQL['ipbldel']['stmt']->execute(
                            array(
                                ':id' => $SQL['ipblock']['single']['id']
                            )
                        );
                        #echo "Vorbei";
                        userLogIn();
                    }
                    else{
                        //block daten(neu) setzen
                        $_SESSION['user']['block']['blocked'] = true;
                        $_SESSION['user']['block']['time'] = $SQL['ipblock']['single']['time'];
                        $_SESSION['user']['block']['until'] = $SQL['ipblock']['single']['until'];
                        $_SESSION['user']['block']['id'] = $SQL['ipblock']['single']['id'];
                        $_SESSION['user']['block']['message'] = $SQL['ipblock']['single']['reason'];
                        #echo "<p>Blocked</p>";
                        userLogOut();
                    }
                }

            }
            else{
                userLogIn();
            }
        }
        catch(PDOException $e) {
            $this->SYS['conf']['db']['status'] = 1;
            $this->SYS['conf']['db']['message'][] = $e->getMessage();
        }#catch
    }
    function userLogIn(){
        echo $this->username.' logged in.';
    }
    function userLogOut(){
        echo $this->username.' logged out.';
    }
}

Alles anzeigen

cottton · 22. April 2014

Geht doch =)

PHP

<?php







class usermag{
    // fillable vars
    var $username;
    var $password;




    public function debug()
    {
        return $this->SYS;
    }




    function __construct(&$SYS){
        //außen + innen verknüpfen
        $this->SYS =& $SYS;
        //filtern
        $this->username = trim($this->username);
        $this->password = trim($this->password);
        $_SESSION['user']['username'] = $this->username;
        //evtl Verbindung erneuern
        try {
            $this->SYS['conf']['db']['conect'] = new PDO('mysql:host='.$SYS['conf']['db']['server'].';dbname='.$SYS['conf']['db']['database'], $SYS['conf']['db']['user'], $SYS['conf']['db']['password'],array(PDO::ATTR_PERSISTENT => true));
        }
        catch(PDOException $e) {
            $SYS['conf']['db']['status'] = 1;
            $SYS['conf']['db']['message'][] = $e->getMessage();
        }#catch
    }
    function checklogIn(){
       #...
    }
    function userLogIn(){
        echo $this->username.' logged in.';
    }
    function userLogOut(){
        echo $this->username.' logged out.';
    }
}




# test:
$SYS['conf']['db']['conect'] = false;
$SYS['conf']['db']['server'] = 'localhost';
$SYS['conf']['db']['database'] = 'keine';
$SYS['conf']['db']['user'] = 'falsch';
$SYS['conf']['db']['password'] = 'auchfalsch';
$obj = new usermag($SYS);




/*DEBUG*/ test($obj->debug(),'$obj->debug() --innen');
/*DEBUG*/ test($SYS,'$SYS --aussen');




# testweise die $SYS aussen auf null setzen
# (sollte dann auch innen null sein)
$SYS = null;
echo '<hr>testweise aussen auf null gesetzt<br>';
/*DEBUG*/ test($obj->debug(),'$obj->debug() --innen');
/*DEBUG*/ test($SYS,'$SYS --aussen');










function test($val,$title=''){
    $br = '<br />';
    echo '<pre># <b>DEBUG</b>: <u>'.$title.'</u>'.$br;
    echo '<i>type:</i> '.gettype($val).$br.'<i>val:</i> ';
    if(is_bool($val)){
        echo ($val === true)?'true':'false';
    }elseif(is_null($val)){
        echo 'NULL';
    }elseif(!is_numeric($val)and empty($val)){
        echo 'empty';
    }elseif(is_resource($val)){
        echo get_resource_type($val);
    }else{
        print_r($val);
    }
    echo $br.'#/DEBUG</pre>';
}
?>

Alles anzeigen

Den Fehler "Fatal error: Call to undefined function sqlInfo() in .. on line 243" bekomme ich nicht. Sehe auch nirgends eine function sqlinfo() genutzt oO?
Aber hier stimmt was nicht:

PHP

# ...
# hier bekommst du die returned rows:          
$SQL['ipblock']['rows'] = $SQL['ipblock']['stmt']->rowCount(); # angenommen 1
echo $SQL['ipblock']['rows'];
while($SQL['ipblock']['fetch'] = $SQL['ipblock']['stmt']->fetch(PDO::FETCH_ASSOC)){
        # und hier erhoehst du sie pro row :D
        $SQL['ipblock']['rows'] ++; # wenn ->rowCount() 1 war, dann sind wir jetzt bei 2 =)
        # besser wäre hier wohl eine var $i=0, die dann hochzaehlt
        $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['id'] = $SQL['ipblock']['fetch']['id'];
        $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['ip'] = $SQL['ipblock']['fetch']['ip'];
        $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['time'] = $SQL['ipblock']['fetch']['time'];
        $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['until'] = $SQL['ipblock']['fetch']['until'];
        $SQL['ipblock']['exec'][$SQL['ipblock']['rows']]['reason'] = $SQL['ipblock']['fetch']['reason'];
}

Alles anzeigen

btw: Du musst noch einen evtl Fehler bei ->execute abfangen:

PHP

$SQL['ipblock']['query'] = "SELECT * FROM `blockedip` WHERE `ip` = :ip;";
$SQL['ipblock']['stmt'] = $this->SYS['conf']['db']['conect']->prepare($SQL['ipblock']['query']);
if(
    !$SQL['ipblock']['stmt']->execute(
        array(
            ':ip' => $_SESSION['user']['ip']
        )
    )
){
    print_r($SQL['ipblock']['stmt']->errorInfo());
    # anstatt print_r dann hier den fehler behandeln.
    # zB:
    $err_info = $SQL['ipblock']['stmt']->errorInfo(); # ist ein array mit infos zum fehler
    # zB $err_info[0] // SQLSTATE ...
    #
    # kann man dann in einen string packen:
    $err_str = '(SQLSTATE: '.$err_info[0].') '.(
        (empty($err_info[2])) # wenn ebene [2] leer
            ?error_get_last_str() # dann get last error
            :$err_info[2] # sonst nimm ebene [2]
    ).' -- Query: '.$SQL['ipblock']['query']; # und die query anhaengen
    #
    # jetzt nur noch den string $err_str in ne logdatei speichern ...
}




# das prinzip noch mal "kleiner":
if(!$stmt->execute(){ # wenn fehler beim ausfuehren
    print_r($stmt->errorInfo()); # dann fehler behandeln
}else{ # sonst
    while($row = $stmt->fetch(PDO::FETCH_ASSOC)){ # hol die die daten
        $result['row'][] = $row; 
    }
}

Alles anzeigen

wolf · 22. April 2014

wie löst man das mit dem Array in der Mail am besten?

PHP: 295

function sqlError(){
    global $SYS;
    if(isset($SYS['conf']['db']['status']) && $SYS['conf']['db']['status'] == 1) {
        // wenn mails an den admin aktiviert sind 
        if ($SYS['conf']['db']['mail'] >= 1) {
            mail(
                $SYS['conf']['admin']['mail'],#to
                'SQL FAIL at '.$_SERVER["SERVER_NAME"].'',#header
                #das Array -- $SYS['conf']['db']['message']) #message
                "error"
            );
        }
        // wenn error reporting über E_STRICT
        if ($SYS['conf']['security']['errors'] >= 8) {
            $SYS['conf']['db']['i'] = 0;
            foreach ($SYS['conf']['db']['message'] as $SYS['conf']['db']['message']['single']) {
                $SYS['conf']['db']['i']++;
                //Wenn es ein fehler bei exqute gab
                if (is_array($SYS['conf']['db']['message']['single'])) {
                    //auch die fehler Nummer vorrausstellen
                    echo "<pre>".$SYS['conf']['db']['i']." -> ";
                    //aus dem fehlerarray daten ausgeben
                    foreach ($SYS['conf']['db']['message']['single'] as $SYS['conf']['db']['message']['single']['key'] => $SYS['conf']['db']['message']['single']['string']) {
                        echo $SYS['conf']['db']['message']['single']['key']." => ".$SYS['conf']['db']['message']['single']['string'];
                    }
                    echo "</pre>";
                }
                //wenns n normaler Fehler war
                else{
                    echo "<pre>".$SYS['conf']['db']['i']." -> ".$SYS['conf']['db']['message']['single']."</pre>";
                }
            }
        }
    }
}

Alles anzeigen

Mein vermeintliches 'nicht außen ankommen' lag am Zeitpunkt, des Aufrufen des Arrays

jaja das mit den Rows --hab ich nachträglich aus Faulheit kurz so gelöst ist schon geändert worden

cottton · 22. April 2014

implode wuerd ich sagen.

PHP

'<br>'.implode('<br>', $SYS['conf']['db']['message']).'<br>'

oh, jetzt haste editiert ... mom ... lesen

Warum global auf einmal?
Wenn die function in der gleichen Klasse ist, dann kannste doch $this->SYS nutzen oO?

Zu dem message array:
wenn Du dort e-msgs sammelst, dann immer so, dass Du eine "Variante" erwarten kannst.
Also nicht einmal ein String, oder manchmal ein Array. Dann besser immer ein Array.
Auch wenn Du im code weißt, dass es nur eine msg wird.
Dann besser [] = array($single_msg)
Denn dann kannst Du immer erwarten, dass es ein Array ist.

und dann halt wie oben schon gesagt: implode()

und warum schickst Du alle vars in das SYS-Array ? Sogar i ?

wolf · 22. April 2014

hmm..
und wie bekomm ich das hier in ein Array...

PHP

$this->SYS['conf']['db']['message'][] = $SQL['ipblock']['stmt']->errorInfo();
$this->SYS['conf']['db']['message'][]['line'] = __LINE__;

Code

Array
(
    [0] => Array
        (
            [0] => 42S02
            [1] => 1146
            [2] => Table 'usermag.blockedips' doesn't exist
        )




    [1] => Array
        (
            [line] => 203
        )




)

Alles anzeigen

global, weil es nicht in der Klasse ist...
kannst du mir das mit dem msg Array mal als Code schreiben? meine Hirndenkfähigkeit ist grad down
wieso ich die alle ins sys Array schicke..? gute frage.
ok, ich weiß grad nicht ob implode rekursiv auf Arrays wirkt..?

cottton · 22. April 2014

implode rec. - nein.
Würde aber an der stelle, wo du ->errorInfo() ausliest, den String zusammenstellen.
Am besten (da immer wieder mal gebraucht wird) in einer eigenen func/methode.

Zu errorInfo():
http://www.php.net/manual/en/pdo.errorinfo.php

Zitat

Element Information
0 SQLSTATE error code (a five characters alphanumeric identifier defined in the ANSI SQL standard).
1 Driver-specific error code.
2 Driver-specific error message.

was Du brauchst ist [0] und [2] mMn.
Ich glaube [1] ist/war oft leer. Kann mich aber irren.

Ich hatte letztens schon mal geposted, was ich nutze:

PHP

$err_str = '(SQLSTATE: '.$err_info[0].') '.( 
        (empty($err_info[2])) # wenn ebene [2] leer
            ?error_get_last_str() # dann get last error
            :$err_info[2] # sonst nimm ebene [2]
    ).' -- Query: '.$SQL['ipblock']['query']; # und die query anhaengen

manchmal war/ist die ebene [2] leer. Daher per error_get_last_str() den letzten error geholt.
Man kann natürlich auch:

PHP

$err_info = $SQL['ipblock']['stmt']->errorInfo();
    echo implode(', ',$err_info);

Aber dann hast Du keine Bezeichnungen.

Also noch mal übersichtlicher:

PHP

$err_info           = $SQL['ipblock']['stmt']->errorInfo();
    $sqlstate_err_code  = $err_info[0]; # SQLSTATE error code 
    $driver_err_code    = $err_info[1]; # Driver-specific error code
    $driver_err_msg     = $err_info[2]; # Driver-specific error message




    if(empty($driver_err_msg)){ # wenn mal keine msg vorhanden, sondern nur der e-code
        $driver_err_msg = error_get_last_str();
    }




    $e_msg = '(SQLSTATE: '.$sqlstate_err_code.')'; # () klammer sind geschmackssache 
    $e_msg .= ' (driver e-code: '.$driver_err_code.')';
    $e_msg .= ' '.$driver_err_msg.')';
    $e_msg .= ' -- Query: '.$SQL['ipblock']['query']; # ich hab immer gern gleich die query vor augen, die das verursacht hat =)

Alles anzeigen

edit:
sorry - vergessen: die func "error_get_last_str" ist ja nicht vordefiniert
hier is sie:

PHP

function error_get_last_str()
{
    if(!function_exists('FriendlyErrorType')){
        function FriendlyErrorType($type)
        { # http://www.php.net/manual/en/errorfunc.constants.php#109430
            switch($type){
                case E_ERROR: // 1 //
                    return 'E_ERROR';
                case E_WARNING: // 2 //
                    return 'E_WARNING';
                case E_PARSE: // 4 //
                    return 'E_PARSE';
                case E_NOTICE: // 8 //
                    return 'E_NOTICE';
                case E_CORE_ERROR: // 16 //
                    return 'E_CORE_ERROR';
                case E_CORE_WARNING: // 32 //
                    return 'E_CORE_WARNING';
                case E_CORE_ERROR: // 64 //
                    return 'E_COMPILE_ERROR';
                case E_CORE_WARNING: // 128 //
                    return 'E_COMPILE_WARNING';
                case E_USER_ERROR: // 256 //
                    return 'E_USER_ERROR';
                case E_USER_WARNING: // 512 //
                    return 'E_USER_WARNING';
                case E_USER_NOTICE: // 1024 //
                    return 'E_USER_NOTICE';
                case E_STRICT: // 2048 //
                    return 'E_STRICT';
                case E_RECOVERABLE_ERROR: // 4096 //
                    return 'E_RECOVERABLE_ERROR';
                case E_DEPRECATED: // 8192 //
                    return 'E_DEPRECATED';
                case E_USER_DEPRECATED: // 16384 //
                    return 'E_USER_DEPRECATED';
                default:
                    return '';
            }
        }
    }
    return ($e=error_get_last())?FriendlyErrorType($e['type']).': '.$e['message'].' in '.$e['file'].' on line '.$e['line']:implode(', ',error_get_last());
}

Alles anzeigen

Klassen und globale Variablen / sql error reporting [lösung]

Jetzt mitmachen!

Teilen