php - foreach loop only runs one

Question

I have this foreach loop:

$d=$dbh->prepare("SELECT * FROM users_rented WHERE since <= unix_timestamp(CURRENT_TIMESTAMP - INTERVAL 14 day) AND clicks_last <= unix_timestamp(CURRENT_TIMESTAMP - INTERVAL 14 day)");
    $d->execute();
        $array = array();
        foreach ($d as $data ) {
                 $array[] = $data['id'];
                     #print_r($new_array);
                     $userToRecycleFor = $data['user_by'];

                    $outcome =  $rentedrefs->_recycleMulti(0, $userToRecycleFor, $array, 1);


        }

The $d query has 2406 results when I run it in the MySQL database.

The foreach loop is only being run 1 time on every page refresh. So instead of updating all 2406 users (as per the SQL query), it updates only 1 per refresh.

What am I doing wrong?

You should fetch the result. Somebody had exactly the same question as you -> stackoverflow.com/questions/16916036/… — pbaldauf
– pbaldauf, Commented Nov 2, 2014 at 16:23
you are looking for PDO::fetch php.net/manual/en/pdostatement.fetch.php — Victory
– Victory, Commented Nov 2, 2014 at 16:23

Asik · Accepted Answer · 2014-11-02 16:24:04Z

1

you need to fetch the data before looping

$result = $d->fetchAll();

foreach ($result  as $data ) {
   $userToRecycleFor = $data['user_by'];
}

answered Nov 2, 2014 at 16:24

Asik

7,9674 gold badges30 silver badges35 bronze badges

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

Comments

Marcio Mazzucato · Accepted Answer · 2014-11-02 16:27:07Z

1

You have to fetch the result before loop over it. Try to change these lines:

$d->execute();

$array = array();

$rows = $d->fetchAll(PDO::FETCH_ASSOC);

foreach ($rows as $data) {
     $array[] = $data['id'];
     $userToRecycleFor = $data['user_by'];

     $outcome =  $rentedrefs->_recycleMulti(0, $userToRecycleFor, $array, 1);
}

answered Nov 2, 2014 at 16:27

Marcio Mazzucato

9,36511 gold badges71 silver badges82 bronze badges

Comments

hazzard · Accepted Answer · 2014-11-02 16:45:58Z

0

Normally, you fetch results like this: (e.g. mysqli)

$mysqli = new mysqli(HOST, USER, PASSWORD, DB);
$query = $mysqli->query("SELECT id, name, store FROM food");

while($array = $query->fetch_array(MYSQLI_ASSOC))
{
   $finalArray = $array;
}

foreach($finalArray as $item)
{
   // your process
}

answered Nov 2, 2014 at 16:45

hazzard

1361 gold badge1 silver badge5 bronze badges

Collectives™ on Stack Overflow

php - foreach loop only runs one

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related