Parse error för { i php-script

11 svar · 447 visningar · startad av m_soderlund 13 mars 2006

m_soderlundMedlem sedan sep. 20026 425 inlägg

Frågan13 mars 2006 12:37#1

Får detta felmess:

Parse error: parse error, unexpected $ in /cust/ec/ec/httpd.www/articles_php/admin/login.php on line 31

Jag har fetmarkerat rad 31 och det är ju inget fel på den raden. Jag tycker att mina villkorssatser ser korrekta ut. Är det nåt med att jag skickar header-data när jag egentligen inte får det?

<?
	
if ($_POST['usr'] == "admin" && $_POST['psw'] == "test")
{	
	session_start();
	session_register("min_session");

	header("Location: index.php");
	
} else {

	if (!empty($_POST['usr']) || !empty($_POST['psw']))
	{
		$login_info = "Fel inloggningsuppgifter.";
	}
	
}

?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">

<head>
<title>Artiklar login</title>

<link rel="stylesheet" href="../../Ec.css" type="text/css" />
</head>
<body>

	<form id="loginForm" name="loginForm" action="login.php" method="post">
		<fieldset>
			[b]<legend>Logga in</legend>[/b]
		
			<p><label for="usr">Användarnamn</label>
			<br /><input type="text" id="usr" name="usr" tabindex="1" /></p>
		
			<p><label for="psw">Lösenord</label>
			<br /><input type="password" id="psw" name="psw" tabindex="2" /></p>
		</fieldset>
	
		<input type="submit" id="btnSubmit" name="btnSubmit" tabindex="3" value="Logga in" />
	</form>
			
	<?
	echo($login_info); ?>
</body>
</html>

tpf

CompusaMedlem sedan jan. 20023 327 inlägg

13 mars 2006 12:48#2

Börja med att byta ut dina taggar från

<? ?>

till

<?php ?>

Inte säkert att det hjälper men en början...

Den kortare varianten <? fungerar endast om man i php.ini satt variabeln short_open_tag till On. Även om detta är standardinställningen är det relativt vanligt att man stänger av det. Den korta varianten kan dessutom ställa till med problem då man i sin kod har en XML-deklaration överst i dokumentet. XML-deklarationen börjar nämligen med <?xml och om short_open_tag då är påslaget kommer PHP avsluta skriptet med ett parse error. Alltså, använd alltid <?php.

m_soderlundMedlem sedan sep. 20026 425 inlägg

13 mars 2006 12:48#3

Jag löste det genom att lägga till ob_start(); högst upp på sidan.

MeleaMedlem sedan juli 20033 147 inlägg

13 mars 2006 12:50#4

Använd session() istället för session_register. Vet inte om det gör någon skillnad men de rekommenderar session() på php.net.

php.net skrev:

Use of $_SESSION (or $HTTP_SESSION_VARS with PHP 4.0.6 or less) is recommended for improved security and code readability. With $_SESSION, there is no need to use the session_register(), session_unregister(), session_is_registered() functions. Session variables are accessible like any other variables.

Annars ser jag inget direkt fel, du kan testa att använda enkelfnuttar/dubbelfnuttar rakt igenom i din if-sats, kan vara någon inställning i php.ini som spökar.

if ($_POST['usr'] == 'admin' && $_POST['psw'] == 'test')

MeleaMedlem sedan juli 20033 147 inlägg

13 mars 2006 12:51#5

Står det någonting innan php-koden?
Har du kollat så att det inte står ett mellanslag innan (<?)?

m_soderlundMedlem sedan sep. 20026 425 inlägg

13 mars 2006 12:58#6

Det står inget innan php-koden. Tack för tipset om sessioner!

ForsbergMedlem sedan juni 200454 inlägg

15 mars 2006 12:09#7

Hej, du gör fel i dina villkorsatser.

Såhär ska det se ut så kommer det nog fungera fint sen!

if (($_POST['usr'] == "admin") && ($_POST['psw'] == "test")) {
        session_start();
        session_register("min_session");

        header("Location: index.php");
	
} else {
        if ((!empty($_POST['usr'])) || (!empty($_POST['psw']))) {
                $login_info = "Fel inloggningsuppgifter.";
        }
}

MeleaMedlem sedan juli 20033 147 inlägg

15 mars 2006 12:13#8

Forsberg: Nej, han skriver alldeles korrekt, och det gör du också, fast med onödigt många paranteser.

ForsbergMedlem sedan juni 200454 inlägg

15 mars 2006 12:22#9

Melea: Är det fler villkor i en sats skall alltid villkoren omslutas i separata paranteser. Precis som i alla andra språk som använder villkorstyrning, dvs i stort sätt alla.

Och förövrigt så behöver du inte använda paranteser när du använder echo.
echo är ingen funktion och ska därför inte användas på det sättet.

Saxat direkt från manualen:

// Because echo is not a function, following code is invalid.
($some_var) ? echo('true'): echo('false');

// However, the following examples will work:
($some_var) ? print('true'): print('false'); // print is a function
echo $some_var ? 'true': 'false'; // changing the statement around

MeleaMedlem sedan juli 20033 147 inlägg

15 mars 2006 12:37#10

Det är ju helt beroende på vilka villkor som ska matcha varandra.

if ($value1 == $value2 && $value3 == $value4 || $value5 == $value6)

Ger inte samma resultat som:

if ($value1 == $value2 && ($value3 == $value4 || $value5 == $value6))

ForsbergMedlem sedan juni 200454 inlägg

15 mars 2006 12:41#11

Nej, just för att du utför det på fel sätt.

Detta ger exakt samma resultat som ditt alternativ, men är ett mycket bättre sätt att skriva det på:

if ( ($value1 == $value2) && ($value3 == $value4) || ($value5 == $value6) )

Det blir mycket lättare att avläsa vad som händer om man skriver på detta sättet.

MeleaMedlem sedan juli 20033 147 inlägg

15 mars 2006 12:43#12

Det ger inte alls samma resultat som min andra villkorssats.
Och varför är det ett bättre sätt att skriva på?