Optimisation du code PHP vs SQL

Bonjour à tous.

En ce moment j'utilise 3 tables dans ma BD ("auteur","livre","auteur_livre");

Dans une page qui permet de mettre à jour un auteur, je dois afficher tous les livres qui existent dans ma BD (j'en ai 460). Sur cette page je dois afficher un flag ou un message qui informe l'utilisateur des livres déjà associés à l'auteur. Je procède donc comme suit:

code:
Code:

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 27 28 29 30 31 32 33 /* charger seulement les id des livres associés à l'auteur qu'on veux mettre à jour*/ $auteurs_livres=loadTable("auteur_livres")->auteur; $stockage_id=array(); // parourir la table; foreach($auteurs_livres as $auteur_livre) { //stoker les livres_id qui se trouvent dans "auteur_livres" et qui sont associès à l'auteur; $stockage_id[]=$auteur_livre["livre_id"]; } // charger la table "livre" $livres=loadTable("livre"); // verifier si les livres de l'auteur existent dans la table livre_auteur foreach ($livres as $key=>$livre) { if(in_array($livre["id"],$stockage_id)) { echo "le livre ".$livre["id"] est déjà associé } else { echo "le livre ".$livre["id"] n'est pas encore associé } }
Mais ce code prends beaucoup de temps pour être traité (entre 5 à 7 sec), donc je veux savoir si il y a un moyen pour mieux optimiser mon code. Si je n'avais pas de message à afficher, il suffit simplement de faire un INNER JOIN et le temps de réponse est plus rapide (moins de 1 sec), mais c'est lors de la vérification que ca prends beaucoup de temps car à chaque ligne trouvé dans le table livre je dois vérfier si elle se trouve dans la table "livre_auteur" et afficher le message adéquat

Merci :)

Pas compris ta problématique.
Donne un exemple de données en entrée et de ce que tu veux en faire.

Citation:

Envoyé par Séb.

Pas compris ta problématique.

Je veux savoir si il y a un moyen pour mieux optimiser mon code, car ca prends entre 5 a 7 sec pour que la page s'affiche et je ne trouve pas ca normal.

Citation:

Envoyé par Séb.

Donne un exemple de données en entrée et de ce que tu veux en faire.

Table auteur_livre contenant 600 entrées.
Code:

1 2 3 4 5 6 auteur_id livre_id 1 2 2 1 2 450 4 5 ...
Table livre contenant 450 entrées
Code:

1 2 3 4 5 6 7 id nom 1 php 2 java 3 asp 4 html ... ..... 450 css
L'auteur dont je veux afficher l'information à l'id 2, et dans la table auteur_livre il a deux livre (1,450), donc "php" et "css"

Ce que je veux afficher à l'écran:

Utilisateur avec id 2:
Code:

1 2 3 4 5 6 7 8 livre livre associée php oui java non asp non html non css oui
Avec le code que j'ai posté ca marche, mais ca prendre beaucoup de temps.

J'espère que j'étais plus clair cette fois-ci

Merci

Mais pourquoi faire cela avec PHP et pas SQL ?

Bonjour,

une requête du style :

Code:

1
2
3
4
5
6
7
8
9
10
SELECT l.idlivre, 
       l.nomlivre, 
       ( CASE 
           WHEN auteur_livre IS NULL THEN 'non' 
           ELSE 'oui' 
         END ) AS acelivre 
FROM   livre AS l 
       LEFT JOIN auteur_livre AS a 
         ON a.idlivre = l.idlivre 
            AND a.idauteur = 2

Citation:

Envoyé par Séb.

Mais pourquoi faire cela avec PHP et pas SQL ?

Et comment pense-tu faire ca avec juste du sql ? à ma connaissance sql ne permet pas d'inclure du code html dans les requêtes.

Citation:
Envoyé par syl2095
Bonjour,

une requête du style :
Code:

1 2 3 4 5 6 7 8 9 10 SELECT l.idlivre, l.nomlivre, ( CASE WHEN auteur_livre IS NULL THEN 'non' ELSE 'oui' END ) AS acelivre FROM livre AS l LEFT JOIN auteur_livre AS a ON a.idlivre = l.idlivre AND a.idauteur = 2
Mais Comment je pourrais passer le text "non" ou "oui" à mon code html ? car je dois donner un style aux 2 messages. Si c'est "oui" on l'affiche en vert, si c'est "non" on l'affiche en rouge. Tu peux regarder mon premier post, tu va comprendre ce que je veux faire

Merci

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
$query = "SELECT l.idlivre, 
       l.nomlivre, 
       ( CASE 
           WHEN auteur_livre IS NULL THEN 'non' 
           ELSE 'oui' 
         END ) AS acelivre 
FROM   livre AS l 
       LEFT JOIN auteur_livre AS a 
         ON a.idlivre = l.idlivre 
            AND a.idauteur = 2";
 
$sql = mysql_query($query);
 
if ( mysql_num_rows($sql) > 0 ) {
 
while ($res = mysql_fetch_array($query)){
 
if ( $res['acelivre'] == 'oui' ) echo '<span style="color:green;">Oui</span>';
if ( $res['acelivre'] == 'non' ) echo '<span style="color:red;">Non</span>';
 
}
 
}

avec ce code normalement, tu devrais pouvoir afficher oui ou non à la couleur que tu veux

par contre le CASE on peu faire un IF a la place

Code:

IF(auteur_livre IS NULL, 'non', 'oui') AS acelivre

ouais c'est vrai les deux sont faisables

Citation:

Envoyé par syl2095

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
$query = "SELECT l.idlivre, 
       l.nomlivre, 
       ( CASE 
           WHEN auteur_livre IS NULL THEN 'non' 
           ELSE 'oui' 
         END ) AS acelivre 
FROM   livre AS l 
       LEFT JOIN auteur_livre AS a 
         ON a.idlivre = l.idlivre 
            AND a.idauteur = 2";
 
$sql = mysql_query($query);
 
if ( mysql_num_rows($sql) > 0 ) {
 
while ($res = mysql_fetch_array($query)){
 
if ( $res['acelivre'] == 'oui' ) echo '<span style="color:green;">Oui</span>';
if ( $res['acelivre'] == 'non' ) echo '<span style="color:red;">Non</span>';
 
}
 
}

avec ce code normalement, tu devrais pouvoir afficher oui ou non à la couleur que tu veux

Merci bien, c'est exactement ce que je cherchais, car je ne savais pas qu'on pouvait inclure des IF ou des CASE dans une requête sql