Issue with returning value in php function

Question

I'm new in programming and especially with php and MySQL. I have to create dynamic web site for homework. My problem is with the function below. I want it to return int value of tag by given tag name (string). But in my case the function returns every time '1'. Can anyone help me to solve this problem, thanks.

public function getTagIdByName(string $tagName) : int
    {
        $statement = self::$db->prepare("SELECT tags.id FROM tags WHERE tags.name = ? ");
        $statement->bind_param("s", $tagName);
        $result = $statement->execute();
        return $result;
    }

Using mysqli prepared statements, you'll need to bind_result to a variable before fetching. See the example here. — Don't Panic
– Don't Panic, Commented Aug 15, 2016 at 20:20

BeetleJuice · Accepted Answer · 2016-08-15 21:19:35Z

1

The problem is that you're returning the result of execute(), but that function doesn't actually give you your query result. You need to fetch results after making sure the execution was successful.

//don't forget to error-check before using query results
$statement->execute() or die($statement->error);
$result = $statement->get_result(); //retrieve results for processing

if(!$result->num_rows) return null;//if the id was not found in the DB    
else return $result->fetch_assoc()['id'];

edited Aug 15, 2016 at 21:19

answered Aug 15, 2016 at 20:13

BeetleJuice

41.1k24 gold badges120 silver badges181 bronze badges

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

6 Comments

Barmar Over a year ago

He's using mysqli, not PDO.

Barmar Over a year ago

Let me know when you fix your answer and I'll remove my downvote.

BeetleJuice Over a year ago

@Barmar I made the change. How did you know that mysqli was used by the OP?

Barmar Over a year ago

Because he's using $statement->bind_param(), not $statement->bindParam().

Barmar Over a year ago

BTW, you can only use fetch_assoc() with a prepared statement if you're using the MYSQLND driver. Otherwise you need to use $statement->bind_result().

|

Rajpal Singh · Accepted Answer · 2016-08-15 20:57:25Z

1

You can achieve easily with

$data = $result->fetch_assoc();
return $data['id']; // you can change with which you want to return with field name

And whichever you can use your returned values.

answered Aug 15, 2016 at 20:57

Rajpal Singh

3111 silver badge13 bronze badges

Collectives™ on Stack Overflow

Issue with returning value in php function

2 Answers 2

6 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related