Exécuter une requête une seule fois dans une boucle

[php_addict]

@php_addict: J'ai pas compris, quelle est la différence avec mon formulaire actuel ?

bah...la différence c'est ton champs name="" dans ton input, exemple:

au lieu de:

Code :

<input type="text" name="unite_1" >
<input type="text" name="unite_2" >
<input type="text" name="unite_3" >
<input type="text" name="unite_4" >

il vaut mieux

Code :

<input type="text" name="unite[1]" >
<input type="text" name="unite[2]" >
<input type="text" name="unite[3]" >
<input type="text" name="unite[4]" >

comme ca au lieu de te retrouver avec

$_POST[unite_1]
$_POST[unite_2]
$_POST[unite_3]
$_POST[unite_4]

tu te retrouves avec un array:

$_POST[unite] = array()

ceci dit je ne suis pas certain que cela ne soit pas dejà le cas dans ton code, mais niveau lisibilité c'est chaud ton code, c'est une remarque constructive, car si dans 2 mois tu retouche ton code tu risque de ne même plus le comprendre...

---

petits conseils:

$req_unites = mysql_query("SELECT * FROM unitesconstruites WHERE id_village = '" . $villageattaquant . "'"); 

ne SELECT pas tout mais juste ce dont tu as besoin et même si tu SELECT tout mieux vaut dire quoi comme ca tu n'a pas besoin de jeter un oeil à phpmyadmin pour savoir se que tu retourne...

dans

while ($unites = mysql_fetch_row($req_unites)) { ?>

<input type="text" name="<?php echo $unites[3]; ?>" /> (Max: <?php echo $unites[5]; ?>)<br />

ne fait pas $unites[3] mais plutot un truc dans le genre $unites['unit_id'] c'est 1000 fois plus clair...car ton index 3 de $unites personne à part toi peut savoir de quoi il s'agit

par convention souvent on met plutôt $i au lieu de $nbr_boucle