nvee Posted October 29, 2009 Share Posted October 29, 2009 I get a nasty error with this one ... yet everything looks so simple ... a chocolate to the person who tells me what stupid mistake im making: <?php $dbx = mysql_connect("localhost","root",""); if(!$dbx) die("The connection to the database could not be made!"); $db = mysql_select_db($dbname); if(!db) die("Cannot connect to database to obtain course values"); $result = mysql_query("SELECT * FROM course"); while ($row = mysql_fetch_array($result)) { echo "<option value='$row[\"cid\"]'>$row[\"$cname\"]</option>"; } ?> Link to comment https://forums.phpfreaks.com/topic/179538-simple-while-problem/ Share on other sites More sharing options...
Maq Posted October 29, 2009 Share Posted October 29, 2009 I get a nasty error with this one ... It would be a good idea to post the error. Link to comment https://forums.phpfreaks.com/topic/179538-simple-while-problem/#findComment-947350 Share on other sites More sharing options...
nvee Posted October 29, 2009 Author Share Posted October 29, 2009 hahaha my bad ... It was a syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING I figured the problem out tho, it was: my echo which was incorrect ... for the record, here is the correct method (or atleast a working one, im sure there will be a smarter person who can help with a better way to put this echo "<option value='" . $row["cid"] . "'>" . $row["cname"] . "</option>"; Link to comment https://forums.phpfreaks.com/topic/179538-simple-while-problem/#findComment-947353 Share on other sites More sharing options...
Maq Posted October 29, 2009 Share Posted October 29, 2009 As far as ease and understandability, I like writing it this way: echo "{$row['cname']}"; Some may argue on behalf of inefficiency but it only produces a minuscule amount of overhead to interpolate variables and arrays in strings. Link to comment https://forums.phpfreaks.com/topic/179538-simple-while-problem/#findComment-947360 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.