It depends on how you plan to read the result set. But in the actual example you have given, you are not interested in reading any returned data. The only thing that interests you is whether there is a record or not.
In that case your code is fine, but it would work equally well with get_result.
The difference becomes more apparent, when you want to get for example the userid of the user with the given email:
SELECT id FROM users WHERE email = ?
If you plan to read out that id with $stmt->fetch
, then you would stick to
store_result
, and would use bind_result
to define in which variable you want to get this id, like this:
$stmt->store_result();
$stmt->bind_result($userid); // number of arguments must match columns in SELECT
if($stmt->num_rows > 0) {
while ($stmt->fetch()) {
echo $userid;
}
}
If you prefer to get a result object on which you can call fetch_assoc()
or any of the fetch_* variant methods, then you need to use get_result
, like this:
$result = $stmt->get_result(); // You get a result object now
if($result->num_rows > 0) { // Note: change to $result->...!
while ($data = $result->fetch_assoc()) {
echo $data['id'];
}
}
Note that you get a result object from get_result
, which is not the case with store_result
. You should get num_rows
from that result object now.
Both ways work, and it is really a matter of personal preference.