php check if mysql select exists or empty

Question

i'm trying to make a check to show content or not:

$stmt = $mysqli->prepare("SELECT *
        FROM friends
        WHERE friendID = ?
        AND userID = ?");

$stmt->bind_param('ii', $id, $connectedUserID);
$stmt->execute();

if (login_check($mysqli) == true && $stmt->num_rows > 0) {
    //do something
} else echo 'you are not allowed to be here';

I want to check if there's any row with those two ids.

But it's not working, i tried many different ways like using === or !== false.

Any help?

Ty!

as said, the bind_param method doesn't runs the uery, you have to use the execute method to run your builded query — kitensei
– kitensei, Commented Sep 30, 2014 at 8:16

meda · Accepted Answer · 2014-10-01 05:59:00Z

1

Most likely login check does not return true, the following should be enough

if(isset($id, $connectedUserID)){
    $stmt = $mysqli->prepare("SELECT *
            FROM friends
            WHERE friendID = ?
            AND userID = ?");

    $stmt->bind_param('ii', $id, $connectedUserID);
    $stmt->execute();

    $stmt->execute();
    if ($stmt->num_rows > 0) {
        //do something
    } else{
        echo 'you are not allowed to be here';
    }
}else{
    //redirect to login
}

If the user need to be logged in then this is not the place to check for that, it should be prior to this query.

edited Oct 1, 2014 at 5:59

answered Oct 1, 2014 at 4:35

meda

45.5k14 gold badges96 silver badges123 bronze badges

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

8 Comments

Chazy Chaz Over a year ago

Yes, the user needs to be logged in, but if $connectedUserID = $_SESSION['user_id']; is empty he won't be able to pass the if. So is that enough, right?

meda Over a year ago

yes exactly, this logic has a flaw, because you need to be connected to get the value for this params, so that should be checked before running the query

Chazy Chaz Over a year ago

ok added a if with login_check function... I don't know if calling too much times this function can broke it or something :P

meda Over a year ago

You just need to make sure the session are set before the query. let me know if you need more help

Chazy Chaz Over a year ago

I have it this way, "if login_check == true" and inside "if $stmt->num_rows > 0". The problem is that I don't know if it works because I'm logged in and the script sends me to the else... i think it's because I'm already using login_check by comparing it to false... I mean, i need to use it many time and not always true.

|

Jonathan Solorzano · Accepted Answer · 2014-09-30 07:05:57Z

1

Try using a simple '=' or declare a boolean variable and set it 'true' and then compare

Collectives™ on Stack Overflow

php check if mysql select exists or empty

2 Answers 2

8 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

8 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related