start.php fout

Door timothy van wilpen op 19-02-2008 21:36

920 views

Heb een script om van login naar stats te gaan genaamd start.php

maar als ik hem upload en probeer te gebruken komt er een fout te staan:
Sesion has expired

<?php include("header.php"); ?>

MAIN 

<?php
Print " Welcome back, $stats[user]!";
?>
 New Players: <a href='help.php'>Learn the Basics!</a> Location of your Knock Out Link:
<?php
Print " <input type=text size=47 value=http://ghetto.woelmuis.nl/page.php?x=$stats[id]>";
?>

 <div class="tiny">Knock out your friends for money!</div> 
<?php
if ($_stat[trap] == 'on') {
print "<table class='darkbox'><tr><td align='center'>Your knock out link is currently set. You can only knock out your friends when your link is set.</a> <a href='start.php?action=trapoff'>Click here to disarm your link.</a></td></tr></table>";
}
if ($_stat[trap] == 'off') {
print "<table class='darkbox'><tr><td align='center'>Your knock out link is currently not set. You can only knock out your friends when your link is set.</a> <a href='start.php?action=trapon'>Click here to set your link.</a></td></tr></table>";
}
?>

<?php
if ($_action == trapon) {
print "Now you can knock out your friends! ";
mysql_query("update players set trap='on' where id=$stat[id]");
}
if ($_action == trapoff) {
print "You turned off your link ";
mysql_query("update players set trap='off' where id=$stat[id]");
}
?>

 <a href=help.php>Learn How To Play</a>

<?php include("footer.php"); ?>

J A

19-02-2008 21:59

Zet eerst maar even code tags, dit is namelijk best wel irritant

TJVB tvb

19-02-2008 22:24

Een paar dingen:
Wat staat er in header.php? en in footer.php? Ik zie hier namelijk niks met sessies staan.

En waar komen je variabelen (zoals $_action $_stap ) vandaan?

Verder is de manier waarop je met query's werkt niet aan te raden.
<?php
if ($_action == trapoff) {
print "You turned off your link ";
mysql_query("update players set trap='off' where id=$stat[id]");
}?>
Wat gebeurt er als de query fout gaat?
Waar wordt gecontroleerd wat $stat[id] is? (ik gok trouwens op $stat['id'])
En haal je variabelen buiten uit je string.
<?php
if ($_action == trapoff && is_numeric($stat['id') ) { //we controleren meteen of id wel numeric is. Ik gok erop dat het bij jou namelijk een getal is
$r_result = mysql_query("update players set trap='off' where id=".$stat['id']);
if( $r_result ){
print "You turned off your link ";
}else{
print 'Helaas er is wat fout gegaan. ';
}
}
?>
Dit is opzich wat basis (het kan nog uitgebreider, er wordt bijvoorbeeld niet gecontroleerd hoeveel regels er gewijzigd zijn. En een limit in je query kan ook handig zijn)

Reageren

Inloggen om te reageren