(how?) die(); on a foreach error

[GeekSupport]

is there a way to kill a script if an error occured?

ex:

function loop_me($array, $obj)
{
    if (emtpy($obj->name))
        die('Name not set in object');

    foreach ($array as $a)
    {
        if ($a > 1)
            echo 'BOOM!';
    }
    return 1;
} 

if the first if statement works, the function will return 1 even if !is_array($array). i know this wasn't the best example, but it serves its purpose.

[edit]
i guess doing an if statement might work (but looks kinda crude)

function loop_me($array, $obj)
{
    $return = 0;

    if (emtpy($obj->name))
        die('Name not set in object');
    if (1 == 1)
    {
        foreach ($array as $a)
        {
            if ($a > 1)
                echo 'BOOM!';
        }
        $return = 1;
    }
    return $return;
} 

[DarkAngelBGE]

Why not use:

function loop_me($array, $obj) 
{ 

    if (empty($obj->name)) 
    {
        die('Name not set in object');
    } 
    else
    {
        foreach ($array as $a) 
        { 
            echo 'BOOM!'; 
        } 
        
    } 
    return true; 
} 

?

[GeekSupport]

because both parts need to be executed ;-P

also, if the foreach fails in the else, the function will still return true;

[lieut_data]

I'm baffled as to where you think the problem is, but won't this work?:

function loop_me($array, $obj) {
    if (emtpy($obj->name))
        die('Name not set in object');
    if (!is_array($array)) 
        die('Array isn\'t an array!');

    foreach ($array as $a) {
        if ($a > 1)
            echo 'BOOM!';
    }
    return 1;
} 

[GeekSupport]

yes, that code would work, but i thought there might be a better way to globally kill execution of scripts if it encountered a problem. my first example above intentionally left out is_array() thinking there was a way to catch errors. if the script had multiple foreach or other looping functions, it would need to check all the variables before it starts the loop.

i guess we'll have to error check everything until php5 comes out. thanks for the help everyone

[DarkAngelBGE]

also, if the foreach fails in the else, the function will still return true;

Hmm..the only way (really?) that a foreach fials is if the argument is not an array. If it's not you could die() it out as lieut showed. If the script dies then it doesn't matter if the function returned true, that's why you can return true; always. Unless I am missing something here...

[lieut_data]

Hmm..the only way (really?) that a foreach fials is if the argument is not an array. If it's not you could die() it out as lieut showed. If the script dies then it doesn't matter if the function returned true, that's why you can return true; always. Unless I am missing something here...

Or even better, since the function seems to do nothing but fail or return true, just

return; 

And don't bother witch checking the return value.

Whey try / catch statements get into full swing, we won't be checking those return values anyway, but will throw errors instead.

[GeekSupport]

hehe yeah. like i said above, it's not the best of examples. the whole point of this post was: how to stop the rest of the script dead in it's track as soon as an error occured (foreach in this case).

[edit]
after thinking it over, i suppose it is best to check for all the different errors. i was hoping i could avoid this.

my initial thought was that there was some $__ERROR_FOUND variable i could envoke before the return statement.

if ($__ERROR_FOUND) die();
return 1; 

[lieut_data]

hehe yeah. like i said above, it's not the best of examples. the whole point of this post was: how to stop the rest of the script dead in it's track as soon as an error occured (foreach in this case).

Ohh -- this was an example (doh!)

See set_error_handler and trigger_error

[GeekSupport]

ah! thanks lieut_data, i think this is what i was looking for.