Discussion :

[yonn_b]

Bonsoir à toutes et tous,

après avoir fouillé le forum et m'être trituré les méninges une bonne partie de la journée (et de la dernière nuit également), je me résous à faire appel à vos lumières ; j'en profite d'ailleurs pour m'excuser par avance si la réponse se trouve déjà dans vos archives.

Je souhaite stocker dans une table des informations susceptibles d'être modifiées dans le temps. Par ailleurs je voudrais que les modifications soient enregistrées sans écraser l'ancienne valeur afin de conserver un historique. Enfin, je veux pouvoir enregistrer à l'avance des modification à compter d'une certaine date.

Dans cette optique j'ai une table qui ressemble, en gros à ça :
+-------+------------------+-------------+-----------------+
|...id...|...parent_id...|...valeur...|.....début.....|
+-------+------------------+-------------+-----------------+
|...1....|........1.........|......10.....|..2012-01-13.| # l'item 1 est créé avec une valeur de 10
|...2....|........2.........|......10.....|..2012-01-13.| # l'item 2 est créé avec une valeur de 10
|...3....|........1.........|......15.....|..2012-01-15.| # l'item 1 prendra la valeur 15
|...4....|........2.........|......50.....|..2012-01-18.| # l'item 2 prendra la valeur 50
|...5....|........1.........|....100.....|..2012-01-20.| # l'item 1 prendra la valeur 100
+-------+------------------+-------------+-----------------+

Premièrement, je souhaite savoir si ma méthode n'a pas un gros vice caché que je n'aurais pas remarqué.

Cette table me donne satisfaction sauf au moment d'afficher la fois la valeur courante et les modifications prévues : je ne parviens pas à sélectionner à la fois les valeurs futures (avec debut>CURDATE() ) et la valeur courante (avec debut<CURDATE() ORDER BY debut DESC LIMIT 0,1 ).

Je ne parviens à mes fins qu'en faisant 2 requêtes et je me dis qu'un cas comme le mien ne doit pas être exceptionnel et qu'une solution doit exister. J'ai bien pensé à rajouter une colonne "fin" mais ça me semble redondant vu qu'on peut la déduire de la colonne "debut".

Bref, j'ai besoin d'un peu d'aide pour monter ma super requête qui sélectionne la valeur courante à la date voulue ainsi que les modifications ultérieures.

Je vous remercie par avance de l'attention que vous me prêterez; et bonne soirée

Yann

[punkoff]

Bonjour,

Une piste mais je ne suis pas satisfait 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
 
select a.parent_id, 
max(case when a.debut = b.dt_cur then valeur end) as cur_val,
max(case when a.dt_debut = b.dt_prev then valeur end) as prev_val
from t_stock a
inner join (
select parent_id, max(case when debut <= current_date then dt_mvt end) as dt_cur,
min(case when debut > current_date then dt_mvt end) as dt_prev
from t_stock 
group by parent_id) b on a.parent_id = b.parent_id and (a.debut = b.dt_cur or a.debut = b.dt_prev)
group by a.parent_id

edit: Sinon avez-vous pensez à utiliser un UNION ALL entre vos deux requêtes ?

[yonn_b]

Merci beaucoup !
Je ne connaissait le principe de l'UNION ; je vais poursuivre mes recherche dans ce sens car ça me semble être une solution plus élégante que les jointures (j'avais un peu tenté ce type de requêtes mais ça vire très rapidement à l'usine à gaz).

Je vous tiens au courant pour mettre un [résolu] dès que j'aurais établi ma requête définitive.

[yonn_b]

Comme prévu, la clause UNION a parfaitement fait son office et je récupère ma sélection à l'aide 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
( SELECT id, valeur
FROM ma_table 
WHERE debut IN 
    ( SELECT max(debut) 
    FROM ma_table 
    WHERE debut<=CURDATE() 
    GROUP BY parent_id )
) UNION ALL ( 
SELECT id, valeur
FROM ma_table 
WHERE  debut>CURDATE() )

Merci beaucoup pour ce coup de main qui m'a permis d'apprendre un nouveau truc au passage.
Yann

[yonn_b]

Bon, finalement, ma requête n'est pas si aboutie : la première partie, chargée de récupérer, pour chaque parent_id la dernière date n'est pas suffisamment sélective : elle identifie, pour un parent_id, la date la plus récente et renvoie toutes les lignes dont la valeur de la colonne 'debut' correspond, quelquesoit la valeur de parent_id... ça sort donc quelques valeurs périmées.
Ma recherche continue donc...

[punkoff]

Bon, finalement, ma requête n'est pas si aboutie : la première partie, chargée de récupérer, pour chaque parent_id la dernière date n'est pas suffisamment sélective : elle identifie, pour un parent_id, la date la plus récente et renvoie toutes les lignes dont la valeur de la colonne 'debut' correspond, quelquesoit la valeur de parent_id... ça sort donc quelques valeurs périmées.
Ma recherche continue donc...

Avez-vous essayez ma proposition ?

[yonn_b]

Bonsoir, merci de m'accorder votre temps.
A vrai dire, j'ai un peu de mal à débrouiller votre requête : l'usage de 'dt_mvt' provoque une erreur du fait que cet alias n'a semble-t-il pas été défini. Et comme je patine un peu, je ne parviens pas à corriger l'erreur.

[punkoff]

Bonsoir, merci de m'accorder votre temps.
A vrai dire, j'ai un peu de mal à débrouiller votre requête : l'usage de 'dt_mvt' provoque une erreur du fait que cet alias n'a semble-t-il pas été défini. Et comme je patine un peu, je ne parviens pas à corriger l'erreur.

hmm effectivement, remplacez dt_mvt par la date "début"

[yonn_b]

La requête que vous m'avez proposée ne renvoie malheureusement que la dernière date passée et pas les dates futures.
Je suis donc retourner peaufiner mon UNION et j'ai enfin retrouvé le bonheur grâce à :

Code :

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

1
2
3
4
5
6
7
8
SELECT parent_id, max(debut) as debut, valeur
FROM ma_table
WHERE debut<=CURDATE()
GROUP BY parent_id
UNION ALL
SELECT parent_id, debut, valeur
FROM ma_table
WHERE debut>CURDATE()

L'alias du premier SELECT n'est là que pour uniformiser la sortie et faciliter le traitement ultérieur.
Au final, la solution n'était pas si compliquée. Encore merci pour votre aide.
Yann

[punkoff]

Code :

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

1
2
3
4
5
 
SELECT parent_id, max(debut) AS debut, valeur
FROM ma_table
WHERE debut<=CURDATE()
GROUP BY parent_id

C'est pas bon ça, MySql va vous remontez n'importe quelle ligne pour la colonne "valeur"

Quand on fait un group by toutes les colonnes du select non encadrées par un fonction d’agrégation (min, max, etc) doivent être présente dans la clause group by

[yonn_b]

... en tous cas, on ne peut pas dire que vous avez manqué de patience avec le mauvais élève que je suis.

Je n'ose vous soumettre ma nouvelle tentative de peur de connaître un nouvel échec.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
SELECT parent_id, debut, valeur
FROM ma_table
WHERE debut=(
    select max(debut) 
    from ma_table as t 
    where t.parent_id=ma_table.parent_id 
    AND debut<=CURDATE() 
    AND ma_table.parent_id=t.parent_id)
UNION ALL
SELECT parent_id, debut, valeur
FROM ma_table
WHERE debut>CURDATE()

A première vue, les tests sur une petite table factice fonctionnent plutôt bien de mon coté, mais le diable se cache dans les détails parait-il, et j'ai forcément omis quelque chose pour pouvoir définitivement passer cette discussion en [RESOLU].
Merci pour votre aide.
Yann

[punkoff]

Bonjour,

votre nouvelle requête me semble pas mal (attention, toute fois, vous avez dupliqué la condition sur le parent_id), même si niveau perf je me demande si ceci ne serai pas mieux :

Code :

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

1
2
3
4
5
6
7
8
9
SELECT parent_id, debut, valeur
FROM ma_table a
inner join (select parent_id, max(debut) as debut
    from ma_table as  t
    where debut<=CURDATE()) b on a.parent_id = b.parent_id and a.debut = b.debut
UNION ALL
SELECT parent_id, debut, valeur
FROM ma_table
WHERE debut>CURDATE()

Je ne connais pas assez l'optimiser de mysql pour savoir quelle solution choisir, à vous de vérifier les plans d'executions

[yonn_b]

J'ai testé votre requête pour comparer mais elle souffre d'un petit défaut : la partie qui sert à la jointure ne retourne qu'un seul résultat (la dernière date passée de la table) et pas la dernière date pour chaque parent_id.
Du coup, je ne reçoit qu'un seul résultat passé alors que j'en voudrais un par parent_id.

Au final, je vais me contenter pour le moment de ma solution ; évidemment si je parviens à optimiser la chose je vous le ferai savoir.

Merci tout de même pour votre aide qui m'a permis d'en apprendre un peu plus sur le requêtage et qui m'a orienté dans la bonne direction.

Yann

[punkoff]

J'ai testé votre requête pour comparer mais elle souffre d'un petit défaut : la partie qui sert à la jointure ne retourne qu'un seul résultat (la dernière date passée de la table) et pas la dernière date pour chaque parent_id.
Du coup, je ne reçoit qu'un seul résultat passé alors que j'en voudrais un par parent_id.

Au final, je vais me contenter pour le moment de ma solution ; évidemment si je parviens à optimiser la chose je vous le ferai savoir.

Merci tout de même pour votre aide qui m'a permis d'en apprendre un peu plus sur le requêtage et qui m'a orienté dans la bonne direction.

Yann

Oui effectivement j'ai oublié le group by dedans, désolé.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
SELECT parent_id, debut, valeur
FROM ma_table a
INNER JOIN (SELECT parent_id, max(debut) AS debut
    FROM ma_table AS  t
    WHERE debut<=CURDATE()
    group by parent_id) b ON a.parent_id = b.parent_id AND a.debut = b.debut
UNION ALL
SELECT parent_id, debut, valeur
FROM ma_table
WHERE debut>CURDATE()