Discussion :

[laurentSc]

Bonjour,

j'ai 2 tables T1 et T2 et je voudrais extraire les données d'une colonne de T1 seulement si la donnée correspondante de T2 vérifie une certaine condition (les 2 tables sont reliées par clé étrangère). Mes essais provoquent plein de messages d'erreur donc je suis loin du compte.

Voici quelques détails :

table "tickets" :
tickets.sql :

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
CREATE TABLE IF NOT EXISTS `tickets` (
  `id` varchar(18) CHARACTER SET latin1 COLLATE latin1_bin NOT NULL,
  `Product_Name` varchar(25) CHARACTER SET latin1 COLLATE latin1_bin DEFAULT NULL,
  `Assigned_Group` varchar(30) CHARACTER SET latin1 COLLATE latin1_bin DEFAULT NULL,
  `Submitted_Date` datetime NOT NULL,
  `Last_Resolved_Date` datetime NOT NULL,
  `Prioritytype` smallint(2) NOT NULL,
  `Statustype` smallint(3) NOT NULL,
  `Area` varchar(10) CHARACTER SET latin1 COLLATE latin1_bin DEFAULT NULL,
  `Country` varchar(10) CHARACTER SET latin1 COLLATE latin1_bin DEFAULT NULL,
  `Submitter_SESA` int(6) NOT NULL,
  `Custommer_SESA` int(6) NOT NULL,
  `Organization` varchar(20) CHARACTER SET latin1 COLLATE latin1_bin DEFAULT NULL,
  `Incident_Type` varchar(20) CHARACTER SET latin1 COLLATE latin1_bin DEFAULT NULL,
  `Reported_Source` varchar(20) CHARACTER SET latin1 COLLATE latin1_bin DEFAULT NULL,
  PRIMARY KEY (`id`),
  KEY `Prioritytype` (`Prioritytype`),
  KEY `Statustype` (`Statustype`)
) ENGINE=InnoDB DEFAULT CHARSET=utf8 COLLATE=utf8_bin;

table "statustype" :
statustype.sql :

Code sql :

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

1
2
3
4
5
6
CREATE TABLE IF NOT EXISTS `statustype` (
  `id` smallint(3) NOT NULL AUTO_INCREMENT,
  `name` varchar(13) CHARACTER SET latin1 COLLATE latin1_bin NOT NULL DEFAULT '',
  PRIMARY KEY (`id`),
  UNIQUE KEY `name` (`name`)
) ENGINE=InnoDB AUTO_INCREMENT=31 DEFAULT CHARSET=utf8 COLLATE=utf8_bin;

et ce que je voudrais faire :

Code :

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

1
2
3
4
5
6
 
IF ((statustype.id != 4)AND(statustype.id)!=5)) THEN
{
SELECT tickets.Last_Resolved_Date FROM tickets
ELSE SELECT NULL //dans ce cas, la valeur n'a aucun sens donc on affiche 0
}

sauf que je n'ai aucune idée de la requête qu''il faudrait faire...

[al1_24]

Ce qui serait intéressant, ce serait de connaître les requêtes testées et les messages d'erreur reçus...

[laurentSc]

en fait, je n'ai rien gardé en mémoire. J'y retravaille donc dans la soirée puis en rendrai compte...

[rawsrc]

et tu continues avec latin1 !!
tu le fais exprès ou quoi ?

[laurentSc]

tu le fais exprès ou quoi ?

Ben oui
Honnêtement, je croyais l'avoir corrigé, mais par exemple, pour tickets.sql, y a que la ligne 19 de corrigée, mais pas toutes celles relatives au colonnes. Je ne vois que le moyen de le changer colonne par colonne. Peut-on le faire directement ?

[laurentSc]

J'ai tenté une requête simplifiée par rapport à mon but, mais déjà une erreur :

Code sql :

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

1
2
3
IF NOT(statustype.id=4)
select DATE_FORMAT(Last_Resolved_Date,"%Y/%m") from tickets 
END IF

1.Type d'énoncé non reconnu. (near "IF" at position 0)

[laurentSc]

Comme je savais atteindre mon but avec du code PHP en plus des requêtes SQL, je l'ai fait, mais je reste intéressé par une méthode full-SQL, ce qui serait plus optimisé...

Code php :

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

1
2
3
4
5
6
$pdo=new PDO();
$res=$pdo->select('SELECT Prioritytype from `tickets`',[]);
if (($res!=4)&&($res!=5))
    $var=$pdo->select('select DATE_FORMAT(Last_Resolved_Date,"%Y/%m") as date from tickets',[]);
else $var=NULL;
echo "var=".$var[0]['date'];

La méthode select (de rawsrc) exécute la requête SQL (avec PDO), puis fait un fetchALL, qu'elle retourne.

[al1_24]

Je n'ai toujours pas compris ce que tu cherchais à faire
Si c'est une requête avec un nombre de colonnes variables, le langage SQl n'est pas adapté.

[laurentSc]

Non, je suis quasiment sûr que c'est faisable en SQL :

voici le pseudo-code :

Code :

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

1
2
3
4
5
6
7
 
IF (NOT(Statustype=4) AND NOT(Statustype=5))
{
    select DATE_FORMAT(Last_Resolved_Date,"%Y/%m") as date from tickets;
}
ELSE
    return 0;

où Statustype est une colonne de la table tickets.

[al1_24]

Dans ce goût là :

Code :

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

1
2
3
4
5
6
7
8
9
select  date_format(last_resolved_date, "%Y/%m") as date
from    tickets
where   statustype  not in (4,5)
union all
select  0
from    tickets
where   statustype  not in (4,5)
having  count(1)    = 0
;

[laurentSc]

On est pas loin. Déjà, si je comprends bien les 3 premières lignes, après, non. Pourrais-tu les expliquer ?

Mais c'est pas parfait ; j'ai fait des essais avec 2 enregistrements.

Je te mets à chaque fois 4 lignes :
1 et 2- les 2 colonnes qui nous intéressent pour les 2 enregistrements (je mets que les années)
3- ce qu'on obtient
4- ce que je souhaiterais.

Last_Resolved_Date	StatusType
2019	5
2020	5

obtenu : 0
souhaité : 0 0

2019	1
2020	5

obtenu : 2019
souhaité : 2019 0

2019	5
2020	1

obtenu : 2020
souhaité : 0 2020

2019	1
2020	1

obtenu : 2019 2020
souhaité : 2019 2020

Merci d'avance.

[al1_24]

Tu présentais une requête avec une seule colonne et maintenant tu attends un résultat sur deux colonnes
Quand tu sauras exprimer clairement ton besoin, tu auras déjà fait la moitié du travail.
En attendant je laisse à d'autres le soin de te répondre. Moi j'abandonne.

[aieeeuuuuu]

bonjour,

Pas trés clair en effet, mais je vais tenter ma chance avec ça :

Code :

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

1
2
SELECT CASE WHEN  statustype  NOT IN (4,5) THEN  DATE_FORMAT(Last_Resolved_Date,"%Y/%m") ELSE 0 END as date 
FROM tickets;

[laurentSc]

besoin pas clair, mais tu l'as quand même compris

[StringBuilder]

bonjour,

Pas trés clair en effet, mais je vais tenter ma chance avec ça :

Code :

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

1
2
SELECT CASE WHEN  statustype  NOT IN (4,5) THEN  DATE_FORMAT(Last_Resolved_Date,"%Y/%m") ELSE 0 END as date 
FROM tickets;

En fait, la réponse d'Alain était bonne aussi... au détail près du having qui n'avait rien à faire là.

A voir maintenant ce qui est le plus rapide... probablement le CASE WHEN ceci dit.

[laurentSc]

Je réouvre cette discussion, car, satisfait, je l'avais mise de côté pour faire autre chose, mais je la reprends et ça ne marche plus !

La requête est :

Code sql :

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

1
2
SELECT CASE WHEN  `statustype`  NOT IN (4,5) THEN  DATE_FORMAT(Last_Resolved_Date,"%Y/%m") ELSE 0 END as date 
      FROM `tickets`

donc la table ticket contient une clé étrangère vers la table statustype et le problème, c'est que même si cette clé vaut 4 ou 5, la requête retourne une date et non 0 ! Cette requête est-elle correcte ?

[laurentSc]

En désespoir de cause, je me tourne vers ma solution qui mixe le PHP et le SQL, mais c'est sûr que c'est pas optimisé, donc si vous avez mieux, je suis preneur...

Code php :

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

1
2
3
4
5
6
7
8
 $res=$pdo->select('SELECT Statustype FROM `tickets` t where t.id="'.$_POST['id'].'"',[]);
 
    $status=$res[0]['Statustype'];
    if (($status!=4)&&($status!=5))
        $var=$pdo->select('select DATE_FORMAT(Last_Resolved_Date,"%Y/%m") as date FROM `tickets` t where t.id="'.$_POST['id'].'"',[]);
    else $var=NULL;
    echo "var=";
    if ($var==NULL) echo "0"; else echo $var[0]['date'];echo "<br/>";

Pour exécuter les requêtes MySql en PHP, j'utilise une classe héritée de la classe PDO.

[escartefigue]

Code sql :

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

1
2
SELECT CASE WHEN  `statustype`  NOT IN (4,5) THEN  DATE_FORMAT(Last_Resolved_Date,"%Y/%m") ELSE 0 END as date 
      FROM `tickets`

donc la table ticket contient une clé étrangère vers la table statustype et le problème, c'est que même si cette clé vaut 4 ou 5, la requête retourne une date et non 0 ! Cette requête est-elle correcte ?

Il faudrait être clair dans votre expression de besoin...
D'une part, vous parlez de la table "statustype", or, statustype est une colonne
D'autre part, votre requête met en forme une date au format "CCAA/MM" pour toute autre valeur de statustype que "4" et "5" et non pas même si les valeurs sont 4 ou 5...
Si vous voulez que les valeurs 4 et 5 produisent un résultat égal à zéro, alors il suffit de remplacer NOT IN (4,5) par IN (4,5).
Enfin, votre code PHP présente pour seule différence de remplacer zéro par null, ce qui peut être avantageusement exécuté directement dans la requête.

[StringBuilder]

Aussi, vous parlez d'une table de référence "statustype"... donc 4 et 5 sont des "valeurs" de "statustype" ou des identifiants de "statustype" ?

[aieeeuuuuu]

le problème, c'est que même si cette clé vaut 4 ou 5, la requête retourne une date et non 0 !

C'est tout à fait normal : un colonne possède un type, bien entendu, le même pour toutes les lignes . Ici, le type de retour du CASE WHEN est celui de Last_Resolved_Date, en raison de la précédence de son type par rapport à à un INT (le 0 ).

vous ne pouvez pas, dans une colonne, renvoyer tantôt des dates, tantôt des entiers.
Comme évoqué par escartefigue, le mieux serait alors de remplacer le 0 par un NULL et de la traiter ensuite en PHP