PHPhulp.nl
35k+ leden Over PHPhulp Offerte opdracht Contact
S

Website OnlineSoldier

Door Steef op 04-02-2008 18:23
472 views
Hallo,

Ik maak samen met 2 andere mensen een website voor een spel wij maken in Game-Maker. Ik zou graag willen vragen wat jullie van de website vinden en wat er eventueel verbeterd/veranderd moet worden.

Op de website komt er nog een login, deze staat samen met het Game-Maker bestand, zodat je moet inloggen om te spelen, er komt een ranglijst (hoeveel XP, HP, favogeweer, gebruikersnaam, online/offline, rank) en nog een aantal dingen.

Ik zou dus graag jullie oordeel horen.

http://onlinesoldier.gethost.nl/

Groet Steef (en Joris en Krister)
-
- -
04-02-2008 18:25
Leuke link! ;-) (klik maar eens)
S
Steef
04-02-2008 18:29
Oke dat hadden we zelf nog niet gevonden, hoe kan je dat precies verhelpen?

We hebben nu dit
if(isset($_GET["p"])){
if(file_exists($_GET["p"].".php")){
$page = $_GET["p"];
}else{
$page = "404";
}
include("pagetitles.php");
} else{
$page = "home";
}
?>
-
- -
04-02-2008 18:32
else if($_GET['p'] == "index")
{
$page = "home";
}

zoiets?
Joren de Wit
04-02-2008 18:34
Niet echt een veilige oplossing. Het is nu mogelijk om elke willekeurige pagina op je server te includen. En zoals je ziet kan dat bijvoorbeeld ook tot die oneindige loop leiden...

Kijk eens naar dit voorbeeld van een veilig script om pagina's te includen.
S
Steef
04-02-2008 19:18
Als het goed is moet het nu opgelost zijn.
-
- -
04-02-2008 20:01
Jup, opgelost!
m
michael
08-02-2008 14:50
Ik vind het een goede site voor de doelgroep! Ik heb eigenlijk niks op te merken. De kleuren zijn goed en de style is goed. Lettertype is goed en de navigatie is ook goed naar mijn mening.

S
Steef
16-02-2008 18:23
Heeft iemand anders nog een opmerking / verbetering zowel aan het uiterlijk als codering?
b
bart van der veen
16-02-2008 19:00
heb niet het idee dat het eigen bild is en zo ja dan is het slordig gedaan...

Reageren

Inloggen om te reageren