11-05-2009, 01:25 PM
Coucou,
Ca n'a rien avoir, mais les GET, ça n'est pas très recommandé. Ca peut attirer des bugs et des failles de sécurité.
Sinon, le message d'erreur est-il ?
"Ce batiment n'est pas à votre portée" ?
Dans ton code, tu fais les tests suivant en batterie.
//on vérifie la distance
//on vérifie si la boutique possède bien l'objet en stock
//on vérifie que le joueur peut payer.
Mais si le test est négatif, tu continues ta page... il faudrait rajouter la fonction exit(); dans ton code. Je modifierais ton code ainsi.
Ca n'a rien avoir, mais les GET, ça n'est pas très recommandé. Ca peut attirer des bugs et des failles de sécurité.
Sinon, le message d'erreur est-il ?
"Ce batiment n'est pas à votre portée" ?
Dans ton code, tu fais les tests suivant en batterie.
//on vérifie la distance
//on vérifie si la boutique possède bien l'objet en stock
//on vérifie que le joueur peut payer.
Mais si le test est négatif, tu continues ta page... il faudrait rajouter la fonction exit(); dans ton code. Je modifierais ton code ainsi.
Code :
//on vérifie la distance
if(!$Batiment->EtreAPortee($PJ->Pe_Caracs->Pe_PosX,$PJ->Pe_Caracs->Pe_PosY,$PJ->Pe_Caracs->Qu_Id))
{
header("location:jouer.php?Erreur=Ce batiment n'est pas à votre portée");
exit ();
}
//on vérifie si la boutique possède bien l'objet en stock
if(!$ObjetInfos = $Batiment->EtreEnStock($ObjetId,$Quantite,$BoutiqueId))
{
header("location:actions_sur_batiment.php?a=entrer&Alerte=Vérifiez la disponibilité de ce produit.");
exit ();
}
$Prix = $ObjetInfos->BAS_PrixAchat * $Quantite;
//on vérifie que le joueur peut payer.
if($PJ->Pe_Caracs->Pe_Thor < $Prix)
{
header("location:actions_sur_batiment.php?a=entrer&boutique_id={$BoutiqueId}&Alerte=Vous n'avez pas assez de Thor pour payer.");
exit ();
}