What is wrong with this PHP script? ASAP+REP!

[myke]

<a href="register.php">Register Here</a>
</td></tr></table></form></center>");
}
if ("$_POST['login'];") 
{
$username=$_POST["username"];
$password=md5($_POST["password"]); 

Error I'm getting:

Parse error: syntax error, unexpected T_STRING in /home/more/public_html/DB/login.php on line 28

Line 28 is the if function

[d17c]

I'm a bit rusty with php, but try this:

<a href="register.php">Register Here</a>
</td></tr></table></form></center>");
}
if (isset($_POST['login'])) {
$username = $_POST["username"];
$password = md5($_POST["password"]); 

I assume you were trying to check if it had been submitted.

[myke]

no, didn't work !

+rep for tring !

[myke]

sorry, can't edit, sorted.

[Protege]

sorry, can't edit, sorted.

Basically in PHP if your using a echo or placing a string (I think your echoing) you have a problem with (ref="register.php">), think about it, you've started it off with echo(" <<-- and your ending it with "); "register.php" Your basically ending it there, and then starting it again, so to fix this;

echo("<a href=\"register.php\">"); 

or

echo('<a href="register.php">'); 

or

echo("<a href=register.php>"); 

all will work, and let you use a href tag in a PHP code.

Hope that helped

I just re-read it and i saw a few more errors in the coding

if statements, I shall explain as you've really and the only word i found to explain this is f**ked it up big time.

if statements are very easy, and its the basic of coding on a whole, once you understand it, you can basically build anything.

$string = "Bob";
$string2 = 5;
## code 1
if($string == "Bob")
{
echo("True");
}
else
{
echo("False");
}

## code 2
if($string2 == 2)
{
echo("False");
}
else
{
echo("True");
} 

Code #1 would be result on the page "True" and Code #2 would result "False" In strings/vars you can place numbers without quotes, but if its text, you need to quote it, and ALWAYS finish off VARS/STRINGS with ; or your code will fail.

What you've done wrong is basically go a bit too far with the syntax

if ("$_POST['login'];") 

the correct version would be

if ($_POST['login']) 

I tend not to do coding like that, so Id probably do

if($_POST['login'] == "") 

^^ That should work, good luck!

[rh4u]

<a href=register.php>Register Here</a>
</td></tr></table></form></center>");
}
if ("$_POST['login'];") 
{
$username=$_POST["username"];
$password=md5($_POST["password"]);  
                 } 

try that =]

[Invent]

@^: Oh come on!

<a href="register.php">Register Here</a>
</td></tr></table></form></center>");
}

if( $_POST["login"]  /* isset() can be used */ ) 
{

    $username = $_POST["username"];
    $password = md5( $_POST["password"] ); 

[Hypertext]

roflmao at this failure
USE ' over " if you want " use escape it with \
<a href='register.php'>Register Here</a>
</td></tr></table></form></center>");
}

if($_POST["login"])
{

$username = $_POST["username"];
$password = md5( $_POST["password"] );

[Navicat]

<a href="register.php">Register Here</a>
</td>
</tr>
</table>
</form>
</center>
<?php
if ($_POST ['login']) {
    $username = $_POST ["username"];
    $password = md5 ( $_POST ["password"] );
    
// DO THE REST OF YOUR **** HERE //


}
?>

[Hypertext]

by the looks of the ); ending in, he's already in an echo/print so you'd have to keep in php