Fatal error: Allowed memory size of...

Version imprimable

Bonjour,

Je rencontre un problème bien #@*'!... lors de l'execution du script chez free.
:evil:

voici mon code d'erreur :

Code:

Fatal error: Allowed memory size of 8388608 bytes exhausted (tried to allocate 4063497 bytes)

et voici pour ceux que ça interesse le code ( que vous pouvez critiquer au passage ) :

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
function makeDate() {
 
 $day=date("d");
 $month=date("n");
 $year=date("Y");
 // Mise en forme des jours
 $res="<td><select>";
 
 for ( $i=01; $i<=31; $i++ ) {
  if ( $day == $i ) {
   $res.=$res."<option selected>".$i."</option>\n";
  }
  else {
   $res.=$res."<option>".$i."</option>\n";    
  }
 }
 // Mise en forme des mois
 $res.="</select></td><td><select>";
 $tabMonth=array("Janvier","Fevrier","Mars","Avril","Mai","Juin","Juillet","Aout","Septembre","Octobre","Novembre","Decembre");
 
 for ( $i=1; $i<=12; $i++) {
  if ( $i == $month ) {
   $res.="<option selected>".$tabMonth[$i-1]."</option>";
  }
  else {
   $res.="<option>".$tabMonth[$i-1]."</option>";
  }  
 }
 
 // Mise en forme des années
 $res.="</select></td><td><option selected>".$year."</option>";
  for ( $i=$year+1; $i< $year+4; $i++) {
   $res.="<option>".$i."</option>";
  }
 $res.="</select></td>";
 
 return $res;
 } 
?>

Si vous avez une solution, Merci !

16/09/2005, 21h19
Webfab

En attendant, ma solution est de remplacer toute les concaténations par des echo... bon je sais, c'est vraiment pas propre mais je vois pas

salut,
ton problème vient de la :

Code:

$res.=$res."<option selected>".$i."</option>\n";

tu concatène 2 fois $res ! Une fois avec l'opérateur ".=" et la 2eme fois en mettant $res à droite du ".=" ...
donc remplace ce code par :

Code:

$res.="<option selected>".$i."</option>\n";

ca marchera mieux ;)
(tu as fait cette erreur à 2 endroits)

ensuite évite d'initialiser $i à 01, mieux vaut l'initialiser à 1 vu que c'est un entier ca revient au même ...
et donc au lieu de récupérer le jour sur 2 chiffres avec date("d"), récupère le sur 1 chiffre ou 2 (1 chiffre pour les jours de 1 à 9 ...) avec date("j")

enfin tu oublie le "<select>" pour la liste des années .

Voila le code modifié :
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 function makeDate() { $day=date("j"); $month=date("n"); $year=date("Y"); // Mise en forme des jours $res="<td><select>"; for ( $i=1; $i<=31; $i++ ) { if ( $day == $i ) { $res.="<option selected>".$i."</option>\n"; } else { $res.="<option>".$i."</option>\n"; } } // Mise en forme des mois $res.="</select></td><td><select>"; $tabMonth=array("Janvier","Fevrier","Mars","Avril","Mai","Juin","Juillet","Aout","Septembre","Octobre","Novembre","Decembre"); for ( $i=1; $i<=12; $i++) { if ( $i == $month ) { $res.="<option selected>".$tabMonth[$i-1]."</option>"; } else { $res.="<option>".$tabMonth[$i-1]."</option>"; } } // Mise en forme des années $res.="</select></td><td><select><option selected>".$year."</option>"; for ( $i=$year+1; $i< $year+4; $i++) { $res.="<option>".$i."</option>"; } $res.="</select></td>"; return $res; }

17/09/2005, 10h11
Webfab

Mais bien sur!
merci !