Warning: count(): Parameter must be an array or an object that implements Countable in C:\wamp\www\news13102020\admin\process.php on line 55 [duplicate]

Question

I write this script in roder to verify if the sql empty or not:

if (isset($_GET['edit'])){
 $id = $_GET['edit'];
$update = true;
$result = $mysqli->query("SELECT * FROM task WHERE id_task=$id") or die($mysqli->error());
$count = count($result);
if($count == 1){
   $row = $result->fetch_array();
   $nom_task = $row['name_task'];
   $id_operation = $row['id_operation'];
   $nom_operation = $row['name_operation'];
}

any suggetions to resove this please? the problem is in $count and $result?????

docl · Accepted Answer · 2020-10-26 10:40:47Z

1

mysqli::query returns a mysqli_result object (see https://www.php.net/manual/en/mysqli.query.php) That object is not countable (meaning it cannot be used as a parameter in the count function)

If you want to know the number of rows returned by your SQL request, use $num_rows property from that mysqli_result object :

$result = $mysqli->query('SELECT ...');
$count = $result->num_rows;

Also as a (very important) side note, your code is vulnerable to SQL Injection, please see : https://stackoverflow.com/a/601524

answered Oct 26, 2020 at 10:40

docl

915 bronze badges

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

1 Comment

Your Common Sense Over a year ago

If you give it a bit of a thought, you never need any counting at all

Prakalp varshney · Accepted Answer · 2020-10-26 10:41:08Z

-2

Hey try this code snippet:

if (isset($_GET['edit'])){
 $id = $_GET['edit'];
$update = true;
$result = $mysqli->query("SELECT * FROM task WHERE id_task=$id") or die($mysqli->error());

if(!empty($result) && count($result) == 1){
   $row = $result->fetch_array();
   $nom_task = $row['name_task'];
   $id_operation = $row['id_operation'];
   $nom_operation = $row['name_operation'];
}

Simply check the array if it's empty or not before count check.

answered Oct 26, 2020 at 10:41

Prakalp varshney

436 bronze badges

3 Comments

Your Common Sense Over a year ago

$result will never be empty

Prakalp varshney Over a year ago

Well, I think they can be check this stackoverflow.com/questions/32322359/…

Your Common Sense Over a year ago

Nope they cannot. It is very important to understand the question you are trying to answer, or a post you are referring to.

Collectives™ on Stack Overflow

Warning: count(): Parameter must be an array or an object that implements Countable in C:\wamp\www\news13102020\admin\process.php on line 55 [duplicate]

2 Answers 2

1 Comment

3 Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

3 Comments

Linked

Related