$selected_ids        = array(1, 2, '3', '#', 4, 4);

$sane_selected_ids   = array_filter($selected_ids, 'is_numeric'); // array(1, 2, '3', 4, 4)

$cast_selected_ids   = array_map('intval', $sane_selected_ids); // array(1, 2, 3, 4, 4)

$unique_selected_ids = array_unique($cast_selected_ids); // array(1, 2, 3, 4)

(10-08-2013, 01:34 PM)Sephi-Chan a écrit : Si on génère un doublon après avoir casté des caractères de toute façon interdits, il y a de toute façon un problème (les caractères interdits).

Je pense qu'il faut développer dans un style fail fast : arrêter le script le plus vite possible en cas de soucis et ne pas essayer de réparer les problèmes.

Si tu veux t'assurer de la bonne forme des données. Tu peux les vérifier et les convertir.

$selected_ids        = array(1, 2, '3', '#', 4, 4);

$sane_selected_ids   = array_filter($selected_ids, 'is_numeric'); // array(1, 2, '3', 4, 4)

$cast_selected_ids   = array_map('intval', $sane_selected_ids); // array(1, 2, 3, 4, 4)

$unique_selected_ids = array_unique($cast_selected_ids); // array(1, 2, 3, 4)

Bien sûr, ça fait de nombreux traitement. Ça ne devrait pas poser de problème sur de faibles volumes.
Et après il faut bien sûr vérifier que l'utilisateur possède bien chacune de ces divisions !


Quand je vois ce genre de choses, je suis bien content d'utiliser un framework ! C'est tellement ennuyeux, répétitif et sans aucune valeur ajoutée !

if(isset($_POST['attack']))

{

	$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(empty($hasDuplicates))

	{

		$count = $_POST['nombre_division_max'];

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

		while ($count > 0)

		{

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

			$count--;

		}

	}

	else

	echo 'Vous avez fait une erreur ! nom de gu, d\'nom de gu';

}

<option value="#">Aucune</option>

• C'est au serveur de savoir le nombre de divisions autorisées, pas au client ;
  
• Tu ne contrôles pas si le joueur a engagé plus de divisions que ce qui est autorisé ;
  
• Tu ne contrôles pas si les divisions engagées sont bien disponibles pour le joueur ;
  
• N'utilises pas empty($has_duplicates) mais directement $has_duplicates ;
  

(10-08-2013, 02:42 PM)Racktor a écrit : PS: j'ai un pb avec un # !
j'ai ajouté un champ a ma liste pour permettre au joueur de ne sélectionner aucune division !

<option value="#">Aucune</option>

sauf que le retraitement n'enlève pas le # ...

(10-08-2013, 03:05 PM)Sephi-Chan a écrit : Je ne comprends pas trop. A quoi sert le code que tu as rajouté ? Le but de ton script n'est-il pas de faire des traitements ?

(10-08-2013, 03:05 PM)Sephi-Chan a écrit :

• C'est au serveur de savoir le nombre de divisions autorisées, pas au client ;
  
• Tu ne contrôles pas si le joueur a engagé plus de divisions que ce qui est autorisé ;
  
• Tu ne contrôles pas si les divisions engagées sont bien disponibles pour le joueur ;

(10-08-2013, 03:05 PM)Sephi-Chan a écrit :

N'utilises pas empty($hasDuplicates) mais directement $hasDuplicates ;

Oki j'ai modifié

(10-08-2013, 02:42 PM)Racktor a écrit : PS: j'ai un pb avec un # !
j'ai ajouté un champ a ma liste pour permettre au joueur de ne sélectionner aucune division !

<option value="#">Aucune</option>

sauf que le retraitement n'enlève pas le # ...

Tu as dû mal faire quelque chose, mon script l'enlève avec le array_filter. La variable saine est dans $cast_selected_ids. Si tu utilises les données de $_POST, elles n'ont pas été assainies.
Par ailleurs, pourquoi mettre un # plutôt que rien du tout (<option value="">Aucune</option>)?

$selected_ids     = array(1, 2); // Le tableau bien filtré qu'on a créé tout à l'heure.

$allowed_ids      = array(1, 2, 3); // Ce qui te vient de la requête SQL mentionnée plus haut.

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

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

Citation :Le nombre importe moins que la complexité des requêtes.

(13-08-2013, 12:42 PM)Racktor a écrit : Par contre Sephi j'ai une petit question : si dans l'array sélectionnant les "bonnes" divisions le joueur ne modifie pas la quantité mais l'id d'une division ? je sais pas si je me fais bien comprendre ...

$selected_ids        = array(1, 2); // Le tableau bien filtré qu'on a créé tout à l'heure.

$allowed_ids         = array(1, 2, 3); // Ce qui te vient de la requête SQL mentionnée plus haut.

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

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