Script étape par étape
Hors ligneAnthony Le 27/09/2005 à 20:04 Profil de Anthony Configuration de Anthony

Admin
Quelques petites broutilles sans importance :

lorsque tu définis un chiffre, inutile de mettre des guillemets simples ou doubles :

<?php      $variable=5; ?>

par exemple :-)

Ensuite préfère l'utilisation des guillemets simples aux guillemets doubles. Consulte ceci : Optimiser PHP

a +
--

Hors ligneSteve76 Le 27/09/2005 à 21:38 Profil de Steve76 Configuration de Steve76

mon script est parfait a part cela ???? lol

y doit bien y avoir des fonction plus simple, ou des confirmations d'action a faire (vérification, validation, en cas d'erreur)

des options

ou une simplification
Hors ligneAnthony Le 27/09/2005 à 22:36 Profil de Anthony Configuration de Anthony

Admin
Si tu indentais ton script on y verrait plus clair pour tenter de te dire ce qu'est potentiellement améliorable. Edite ton message et on regardera après :-)
--

Hors ligneSteve76 Le 28/09/2005 à 12:34 Profil de Steve76 Configuration de Steve76

indentais ????

ca veux dire quoi
Hors ligneAnthony Le 28/09/2005 à 15:07 Profil de Anthony Configuration de Anthony

Admin
indenter ça veut dire mettre des tabulations en fonction de ton code pour le rendre lisible.

exemple :

<?php if(fonction()) {      //code } ?>
est plus lisible que

<?php if(fonction()) { //code } ?>
--

Hors ligneSteve76 Le 28/09/2005 à 18:31 Profil de Steve76 Configuration de Steve76

C'est mieux comme ca ?

<HTML> <HEAD> <TITLE>Test</TITLE> </HEAD> <BODY> <?php $nb_aleatoire="4"; $rep ="includes/"; $rep_cache = 'cache/'; $fichier_cache = $rep_cache . basename(dirname($_SERVER["PHP_SELF"])) . '.php'; function creer_fichier_nbjour($fichier,$ExpDate,$NewValue) {      $txt = '<?php      $Value=' . $NewValue . ';      $ExpDate=' . $ExpDate . ';      ?>';      $fp = fopen ($fichier, 'w');      fputs ($fp, $txt);      fclose ($fp);      } $fichier='nbjour.php'; $MustRecreate = false;    if( file_exists($fichier) ) {            include($fichier);           if( time() >= $ExpDate ) {                  $MustRecreate = true;                                  }                                  } else {                                  $MustRecreate = true;                                   } if ($MustRecreate) {        $DateExp=mktime(0,0,0,date('m'),date('d')-1,date('Y'));        $Valeur = mt_rand(1,100);           creer_fichier_nbjour($fichier,$DateExp,$Valeur);           include($fichier);           $txt_fichier = "<?php\n";                     if ($handle = opendir($rep)) {                            $fichiers = array();                            while (false !== ($file = readdir($handle))) {                                                  if ($file != "." && $file != "..") {                                                          $fichiers[] = $rep . $file;                                                           }                                                  }                           closedir($handle);                           }         srand ((double) microtime() * 10000000);         $include_aleatoire = array_rand ($fichiers, $nb_aleatoire);         shuffle($include_aleatoire);         for($i=0;$i<count($include_aleatoire);$i++){                  $txt_fichier .= "include('".$fichiers[$include_aleatoire[$i]]."');\n";                 }      $txt_fichier .= '?>';      $f=fopen($fichier_cache,'w');      fwrite($f,$txt_fichier);      fclose($f);             } ?> crée un chiffre au hazard : <?php echo($Value); ?> <BR> <?php include($fichier_cache); ?> <BR> La prochaine mise a jour se fera le : <? echo (date('r',$ExpDate)); ?> </BODY> </HTML>

est il possible d'avoir des conseil ou critiques pour améliorer ce script
Vous avez résolu votre problème avec VIC ? Faites-le savoir sur les réseaux sociaux !
Vulgarisation-informatique.com
Cours en informatique & tutoriels