Keskustelu: Nettisivujen teko: Parse error: syntax error, unexpected T_VARIABLE on line 42!

henkilo [28.09.2009 11:26:46]

Hei!

Tarvitsisin vähän apua. Mitähän tällainen herja mahtaa tarkoittaa:
Parse error: syntax error, unexpected T_VARIABLE on line 42?
Muuttujassa on jotain vikaa? Häh? Ei pitäs olla, en ymmärrä...

Koodi on tällainen:

<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Osoitekirja</title>
</head>

<body>
<?php
require_once('config.php');
require_once('dbopen.php');
require_once('lib_osoitekirja.php');

$action  	= (isset($_REQUEST['action']))  ?
				$_REQUEST['action'] : '';
$id 	 	= (isset($_REQUEST['id']))  	?
				$_REQUEST['id'] : '';
$tunnus  	= (isset($_REQUEST['tunnus']))  ?
				$_REQUEST['tunnus'] : '';
$sukunimi  	= (isset($_REQUEST['sukunimi']))?
				$_REQUEST['sukunimi'] : '';
$etunimi  	= (isset($_REQUEST['etunimi'])) ?
				$_REQUEST['etunimi'] : '';
$osoite  	= (isset($_REQUEST['osoite']))  ?
				$_REQUEST['osoite'] : '';
$puhnro  	= (isset($_REQUEST['puhnro']))  ?
				$_REQUEST['puhnro'] : '';
$email  	= (isset($_REQUEST['email']))   ?
				$_REQUEST['email'] : '';

tulosta_sivun_alku();
navigointivalikko($sukunimi);
switch ($action) {
	case "Muokkaa":
		hae_muokattava_data($id);
		break;	             //virhe on muka tässä!!!!
	case "Lisaa":
		hae_muokattava_data($id);
		break;
	case "Tallenna":
		lisaa_uusi_tietue($tunnus, $sukunimi, $etunimi
				         $osoite, $puhnro, $email);
		break;
	case "Tallenna muutokset":
		paivita_tietue($tunnus, $sukunimi, $etunimi
						  $osoite, $puhnro, $email);
		break;
	case "Poista":
		poista_tietue($tunnus);
		break;
	default:
		listaa_henkilot($sukunimi);
	}
?>


</body>
</html>

Mod. lisäsi kooditagit

Teuro [28.09.2009 11:40:39]

Virhe löytynyt pienen etsinnän jälkeen riviltä 39, josta voisit tarkistaa onko siellä tarpeeksi "," merkkejä. Sama virhe on myös hieman edempänä.

Metabolix [28.09.2009 11:44:03]

"Parse error" tarkoittaa yleisesti ottaen, että koodista puuttuu sulkuja, pilkkuja, puolipisteitä tms. tai että niitä on jossain kohti liikaa.

Pitkän, toistuvan isset-viritelmän tilalle voisin suositella vaikka funktiota:

<?php
// Tehdään ensin pieni apufunktio...
function from_request($x) {
  if (!isset($_REQUEST[$x])) {
    return null;
  }
  return $_REQUEST[$x];
}

// Nyt olennaisesta koodista tulee selkeämpää:
$tunnus = from_request("tunnus");
$salasana = from_request("salasana");
$email = from_request("email");

henkilo [28.09.2009 11:46:05]

Kiitos! Niinpäs puuttuikin pilkkuja, nyt toimii!
Ja kiitos vaihtoehtoisesta tavasta ilmaista sama asia, ei olisi tullut tuollainen
mieleenikään.

Vastaus

Aihe on jo aika vanha, joten et voi enää vastata siihen.

Kirjautuminen