herghost Posted October 23, 2009 Share Posted October 23, 2009 Hi all, I have this: You currently have <?php echo $row['quant'] ;?> stocks in <?php echo $rows['stock_name']; ?> available to sell. The current selling price is $<?php echo $rows['t_val'];?> each.<br> <br> Please select how many of these stocks you wish to sell. <form action="" method="post"> <label>Quantity</label> <select name="quantity"> <?php if ($q > '0' && $q < '6'); { echo '<option value="'; echo $q ; echo '">All ('; echo $q ; echo ')</option>'; echo '<option value="1">1</option>'; echo '<option value="5">5</option>'; echo '<option value="10">10</option>'; } ?> </select> </form> Which is meant to display the form if the amount of stocks held are between 1 and 5, however it displays regardless of the amount of stocks held! Whats going on?! Link to comment https://forums.phpfreaks.com/topic/178719-solved-if-statement-help/ Share on other sites More sharing options...
Bricktop Posted October 23, 2009 Share Posted October 23, 2009 Hi herghost, Remove the ; from the end of the if statement, i.e.: if ($q > '0' && $q < '6'); Also, have you correctly defined $q? For example: $q = $row['quant']; Your code currently does not show it being defined but that may be because you did not include that part of the code here. Hope this helps. Link to comment https://forums.phpfreaks.com/topic/178719-solved-if-statement-help/#findComment-942741 Share on other sites More sharing options...
taquitosensei Posted October 23, 2009 Share Posted October 23, 2009 also remove the quotes around numbers. It's changing it to a string then back to an integer to compare it. Just leave it as an integer by getting rid of the quotes. Link to comment https://forums.phpfreaks.com/topic/178719-solved-if-statement-help/#findComment-942760 Share on other sites More sharing options...
herghost Posted October 23, 2009 Author Share Posted October 23, 2009 Thanks to both of you Link to comment https://forums.phpfreaks.com/topic/178719-solved-if-statement-help/#findComment-942761 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.