par
Henri » 09 août 2006, 18:29
1) Tu as bien tiré ton nombre au hasard avec l'instruction rand ? Que veux-tu de plus ?
2) regarde la syntaxe switch case qui sera beaucoup plus lisible que les suites de if elseif ...
switch ($sort) {
case 1: ...
break;
case 2: ...
break;
...
default: ...
break;
}
3) Factorise dans ton code tout ce qui peut l'être : le nom du sort, le nombre de PV perdus.
switch ($sort) {
case 1: $nomsort = "Incendio";
$deltaPV = 5;
$action = true;
break;
case 2: $nomsort = "Stupéfix";
$deltaPV = 10;
$action = true;
break;
...
default: echo "Heu... petit problème ... ";
$action = false;
break;
}
if ($action) {
echo "$nomsort !"
$sql = 'UPDATE siteweb SET `points_vie` = "'.$points_vie." - $deltaPV";
$req = mysql_query($sql) or die('Erreur SQL !<br>'.$sql.'<br>'.mysql_error());
echo "Vous avez perdu $deltaPV points de vie. <a href=\"choisir_adversaire.php\"> Jouer </a>";
}
4) Laisse tomber les if et les switch case et met tous tes noms de sorts et pertes de point de vie dans un tableau : ça ira plus vite.
5) Laisse tomber le tableau codé en dur dans le code et met les sorts dans une table : c'est ce qu'il y a de plus propre et de plus évolutif.
1) Tu as bien tiré ton nombre au hasard avec l'instruction rand ? Que veux-tu de plus ?
2) regarde la syntaxe switch case qui sera beaucoup plus lisible que les suites de if elseif ...
[php]
switch ($sort) {
case 1: ...
break;
case 2: ...
break;
...
default: ...
break;
}
[/php]
3) Factorise dans ton code tout ce qui peut l'être : le nom du sort, le nombre de PV perdus.
[php]
switch ($sort) {
case 1: $nomsort = "Incendio";
$deltaPV = 5;
$action = true;
break;
case 2: $nomsort = "Stupéfix";
$deltaPV = 10;
$action = true;
break;
...
default: echo "Heu... petit problème ... ";
$action = false;
break;
}
if ($action) {
echo "$nomsort !"
$sql = 'UPDATE siteweb SET `points_vie` = "'.$points_vie." - $deltaPV";
$req = mysql_query($sql) or die('Erreur SQL !<br>'.$sql.'<br>'.mysql_error());
echo "Vous avez perdu $deltaPV points de vie. <a href=\"choisir_adversaire.php\"> Jouer </a>";
}
[/php]
4) Laisse tomber les if et les switch case et met tous tes noms de sorts et pertes de point de vie dans un tableau : ça ira plus vite.
5) Laisse tomber le tableau codé en dur dans le code et met les sorts dans une table : c'est ce qu'il y a de plus propre et de plus évolutif.