retrieve from MySQL without notices

[dmorris]

Hi,

I have the following code, which whilst works gives me notices, which I want to avoid - without changing the error warnings!

while ($row = $this->db->fetchAssoc()){
    if( array_key_exists("resultID", $row)  &&
        array_key_exists("date", $row) &&	
        array_key_exists("competition", $row) &&
        array_key_exists("hteam", $row) &&
        array_key_exists("hscore", $row) &&
        array_key_exists("ateam", $row) &&
        array_key_exists("ascore", $row) &&
        array_key_exists("report", $row)	
     ){
   $this->resultList[] = new Result($row[resultID], $row[date], $row[competition], $row[hteam], $row[hscore], $row[ateam], $row[ascore], $row[report]);
} else {
    echo "Error creating Result Object";	
}
}

The notice says that I am assuming resultID ...

Cheers in advance

[mikesgt2]

<?php

error_reporting(E_ALL & ~E_NOTICE);

...

?>

You can also set this in php.ini if you have access to it.

[dmorris]

Cheers,

I am still a relative newbie and was hoping to understand why it gave me these notices, and try to get in the habit of coding correctly.

Is there something I should be doing to avoid the notices, instead of just simply hiding the fact that the notices are present.

I assumed that since php gives a notice that there would be a "correct" way of doing it where there we no notices given.

Cheers for now

[twigletmac]

You just need to put the array element names into quotes:

$row['resultID']

instead of

$row[resultID]

If they are not quoted then PHP first checks to see if it's a constant that you are referencing and when it can't find a constant with that name assumes that you meant to quote the element name. So the 'correct way' would be with quotes since PHP then doesn't have to do the extra work of looking for constants that don't exist.

Hope it helps.

Mac

[[deleted]]

Indeed.

Do NOT change the error reporting levels, that is a fool's way of getting rid of errors.

PHP never gives you a notice or warning without a reason, making the notices go away does not make the cause of the notices go away.

[dmorris]

cheers guys.

D