Parse error: syntax error, unexpected T_STRING

cimanyenje

I have this code which I want to show the records of a table and it's giving me a "Parse error: syntax error, unexpected T_STRING", it's saying the error is on this line;

echo "<tr><th align=left>Student Name</th><th align=left>Address</th><th align=left>Phone Number</th><th align=left>Reg-Number</th></tr>\n";
but I cant see it. what is the problem.

<?php
$db = mysql_connect("localhost", "root");
mysql_select_db("membership",$db);
$query = "SELECT  * FROM members;
$result = mysql_query($query) or die(mysql_error());
if ($mycolumn = mysql_fetch_array($result)) {
echo "<table border=1 align=center cellspacing=0\n";
echo "<tr><th align=left>Student Name</th><th align=left>Address</th><th align=left>Phone Number</th><th align=left>Reg-Number</th></tr>\n";
do {
printf("<tr><td>%s %s %s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</td></tr>\n",  $mycolumn["title"], $mycolumn["firstname"], $mycolumn["lastname"], $mycolumn["address"],
$mycolumn["phonenumber"], $mycolumn["regnumber"]);
} while ($mycolumn = mysql_fetch_array($result));
//echo "<BR>";
echo "<h3>REGISTERED MEMBERS</h3>";
echo "<BR>";
echo "</table>\n";
} else {
echo "Sorry, no records were found!";
}
?>

NogDog

You left out the closing quote on the line where you define $query. (The line number in a parse error message is the line where PHP determines something is wrong, but is not necessarily where the actual error is.) Using an editor with syntax highlighting can help you find these things, much as using this forum's [noparse]

...

[/noparse] tags around your code makes it fairly obvious where things went wrong in this case.

cimanyenje

Thank you NopDog. That went well. but then I encountered another prob after that collection, It says

Warning: printf() [function.printf]: Too few arguments on line 157

Am using an editor but cant really detect the prob (Using Notepad ++)

bradgrafelman

cimanyenje;11028329 wrote:
Warning: printf() [function.printf]: Too few arguments on line 157

The problem is just as the error message states; you've given [man]printf/man a total of 8 argument placeholders in the format string, yet you then only pass it 6 arguments. (See the manual page for [man]sprintf/man for a better idea of how printf() works.)

cimanyenje

I was thinking of combining (concatenating) the title, firstname and last name to be in one column, any idea how I can do that?

Weedpacket

You are doing it.

But look at your table row. How many cells is it supposed to have? Count them. How many are there really?