Discussion :

[ClayStone]

Bonjour,
Je cherche à déterminer si une ligne existe ou non dans une table (pas besoin de connaître le nombre de fois ou elle existe, pas besoin non plus d'en récupérer les données).
J’obtiens le résultat attendu de deux manières. Laquelle me conseillez-vous? Ou y-a-t-il mieux à faire?

Option COUNT():

Code PHP :

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
 
$reqCount = <<<SQL
	SELECT COUNT(*)
	FROM table        
	WHERE table.colonne1 = ?
		    AND table.colonne2 = ?;
SQL;
try{
	$stmtCount = $db->prepare($reqCount) ;
	$stmtCount -> bindParam (1, $col1,PDO::PARAM_STR);
	$stmtCount -> bindParam (2, $col2,PDO::PARAM_STR);
	$stmtCount -> execute()
	$resCount = $stmtCount -> fetch();
 
	if (intval($resCount["COUNT(*)"])==0){
		// n’existe pas traitement;
		}
	else{
		// existe traitement;
                }

Option EXISTS()

Code PHP :

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
 
$reqExist = <<<SQL
	SELECT *
	FROM table
	WHERE EXISTS (
		SELECT *
		FROM table
		WHERE table.col1 = ?
		            AND table.col2 = ?
                );
SQL;
 
try{
	$stmtExist = $db->prepare($reqExist) ;
	$stmtExist -> bindParam (1, $col1,PDO::PARAM_STR);
	$stmtExist -> bindParam (1, $col2,PDO::PARAM_STR);
	$stmtExist -> execute()
	$resExist = $stmtExist -> fetch();
 
	if ($resExist<>FALSE){
		//existe traitement;
		}
	else{
		//n’existe pas traitement;
	        }

J'étais aussi tenté par rowCount() sur un SELECT (code plus court à écrire - pas sûr de la validité le l'argument ), mais la doc semble le déconseiller…?

Question subsidiaire, les tests :
if (intval($resCount["COUNT(*)"])==0){} (ligne 15 pour COUNT)
et
if ($resExist<>FALSE){}(ligne 20 pour EXISTS)
sont-ils écrit proprement? Je n’y suis arrivé qu’après de nombreux tâtonnements

Merci pour vos conseils

[kevin254kl]

Salut,

Il me semble que Exist est plus performant que Count, dans ton cas puisque tu test l'existence ça fait sens.
Pour les comparaisons

Code :

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

if (intval($resCount["COUNT(*)"])==0){}

plutôt

Code :

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

if (intval($resCount["COUNT(*)"])===0){}

qui va tester le type, pas besoin du intval logiquement la requête retourne un entier non?
Pour

Code :

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

f ($resExist<>FALSE)

plutôt

Code :

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

f ($resExist!==0)

[ClayStone]

Hello,

Il me semble que Exist est plus performant que Count, dans ton cas

Super ! Merci !

Pour les comparaisons, j'ai du mal à comprendre, je tâtonne toujours :
pour COUNT(),

pas besoin du intval logiquement la requête retourne un entier non?

La requête SQL en elle même, oui, mais :

Code :

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

1
2
3
4
5
 
// si 3 lignes trouvées:
var_dump($resCount) // -> array(2) { ["COUNT(*)"]=> string(1) "3" [0]=> string(1) "3" } 
// si ligne n'existe pas
var_dump($resCount) // -> array(2) { ["COUNT(*)"]=> string(1) "0" [0]=> string(1) "0" }

if (intval($resCount["COUNT(*)"])===0){} qui va tester le type

Merci pour l'astuce

Pour EXISTS(), ta proposition if ($resExist!==0) me retourne toujours TRUE :

Code :

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

1
2
3
4
5
6
7
 
//si une ligne est trouvée :
var_dump($resExist) // array(10) { tableaux indexés et associatifs des 5 colonnes de la première ligne}
var_dump($resExist!==0) // bool(true)
//si aucune ligne n'existe,
var_dump($resExist) // bool(false)
var_dump($resExist!==0) // bool(true)

[kevin254kl]

Cava dépendre du retour de ta requête, la plus part du temps j'ai des orm qui font le travaille, si tu as bien comme retour

Code :

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

1
2
3
4
5
6
//si une ligne est trouvée :
var_dump($resExist) // array(10) { tableaux indexés et associatifs des 5 colonnes de la première ligne}
var_dump($resExist!==0) // bool(true)
//si aucune ligne n'existe,
var_dump($resExist) // bool(false)
var_dump($resExist!==0) // bool(true)

C'est encore plus simple
tu as juste à faire

Code :

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

1
2
3
4
5
if($resExist){
 
} else { 
 
}

En passant utilise des fonctions c'est plus propre pour découpler.
Dès que tu vois un else c'est que généralement tu peux utiliser une fonction pour refactorer

Code :

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

1
2
3
4
5
6
7
8
 
function process(?array $exist) : void
{
     if(!$exist)
        return ;
 
    // ici ton process
}

[ClayStone]

Kello Kevin,

C'est encore plus simple
tu as juste à faire

Code :

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

1
2
3
if($resExist){
} else { 
}

C'est nickel !

En passant utilise des fonctions c'est plus propre pour découpler.
Dès que tu vois un else c'est que généralement tu peux utiliser une fonction pour refactorer

Merci pour le conseil !
Résolu

[ClayStone]

J'ai présumé de mes capacités... à quoi sert : void ?

En passant utilise des fonctions c'est plus propre pour découpler.
Dès que tu vois un else c'est que généralement tu peux utiliser une fonction pour refactorer

Code :

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

1
2
3
4
5
6
7
function process(?array $exist) : void
{
     if(!$exist)
        return ;
 
    // ici ton process
}

L'utilisation des fonctions, c'est à la fois plus pratique et plus lisible en effet, merci !
Mais à quoi sert : void ? J'ai lu dans la doc :

void comme type retourné signifie que la valeur retournée est inutile.

Qu'est-ce qui est inutile ici ?