PDO query not returning results in PHP foreach loop

Question

I have a function (below) which isn't returning a result set from a PDO query inside a foreach loop. Not sure what could be the problem. To explain a bit, I'm running PHP v5.6.3 and the $csv string passed to the function is something like:

$csv = "red, brown, blue";

After the str_getcsv is ran, print_r($colors) works as expected. I've tried preparing the query and binding values inside or outside the loop with no success, and if I pull the query outside of the loop (and run it once) it works.

function generateColors($csv) {
    global $db; // Connect DB
    $colors = str_getcsv($csv, ","); //Converts the CSV to an array
    $query = $db->prepare("SELECT * FROM colors WHERE slug = :slug");
    foreach ($colors as $slug) {
        $query->bindValue(':slug', $slug, PDO::PARAM_STR);
        $query->execute();
        $result = $query->fetch(PDO::FETCH_ASSOC);
        echo "COLOR: " .$slug;
        echo "<br>";
        var_dump($result);
    }
}

The output (below) shows that the loop is working, just the query seems to stop returning values after 1 iteration. Any help on why I'm getting a bool(false) instead of an array set like [column 1]=>value 1, [column 2]=>value 2, etc., would be greatly appreciated.

COLOR: red
array -> [column]=>value, [column]=value, etc. // returned as expected.
COLOR: brown
bool(false)
COLOR: blue
bool(false)

THE PROBLEM IS you are thinking that $slug of foreach will automaticlly equalizie $slug of bindvalue which is not going to happen. you need to put your bindValue code in foreach and then need to check — Death-is-the-real-truth
– Death-is-the-real-truth, Commented May 12, 2015 at 18:31
Edited function above to put bindValue back inside loop. Unfortunately, still resulting in a bool(false). — PanicRev
– PanicRev, Commented May 12, 2015 at 18:39
Actually... putting it back inside the loop, I've noticed it returns the first result set properly (for the first loop run), however, all subsequent values in the loop are returning the bool(false). — PanicRev
– PanicRev, Commented May 12, 2015 at 18:41
that means you have record for first slug but not for others? I think i am clear? — Death-is-the-real-truth
– Death-is-the-real-truth, Commented May 12, 2015 at 18:43
Correct, my output is like this now: COLOR: red array(11) { ["slug"]=> string(15) "red" ["name"]=> string(15) "Red", etc... COLOR: brown bool(false) etc.. — PanicRev
– PanicRev, Commented May 12, 2015 at 18:46

user4887838 · Accepted Answer · 2015-05-12 18:31:48Z

1

You should bind the params inside the foreach. Before the foreach your variable $slug is not defined.

function generateColors($csv) {
    global $db; // Connect DB
    $colors = str_getcsv($csv, ","); //Converts the CSV to an array
    $query = $db->prepare("SELECT * FROM colors WHERE slug = :slug");
    foreach ($colors as $slug) {
        $query->bindValue(':slug', $slug, PDO::PARAM_STR);
        $query->execute();
        $result = $query->fetch(PDO::FETCH_ASSOC);
        echo "COLOR: " .$slug;
        echo "<br>";
        var_dump($result);
    }
}

answered May 12, 2015 at 18:31

user4887838

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

1 Comment

PanicRev Over a year ago

I do see your point. In fact, I've been tinkering so long with this that I mistakenly should've put it back inside before posting ;) However, even putting the bindValue back inside the loop, I'm still getting a bool(false). :-/

Collectives™ on Stack Overflow

PDO query not returning results in PHP foreach loop

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related