25

I am trying to quickly determine if a user_ID is the owner of a 'goal'. I believe my SQL query is good, but I'm trying to find a nice way of checking the result!

In this case, no matter what I put for $obj_id or $user_id, my function returns true. I assume it's because mysql_num_rows is counting even a false result as a row? So what PHP code should I use to check to see if the result exists or not?

Note that I want something short and elegant! I know I could do it the long way (check count(*), return mysql_assoc then check the count value...) but that is long winded and ugly.

Any ideas? Thanks!

$query = "SELECT EXISTS (SELECT * FROM goals WHERE goal_ID='$obj_id' AND user_ID='$user_id')"; if (@mysql_num_rows(mysql_query($query))!=1) { return false; } else { return true; } 
1
  • Using count(*) should be preferred because it returns just one row instead of potentially millions. Commented Oct 9, 2013 at 15:55

8 Answers 8

42

Don't bother with EXISTS. The in-line exists will always give one row containing "true" or "false".

You're looking for either "zero rows" or "at least one row" so change the query to something like this and then check how many rows are returned

SELECT 1 FROM goals WHERE goal_ID='$obj_id' AND user_ID='$user_id' LIMIT 1 
Sign up to request clarification or add additional context in comments.

2 Comments

Should I use "SELECT 1"? What is this 1 here?
So how should I extract that "true or false" from EXISTS?
10

I like gbn's answer the best, but I wanted to point out that this:

if (@mysql_num_rows(mysql_query($query))!=1) { return false; } else { return true; } 

can be simplified to:

return @mysql_num_rows(mysql_query($query)) == 1; 

1 Comment

I have some other stuff in my "if / else" sections in the real page, so I can't simplify quite that much, but I didn't realize you could just add the '==' like that. Thanks, that's useful!
2

This way is probably faster.

$query = "SELECT EXISTS (SELECT * FROM goals WHERE goal_ID='$obj_id' AND user_ID='$user_id')"; if(mysql_num_rows(mysqli_query($query)) < 1) { // Nothing found! } 

1 Comment

2

Counting how many rows match the criteria should be easier:

$sql = SELECT COUNT(*) FROM goals WHERE goal_ID='$obj_id' AND user_ID='$user_id' $query = mysql_query($sql); $result = mysql_fetch_row($query); return $result[0] >= 1; 

Comments

1

SELECT EXISTS always returns a row! The following code uses the fastest MySql query(according to this answer: https://stackoverflow.com/a/10688065/3710053) and gives in PHP the correct result:

$link = mysqli_connect($DB_SERV, $DB_USER, $DB_PASS, $DB_NAME); $query = "SELECT EXISTS (SELECT * FROM goals WHERE goal_ID='$obj_id' AND user_ID='$user_id' LIMIT 1) as `row_exists`"; if(mysqli_fetch_assoc(mysqli_query($link,$query))['row_exists'] ===0) { // Nothing found! } 

Comments

1

Really working config for MySQL:

$sql="SELECT EXISTS (Select * FROM $dbname.$dbrel WHERE Index_r=$idx AND ... LIMIT 1)"; $r1=mysqli_fetch_row(mysqli_query($conn, $sql)); if (current($r1) == 0) {when no recoeds} else {when exist records } 

Comments

0

what about this:

$query = "SELECT EXISTS (SELECT * FROM goals WHERE goal_ID='$obj_id' AND user_ID='$user_id')"; return mysql_query($query) ? false : true; 

Comments

0
mysql_result(mysql_query("SELECT EXISTS (SELECT * FROM goals WHERE goal_ID='$obj_id' AND user_ID='$user_id')"),0); 

Comments

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.