besion aide optimisation php mysql

besion aide optimisation php mysql - PHP - Programmation

Marsh Posté le 23-10-2007 à 13:27:49    

salut a tous , voilà j'ai un petit soucis avec le code suivant :
 

Code :
  1. <?
  3. $out.="<th class=column2 style='font-weight:bold'>Action</th></tr></thead>";
  4. $tabech=array();
  5. $tabidt=array();
  6. $nextmonth=date("Y-m-d",mktime(0,0,0,date("m" )+1,date("d" ),date("Y" )));
  7. $today=date("Y-m-d",mktime(0,0,0,date("m" ),date("d" ),date("Y" )));
  10. //REQUETE SUR ID CANDIDATS
  11. $result = db_query("SELECT distinct(c.id),c.nachname,c.vorname,t.status FROM candidats c, todo t, parametres p
  12. WHERE t.deadline>'2007-05-05' and t.candidat=c.ID and c.kategorie =p.id and p.nom like 'statut' and
  13. ((p.id!=995 and p.id!=48 and t.status <>3) or (p.id=995 or p.id=48 and t.status=3))
  14. and t.ext=".$user_ID." ".$wherec." and t.echeance is not null order by t.echeance ASC " )
  15. or db_die();
  17. $i=0;
  19. while($row = db_fetch_row($result)){
  20. $tab_rech=array();
  21. $reqza = db_query("SELECT count(ID),recherche FROM ".DB_PREFIX."todo
  22. WHERE suite like 'sans%' and candidat=".$row[0]." Group BY recherche" )
  23. or db_die();
  24. while($resza=db_fetch_row($reqza)){
  25. if($resza[0]>0 and $resza[1] !="" ) $tab_rech[]=$resza[1];
  27. }
  29. $conpteur=count($tab_rech);
  31. if($conpteur>0){
  32. $req10 = db_query("SELECT t.echeance,t.status,t.id FROM ".DB_PREFIX."todo t, parametres p
  33. WHERE p.valeur = t.remark and
  34. (p.nom like 'action_process%' or p.nom like'action_cv' ) and t.candidat = ".$row[0]." and t.suite not like 'sans suite%' ".$wherec." and t.recherche NOT IN (".implode(",",$tab_rech)." ) ORDER BY t.deadline desc,p.ordre DESC" )
  35. or db_die();
  36. }
  37. else{
  38. $req10 = db_query("SELECT t.echeance,t.status,t.id FROM ".DB_PREFIX."todo t, parametres p
  39. WHERE p.valeur = t.remark and
  40. (p.nom like 'action_process%' or p.nom like'action_cv' ) and t.candidat = ".$row[0]." and t.suite not like 'sans suite%' ".$wherec." ORDER BY t.echeance desc,p.ordre desc,t.deadline DESC" )
  41. or db_die();
  42. }
  43. $res10=db_fetch_row($req10);
  44. if($res10[1]!=3){
  45. $tabec[]=$res10[0];
  46. }
  47. else if ($res10[1]==3){
  48. $req00 = db_query("SELECT deadline FROM ".DB_PREFIX."todo
  49. WHERE id=".$res10[2])
  50. or db_die();
  51. $res00=db_fetch_row($req00);
  52. $tabec[]=$res00[0];
  53. }
  54. $tabidt[]="0-".$row[0]."-".$row[2]." ".$row[1];
  56. }
  58. //REQUETE ID CONTACTS
  59. $result2 = db_query(" SELECT contact,deadline FROM todo WHERE ext = $user_ID and status = '3' and candidat=0 ORDER BY deadline ASC " );
  60. while($row2=db_fetch_row($result2)){
  61. $tabec[]=$row2[1];
  62. $tabidt[]="1-".$row2[0];
  63. }
  64. $ar = array($tabec,$tabidt);
  65. array_multisort($ar[0], SORT_ASC, SORT_STRING, $ar[1]);
  67. $cpt=count($ar[1]);
  68. $i=0;
  69. $k=0;
  70. for($j=0;$j<$cpt;$j++){
  71. $tab_rech=array();
  72. $var = explode("-", $ar[1][$j]);
  74. //si candidat
  75. if($var[0]=="0" ){
  76. //recherche des recherches où le candidats est ouvert
  77. $reqza = db_query("SELECT count(ID),recherche FROM ".DB_PREFIX."todo
  78. WHERE suite like 'sans%' and candidat=".$var[1]." Group BY recherche" )
  79. or db_die();
  81. while($resza=db_fetch_row($reqza)){
  82. if($resza[0]>0 and $resza[1] !="" ) $tab_rech[]=$resza[1];
  83. }
  84. $conpteur=count($tab_rech);
  86. //si candidat ouvert dans au moins une recherche
  87. if($conpteur>0){
  88. $req1 = db_query("SELECT t.remark, t.origine_cv, t.recherche,t.echeance, t.ID,t.status,t.deadline,t.ext FROM ".DB_PREFIX."todo t, parametres p
  89. WHERE p.valeur = t.remark and
  90. (p.nom like 'action_process%' or p.nom like'action_cv' ) and t.candidat = ".$var[1]." and t.suite not like 'sans suite%' ".$wherec." and t.recherche NOT IN (".implode(",",$tab_rech)." ) ORDER BY t.deadline desc,p.ordre DESC" )
  91. or db_die();
  92. $tmp=0;
  93. $res1=db_fetch_row($req1);
  94. $req10 = db_query("SELECT t.remark,t.origine_cv ,t.echeance,t.ext FROM ".DB_PREFIX."todo t, parametres p
  95. WHERE p.valeur = t.remark and
  96. (p.nom like 'action_process%' or p.nom like'action_cv' ) and t.candidat = ".$var[1]." and t.suite not like 'sans suite%' ".$wherec." and t.recherche NOT IN (".implode(",",$tab_rech)." ) ORDER BY t.echeance desc,p.ordre desc,t.deadline DESC" )
  97. or db_die();
  98. $res10=db_fetch_row($req10);
  99. $req101 = db_query("SELECT t.remark,t.origine_cv ,t.echeance,t.ext FROM ".DB_PREFIX."todo t, parametres p
  100. WHERE p.valeur = t.remark and
  101. (p.nom like 'action_process%' or p.nom like'action_cv' ) and t.candidat = ".$var[1]." and t.suite not like 'sans suite%' ".$wherec." and t.recherche NOT IN (".implode(",",$tab_rech)." ) ORDER BY p.ordre desc,t.deadline DESC" )
  102. or db_die();
  103. $res101=db_fetch_row($req101);
  104. }
  105. //sinon
  106. else{
  107. $req1 = db_query("SELECT t.remark, t.origine_cv, t.recherche,t.echeance, t.ID,t.status,t.deadline,t.ext FROM ".DB_PREFIX."todo t, parametres p
  108. WHERE p.valeur = t.remark and
  109. (p.nom like 'action_process%' or p.nom like'action_cv' ) and t.candidat = ".$var[1]." and t.suite not like 'sans suite%' ".$wherec." ORDER BY t.deadline desc,p.ordre DESC" )
  110. or db_die();
  111. $tmp=0;
  112. $res1=db_fetch_row($req1);
  114. $req10 = db_query("SELECT t.remark,t.origine_cv,t.echeance,t.ext FROM ".DB_PREFIX."todo t, parametres p
  115. WHERE p.valeur = t.remark and
  116. (p.nom like 'action_process%' or p.nom like 'action_cv' ) and t.candidat = ".$var[1]." and t.suite not like 'sans suite%' ".$wherec." ORDER BY t.echeance desc,p.ordre desc,t.deadline DESC" )
  117. or db_die();
  118. $res10=db_fetch_row($req10);
  120. $req101 = db_query("SELECT t.remark,t.origine_cv ,t.echeance,t.ext,t.von FROM ".DB_PREFIX."todo t, parametres p
  121. WHERE p.valeur = t.remark and
  122. (p.nom like 'action_process%' or p.nom like'action_cv' ) and t.candidat = ".$var[1]." and t.suite not like 'sans suite%' ".$wherec." ORDER BY p.ordre desc,t.deadline DESC" )
  123. or db_die();
  124. $res101=db_fetch_row($req101);
  126. }
  127. if($res1[0] !='' and $ $res1[0] != 'CACO' and $tmp != $res10[2] and $res101[3]==$user_ID and ($res10[2]<$nextmonth) and ($res1[6]<$nextmonth)){
  129. //requete pour le projet auquel le candidat est affecté
  130. $req3 = db_query("SELECT remark,project,contact,firma,ID,progress FROM ".DB_PREFIX."recherche
  131. WHERE ID = ".$res1[2])
  132. or db_die();
  133. $res3 = db_fetch_row($req3);
  135. $tmp=$res1[2];
  136. if(!ereg("N".$res3[4],$res2[3]) and $res3[3]!='' and $res3[5]==1){
  138. //requete pour la recherche auquel le candidat est affecté
  139. $req4 = db_query("SELECT firma FROM ".DB_PREFIX."contacts
  140. WHERE ID = ".$res3[3])
  141. or db_die();
  142. $res4 = db_fetch_row($req4);
  144. //.....code d'affichages des candidats
  146. }
  147. //si contact
  148. elseif($var[0]=="1" and !$recherche){
  149. $req1 = db_query("SELECT t.remark, t.origine_cv, t.recherche,t.echeance, t.ID,t.status,t.deadline,t.ext FROM ".DB_PREFIX."todo t
  150. WHERE t.contact = ".$var[1]." ".$wherec." ORDER BY t.deadline desc" )
  151. or db_die();
  152. $tmp=0;
  153. $res1=db_fetch_row($req1);
  154. if($res1[7]==$user_ID and ($res1[6]<$nextmonth)){
  155. $req2 = db_query("SELECT vorname, nachname,firma FROM ".DB_PREFIX."contacts
  156. WHERE ID = ".$var[1])
  157. or db_die();
  158. $res2 = db_fetch_row($req2);
  159. $firme=$res2[2];
  160. $req3 = db_query("SELECT ID FROM ".DB_PREFIX."contacts
  161. WHERE firma like '$firme' and div1 like 'e'" )
  162. or db_die();
  163. $res3 = db_fetch_row($req3);
  164. //...code d'affichage des contacts
  165. }
  167. }
  168. }
  170. ?>


 
pour résumé la page met environs une minute à ce charger et il faut réactualiser la page jusqu'a cinq fois pour avoir la totalité des données affichés
si vous avez des suggestion je suis preneur.
Merci d'avance à tous


Message édité par zaka47 le 23-10-2007 à 13:49:05
Reply

Marsh Posté le 23-10-2007 à 13:27:49   

Reply

Marsh Posté le 23-10-2007 à 13:36:37    

commence par eregarder le temps d'execution des requetes ( avec un debuguer , ou plus simplement avec time )  
 
 
mais sinon, en vitesse :  
dans le premier cas , tu fais un COUNT* pour chacune des lignes de ta base de donnees, c'est un peu bourrin  
j'adore les commentaire du genre //sinon  
tu devrai mettre ton code entre des balkse code au lieu de quote

Reply

Marsh Posté le 23-10-2007 à 13:45:13    

salut,
ce n'est pas les requetes en elle meme qui sont longue , c'est le fait de faire une boucle for sur des requetes qui est assez lourd en faite (on peut boucler jusqu'a 100 fois) .
 
ensuite le count me sert a verifier a combien de recherches differentes un candidat est il associe (je ne vois pas comment faire autrement dans mon cas)
 
et je voulais savoir ce que tu entends par :
"tu devrai mettre ton code entre des balkse code au lieu de quote" ?

Reply

Marsh Posté le 23-10-2007 à 13:46:24    

clic sur le bouton http://forum-images.hardware.fr/themes_static/images/silk/viewbbcode.gif au dessus de mon post pour voir les balises que j'ai mise autour de mon code

 
Code :
  1. if($monCode est bien presente)
  2.    $nbReponse++;


Message édité par flo850 le 23-10-2007 à 13:46:43
Reply

Marsh Posté le 23-10-2007 à 13:48:51    

est ce que tu as mis des index dans tes tables ?
 
 
tu ne peux pas transformer  
SELECT distinct(c.id),c.nachname,c.vorname,t.status FROM candidats c, todo t, parametres p
WHERE
 
en  
SELECT c.id,c.nachname,c.vorname,t.status,COUNT(DISTINCT recherche.id)
 FROM candidats c, todo t, parametres p,recherche
WHERE GROUP BY c.id

Reply

Marsh Posté le 23-10-2007 à 13:57:34    

heu pas trop en faite c encore plus long car je v taper dans une table en plus(une tres grosse table)

Reply

Marsh Posté le 23-10-2007 à 15:26:27    

ben j'ai trouvé des petits tuto sympas
www.vulgarisation-informatique.com...
et
www.estvideo.com...
 
j'ai donc remplacé mes for par des while, les doucles cotes par des simples et revue vite fait mes requetes (enlevé deux ou trois champs qui ne servaient à rien)
et au final rien qu'avec ca j'ai gagné environs 30sed dans le meilleurs des cas.
mais bon comme on peut le voir le code fait assez peur en lui même et je cherche toujours des suggestions d'optimisation

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed