how do I use mysql_num_rows here?

ejwf

I have a whole load of code that basically sorts through the data received from a user input form and based on 4 clauses does one of 4 things. The first three are echo an error message, the last being insert data into the mysql database.......
Here's the thing, if there are 3 rows that match the first error message clause, it echos the error message 3 times, which I don't want. How do I make it only echo it once?

I was thinking mysql_num_rows, but cannot make it work.

Here is the code I have so far. When I run the code I get the error message mysql_num_rows(): supplied argument is not a valid MySQL result resource for the lines where the data from the forms matches the if / else if clauses on the lines where it has if ($Count_Check >0)

Can anyone make any suggestions on how I get this to work - thanks

if (is_array($_POST['Size'])  && count($_POST['Size'])>0) 
{ 
  $arr_size = count($_POST['Size']);  

  for ($i=0; $i< $arr_size; $i++)  

  {  

// if the user has put the size in but not thecode   
if (!empty($_POST['Size'][$i]) && empty($_POST['Id'][$i])) 
{   $count_check = mysql_num_rows($i); 

		if ($count_check > 0) 
		{ 
        echo "<br><br><br><br>" . 
             "<font face=\"Arial, Helvetica, sans-serif\">" . 
             "Sorry, it appears that a <b>Code</b> is missing.<br>" . 
         " Please kindly return to the " . 
             "input form, and re-enter this code.  " . 
             "<br>Thank you" . 
             "</font>"; 
		}	 
} 

   // if the user has put neither the size in nor the code   

     else if (empty($_POST['Size'][$i]) && empty($_POST['Id'][$i])) 
    {   	$count_check = mysql_num_rows($i); 

		if ($count_check > 0) 
		{ 
      echo"<br><br><br><br><font face=\"Arial, Helvetica, sans-serif\">" . 
          "You have chosen not to enter data today " . 
          "<br>Please ensure that you log back in when " . 
          " you have the remaining information" . 
          "<br>Thank you</font>" ;  
		 } 	
} 


   // if the user has put the code in but not the data   

  else if (empty($_POST['Size'][$i]) && !empty($_POST['Id'][$i])) 
    { $count_check = mysql_num_rows($i); 

		if ($count_check > 0)
		{ 
      echo "<br><br><br><br><font face=\"Arial, Helvetica, sans-serif\">" . 
           "Sorry, it appears that the information " . 
           "are missing.<br><br>Please click on the back " . 
           " arrow of your browser to return to the " . 
           "input form, and re-enter this code.  <br>Thank you</font>";  
		 }  
} 

   // insert the record 
    else     

    { 
	$SectionRef = $_POST['SectionRef']; 

// sets date into correct format to match database
$today = date('YmdHis');

$Size = strip_tags(trim($_POST['Size'][$i])); 
$Colour = strip_tags(trim($_POST['Colour'][$i])); 
$Id = ($_POST['Id'][$i]); 

$query = "INSERT INTO tblResults (DateRef,MRef,SRef,CRef,Size,Colour,LoadedBy)
		VALUES ('$DateRef','$Id','$SRef','$CRef','$Size','$Colour','$UserId')";
$result = mysql_query($query)
or die(mysql_error());
    }  

  }  // close for i < $arr_size 
} //close if count > 0

laserlight

mysql_num_rows() takes a resource, not an array index.

Use a flag, e.g. $error_flag, to determine if an error has occured.

ejwf

Hi - thanks for your reply, but I have never come across using a flag before, so not sure exactly what this is / what you mean.
I have done a search on these forum and it doesn't really show me what I need to do.
I would be grateful if you could explain a bit further - thanks

andrewtayloruk

mysql_num_rows() expects a result identifier, perhaps you need to use sizeof() ?

mysql_num_rows() example:

$query = 'SELECT field1, field2 FROM table';
$result = mysql_query($query);

if($result) {

$number_of_rows_returned_from_query = mysql_num_rows($result);
}

sizeof() example:

$array = Array(1,2,3,4,5,6);
$number_of_items_in_array = sizeof($array);
echo $number_of_items_in_array; // Will be 6;

Does this make sense?

Andrew

ejwf

thank you andrewtayloruk - I don't get any error message with sizeof(), but it isn't doing what I would expect.
For each of the if / else clauses if I have got

// if the user has put the reading in but not the code   

    if (!empty($_POST['Size'][$i]) && empty($_POST['Id'][$i])) 
    {   $count_check = sizeof($i); 

if ($count_check > 0) 
  { 
            echo $count_check;
	echo "<br><br><br><br>" . 
                 "<font face=\"Arial, Helvetica, sans-serif\">" . 
                 "Sorry, it appears that a <b> Code</b> is missing.<br>" . 
         " Please kindly return to the " . 
                 "input form, and re-enter this code.  " . 
                 "<br>Thank you" . 
                 "</font>"; 
			}	 
    }

I have added echo $count_check to see what happens but it doesn't output anything, so obviously the if ($count_check > 0) {} isn't going to work, and I am still getting an error message for each row instead of just one error message - any ideas? - thanks

andrewtayloruk

Could you please alter

for ($i=0; $i< $arr_size; $i++)

{

// if the user has put the size in but not thecode    
if (!empty($_POST['Size'][$i]) && empty($_POST['Id'][$i]))

to be

print_r($_POST);
for ($i=0; $i< $arr_size; $i++)

{

// if the user has put the size in but not thecode    
if (!empty($_POST['Size'][$i]) && empty($_POST['Id'][$i]))

And post the results, print_r will show us what information is in $_POST as an array, it will allow me to better understand what you are trying to do.

Thanks

Andrew

ejwf

Andrew - I get

Array ( [SRef] => 137

=> [Id] => Array ( [0] => 205 [1] => [2] => 201 [3] => [4] => ) [Size] => Array ( [0] => 4 [1] => 4 [2] => 4 [3] => [4] => ) [Colour] => Array ( [0] => 4 [1] => 4 [2] => 4 [3] => [4] => ) [submit] => Submit ) 1 [/b]// is this sizeof() ?

Sorry, it appears that a Code is missing.
Please kindly return to the input form, and re-enter this code.
Thank you
1 // is this sizeof() ?

You have chosen not to enter data today
Please ensure that you log back in when you have the remaining information
Thank you
1 // is this sizeof() ?

You have chosen not to enter data today
Please ensure that you log back in when you have the remaining information
Thank you