[PHP] Undefined index & propertie

[TangLeFuzZ]

Hey,

als ik error_reporting op E_ALL heb staan, krijg ik de volgende notices;

Notice: Undefined property: stdClass::$userid in ........

Nu weet ik even geen nette manier om die notice weg te krijgen zeg maar, vind het ook een beetje onlogisch.

De notice komt voort uit een while loop van een query, waarin ik de vars uit de database in de class vars gooi.

PHP:

while ($record = mysql_fetch_object($result)) 
{
    $this->checkLogin = 1;
    $this->userId = $record->userid;
}

$record->userid; mag dus niet zomaar volgens PHP... maar ik kan die toch niet zomaar ergens netjes initialiseren of wel?

Hoe doen jullie dit?

[ Voor 4% gewijzigd door TangLeFuzZ op 19-02-2005 14:33 ]

[djluc]

Sowieso is mysql_fetch_assoc een stukje sneller. Verder mis ik een belangrijk stukje code: je query. Selecteer je wel het veld userid?

[ACM]

$record->userid mag prima, mits er ook daadwerkelijk een userid-property is... Als ie userId, userID of UserID heet of gewoon niet bestaat krijg je uiteraard die waarschuwing.
En de notice betekent niet dat het "niet zomaar mag" alleen maar dat je naar een property van een object verwijst die niet bestaat... Notice is alleen maar een melding van iets dat fout is, maar verder geen invloed op de executie van de code heeft (wel eventueel op het resultaat van die executie).

[Michali]

waarom wil je velden uit een record initialiseren? dat lijkt me toch niet nodig als je hem gewoon goed ophaalt? doe anders eens een var dump op $record. je kunt ook gewoon in je query zien wat je ophaalt.

[GlowMouse]

djluc schreef op zaterdag 19 februari 2005 @ 14:40:
Sowieso is mysql_fetch_assoc een stukje sneller.

In dit geval lijkt het alsof de TS slechts 1 rij nodig heeft (loginscript ofzo). In dat geval zijn beide methoden overkill.

TS: kun je niet beter wat doen met mysql_result, in plaats van een mysql_fetch_object op 1 rij?

[JHS]

GlowMouse schreef op zaterdag 19 februari 2005 @ 15:32:
[...] TS: kun je niet beter wat doen met mysql_result, in plaats van een mysql_fetch_object op 1 rij?

Of mysql_fetch_row . Er zijn er nog wel wat denk ik .

[GlowMouse]

JHS schreef op zaterdag 19 februari 2005 @ 15:52:
Of mysql_fetch_row . Er zijn er nog wel wat denk ik .

While() is eigenlijk het meest overbodig.

[Genoil]

Even afgezien van het feit dat een functie als bovenstaande geen notices zou hoeven genereren, is er wel een manier om specifieke Notices af te vangen:

PHP:

define("ERR_LOG_SCREEN", 1);
define("ERR_LOG_FILE"  , 2);

define("ERR_LOG_LEVEL"  , E_USER_NOTICE);
define("ERR_LOG_METHOD" , ERR_LOG_SCREEN);

function errorHandler($errorType, $errorMessage, $filePath, $lineNo) {
    if($errorType <= ERR_LOG_LEVEL) {
        $errStr = $errorMessage.", in <b>".$filePath."</b> on line <b>".$lineNo."</b><br />";
        switch($errorType) {
            case E_NOTICE:
                if(strstr($errorMessage, "Undefined property") !== FALSE) return;
            case E_USER_NOTICE:
                $errStr = "<b>Notice:</b> ".$errStr;
                break;
            case E_WARNING:
            case E_USER_WARNING:
                $errStr = "<b>Warning:</b> ".$errStr;
                break;
            case E_ERROR:
            case E_USER_ERROR:
                $errStr = "<b>Fatal error:</b> ".$errStr;
                break;
        }
        
        if(ERR_LOG_METHOD & ERR_LOG_SCREEN) {
            print($errStr);
        }
        
        if(ERR_LOG_METHOD & ERR_LOG_FILE) {
            $errStr = strip_tags($errStr);
            $GLOBALS['errorlog']->msg($errStr);
        }
    }
}

error_reporting(E_ALL);
set_error_handler("errorHandler"); 

$foo = new stdclass;
echo $foo->bar;

$moo = array();
echo $moo[0];

trigger_error("dit is gewoon fout!", E_USER_ERROR);

Normaalgesproken zou op regel 41 een Notice gegenereerd worden, maar Notices van het type "Undefined property", worden niet gelogged (regel 12). De andere Notice op regel 44 wordt gewoon gemeld en met trigger_error (46) kun je ook nog eens custom error messages genereren.