Erreur time out code

Version imprimable

Bonjour, j'ai le message d'erreur suivant qui survient lors que j'execute ce scripts ci-bas :

Citation:

Fatal error: Maximum execution time of 90 seconds exceeded in I:\phplogiciel\ArchiveDirm\chargement\chargement_radio.php on line 35

Voici mon 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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
<?
include("../connexion/connexion_bd.php");
 
//chargement de la table radio
 
		echo "	*************************** Debut du Chargement des patients ******************************	";
		?> <br/><?
		$requete = mysql_query("SELECT num_radio, nom_pat, prenom_pat,nomjf_pat, datenaiss_pat,Sexe_pat, Cliche_pat, Secteur,Date_radio FROM chargement_radio") OR die("erreur requete 1er SELECT");
 
 
		while ($R_requete = mysql_fetch_array($requete)) { 
 
			// Chargement des données dans les variables
			$numradio = $R_requete['num_radio'];
			$nom = $R_requete['nom_pat'];
			$nomjf = $R_requete['nomjf_pat'];
			$sexe = $R_requete['Sexe_pat'];
			$prenom = $R_requete['prenom_pat'];
			$datenaiss = $R_requete['datenaiss_pat'];
			$cliche = $R_requete['Cliche_pat'];
			$secteur = $R_requete['Secteur'];
			$Date_radio = $R_requete ['Date_radio'];
 
 
 
			//Vérification du sexe des patients
			if (ereg("1",@$sexe)) { $sexe = 'M'; }
			elseif (ereg("2",@$sexe)) { $sexe = 'F'; }
			else { $sexe = 'NULL'; }
 
 
 
			// Détection des doublons
 
			$R_patient = mysql_query("SELECT * FROM patients WHERE nom='$nom' AND prenom='$prenom' AND date_naiss=str_to_date('$datenaiss','%Y-%m-%d')") OR die("erreur requete 2ième SELECT");
			$annee = substr($datenaiss, -4);
			// Si pas de doublon . . .
			if (mysql_num_rows($R_patient) == 0)   {			
				mysql_query("INSERT INTO patients(ipp, nom, prenom, nom_de_jf, date_naiss,annee_naiss, ancienne_date, dernier_soin, sexe, decede, precisions ) VALUES( NULL, '$nom', '$prenom', '$nomjf', str_to_date('$datenaiss','%Y-%m-%d'), '$annee', ' ', ' ', '$sexe', ' ', ' ')") OR die(mysql_error());
			}
				?> <br/><?
				echo "$nom";
}		
 
 
		echo "	*************************** Fin de l'enregistrement des nouveaux patients ******************************	";
 
?>

Je ne trouve pas que mon code est trop complexe, peut-etre une optimisation que je n'ai pas vu ?

Merci d'avance,

Anthony

11/01/2010, 14h56
CinePhil

Le code n'est pas particulièrement complexe mais le traitement qu'il effectue peut être très long s'il y a beaucoup de lignes dans les tables radio et patient.
11/01/2010, 15h00
Anthony_

Merci de ta rapidité,

J'ai 3376 entrées dans les radios et 83074 dans les patients,

Cela explique-t-il la lenteur d'execution du script ?

Mais ce que je ne comprend pas, c'est que cela fonctionne sur un serveur test.

Citation:

Envoyé par Anthony_

J'ai 3376 entrées dans les radios

Cette requête va dons s'exécuter 3376 fois :
Code:

1 2 3 4 5 SELECT * FROM patients WHERE nom='$nom' AND prenom='$prenom' AND date_naiss = str_to_date('$datenaiss','%Y-%m-%d')
Les colonnes nom et prenom sont-elles indexées ?

Le fait qu'il y ait une conversion en date à faire dans la requête n'arrange pas les choses. Tu devrais faire la conversion en PHP et envoyer la variable formatée 'aaa-mm-jj' dans la requête.

S'il n'y a pas beaucoup de "doublons", il y aura beaucoup d'insertions parmi les 3376 fois.
Avec à chaque fois une réorganisation de l'index de la table.

Il serait plus sage de faire une jointure entre les deux tables pour trouver les patients de la table radio non présents dans la table patient et d'injecter le résultat dans la table patients :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 CREATE TEMPORARY TABLE temp SELECT r.nom_pat, r.prenom_pat, r.nomjf_pat, STR_TO_DAT(r.datenaiss_pat, '%Y-%m-%d') AS date_naiss_pat, EXTRACT(YEAR FROM STR_TO_DAT(r.datenaiss_pat, '%Y-%m-%d')) AS r.annee_naiss_pat, r.Sexe_pat FROM radio AS r LEFT OUTER JOIN patients AS p ON r.nom_pat = p.nom AND r.prenom_pat = p.prenom_pat AND STR_TO_DAT(r.datenaiss_pat, '%Y-%m-%d') = p.date_naiss WHERE p.ipp IS NULL; INSERT INTO patients(nom, prenom, nom_de_jf, date_naiss, annee, sexe) SELECT nom_pat, prenom_pat, nomjf_pat, date_naiss_pat, annee_naiss_pat, Sexe_pat FROM temp
J'ai supposé ci-dessus que ipp est l'identifiant du patient.

A noter qu'en php, ces deux requêtes doivent être exécutées successivement par la même connexion et grâce à deux instructions mysql_query.

Dans phpMyAdmin, les deux requêtes (la première se terminant par un point-virgule) peuvent être directement entrées dans la fenêtre SQL et lancées.