Warning: mysql_num_rows() expects parameter 1 - при передаче символов

[Женя32]

Здравствуйте!
Подскажите пожалуйста, почему когда через url передаются только цифры, ошибок нет, но, когда в строке присутствуют другие символы, выходит такая ошибка:


Warning: mysql_num_rows() expects parameter 1 to be resource, boolean given in C:\Programming\xampp\htdocs\shop\ac tivation.php on line 12

PHP код:

<?php
    include("blocks/db_connect.php");
    include("functions/functions.php");    

$msg = '';
if(!empty($_GET['code']) && isset($_GET['code']))
{
    $code=clear_string($_GET['code']);

    $c=mysql_query("SELECT id FROM reg_user WHERE email_active_code=$code");

    if(mysql_num_rows($c) > 0)
    {

        $count=mysql_query("SELECT * FROM reg_user WHERE email_active_code=$code and email_status='unconfirme'");

        if(mysql_num_rows($count) > 0)
        {
        mysql_query("UPDATE reg_user SET email_status='confirm' WHERE email_active_code='$code'");
        $msg="Ваш e-mail успешьно подтвержден";    
        $reg_mess_class = 'reg_message_good';
        }
        else
        {
          $msg ="Ваш e-mail уже подтвержден";
        $reg_mess_class = 'reg_message_good';
        }

    }
    else
    {
    $msg ="Неверный код активации.";
    $reg_mess_class = 'reg_message_error';
    }

}
        
?>

[ADSoft]

да наверное потому что поле - email_active_code имеет числовой тип, и при попытке сравнения с символами запрос дает ошибку - но у вас этот случай не предусмотрен (вывода ошибок запроса), кроме того, думаю ваш код небезопасен для SQL инъекций.
Не знаю что там clear_string у вас творит, но если вместо него использовать intval() то на выходе всегда будет число и ваша ошибка исчезнет

[Женя32]

Сообщение от ADSoft

да наверное потому что поле - email_active_code имеет числовой тип, и при попытке сравнения с символами запрос дает ошибку - но у вас этот случай не предусмотрен (вывода ошибок запроса), кроме того, думаю ваш код небезопасен для SQL инъекций.
Не знаю что там clear_string у вас творит, но если вместо него использовать intval() то на выходе всегда будет число и ваша ошибка исчезнет

Нет, поле email_active_code - символьный тип.

clear_string:

PHP код:

<?php

function clear_string($cl_str)
  {
    $cl_str = strip_tags($cl_str);
    $cl_str = mysql_real_escape_string($cl_str);
    $cl_str = trim($cl_str);  
    
    return $cl_str;
     
  }

?>

[Serge_Bliznykov]

строковые значения должны быть в кавычках:

$c=mysql_query('SELECT id FROM reg_user WHERE email_active_code="'.$code.'"'); 
if($c && (mysql_num_rows($c) > 0)) 
.....

от библиотеки mysql нужно отказываться, она уже устарела (в PHP7.x её совсем уже убрали),
переходите на библиотеку mysqli или PDO

[Женя32]

Сообщение от Serge_Bliznykov

строковые значения должны быть в кавычках:

$c=mysql_query('SELECT id FROM reg_user WHERE email_active_code="'.$code.'"'); 
if($c && (mysql_num_rows($c) > 0)) 
.....

от библиотеки mysql нужно отказываться, она уже устарела (в PHP7.x её совсем уже убрали),
переходите на библиотеку mysqli или PDO

Большое спасибо!!!