Iterating PDO queries with php and mysql

Question

I have this problem I can't figure out, I query the DB this way:

$stmt = $this->pdo->prepare('SELECT customer_name FROM active_users WHERE a_id= ?');
$stmt->execute(array($a_id));
$c_name = $stmt->fetch(PDO::FETCH_OBJ);
API::writeToLog('CMD=leave , Customer_Name = ' . $c_name->customer_name, $customer_name);

This: $c_name->customer_name gives the following error :

Notice: Trying to get property of non-object

How can I iterate my database, and whats the difference between fetch and fetchAll ? which should I use?

This is the DB schema:

+---------------+--------------+------+-----+---------+-------+
| Field         | Type         | Null | Key | Default | Extra |
+---------------+--------------+------+-----+---------+-------+
| a_id          | varchar(40)  | NO   | PRI | NULL    |       |
| token         | varchar(64)  | NO   |     | NULL    |       |
| nick          | varchar(255) | NO   |     | NULL    |       |
| ip            | varchar(32)  | NO   |     | NULL    |       |
| customer_name | varchar(255) | YES  |     | NULL    |       |
+---------------+--------------+------+-----+---------+-------+

thanx!

EDIT: fixed a typo..

xdazz · Accepted Answer · 2012-05-29 11:03:42Z

3

You should add an if condition to check whether the record exists.

$stmt = $this->pdo->prepare('SELECT customer_name FROM active_users WHERE udid = ?');
$stmt->execute(array($udid));
if ($c_name = $stmt->fetch(PDO::FETCH_OBJ)) {
  API::writeToLog('CMD=leave , Customer_Name = ' . $c_name->customer_name, $customer_name);
}

For fetchAll, it returns all the records at once.

As @Darragh pointed out, you have a typo for $c_name.

edited May 29, 2012 at 11:03

answered May 29, 2012 at 10:58

xdazz

161k38 gold badges255 silver badges278 bronze badges

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

3 Comments

Darragh Enright Over a year ago

+1 always good practice to check your returned assignment values

Li3ro Over a year ago

when using the fetchAll like this: $messages = $stmt->fetchAll(PDO::FETCH_OBJ); should I use 'if' to check the returned value as well? if so, iterating over the pdo objects will be as followed ? : foreach ($messages as $message) { ... }

xdazz Over a year ago

@Li3ro You just need iterate it.

Darragh Enright · Accepted Answer · 2012-05-29 11:00:12Z

0

Typo? Should it not be $cname->customer_name instead of $c_name->customer_name?

EDIT

To be specific, you're assiging your return object to $cname:

$cname = $stmt->fetch(PDO::FETCH_OBJ);

But you're referring to $c_name on the next line:

API::writeToLog('CMD=leave , Customer_Name = ' . $c_name->customer_name, $customer_name);

answered May 29, 2012 at 11:00

Darragh Enright

14.2k8 gold badges45 silver badges49 bronze badges

1 Comment

Li3ro Over a year ago

You right, its a typo when editing the question, not in code.. sry

Collectives™ on Stack Overflow

Iterating PDO queries with php and mysql

2 Answers 2

3 Comments

1 Comment

Your Answer

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

Related