Discussion :

[moimp]

Bonjour,

J'ai une fonction de requête avec PDO. Les critères de requêtes proviennent d'un formulaire. Je n'ai pas voulu alourdir en mettant le code du formulaire mais si vous en avez besoin, je vous le donnerai.

J'ai deux soucis:

• La valeur 0 lorsqu'elle est toute seule n'est pas prise en compte dans le code postal ce qui est gênant pour les pays où les codes postaux sont affectés géographiquement.
• Les critères de sélection de plage (min et max) ne fonctionnent pas. Si l'un des deux critères est renseigné le résultat de la requête est vide.

Voici la fonction de traitement du formulaire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
function thisFilter() {
	$_POST = array_map('strip_tags', $_POST);
	$criteria = empty($_POST) ? []: $_POST;
 
	// Elimination des champs qui ne sont pas des critères de sélection
	foreach ($criteria as $key=>$value) {
		if (in_array($key, ['filter', 'home'])) unset($criteria[$key]);
	}
	unset($key, $value);
 
	// Suppression des critères vides
	foreach ($criteria as $key=>&$value) {
		if ( empty($criteria[$key]) ) { unset($criteria[$key]); }
	}
 
	return getAddresses($criteria);
}

Et la composition de la requête:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
function getAddresses($criteria = []) {
	global $db, $language;
	$countryCol	= $language.'_name';
	$select = "
		SELECT a.id, nom, prenom, $countryCol AS pays, cp, ville,
			courriel, tel_domicile, tel_portable
		FROM dat_addresses a
			LEFT JOIN lst_countries c
				ON a.pays=c.id
		";
 
	// Composition de la clause WHERE et du tableau des paramètres
	print_r($criteria); echo '<br/>';
	$addClauses[0]		= "id_user=:id_user";
	$params['id_user']	= $_SESSION['user']['id_user'];
	if ($criteria) {
		if ( !empty($criteria['min']) and empty($criteria['max']) ) {
			$criteria['max'] = '%';
		}
		if ( !empty($criteria['max']) and empty($criteria['min']) ) {
			$criteria['min'] = '_';
		}
		if ( isset($criteria['min'], $criteria['max']) ) {
			$addClauses[] = "nom BETWEEN :min AND :max";
			$params['min'] = $criteria['min'];
			$params['max'] = $criteria['max'];
		}
		if ( !empty($criteria['pays']) ) {
			$addClauses[] = "pays=:pays";
			$params['pays']= $criteria['pays'];
		}
		if ( !empty($criteria['cp']) ) {
			$addClauses[] = "cp LIKE :cp";
			$params['cp']= $criteria['cp'].'%';
		}
		if ( !empty($criteria['ville']) ) {
			$addClauses[] = "ville LIKE :ville";
			$params['ville']= $criteria['ville'].'%';
		}
		print_r($addClauses); echo '<br/>';
	}
	$strWhere = "WHERE " . implode(' AND ', $addClauses);
	// Requête globale
	$query = $select . $strWhere . " ORDER BY nom, prenom";
	echo $query, '<br/>';
	print_r($params); echo '<br/>';
	// Traitement
	$result = $db->prepare($query);
	$result->execute($params);
	return $result->fetchAll();
}

[rawsrc]

Salut,

n'oublie pas que empty() renvoie true quand la valeur testée est égale à zéro. Regarde ici. Donc si ta valeur peut être zéro alors ce n'est pas empty() qu'il faut utiliser
Ce code est inutile :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
$_POST = array_map('strip_tags', $_POST);
$criteria = empty($_POST) ? []: $_POST;

$_POST est toujours défini (vide ou pas) et strip_tags est non compatible avec l'encodage unicode.

[Invité]

Bonjour,

Le minimum est de MONTRER les éléments du code du formulaire HTML, en rapport avec la(les) problématique(s).

Tu parles de min/max (par exemple) : on ne sait pas comment c'est appliqué.

strip_tags est non compatible avec l'encodage unicode.

@rawsrc
Ah bon ? (strip_tags)

Tu as un exemple ?

[rawsrc]

Salut jreaux62

effectivement, si la chaîne est encodée correctement en UTF-8 (pas UTF-16 ou UTF-32), strip_tags() est compatible UTF-8 dans la mesure ou aucun autre caractère UTF-8 ne contient la séquence encodant le < et le >.

[moimp]

Merci pour votre aide.

n'oublie pas que empty() renvoie true quand la valeur testée est égale à zéro

Je le savais mais je me suis laissé surprendre, d'autant qu'ailleurs c'était bien empty qu'il fallait que j'utilise (pas de chaîne vide, pas de 0).
Le problème du code postal est donc réglé et fonctionne.
Par contre, il me reste le fonctionnement entre les bornes min et max. Le résultat est vide si:

• L'une des bornes n'est pas renseignée,
• La borne de fin ('max') commence par la même lettre que celle de début ('min'). Exemple min='b', max='bi'.

Je suppose que ce sont les jokers qui sont mal utilisés mais je ne vois pas comment faire.
Je n'ai trouvé aucune explications sur la compatibilité de strip_tags avec l'encodage UTF-8.

Code html :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
	<form method="post" action="">
		<fieldset><legend>Filtre</legend>
			<label for="min" class="short">De</label><input type="text" name="min" id="min" placeholder="a" />
			&nbsp;<label for="max" class="short">à</label><input type="text" name="max" id="max" placeholder="z" /><br/>
			<label for="pays">Pays</label><select name="pays" id="pays"><option></option><option value="AL">Albanie</option><option value="FR" selected="selected">France</option><option value="CZ">Tchèque, République </option></select><br/>
			<label for="cp">Code postal</label><input type="text" name="cp" id="cp" /><br/>
			<label for="ville">Ville</label><input type="text" name="ville" id="ville" /><br/>
			<p><input type="submit" name="filter" value="Filtrer" /> <input type="submit" name="home" value="Accueil" formaction="home.php" /></p>
		</fieldset>		
	</form>

[Celira]

On parle bien de ce code pour les bornes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
if ( isset($criteria['min'], $criteria['max']) ) {
	$addClauses[] = "nom BETWEEN :min AND :max";
	$params['min'] = $criteria['min'];
	$params['max'] = $criteria['max'];
}

J'ignorais qu'on pouvait utiliser BETWEEN pour des chaines.
Cela dit, le fait que tes 2 bornes commencent par la même lettre n'a a priori pas d'impact, tant que le texte est effectivement entre les deux mots
Par exemple : "bi" est entre "b" et "bu", mais pas "bien".

[Invité]

Salut jreaux62

effectivement, si la chaîne est encodée correctement en UTF-8 (pas UTF-16 ou UTF-32), strip_tags() est compatible UTF-8 dans la mesure ou aucun autre caractère UTF-8 ne contient la séquence encodant le < et le >.

@rawsrc
Merci pour la précision (je n'ai jamais employé UTF-16 ou UTF-32)