[RESOLVED] I don't know what's going wrong

gr-games

I have a code, but I don't know what's wrong with it. When I execute the script, it says: "Parse error: unexpected $ in line 105". That's the line with "</html>"...

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<?php
if($num==1||$num==2||$num==3||$num==4){
switch($num){
case 1 : {
$type="TI-83";
$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83/\" target=\"_blank\">Ticalc.org</a>.";
$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>");
$soort=array(0 => "Basic","Basic");
$grootte=array(0 => "between 7.119 and 10.167","Between 2.944 and 4.053");
$levels=array(0 => "0","0");
}
break;
case 2 : {
$type="TI-83+";
$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83plus/\" target=\"_blank\">Ticalc.org</a>.";
$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>","<a href=\"spelletjes/lemmings.htm\" target=\"_blank\">Lemmings</a>");
$soort=array(0 => "Basic","Basic","MirageOS");
$grootte=array(0 => "between 7.119 and 10.167","Between 2.944 and 4.053","12.349 plus levels");
$levels=array(0 => "0","0","17");
}
break;
case 3 : {
$type="TI-84+";
$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83plus/\" target=\"_blank\">Ticalc.org</a>.";
$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>","<a href=\"spelletjes/lemmings.htm\" target=\"_blank\">Lemmings</a>");
$soort=array(0 => "Basic","Basic","MirageOS");
$grootte=array(0 => "Tussen 7.119 en 10.167","Tussen 2.944 en 4.053","12.349 plus levels");
$levels=array(0 => "0","0","17");
}
break;
case 4 : {
$type="Casio 9850(+)";
}
break;
}
$titel = "GR-Games NL - ".$type;
$bericht="<table width=\"100%\">\n<tr>\n<td><i>Naam:</i></td>\n<td><i>Soort:</i></td>\n<td><i>Grootte:</i></td>\n<td><i>Extra levels</i></td>\n</tr>\n</table>";
$bericht .= $ticalc;
if($num==4)
{
$bericht = "Helaas, er zijn nog geen spelletjes beschikbaar voor de ".$type."<br>\nIk werk er wel aan.";
}
?>
<link href="style.css" rel="stylesheet" type="text/css">
<title><?php echo($titel) ?></title>
</head>
<body class="main">
<table width="100%" border="0" align="left">
<tr>
  <td width="192" height="130"><img src="images/mario.jpg" width="192" height="128"></td>
  <td align="left" valign="middle" class="kop"> <p>GR-Games:<br>
    Spelletjes op je GR</p>
    </td>
</tr>
<tr>
  <td height="10"></td>
  <td height="10"></td>
</tr>
<tr>
  <td height="209" valign="top"><p><a href="index.htm">&gt; Home</a><br>
      <a href="shells.htm" target="_self">&gt; Shells</a><br>
        <?php
		if($type=="TI-83")
		{
		echo("<em><a href=\"spel.php?num=1\" target=\"_self\">&gt; TI-83</a><br></em>\n");
		echo("<a href=\"spel.php?num=2\" target=\"_self\">&gt; TI-83+<br></a>\n");
		echo("<a href=\"spel.php?num=3\" target=\"_self\">&gt; TI-84+<br></a>\n");
		echo("<a href=\"spel.php?num=4\" target=\"_self\">&gt; Casio 9850(+)<br></a>\n");
		}
		if($type=="TI-83+")
		{
		echo("<a href=\"spel.php?num=1\" target=\"_self\">&gt; TI-83</a><br>\n");
		echo("<em><a href=\"spel.php?num=2\" target=\"_self\">&gt; TI-83+<br></a></em>\n");
		echo("<a href=\"spel.php?num=3\" target=\"_self\">&gt; TI-84+<br></a>\n");
		echo("<a href=\"spel.php?num=4\" target=\"_self\">&gt; Casio 9850(+)<br></a>\n");
		}
		if($type=="TI-84+")
		{
		echo("<a href=\"spel.php?num=1\" target=\"_self\">&gt; TI-83</a><br>\n");
		echo("<a href=\"spel.php?num=2\" target=\"_self\">&gt; TI-83+<br></a>\n");
		echo("<em><a href=\"spel.php?num=3\" target=\"_self\">&gt; TI-84+<br></a></em>\n");
		echo("<a href=\"spel.php?num=4\" target=\"_self\">&gt; Casio 9850(+)<br></a>\n");
		}
		if($type=="Casio 9850(+)")
		{
		echo("<a href=\"spel.php?num=1\" target=\"_self\">&gt; TI-83</a><br>\n");
		echo("<a href=\"spel.php?num=2\" target=\"_self\">&gt; TI-83+<br></a>\n");
		echo("<a href=\"spel.php?num=3\" target=\"_self\">&gt; TI-84+<br></a>\n");
		echo("<em><a href=\"spel.php?num=4\" target=\"_self\">&gt; Casio 9850(+)<br></a></em>\n");
		}
		?>
        <a href="voegtoe.htm">&gt; Voeg een spel toe</a><br>
        <a href="contact.htm">&gt; Contact</a><br>
        <a href="zoek.htm">&gt; Zoek op deze website</a></p>
    </td>
<td valign="top"><p><strong> &gt; <?php echo($type) ?></strong></p>
  <p><?php echo($bericht) ?></p></td>
</tr>
</table>
</body>
</html>

dmacman1962

You missed the semi colon ; in this line.

<?php echo($bericht) ?>

change it to

<?php echo($bericht); ?>

I did not review all your code, but spotted that right off.

Try it.

Good luck,

Don

bpat1434

This is where indenting your code helps....

You are missing a closing "}" with your main if() statement.

if($num==1||$num==2||$num==3||$num==4){
switch($num){
case 1 : {
$type="TI-83";
$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83/\" target=\"_blank\">Ticalc.org</a>.";
$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>");
$soort=array(0 => "Basic","Basic");
$grootte=array(0 => "between 7.119 and 10.167","Between 2.944 and 4.053");
$levels=array(0 => "0","0");
}
break;
case 2 : {
$type="TI-83+";
$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83plus/\" target=\"_blank\">Ticalc.org</a>.";
$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>","<a href=\"spelletjes/lemmings.htm\" target=\"_blank\">Lemmings</a>");
$soort=array(0 => "Basic","Basic","MirageOS");
$grootte=array(0 => "between 7.119 and 10.167","Between 2.944 and 4.053","12.349 plus levels");
$levels=array(0 => "0","0","17");
}
break;
case 3 : {
$type="TI-84+";
$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83plus/\" target=\"_blank\">Ticalc.org</a>.";
$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>","<a href=\"spelletjes/lemmings.htm\" target=\"_blank\">Lemmings</a>");
$soort=array(0 => "Basic","Basic","MirageOS");
$grootte=array(0 => "Tussen 7.119 en 10.167","Tussen 2.944 en 4.053","12.349 plus levels");
$levels=array(0 => "0","0","17");
}
break;
case 4 : {
$type="Casio 9850(+)";
}
break;
}

SHould be:

if($num==1||$num==2||$num==3||$num==4){
	switch($num){
		case 1 : {
			$type="TI-83";
			$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83/\" target=\"_blank\">Ticalc.org</a>.";
			$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>");
			$soort=array(0 => "Basic","Basic");
			$grootte=array(0 => "between 7.119 and 10.167","Between 2.944 and 4.053");
			$levels=array(0 => "0","0");
		}
		break;

	case 2 : {
		$type="TI-83+";
		$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83plus/\" target=\"_blank\">Ticalc.org</a>.";
		$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>","<a href=\"spelletjes/lemmings.htm\" target=\"_blank\">Lemmings</a>");
		$soort=array(0 => "Basic","Basic","MirageOS");
		$grootte=array(0 => "between 7.119 and 10.167","Between 2.944 and 4.053","12.349 plus levels");
		$levels=array(0 => "0","0","17");
	}
	break;

	case 3 : {
		$type="TI-84+";
		$ticalc="\n<br>Voor andere spelletjes, zie <a href=\"http://www.ticalc.org/pub/83plus/\" target=\"_blank\">Ticalc.org</a>.";
		$naam=array(0 => "<a href=\"spelletjes/yahtzee.htm\" target=\"_blank\">Yahtzee</a>","<a href=\"spelletjes/poker.htm\" target=\"_blank\">Poker</a>","<a href=\"spelletjes/lemmings.htm\" target=\"_blank\">Lemmings</a>");
		$soort=array(0 => "Basic","Basic","MirageOS");
		$grootte=array(0 => "Tussen 7.119 en 10.167","Tussen 2.944 en 4.053","12.349 plus levels");
		$levels=array(0 => "0","0","17");
	}
	break;

	case 4 : {
		$type="Casio 9850(+)";
	}
	break;
}
}

Indent your code, it helps a lot.

And although you didn't include the closing ";" on the one liners, it helps to just be sure. In your case, it doesn't seem to make a difference (i.e. it works with or without them) but for future reference, always end your lines with a semicolon (😉.

~Brett

gr-games

Thanx