Re: If statement duplicating mysql records?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Jason Pruim wrote:

On Jun 19, 2007, at 4:20 PM, Jim Lucas wrote:

Jason Pruim wrote:
Okay, so I have a question... Probably pretty easy, but why would my if statement show more records then what are in the database?
if($row[5] =='Level4'){ // White Highlight
            echo "<TR><TD bgcolor=".$Level4.">$row[0] </td>";
            echo "<td bgcolor=".$Level4.">$row[1]  </td>";
echo "<td bgcolor=".$Level4."><A href ='$row[2]'>Instructions</A></td>"; echo "<TD bgcolor=".$Level4."><A href='update.php?taskid=$row[0]'>Click here!</A>";
        }// End of Level 4
                             else
               {// Green Highlight
            echo "<TR><TD bgcolor=".$unclassified.">$row[0] </td>";
            echo "<td bgcolor=".$unclassified.">$row[1]  </td>";
echo "<td bgcolor=".$unclassified."><A href ='$row[2]'>Instructions</A></td>"; echo "<TD bgcolor=".$unclassified."><A href='update.php?taskid=$row[0]'>Click here!</A>";
                   }// End of Unclassified

Why not do it this way instead.

while( $row = mysql_fetch_array($results) ) {

    $rowColor = $unclassified;

    // White Highlight
    if($row[5] =='Level4'){
        $rowColor = $Level4;
    }

    echo "<TR><TD bgcolor=".$rowColor.">$row[0] </td>";
    echo "<td bgcolor=".$rowColor.">$row[1]  </td>";
echo "<td bgcolor=".$rowColor."><A href='$row[2]'>Instructions</A></td>"; echo "<TD bgcolor=".$rowColor."><A href='update.php?taskid=$row[0]'>Click here!</A>";

}

And that just goes to show how much I have yet to learn! Once I typed that out I went from like, 86 lines of code down to 55... And it works as well!

Just for my own knowledge, Do you know why the else was being applied after the condition was met?

Thanks Jim!



nope, couldn't guess why.

I would look at your result set and see if you had multiple lines returned. One person mentioned that you might be using a JOIN in you SQL call. This could lead to multiple lines.

Start investigating your SQL statement I would suggest.

--
Jim Lucas

   "Some men are born to greatness, some achieve greatness,
       and some have greatness thrust upon them."

Unknown

--
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php


[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux