Discussion :

[laurentSc]

Bonjour,

je cherche à tester la présence d'une valeur dans plusieurs tables. En fait l'idée est de faire une seule fonction plutôt que une par table. Donc une seule fonction pour tester si une valeur appartient à la table A ou à la table B, soit la fonction isFromAorB :

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
 
function isFromAorB($val):bool
{
    $bdd=new MYPDO();
    $sql="SELECT COUNT(name) as yes_or_no  FROM `A`,`B` WHERE LOWER(name) = :val";
    $prio=$bdd->select( $sql, [':val' => mb_strtolower($val)] );
    $yes_or_no=$prio[0]['yes_or_no'];
    var_dump($yes_or_no);
 
    if ($yes_or_no) return(TRUE); else return(FALSE);
}

Voici l'idée du script mais comme le var_dump retourne NULL, y a une erreur, mais où ?

[rawsrc]

salut

c'est quoi cette syntaxe ?

Code sql :

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

FROM `A`,`B`

arrête d'utiliser return(TRUE), préfère simplement return true;.
D'ailleurs :

Code :

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

if ($yes_or_no) return(TRUE); else return(FALSE);

est remplaçable par return (bool)$yes_or_no;

[laurentSc]

c'est quoi cette syntaxe ?

Je sais bien qu'elle pas bonne mais c'était pour montrer ce que je veux faire et qu'on me montre comment on peut faire. Pour le reste, merci.

[Toufik83]

Salut laurent,

Essaies d'abord d'exécuter cette requête dans phpmyadmin pour t'assurer du résultat :

Code sql :

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

1
2
3
4
5
6
7
 
SELECT 
count( distinct( a.id ) ) as in_a,
count( distinct( b.id ) ) as in_b  
FROM `table_a` a
inner join `table_b` b 
on a.name=b.name and lower(a.name)=:val

dont a.id et b.id sont les identificateurs uniques des tables a et b.

[laurentSc]

Salut Toufik,

j'ai essayé ta requête sur mes 2 tables (priority et status) :

Code sql :

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

1
2
3
4
5
6
7
 
SELECT 
count( distinct( p.id ) ) as in_p,
count( distinct( s.id ) ) as in_s  
FROM `priority` p
inner join `status` s
on p.name=s.name and lower(p.name)='low'

priority :

Code sql :

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
 
 
--
-- Structure de la table `priority`
--
 
DROP TABLE IF EXISTS `priority`;
CREATE TABLE IF NOT EXISTS `priority` (
  `id` smallint(2) NOT NULL AUTO_INCREMENT,
  `name` varchar(6) COLLATE utf8_bin NOT NULL DEFAULT '',
  PRIMARY KEY (`id`),
  UNIQUE KEY `name` (`name`)
) ENGINE=InnoDB AUTO_INCREMENT=3 DEFAULT CHARSET=utf8 COLLATE=utf8_bin;
 
--
-- Déchargement des données de la table `priority`
--
 
INSERT INTO `priority` (`id`, `name`) VALUES
(2, 'high'),
(0, 'low'),
(1, 'medium');
COMMIT;

status :

Code sql :

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
 
 
--
-- Structure de la table `status`
--
 
DROP TABLE IF EXISTS `status`;
CREATE TABLE IF NOT EXISTS `status` (
  `id` smallint(3) NOT NULL AUTO_INCREMENT,
  `name` varchar(13) COLLATE utf8_bin NOT NULL DEFAULT '',
  PRIMARY KEY (`id`),
  UNIQUE KEY `name` (`name`)
) ENGINE=InnoDB AUTO_INCREMENT=6 DEFAULT CHARSET=utf8 COLLATE=utf8_bin;
 
--
-- Déchargement des données de la table `status`
--
 
INSERT INTO `status` (`id`, `name`) VALUES
(0, 'assigned'),
(1, 'cancelled'),
(2, 'closed'),
(3, 'in_progress'),
(4, 'pending'),
(5, 'resolved');
COMMIT;

ca donne 0 pour in_p au lieu de 1.
Le souci est que justement name est la clé unique des 2 tables...

[Invité]

Bonjour,

...En fait l'idée est de faire une seule fonction plutôt que une par table...

Dès qu'il s'agit de trouver des idées de problématiques farfelues et/ou grotesques, on peut compter sur Laurent !

• Si tu veux te compliquer la vie : BRAVO ! Tu as gagné !
• Si tu espères un "gain de performance", c'est ridicule (négligeable)

Ici, on part du principe que les tables A et B sont totalement indépendantes.
Donc : pas de INNER JOIN ici, mais plutôt UNION.

[Invité]

Une (meilleure) solution :

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
<?php
function isValInTable($val, $table):bool
{
    global $bdd;
    $sql="SELECT COUNT(name) as total FROM `".$table."` WHERE LOWER(name) = :val";
    $prio=$bdd->select( $sql, [':val' => mb_strtolower($val)] );
    $total=$prio[0]['total'];
 
    return (bool)$total; // false si 0 (rien trouvé), true sinon
}
function isValInTableArray($val, $tableArr):bool
{
	foreach( $tableArr as $table )
	{
		if(isValInTable($val, $table)){ // on trouve une correspondance : true
			return true;
		}
	}
	return false; // sinon false
}

Utilisation :

Code :

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

1
2
3
<?php
$val = 'machin';
$yes_or_no = isValInTableArray($val, ['A','B']);

Remarque :

Code :

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

    global $bdd;

Il est inutile de faire une nouvelle connexion à chaque nouvelle fonction.
On la fait une fois pour toutes en début de script $bdd=new MYPDO();, puis dans les fonctions on utilise la même : global $bdd;.

[rawsrc]

Salut,

Remarque :

Code :

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

    global $bdd;

Il est inutile de faire une nouvelle connexion à chaque nouvelle fonction.
On la fait une fois pour toutes en début de script $bdd=new MYPDO();, puis dans les fonctions on utilise la même : global $bdd;.

Non, si c'est ma classe qu'il utilise, tu peux faire autant de new MYPDO() que tu veux, tu auras toujours qu'une seule et même connexion/instance de PDO

[Toufik83]

Oups laurent, je n'avais pas pris en considération que la valeur peut exister dans une table et pas l'autre, donc + la remarque de jreaux62 essaies ça :

Code sql :

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

1
2
3
4
5
6
7
 
select * from 
(
    (select count(id) as result from priority where lower(name)=:val)  
    union 
    (select count(id) from `status` where lower(name)=:val) 
) as t

ou bien directement et sans UNION (pour avoir deux colonnes):

Code sql :

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

1
2
3
4
 
SELECT 
(select count(id) from priority where lower(name)=:val) as inP ,
(select count(id) from `status` where lower(name)=:val) as inS

[laurentSc]

Oui, c'est vrai rawsrc. J'avais vu ça dans ton code mais l'avais oublié.

Code php :

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

1
2
3
4
5
6
7
8
 
 public function __construct()
    {
        if (self::$pdo === null) {
            self::connect();
        }
        return self::$pdo;
    }

J'avais déjà implémenté le conseil de jreaux62 et m'apprêtais à le remercier mais suis revenu en arrière vu que pas la peine (je viens à l'instant de te lire).
Comme j'étais depuis hier soir sur un autre bug que je viens seulement de corriger, je vais maintenant me pencher sur le reste de la réponse de jreaux62 (et aussi sur le dernier post de toufik83).

N'en déplaise à Jérôme, ce n'était ni pour compliquer, ni pour gagner en performance, mais pour simplifier (une seule fonction qui fait tout), sauf que ça n'a pas l'air de simplifier !

[Invité]

...ce n'était ni pour compliquer, ni ..., ..., sauf que ça n'a pas l'air de simplifier !

Ben non...
Et c'est NORMAL !

Tu gagnerais ENORMEMENT de TEMPS (et nous aussi) si tu te contentais de choses SIMPLES, SIMPLES et SIMPLES !
Dans le code que j'ai proposé, chaque fonction effectue une/des action(s) SIMPLES.
Bref : fais SIMPLE.

[laurentSc]

Le meilleure solution de jreaux62 et la requête sans union de Toufik83 fonctionnent toutes les 2, mais vu que l'objectif de simplification n'est pas atteint (même si les fonctions de Jérôme sont simples, faire 2 fonctions est encore plus simple (pour ma petite tête)), donc je reste sur les 2 fonctions.
Désolé de vous prendre du temps et merci.

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
 
 
static public function isPriority($val):bool
{
    $bdd=new MYPDO();
    $sql="SELECT COUNT(name) as yes_or_no  FROM `Priority` WHERE LOWER(name) = :val";
    $prio=$bdd->select( $sql, [':val' => mb_strtolower($val)] );
    $yes_or_no=$prio[0]['yes_or_no'];//echo "validator 57 sql=".$sql."/prio=".$yes_or_no."/val=".$val."<br/>";
 
    return (bool)$yes_or_no;
}
 
static public function isStatus($val):bool
{
    $bdd=new MYPDO();
    $sql="SELECT COUNT(name) as yes_or_no  FROM `Status` WHERE LOWER(name) = :val";
    $prio=$bdd->select( $sql, [':val' => mb_strtolower($val)] );
    $yes_or_no=$prio[0]['yes_or_no'];
 
    return (bool)$yes_or_no;
}