Discussion :

[kabkab]

Bonjour,
Désirant optimiser les accès à ma base de données, je me demandais si c'était possible de le faire pour cette série de requêtes de façon à n'aboutir qu'à une seule requête.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 2 ORDER BY `newsdate` DESC LIMIT 4;
 
SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 5 ORDER BY `newsdate` DESC LIMIT 4;
 
SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 4 ORDER BY `newsdate` DESC LIMIT 4;
 
SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 3 ORDER BY `newsdate` DESC LIMIT 4;
 
SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 7 ORDER BY `newsdate` DESC LIMIT 4;
 
SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 8 ORDER BY `newsdate` DESC LIMIT 4;

[mohamed]

Essaie d'utiliser les "union".

Code :

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

1
2
3
4
5
6
select .. from where ....
union
select .. from where ....
union
select .. from where ....
order by .....

[kabkab]

Merci mais ça ne donne pas.

[kabkab]

J'ai parlé trop vite
Il fallait metre ça

Code :

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

(SELECT `newsid`,`newsheadline`,`newsdate`,`surtitle`, `newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 2 ORDER BY `newsdate` DESC LIMIT 4) UNION (SELECT `newsid`,`newsheadline`,`newsdate`,`surtitle`, `newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 5 ORDER BY `newsdate` DESC LIMIT 4) UNION (SELECT `newsid`,`newsheadline`,`newsdate`,`surtitle`, `newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 4 ORDER BY `newsdate` DESC LIMIT 4) UNION (SELECT `newsid`,`newsheadline`,`newsdate`,`surtitle`, `newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 3 ORDER BY `newsdate` DESC LIMIT 4) UNION (SELECT `newsid`,`newsheadline`,`newsdate`,`surtitle`, `newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 7 ORDER BY `newsdate` DESC LIMIT 4) UNION (SELECT `newsid`,`newsheadline`,`newsdate`,`surtitle`, `newscat` FROM news WHERE `newsdate` <= '2007-07-24' and `newscat` = 8 ORDER BY `newsdate` DESC LIMIT 4)

N'empêches, si quelqu'un a une autre solution, je suis preneur.

[ddelec24]

Salut,

pourquoi ne fais-tu pas plutôt pas:

Code :

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

SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' AND `newscat` BETWEEN 2 AND 8 ORDER BY `newsdate` DESC LIMIT 4;

et ensuite au niveau du traitement php tu sépare tes éléments?
enfin c'est ce que j'aurais fais perso... pour 4 éléments le traitement sera plus rapide que les requêtes a mon avis.

EDIT: non je viens de réfléchir ca ne peux pas marcher
Ca sélectionnerais que les 4 premieres occurences trouvées, et pas au max 4 pour chaque valeur de newscat...
peut etre avec une requete imbriquée mais c'est pas mon fort
désolé

[mohamed]

Ou aussi:

Code :

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

SELECT `newsid`,`newsdate`,`newscat` FROM news WHERE `newsdate` <= '2007-07-24' AND `newscat` IN (2,5,4,3,7,8) ORDER BY `newsdate` DESC LIMIT 4;

Par contre la requete de ddelec24 est plus optimale.

[kabkab]

mohamed et ddelec24, vos requêtes ne sont pas fausses mais le résultat ne donne pas ce qu'il faut.

J'ai en retour que des newscat = 2, pour l'instant il n'y a que la requête que j'ai trouvé comme solution qui fonctionne.

[ced]

Bonjour,

En fait, dans ce cas, il vaut peut-être mieux utiliser l'opérateur IN que l'opérateur BETWEEN, parce que, dans ton exemple, tu ne ramènes pas les lignes pour lesquelles newscat = 6 (si ça existe).
Pour faire ça en une seule requête, essaie un truc du genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
SELECT `newsid`,`newsdate`,`newscat` 
FROM news n1
WHERE `newsdate` <= '2007-07-24' 
AND newscat in (2, 3, 4, 5, 7, 8)
AND 
(SELECT count(*)
FROM news n2
WHERE n1.newscat = n2.newscat
AND n1.newsdate < n2.newsdate) < 4
order by newscat, newsdate

J'ai pas testé... Mais ça doit être une solution de ce type là...

ced

[kabkab]

ced, je ne peux te répondre que par : cette requête me donne une belle plantation de can abis.

[ced]

Quelle version de MySQL ?
Si c'est en-dessous de la 4.1, c'est cuit... Les requêtes imbriquées ne passent pas...
Sinon, quel est le message d'erreur de la plantation ?

ced

[kabkab]

C'est simple, la requête fais rouler le mysql à ne plus en finir avec des pics d'utilisation à 100% du processeur.
Je n'ai pas laissé finir pour vérifier le temps que ça demandait, j'ai redémarré.

[pop_up]

as tu un index sur le couple newsdate, newscat ?

[kabkab]

Je m'excuse si j'ai induit les amis en erreur. Vous remarquez qu'à ma première demande j'incluait uniquement `newsid`,`newsdate`,`newscat` en SELECT pour faciliter la lecture de ma requête. En fait, les colonnes dont j'ai besoin sont `newsid`,`newsheadline`,`newsdate`,`surtitle`, `newscat`.

Ainsi, je ne peux me permettre de mettre des index sur toutes ces colonnes au vu de la structure de la table.

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
CREATE TABLE news (
  newsid int(11) NOT NULL auto_increment,
  newsheadline varchar(150) collate latin1_general_ci NOT NULL default '',
  newsbody text collate latin1_general_ci NOT NULL,
  newsdate date NOT NULL default '0000-00-00',
  newsauthor int(11) NOT NULL default '0',
  newscat int(11) NOT NULL default '0',
  newscomments tinyint(1) NOT NULL default '0',
  author varchar(50) collate latin1_general_ci NOT NULL default '',
  abstract text collate latin1_general_ci NOT NULL,
  photo varchar(200) collate latin1_general_ci NOT NULL default '',
  photoauthor varchar(50) collate latin1_general_ci NOT NULL default '',
  surtitle varchar(150) collate latin1_general_ci NOT NULL default '',
  exerg text collate latin1_general_ci NOT NULL,
  photolegend text collate latin1_general_ci NOT NULL,
  page tinyint(2) NOT NULL default '0',
  Art_Lu int(11) unsigned NOT NULL default '0',
  PRIMARY KEY  (newsid),
  KEY newsdate (newsdate)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 COLLATE=latin1_general_ci;