Error Showing Up in Server Logs...How Do I Stop It?

Question

I'm getting an error and I've googled everything I can think of with barely any insights. Maybe you can help with a fresh set of eyes.

I have a function that returns an array of objects. The code that calls this function then loops through this array and does what it needs to do with each object. It seems to work visually when it spits out the HTML, but the server logs still give an error "Invalid argument supplied for foreach()". Yeah, it's great that it works technically, but I don't want this error to pop up anymore. Thoughts?

<?php

// The class whose method returns an array of library_Label objects
class library_Label
{
  public $id = null;
  public $name = null;

  public function getChildren()
  {
    // $children is declared earlier
    return $children;
  }
}


// main code
$columns = new library_Label(1);
foreach ($columns->getChildren() as $label) echo $label->Name;

?>

Can you show us some code, please? In particular, what you're returning and what the foreach looks like. — strager
– strager, Commented Mar 21, 2009 at 6:48
It is PHP. Sorry, that does seem like an important bit of information, doesn't it? — Iwasakabukiman
– Iwasakabukiman, Commented Mar 21, 2009 at 6:51

benlumley · Accepted Answer · 2009-03-21 08:29:20Z

1

This is fairly common, my solution is always to change this:

foreach ($columns->getChildren() as $label) echo $label->Name;

to

$children = $columns->getChildren();
if (is_array($children)) foreach ($children as $label) echo $label->Name;

It should cover all situations.

You can expand it to a full if statement with braces as well and then include an else block to handle the situation when children is empty if you want to.

answered Mar 21, 2009 at 8:29

benlumley

11.3k2 gold badges43 silver badges40 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Community · Accepted Answer · 2023-11-17 19:27:51Z

1

You don't say what language you're using, but that looks like a PHP error. If so, check out PHP's error reporting levels. You should probably actually change the values in your php.ini, though, so that you don't dump out other errors into your server logs.

Of course, you'll want to find out why that error is being thrown regardless.

edited Nov 17, 2023 at 19:27

CommunityBot

11 silver badge

answered Mar 21, 2009 at 6:49

John Biesnecker

3,8322 gold badges36 silver badges43 bronze badges

Comments

paxdiablo · Accepted Answer · 2009-03-21 08:25:33Z

The warning "Invalid argument supplied for foreach()" happens when you try to do a foreach on an variable that isn't an array.

You need to check that the return value from getChildren() actually is an array.

We can't tell this from your context since you've provided neither the constructor source nor the code that actually creates $children. If that were forthcoming, it might be easier to help you out.

One thing you could try is printing out the return value before executing the foreach, with "print_r($columns->getChildren())". This should be clear indication as to whether it's an array or not.

I'd also suggest checking to see if the array is empty, using empty() or count() - some sites on the net state that empty arrays will also cause this warning but I don't have a PHP environment at home to test that.

Collectives™ on Stack Overflow

Error Showing Up in Server Logs...How Do I Stop It?

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related