IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

Langage PHP Discussion :

Amélioration de code


Sujet :

Langage PHP

Vue hybride

Message précédent Message précédent   Message suivant Message suivant
  1. #1
    Membre éclairé
    Homme Profil pro
    Étudiant
    Inscrit en
    Février 2008
    Messages
    391
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 38
    Localisation : France

    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Février 2008
    Messages : 391
    Par défaut Amélioration de code
    Bonjour à tous,

    J'ai une petite question pour améliorer la rapidité d'affichage.
    J'ai une requete qui va chercher environ 5000 données dans une table et je la met en forme dans un tableau. Mais à chaque ligne je réalise de nouveau une requête pour avoir des informations complèmentaires dans une autre table. Comment faire pour améliorer tout cela ?

    Voici le code
    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
    21
    22
    23
    24
    25
    26
    27
    28
    29
    30
    31
    32
    33
    34
    35
    36
    37
    38
    39
    40
    41
    42
    43
    44
    45
    46
    47
    48
    49
    50
    51
    52
    53
    54
    55
    56
    57
    58
    59
    60
    61
    62
    63
    64
    65
    66
    67
    68
    69
    70
    71
    72
    73
    74
     
    <?php
    	//Requete sélection des adherents
    	$req_adh = 'SELECT id_adh, nom_adh, prenom_adh, tel_adh, etat_adh FROM adherents ORDER BY id_adh ASC';
    	$res_adh = mysql_query($req_adh);
    ?>
    	<table align="center" width="100%" id="table_adh">
    		<thead>
    		<tr>
    			<th>N&deg; Adh</th>
    			<th>Nom</th>
    			<th>Pr&eacute;nom</th>
    			<th>T&eacute;l&eacute;phone</th>
    			<th>Date Inscription</th>
    			<th>Dur&eacute;e</th>
    			<th>Montant</th>
    			<th>Date de fin</th>
    			<th>Etat</th>
    			<th>&nbsp;</th>
    		</tr>
    		</thead>
    		<tbody>
     
    		<?php
    		while($adherent = mysql_fetch_array($res_adh))
    		{
    		?>
    		<tr align="center">
    			<td><?php echo $adherent['id_adh'];?></td>
    			<td><a href="?page=detail_adh&id=<?php echo $adherent['id_adh'];?>" class="action"><?php echo $adherent['nom_adh'];?></a></td>
    			<td><a href="?page=detail_adh&id=<?php echo $adherent['id_adh'];?>" class="action"><?php echo $adherent['prenom_adh'];?></a></td>
    			<td><?php echo $adherent['tel_adh'];?></td>
    			<?php
    			$sql_fact = 'SELECT DISTINCT id_fact, date_fact, duree_fact, montant_fact, id_adh_fact FROM facture WHERE id_adh_fact='.$adherent['id_adh'].' ORDER BY date_fact DESC LIMIT 1'; 
    			$req_fact = mysql_query($sql_fact) or die('Erreur SQL !<br>'.$sql_fact.'<br>'.mysql_error());
     
    			if(mysql_num_rows($req_fact)>0)
    			{//Si on a un résultat
    				while($facture = mysql_fetch_assoc($req_fact))
    				{
    					$date_facture_fr = explode('-',$facture['date_fact']);
     
    					echo "<td>".$date_facture_fr[2].'/'.$date_facture_fr[1].'/'.$date_facture_fr[0]."</td>";
    					echo "<td>".$facture['duree_fact']."</td>";
    					echo "<td>".$facture['montant_fact']." &euro;</td>";
     
    					$date_fin =  mktime(0,0,0, $date_facture_fr[1] + $facture['duree_fact'], $date_facture_fr[2], $date_facture_fr[0]);
    					$date_fin_fr=date('d/m/Y',$date_fin);
     
    					echo "<td>".$date_fin_fr."</td>";
    				} 
    			}
    			else
    			{
    				echo "<td>0000/00/00</td>";
    				echo "<td>0</td>";
    				echo "<td>0</td>";			
    				echo "<td>0000/00/00</td>";
    			}
    			?>
    			<td><?php echo $adherent['etat_adh'];?></td>
    			<td align="center">
    				<a href="pages/adherents/modif_rapid_adherent.php?id=<?php echo $adherent['id_adh'];?>" rel="action" title="Modifier l'adh&eacute;rent rapidement">
    				<span class="ui-state-default ui-corner-all ui-icon ui-icon-pencil"></span></a>
    			</td>
    		</tr>
    		<?php
    		}
    		?>
    		</tbody>
    	</table>
    <?php
    }//Vérification qu'une session existe bien
    ?>

  2. #2
    Membre expérimenté
    Profil pro
    Responsable de service informatique
    Inscrit en
    Août 2006
    Messages
    174
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations professionnelles :
    Activité : Responsable de service informatique
    Secteur : Industrie

    Informations forums :
    Inscription : Août 2006
    Messages : 174
    Par défaut
    Bonjour,

    Tout récupérer en une seule requète.

    Mais pas certains que cela accélère l'affichage.

    Natso

  3. #3
    Membre émérite Avatar de tdutrion
    Homme Profil pro
    Architecte technique
    Inscrit en
    Février 2009
    Messages
    561
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 37
    Localisation : France, Côte d'Or (Bourgogne)

    Informations professionnelles :
    Activité : Architecte technique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Février 2009
    Messages : 561
    Par défaut
    Pourquoi ne pas paginer les donnees ?

    Si tu as du temps, tu peux essayer d'utiliser jquery datatables, en recuperant les donnees en ajax, 20 lignes par 20 lignes environ...

  4. #4
    Membre expérimenté
    Profil pro
    Responsable de service informatique
    Inscrit en
    Août 2006
    Messages
    174
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations professionnelles :
    Activité : Responsable de service informatique
    Secteur : Industrie

    Informations forums :
    Inscription : Août 2006
    Messages : 174
    Par défaut
    Donc possible en une requète.

    Sinon pour l'affichage, voir avec la pagination.

    Je pencherais pour: revoir le besoin initial car afficher 5000 données ça commence à faire.

    Natso

  5. #5
    Membre éclairé
    Homme Profil pro
    Étudiant
    Inscrit en
    Février 2008
    Messages
    391
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 38
    Localisation : France

    Informations professionnelles :
    Activité : Étudiant

    Informations forums :
    Inscription : Février 2008
    Messages : 391
    Par défaut
    La pagination est faite directement avec jquery mais il faut charger les données entièrement.

    J'ai une erreur avec la requete Sabotage :

    #1054 - Unknown column 'last_facture.id_adh_fact' in 'on clause'

  6. #6
    Membre expérimenté
    Profil pro
    Responsable de service informatique
    Inscrit en
    Août 2006
    Messages
    174
    Détails du profil
    Informations personnelles :
    Localisation : France

    Informations professionnelles :
    Activité : Responsable de service informatique
    Secteur : Industrie

    Informations forums :
    Inscription : Août 2006
    Messages : 174
    Par défaut
    Il ne faut pas nécessairement faire des copier/coller des bouts de code donnés.

    Mais tu peux voir que sabotage utilise des INNER. Cela devrait te mettre sur la voie

    Euh, quel est le besoin ?

    Non pas que je veux te faire changer d'avis mais juste pour comprendre l'utilité.

    Cordialement,

    Natso.

  7. #7
    Membre émérite Avatar de tdutrion
    Homme Profil pro
    Architecte technique
    Inscrit en
    Février 2009
    Messages
    561
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 37
    Localisation : France, Côte d'Or (Bourgogne)

    Informations professionnelles :
    Activité : Architecte technique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Février 2009
    Messages : 561
    Par défaut
    Si tu regardes ma réponse, ce que je propose est effectivement une pagination avec jQuery mais avec chargement des données en ajax...

    Avec 5000 lignes, tu as beau chercher ça galerera forcément...

  8. #8
    Modérateur
    Avatar de sabotage
    Homme Profil pro
    Inscrit en
    Juillet 2005
    Messages
    29 208
    Détails du profil
    Informations personnelles :
    Sexe : Homme

    Informations forums :
    Inscription : Juillet 2005
    Messages : 29 208
    Par défaut
    En une seule requête est en fermant les yeux

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    9
    SELECT id_adh, nom_adh, prenom_adh, tel_adh, etat_adh, f.id_fact, f.date_fact, f.duree_fact, f.montant_fact, f.id_adh_fact
     
    FROM adherents 
     
    JOIN (SELECT id_fact, MAX(date_fact) as max_date_fact FROM facture GROUP BY id_fact) last_facture ON last_facture.id_adh_fact = adherents.id_adh
     
    JOIN facture f ON last_facture.id_facture = f.id_facture AND last_facture.max_date_fact = f.date_fact
     
    ORDER BY id_adh ASC
    N'oubliez pas de consulter les FAQ PHP et les cours et tutoriels PHP

Discussions similaires

  1. Amélioration de code
    Par Devilju69 dans le forum Langage
    Réponses: 3
    Dernier message: 07/10/2008, 15h01
  2. Optimisations et Améliorations de code
    Par womannosky dans le forum Langage
    Réponses: 19
    Dernier message: 02/07/2008, 15h05
  3. [Optimisation]Peut-on améliorer ce code ?
    Par progfou dans le forum Algorithmes et structures de données
    Réponses: 6
    Dernier message: 09/08/2007, 15h43
  4. [Tableaux] Amélioration de code (if)
    Par Sir Tengu dans le forum Langage
    Réponses: 3
    Dernier message: 24/12/2006, 00h26
  5. [Sécurité] Comment amélioré mon code ?
    Par Nadd dans le forum Langage
    Réponses: 14
    Dernier message: 03/03/2006, 20h13

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo