Using foreach with mysql results array

Question

I want to search for name in text but I have this error:

Warning: Invalid argument supplied for foreach()

    $users = $db->query_read(" SELECT username FROM " . TABLE_PREFIX . " user "); 
    $nameuser=array();
    while ($results = $db->fetch_array($users, MYSQLI_ASSOC)) 
    {
        $nameuser[] = $results['username'];
    }
    foreach ($nameuser as $name)
    {
        if(strstr($post['message'], $name))
        {
            // create new PM
            $pmdm =& datamanager_init('PM', $vbulletin, ERRTYPE_ARRAY);
            $pmdm->set_info('is_automated', true);
            $pmdm->set('fromuserid', 1);
            $pmdm->set('fromusername', admin);
            $pmdm->set('title', 'some title');
            $pmdm->set('message', 'some text');
            $botpermissions['adminpermissions'] = 2;
            $pmdm->set_recipients($name, $botpermissions);
            $pmdm->set('dateline', TIMENOW); 
            $pmdm->save(); 
        }
    }

Edit : use stripos Instead of strstr

The argument to foreach has to be an array. $names is a string that you just created with implode. Use $nameuser. — Barmar
– Barmar, Commented Feb 20, 2015 at 16:12
@Barmar thanks brother , you are right .. now no error , but the code don't do any result .. can you know why ? — omar dealo
– omar dealo, Commented Feb 20, 2015 at 16:22
try echo(count($nameuser)); right before foreach line to see if any was found — Alex
– Alex, Commented Feb 20, 2015 at 16:51
@KimAlexander thanks brother i found th sloution with use stripos Instead of strstr — omar dealo
– omar dealo, Commented Feb 20, 2015 at 17:04

Alex · Accepted Answer · 2015-02-20 16:13:30Z

2

Why do you:

$names = implode(',', $nameuser);
foreach ($names as $name)

just :

foreach ($nameuser as $name)

answered Feb 20, 2015 at 16:13

Alex

17.3k1 gold badge31 silver badges52 bronze badges

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

6 Comments

omar dealo Over a year ago

yeah... thanks brother , you are right .. now no error , but the code don't do any result .. can you know why ?

Anil Baweja Over a year ago

Not this one the code which is not showing any error i mean the current code which you are saying providing no output. .

Alex Over a year ago

@omardealo mustn't it be: `" . TABLE_PREFIX . " user` ") with ticks? is your table name include space?? or you should use " . TABLE_PREFIX . "user") if there is no space in table name

omar dealo Over a year ago

@KimAlexander , my table name don't include space , i can use query like (" SELECT username FROM user ") There is no difference or a problem in both cases

Alex Over a year ago

there is a difference, in your current code you use space character after TABLE_PREFIX before user. so if TABLE_PREFIX = 'wp' real query will be SELECT username FROM wp user which is not correct

|

Collectives™ on Stack Overflow

Using foreach with mysql results array

1 Answer 1

6 Comments

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

6 Comments

Related