[Help] Parse Error

March 10, 2004

Well I will first of all post the error...

Parse error: parse error, unexpected T_VARIABLE in C:\Program Files\Apache\Apache2\htdocs\websitetest\register.php on line 11

And here is the code...

(Register.html)

&lt;HTML&gt;
&lt;HEAD&gt;
&lt;TITLE&gt;Register&lt;/TITLE&gt;
&lt;/HEAD&gt;
&lt;BODY&gt;

// Form to register with

&lt;FORM METHOD="post" ACTION="register.php"&gt;
&lt;p&gt;Name:
&lt;INPUT TYPE="text" Name="Name" Size=30&gt; MAXLENGTH=25&lt;/p&gt;
&lt;p&gt;Password:
&lt;INPUT TYPE="text" Name="Password" Size=30&gt; MAXLENGTH=12&lt;/p&gt;
&lt;p&gt;Confirm Password:
&lt;INPUT TYPE="text" Name="CPassword" Size=30&gt; MAXLENGTH=12&lt;/p&gt;
&lt;p&gt;E-Mail:
&lt;INPUT TYPE="text" Name="EMail" Size=30&gt; MAXLENGTH=50&lt;/p&gt;
&lt;p&gt;Website:
&lt;INPUT TYPE="text" Name="Website" Size=30&gt; MAXLENGTH=50&lt;/p&gt;
&lt;p&gt;Location:
&lt;INPUT TYPE="text" Name="Location" Size=30&gt; MAXLENGTH=50&lt;/p&gt;
&lt;p&gt;Occupation:
&lt;INPUT TYPE="text" Name="Occupation" Size=30&gt; MAXLENGTH=50&lt;/p&gt;
&lt;p&gt;ICQ:
&lt;INPUT TYPE="text" Name="ICQ" Size=30&gt; MAXLENGTH=10&lt;/p&gt;
&lt;p&gt;MSN Address:
&lt;INPUT TYPE="text" Name="MSN" Size=30&gt; MAXLENGTH=25&lt;/p&gt;
&lt;p&gt;AIM Address:
&lt;INPUT TYPE="text" Name="AIM" Size=30&gt; MAXLENGTH=25&lt;/p&gt;
&lt;p&gt;Yahoo Address:
&lt;INPUT TYPE="text" Name="Yahoo" Size=30&gt; MAXLENGTH=25&lt;/p&gt;
&lt;P&gt;&lt;INPUT TYPE="submit" NAME="submit" VALUE="Submit"&gt;&lt;/p&gt;
&lt;/FORM&gt;
&lt;/BODY&gt;
&lt;/HTML&gt;

(Register.php)

&lt;?
	if ((!$name) || (!$Password) || (!$CPassword) || (!$EMail)) {
  header( "Location: register.html");
  exit;
	}
$db_name = "websitetest";
$table_name = "$name";

$connection = @mysql_connect("localhost", "websitetest", "") or die("couldn't Connect to Database.");
$db = @mysql_select_db($db_name, $connection) or die("Couldn't select database.");
$sql = "INSERT INTO $name (Name,Password,CPassword,EMail,Website,Location,Occupation,ICQ,MSN,AIM,Yahoo) VALUES ("$Name",password($Password),"$CPassword","$EMail","$Website","$Location","$Occupation","$ICQ","$MSN","$AIM","$Yahoo")";
$result = @mysql_query($sql,$connection) or die("Couldn't execute query.");
?&gt;

&lt;HTML&gt;
&lt;HEAD&gt;
	&lt;TITLE&gt;Register&lt;/TITLE&gt;
&lt;/HEAD&gt;
&lt;BODY&gt;
	&lt;P&gt;&lt;STRONG&gt;Name:&lt;/STRONG&gt;
  &lt;? print "$Name"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;Password:&lt;/STRONG&gt;
  &lt;? print "$Password"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;E-Mail:&lt;/STRONG&gt;
  &lt;? print "$EMail"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;Website:&lt;/STRONG&gt;
  &lt;? print "$Website"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;Location:&lt;/STRONG&gt;
  &lt;? print "$Location"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;Occupation:&lt;/STRONG&gt;
  &lt;? print "$Occupation"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;ICQ:&lt;/STRONG&gt;
  &lt;? print "$ICQ"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;MSN Address:&lt;/STRONG&gt;
  &lt;? print "$MSN"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;AIM Address:&lt;/STRONG&gt;
  &lt;? print "$AIM"; ?&gt;&lt;/P&gt;
	&lt;P&gt;&lt;STRONG&gt;Yahoo Address:&lt;/STRONG&gt;
  &lt;? print "$Yahoo"; ?&gt;&lt;/P&gt;
&lt;/BODY&gt;
&lt;/HTML&gt;

Do you know what is causing this?

(And yes I am not using a database password)

Thanks,

Esoteric

March 10, 2004

On the line with $sql, remove the "s around the variables or put periods between the variables and the quotes, eg: $string = "The value of your variable is " . $variable . ", ok?";

March 10, 2004

Ok I tried that, however now it just recycles, and nothing gets added to database....

March 10, 2004

Sorry, my mistake, you need to put a \ infront of the quotes so they don't get treated as the end of the string.

$db = @mysql_select_db($db_name, $connection) or die("Couldn't select database.");

$sql = "INSERT INTO $name (Name,Password,CPassword,EMail,Website,Location,Occupation,ICQ,MSN,AIM,Yahoo) VALUES (\"$Name\",password($Password),\"$CPassword\",\"$EMail\",\"$Website\",\"$Location\",\"$Occupation\",\"$ICQ\",\"$MSN\",\"$AIM\",\"$Yahoo\")";

$result = @mysql_query($sql,$connection) or die("Couldn't execute query.");

March 10, 2004

haha, that is what I tried, but I showed it to a friend of mine (who has been working with php for about 6 months) and told me I didnt need them, now I get to go rub this in his face :p

Edit: Still Recycles :(

Edited March 10, 2004 by Esoteric

17,624 · March 10, 2004

One more time:

$sql = "INSERT INTO $name (Name,Password,CPassword,EMail,Website,Location,Occupation,ICQ,MSN,AIM,Yahoo) VALUES ('$Name',password('$Password'),'$CPassword','$EMail','$Website','$Location','$Occupation','$ICQ','$MSN','$AIM','$Yahoo')";
$result = @mysql_query($sql,$connection) or die("Couldn't execute query. Reason: " . mysql_error() );

I changed all those to single quotes, and added Reason: " . mysql_error() to the die for the query so you'd get better error results.

March 10, 2004

Still recycles...

2,511 · March 10, 2004

What do you mean by "Recycles"?

March 10, 2004

it just reloads the page

I am just learning php, so I don't know much at all.

oh and if you want you can see the script on my self hosted webserver...

http://esoteric.sytes.net/websitetest/

2,511 · March 10, 2004

OK, I get it now.

The problem would appear to be here:

if ((!$name) || (!$Password) || (!$CPassword) || (!$EMail)) {
 header( "Location: register.html");
 exit;
}

It's not finding some or all of those. First off, $name should be $Name to fit your form (it *probably* wouldn't matter) - even better would be to make all data names lowercase as this is generally the used method.

At a guess the problem is that your server isn't set up to automatically set passed data. Try this at the start:

$name = $_POST['Name'];
$password = $_POST['Password'];
$cpassword = $_POST['CPassword'];
// Continue for all 12 or so bits of data

if ((!$name) || (!$password) || (!$cPassword) || (!$email)) {
 header( "Location: register.html");
 exit;
}

// And so on...

March 11, 2004

still recycles :|

2,511 · March 11, 2004

Print out $name and such like, see what they contain and compare that to what they should contain.

8,600 · March 11, 2004

is this intentional?

&lt;html&gt;
&lt;INPUT TYPE="text" Name="ICQ" Size=30&gt; MAXLENGTH=10&lt;/p&gt;
&lt;INPUT TYPE="text" Name="ICQ" Size=30 MAXLENGTH="10"&gt; &lt;/p&gt; &lt;!-- was this what you intended? --!&gt;
&lt;/html&gt;

17,624 · March 11, 2004

How about:

if (($name == '') || ($Password == '') || ($CPassword == '') || ($EMail == '')) {

header( "Location: register.html");

exit;

}

March 11, 2004

is this intentional?

&lt;html&gt;
&lt;INPUT TYPE="text" Name="ICQ" Size=30&gt; MAXLENGTH=10&lt;/p&gt;
&lt;INPUT TYPE="text" Name="ICQ" Size=30 MAXLENGTH="10"&gt; &lt;/p&gt; &lt;!-- was this what you intended? --!&gt;
&lt;/html&gt;

No that wasn't intentional, however I did notice it about the time when I first posted this topic and so I changed the code, just forgot to update it on here...

And I will try that timdorr when i get home (I am at school currently)

March 12, 2004

How about:
if (($name == '') || ($Password == '') || ($CPassword == '') || ($EMail == '')) {

header( "Location: register.html");

exit;

}

Tried that, however instead of recycling, it gave me the parse error again...

Parse error: parse error, unexpected T_VARIABLE in C:\Program Files\Apache\Apache2\htdocs\websitetest\register.php on line 11

3,130 · March 21, 2004

I haven't been using PHP/MySQL long, but don't you have to set the POST data to variables first?

try putting the following before you test for empty variables:

$name = $_POST['name'];

$Password = $_POST['Password'];

$CPassword = $_POST['CPassword'];

$EMail = $_POST['EMail];

2,511 · March 21, 2004

Welcome to last week...

Have you fixed this problem?

March 22, 2004

No, I was away (spring break) and all of the posts about what to try and everything are confusing me, whether I should try them with the original (which I did) or all together...

3,130 · March 22, 2004

I didn't notice this at first:

$sql = "INSERT INTO $name ...

It looks like you want to have a different table for every username. Since the table you want to add to is based on user input, your INSERT query is inserting into a non-existant table. You'd have to create a new table every time.

It would be a lot easier to keep everything in one table (assuming you were trying to put them in separate tables). So just replace $name in that instance with whatever you would call the table, or better yet, put the name after "$table_name = ".

If I confused you, here's what I'm telling you to try:

&lt;?
$Name = $_POST['name'];
$Password = $_POST['Password'];
$CPassword = $_POST['CPassword'];
$EMail = $_POST['EMail];

if ((!$Name) || (!$Password) || (!$CPassword) || (!$EMail)) {
 header( "Location: register.html");
 exit;
}
$db_name = "websitetest";
$table_name = "your_table_name";

$connection = @mysql_connect("localhost", "websitetest", "") or die("couldn't Connect to Database.");
$db = @mysql_select_db($db_name, $connection) or die("Couldn't select database.");
$sql = "INSERT INTO $table_name (Name,Password,CPassword,EMail,Website,Location,Occupation,ICQ,MSN,AIM,Yahoo) VALUES ('$Name',password($Password),'$CPassword','$EMail','$Website','$Location','$Occupation','$ICQ','$MSN','$AIM','$Yahoo')";
$result = @mysql_query($sql) or die("Couldn't execute query.".mysql_error());

mysql_close($connection);
?&gt;

I didn't test it, but it should work. Just create a table that "websitetest" can access and replace "your_table_name" above with it. It should work with the rest of Register.php

Sign In

[Help] Parse Error

Question

Esoteric

Link to comment

Share on other sites

19 answers to this question

Recommended Posts

Banjo

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

Banjo

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

Tim Dorr Veteran

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

Quboid

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

Quboid

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

Quboid

Link to comment

Share on other sites

john smith 1924 Veteran

Link to comment

Share on other sites

Tim Dorr Veteran

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

dnast

Link to comment

Share on other sites

Quboid

Link to comment

Share on other sites

Esoteric

Link to comment

Share on other sites

dnast

Link to comment

Share on other sites

Recently Browsing 0 members