PHP PDO Fetch not working?

Question

I can successfully select an object, but I cannot fetch all rows from the database using the following code, can anyone see any obvious errors?

    $sql2 = "SELECT ID, Latitude, Longitude, Name FROM Countries";
    $stmt2 = $pdo->prepare($sql2);
    $stmt2->execute();

    while ($row = $stmt2->fetch(PDO::FETCH_ASSOC)) {
        echo $countryID = $row->ID;
        echo $countryName= $row->Name;
        echo $longitude2 = $row->Longitude;
        echo $latitude2 = $row->Latitude;
    }

You should use those arrays as: $row['key'], not $row->key. — Tibor B.
– Tibor B., Commented Jul 14, 2015 at 9:37
PDO::FETCH_ASSOC fetches an assoc array, which you use as an object, use PDO::FETCH_OBJ or change $row->ID to $row['ID']. There's also no params in your query, you needn't prepare it if it really is a hard-coded query string — Elias Van Ootegem
– Elias Van Ootegem, Commented Jul 14, 2015 at 9:40

Nishant Ghodke · Accepted Answer · 2015-07-14 10:10:10Z

2

The parameter PDO::FETCH_ASSOC tells PDO to return the result as an associative array. SO you can fetch array not object

 while ($row = $stmt2->fetch(PDO::FETCH_ASSOC)) {
        echo $countryID = $row['ID'];
        echo $countryName= $row['Name'];
         //Rest of the code

    }

edited Jul 14, 2015 at 10:10

Nishant Ghodke

9231 gold badge12 silver badges21 bronze badges

answered Jul 14, 2015 at 9:38

Saty

22.5k7 gold badges35 silver badges52 bronze badges

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

3 Comments

Frog82 Over a year ago

Ahhh arrays - thanks, will accept the answer when I can

Saty Over a year ago

You are welcome @Frog82 !! enjoy . Who has given downvote at least dare to give reason.

Elias Van Ootegem Over a year ago

@Saty: I suspect the -1 is because, when taken literally, your answer might be seen as incorrect ("So you can fetch array not object" could be taken to mean PDO::FETCH_OBJ doesn't exist). Another reason why one might down-vote is you failed to address other (less obvious) issues like the pointless PDO::prepare call and the redundant assignments (echo $countryID = $row['ID']; vs echo $row['ID'];). I'm just guessing here, though, seeing as I didn't vote down, but I, too, dislike down-votes without a comment explaining why.

Your Common Sense · Accepted Answer · 2015-07-14 09:46:43Z

1

You have to realize that PDO::FETCH constants are on purpose. And if you want to use object notation, you have to specify PDO::FETCH_OBJ instead of ...ASSOC.

Anyway, PDO::FETCH_LAZY should be most preferred way, as it will let you use ANY notation:

while ($row = $stmt2->fetch(PDO::FETCH_LAZY)) {
    echo $row->ID;    // all
    echo $row['ID'];  // three
    echo $row[0];     // works
}

with even less memory overhead than any other method. With no memory overhead at all, to be exact.

edited Jul 14, 2015 at 9:46

answered Jul 14, 2015 at 9:40

Your Common Sense

158k42 gold badges226 silver badges374 bronze badges

1 Comment

Elias Van Ootegem Over a year ago

Not sure about the "less memory overhead" part of your answer. Have you checked the implementation, and compared the overhead? If so, how? The PDORow instance it returns has quite a few handlers... Or are you referring to memory overhead caused by using all possible notations, in which case I agree, but the handlers do have some (negligible/micro-optimizable) overhead

Collectives™ on Stack Overflow

PHP PDO Fetch not working?

2 Answers 2

3 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related