Posted by J.O. Aho on 05/28/07 17:31

Patrick wrote:
> Hi
>
> I am trying to write a simple database class to encapsulate all the
> database functions in one. Howerver I am having problems with
> while($row = mysql_fetch_array($IsResult,MYSQL_NUM)) line it never
> executes the loop. $IsResult is reeves no value.

Then the mysql_query() returned a false, you should check for that before you
start to loop, if it's false you should fetch the mysql_error().


> function sqlQuery($query)
> {
> $numberofResults = 0;
> $IsResult = mysql_query($query);
>
> echo "IsResult= '".$IsResult."' <br/>\n";
>
> while($row = mysql_fetch_array($IsResult,MYSQL_NUM))

Why not use MYSQL_BOTH (this is the default, so you don't need to assign it to
mysql_fetch_array())? This way you can seach your mResult array with both the
column name and with the column number.

> {
> $this->mResult[] = $row;
> $numberofResults++;
> echo $numberofResults++;
> echo "loop";
> }
>
> return $numberofResults;

You don't have to count, as you have mysql_num_rows() which will give you the
amount of rows directly after you done the mysql_query()



--

//Aho

[Back to original message]