Optimisation du code PHP vs SQL

[persia]

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

[Séb.]

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

[persia]

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.

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

[Séb.]

Mais pourquoi faire cela avec PHP et pas SQL ?

[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

[persia]

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.

[persia]

Envoyé par syl2095

Bonjour,

une requête du style :

Code :

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

[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

[stealth35]

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

Code :

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

[syl2095]

ouais c'est vrai les deux sont faisables

[persia]

Envoyé par syl2095

Code :

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