expects parameter warning 1 to be resource

Question

This below code is giving me mysql_fetch_array() expects parameter

<?php
    $query = $forumdb->prepare("SELECT COUNT(*) as cnt FROM smf_personal_messages");
    $query->execute();
    $num_rows = mysql_fetch_array($query);

    if ($query->rowCount() == 0) {
        echo "<tr><td colspan='6'><small>No rows found</small></td></tr>";
    }

    echo ($num_rows['cnt']);
?>

Warning: mysql_fetch_array() expects parameter 1 to be resource, object given in /home/gamin1/public_html/ucp/forumstats.php on line 127

What database interface are you using? Because you can't play pick-a-mix with MySQL, MySQLi and PDO — Mark Baker, Sep 19 '15 at 10:20
"pick-a-mix", I like that. [Is that for here or to go?](https://www.youtube.com/watch?v=7fHj_YWsI-I&t=0m15s) :-) — VolkerK, Sep 19 '15 at 10:33
http://stackoverflow.com/questions/12859942/why-shouldnt-i-use-mysql-functions-in-php — Alessandro Da Rugna, Sep 19 '15 at 11:06

VolkerK · Answer 1 · 2015-09-19T15:24:13.427

1

Seems to be a reasonable assumption that you want to use PDO. As has already been mentioned, you can't mix pdo(_msyql) with the other mysql apis.
But there are some other issues with your script:

<?php
// not an error per se, but:
// no placeholders in the query + sent only once -> prepared statement superfluous
$result = $forumdb->query("SELECT COUNT(*) as cnt FROM smf_personal_messages");
// assuming PDO::ATTR_ERRMODE=>PDO::ERRMODE_EXCEPTION, so no error handling here....

// Count(*) with no explicit grouping always returns exactly one record, so
// ->rowCount() will do you no good
// instead fetch that one record and read the field holding the value of Count(*) as cnt
$row = $result->fetch();
if ( 0==$row['cnt'] ) {
    echo "<tr><td colspan='6'><small>table cotains no records</small></td></tr>";
}
else {
    echo '# of rows: ', row['cnt'];
}

edited Sep 19 '15 at 15:24

answered Sep 19 '15 at 10:33

VolkerK

95,432
20
163
226

OP reposted http://stackoverflow.com/questions/32668158/query-fetching-issue and used one of the answers in their new code. I voted to close the other one as unclear, since we have no idea which MySQL API they're using to connect with. Edit: My guess http://stackoverflow.com/questions/32666137/few-warnings-expects-parameter they're still using mysql_ to connect with. – Funk Forty Niner Sep 19 '15 at 13:12
Exactly. I think/feel the OP needs a good jolt with tutorials. They're not far off though ;-) – Funk Forty Niner Sep 19 '15 at 15:29

score 0 · Answer 2 · edited Sep 19 '15 at 10:26

0

Use $query->fetch() instead of mysql_fetch_array($query).

You are using PDO here: http://php.net/manual/fr/pdostatement.fetch.php

edited Sep 19 '15 at 10:26

Tim Biegeleisen

502,043
27
286
360

answered Sep 19 '15 at 10:24

Alexandre Keller

61
6

expects parameter warning 1 to be resource

2 Answers2

Linked