Keskustelu: Nettisivujen teko: HTML ja PHP ongelma

pake10 [29.08.2009 15:40:34]

Niin, eli nyt sain sivupohjan. Pohjana käytän tätä. Mutta kun yritin laittaa tuohon pohjaan taustakuvan, niin laitoin index.php tiedostoon näin:

Alkuperäinen:

<?php
if (@strpos($_GET['sivu'],"../") > -1){ die("Yritätkö haxoroida?");
include("asetukset.php");
include("header.php");
if(!isset($_GET['sivu'])){
include("html/Etusivu.php");
}else{
if(!@include("html/".$_GET['sivu'].".php")){
echo "Sivua ei löydy! Ota yhteyttä (".$admin.")";
}
}
include("footer.php");
?>

Muokattu:

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<title>Etusivu</title>
</head>
<body>
style type="text/css">
body {
background-image:url('http://n4.backgroundsarchive.net/pub/0/967mi4ib14e6d.jpg');
}
</style>
<?php
if (@strpos($_GET['sivu'],"../") > -1){ die("Yritätkö haxoroida?");
include("asetukset.php");
include("header.php");
if(!isset($_GET['sivu'])){
include("html/Etusivu.php");
}else{
if(!@include("html/".$_GET['sivu'].".php")){
echo "Sivua ei löydy! Ota yhteyttä (".$admin.")";
}
}
include("footer.php");
?>
</body>
</html>

Mutta tuolla muokatulla ei toimi, tulee vain näin:

Parse error: syntax error, unexpected $end in /home/jiminet/www/sivupohja/index.php on line 26

Olen kokeillut myös ulkoisella CSS-tiedostolla mutta tulee sama virhe. Mitenkäs korjataan?

(Tiedän, tuo sivupohja on vanha, 2006.)

Metabolix [29.08.2009 15:46:59]

Vaikka et selvästikään osaa yhtään koodata, voisit yrittää edes kopioida oikein. Heti ensimmäisen PHP-koodirivin lopusta puuttuu nimittäin sulkeva aaltosulku, }.

Sitä paitsi oikea käyttötapa olisi sijoittaa tuo sivun yläreunan html-koodi tiedostoon header.php ja lopun koodi taas tiedostoon footer.php.

Grez [29.08.2009 15:47:14]

No tuohan kertoo ihan selvästi, että tiedosto loppuu, vaikka jotain on vielä "auki".

Eli korjaa se auki oleva juttu niin ongelma poistuu. (Metabolix näköjään ehtikin kommentoimaan mitä siellä on auki)

Teuro [29.08.2009 15:50:00]

Ei liity css:n mitenkään, mutta laskepa noiden aaltosulkujen määrä. Lisäksi ei kannata muokata, ennen kuin on varma mitä on tekemässä.

Vastaus

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

Kirjautuminen