if(isset($_POST['attack']) && !empty($_POST['attack']['lines']))

{

	$selected_ids  = array_values($_POST['attack']['lines']);

	$sane_selected_ids   = array_filter($selected_ids, 'is_numeric'); 

	$cast_selected_ids   = array_map('intval', $sane_selected_ids);	

	$hasDuplicates = count($cast_selected_ids) != count(array_unique($cast_selected_ids));

		

	if($hasDuplicates == 0)

	{

		$allowed_ids = array();

		$requete_liste_id_division = "SELECT id FROM ". PREFIX ."militaire_division WHERE pseudo_proprio = '". $pseudo ."' AND statut = 'Prête au combat'";	

		$sql_liste_id_division = mysql_query($requete_liste_id_division) or die (GENERAL_ERREUR_CONNEXION_BDD);	//Execution de la requete

		$resultat_liste_id_division = mysql_fetch_array($sql_liste_id_division);	//Triage de la requète

		

		while($resultat_liste_id_division = mysql_fetch_assoc($sql_liste_id_division))

		{

			$allowed_ids[] = $resultat_liste_id_division['id'];

		}

		$union = array_unique(array_merge($cast_selected_ids, $allowed_ids)); // L'union des deux ensembles, sans doublons.

		$allIdsAreAllowed = count($allowed_ids) == count($union);

		

		if($allIdsAreAllowed == 0)

		{

			echo $_POST['attack']['target'].'<br />';

			$new_string='';

			foreach ($cast_selected_ids as $val) 

			{

				$new_string .= $val.'-';

			}

		echo substr($new_string,0,-1);

		}

		else 

		{

			redirection(2);

		}

	}

	else

	{

		echo '<b>Vous avez fait une erreur ! nom de gu, d\'nom de gu !</b>';

	}

}

• Tu devrais utiliser une seule langue (l'anglais) pour coder (les nom des tables, de colonnes, de variables, idéalement les commentaires, etc.) ;
  
• Tu devrais utiliser une seule syntaxe pour tes variable ($camelCase ou $snake_case) et n'utiliser qu'elle (là on voit clairement que tu as fait du copier-coller) ;
  
• Tu devrais aligner tes = ou ne pas le faire, mais pas mélanger les deux styles (là aussi on voit le copier-coller) ;
  
• Tu ne devrais pas utiliser d'abréviation (comme proprio) mais toujours les mêmes mots, cohérents ;
  
• Tu ne devrais pas utiliser de pseudo comme clé (ici "pseudo_proprio") : seulement des ID (par exemple "owner_id");
  
• Tu ne devrais pas utiliser une chaîne complexe pour stocker tes états (ici "Prête au combat") mais une simple chaîne (comme "ready" ou bien un nombre) stockée dans une constante PHP, puis toujours utiliser cette constante (un peu comme tu le fais pour PREFIX) ;
  
• Tu devrais utiliser sprintf pour générer tes requêtes SQL et ainsi éviter des concaténation au milieu de la requête ;
  
• Tu ne devrais pas mettre de commentaires triviaux comme "Execution de la requete" ou "Triage de la requète", qui est mal nommé (ce n'est pas un "triage") en plus de contenir une faute de français (requête, pas requète) ;
  
• Tu devrais utiliser seulement mysql_fetch_assoc ;
  
• Tu devrais utiliser un format de sérialisation (json_encode et json_decode) plutôt que de faire la tienne à base de tirets ;
  
• echo $_POST['attack']['target'].''; quel est l'intérêt de concaténer une chaîne vide ?
  
• Tu devrais utiliser les guillemets doubles quand tu écris une chaîne qui nécessite de l'échappement (ou fermer les balises PHP quand tu génères du HTML) ;
  

Citation :Bonjour à tous !

Bon j'ai fini mon script de combat, il est donc fonctionnel maintenant je fais l'interface pour le joueur.

Bhon alors le formulaire en lui-même pas franchement de souci sauf pour la récupération des données : imaginez que mon formulaire est aussi dynamique !

le joueur doit sélection les divisions qu'il souhaite engager dans le combat, le nombre de division maximum est fonction d'un niveau que le chef d'état major a. Donc on répète la même liste de division autant de fois que le niveau le permet logique... mais comment récupérer le résultat des listes ainsi crées ? on prend le niveau est on fait le chemin en sans inverse logiquement ?
une boucle qui récupère les variables POST...

Ensuite je crée une chaine de caractère du système de ce type la : 1-5-6-9
En gros les chiffres correspondent aux id des divisions utilisées, bhon après pour le traitement j’utilise la fonction qui permet de couper le chaine en fonction d'un caractère (ici -). C'est peut être pas trés propre mais bon ca marche 26

Le problème c'est que cette chaine de caractère doit être vérifier : bha oui le joueur peut mettre plusieurs fois la même division ! alors comment détecter dans ce cas les doublons ?

Merci par avance pour le temps que vous consacrerez à mes questions.

<?php 

$requete_liste_id_division = sprintf("SELECT `id` FROM `%smilitaire_division` WHERE `id_membre` = %d AND `statut` = %s", PREFIX, $id, "Prête au combat"); 

<?php 

$requete_liste_id_division = sprintf(

            "SELECT `id` FROM `%s` WHERE `%s` = %d AND `%s` = %d", 

                \constante\MySQL\divisionMilitaire\table, 

                \constante\MySQL\divisionMilitaire\idMembre, 

                $id, 

                \constante\MySQL\divisionMilitaire\statut, 

                \constante\divisionMilitaire\statut\pretAuCombat

        ); 

$requete_liste_id_division = sprintf(

  "SELECT id

  FROM    %smilitaire_division

  WHERE   id_membre = '%s' AND statut = '%s';",

  PREFIX,

  (int) $id,

  READY

);