PHPfrance

la communauté d'entraide francophone dédiée au PHP

Probleme de script

11 messages   •   Page 1 sur 1
   Outils de sujet
stever
Eléphanteau du PHP | 36 Messages

08 sept. 2006, 14:26

Bonjour cela fais 3 jours que je suis sur ce script et le tourne en boucle je commence a me mélanger grave.

Si je pouvez avoir vos avis extérieurs

Code : Tout sélectionner

class parser { var $filename; function parser($filename) { $this->filename=$filename; } function get_nom_commune() { $file=fopen($this->filename,"r"); if (!$file) { echo "<p>Impossible d'ouvrir le fichier</p>.\n"; exit; } $rechercher = 0; while (!feof ($file)) { $ligne=fgetss($file,4096); if ( strstr($ligne,'Nom de la commune') <> "") $rechercher = 1; // Si on trouve donneesprincipales, on peut commencer la recherche des elements if ( $rechercher == 1 ) { echo $lnb,' '; if ( strstr($ligne,'Nom de la commune')) {$nomcommune = substr(strstr($ligne,'Nom de la commune'),17);} if ( strstr($ligne,'Région')) {$region = substr(strstr($ligne,'Région'),6);} if ( strstr($ligne,'Population')) {$population = substr(strstr($ligne,'Population'),10);} if ( strstr($ligne,'Altitude')) {$altitude = substr(strstr($ligne,'Altitude'),8);} $rechercher = 0; // On arrete la recherche apres cette ligne echo $region.$population; } } } }
ViPHP
ViPHP | 3607 Messages

08 sept. 2006, 14:38

bonjour,
Où est le problème, sur quoi veux tu que nous donnions un avis?
Il ne marche pas ce script? qu'es-ce qu'il est censé faire ?
Eléphanteau du PHP | 36 Messages

08 sept. 2006, 16:41

c'est au niveau des { }
je n'arrive plus a le smettre au bon endroit
Mammouth du PHP | 19672 Messages

08 sept. 2006, 16:48

Aligne ton code de façon à pouvoir le suivre sinon tu vas te perdre dns le code, voici ton code simplement ermis en forme sans la moindre modification: 
<?php
class parser
{
    var $filename;

    function parser($filename)
    {
        $this->filename=$filename;
    }

    function get_nom_commune()
    {
        $file=fopen($this->filename,"r");
        if (!$file)
        {
            echo "<p>Impossible d'ouvrir le fichier</p>.\n"; 
            exit; 
        }
        $rechercher = 0;
        while (!feof ($file)) 
        {
            $ligne=fgetss($file,4096);

            if ( strstr($ligne,'Nom de la commune') <> "")
            {
                $rechercher = 1;
            }
            // Si on trouve donneesprincipales, on peut commencer la recherche des elements
            if ( $rechercher == 1 ) 
            {
                echo $lnb,' ';
                if ( strstr($ligne,'Nom de la commune'))
                {
                    $nomcommune = substr(strstr($ligne,'Nom de la commune'),17);
                }
                if ( strstr($ligne,'Région'))
                {
                    $region = substr(strstr($ligne,'Région'),6);
                }
                if ( strstr($ligne,'Population'))
                {
                    $population = substr(strstr($ligne,'Population'),10);
                }
                if ( strstr($ligne,'Altitude'))
                {
                    $altitude = substr(strstr($ligne,'Altitude'),8);
                }
                $rechercher = 0; // On arrete la recherche apres cette ligne
                echo $region.$population;
            }
        }
    }
}
?>
Codez en pensant que celui qui maintiendra votre code est un psychopathe qui connait votre adresse :axe:
ViPHP
ViPHP | 3607 Messages

08 sept. 2006, 16:50

Il faut essayer de mieu indenter ton code:
je l'ai fait pour toi, mais je ne vois pas d'accolades manquantes?
Qu'est-ce qui ne fonctionne pas?
voici le code réindenté
class parser {
  var $filename;

  function parser($filename) {
    $this->filename=$filename;
  }

  function get_nom_commune() {
    $file=fopen($this->filename,"r");
    if (!$file) {
      echo "<p>Impossible d'ouvrir le fichier</p>.\n"; 
      exit; 
    }
    $rechercher = 0;
    while (!feof ($file)) {
     $ligne=fgetss($file,4096);
      if ( strstr($ligne,'Nom de la commune') <> "") 
        $rechercher = 1;
        // Si on trouve donneesprincipales, on peut commencer la recherche des elements
      if ( $rechercher == 1 ) {
        echo $lnb,' ';
        if ( strstr($ligne,'Nom de la commune')){
          $nomcommune = substr(strstr($ligne,'Nom de la commune'),17);
        }
        if ( strstr($ligne,'Région')){
          $region = substr(strstr($ligne,'Région'),6);
        }
        if ( strstr($ligne,'Population')){
          $population = substr(strstr($ligne,'Population'),10);
        }
        if ( strstr($ligne,'Altitude')){
          $altitude = substr(strstr($ligne,'Altitude'),8);
        }
        $rechercher = 0; // On arrete la recherche apres cette ligne
        echo $region.$population;
      }                       
    }
  }
}
EDIT: je me suis fait doubler :oops: et en plus ton code est plus lisible :cry:
Modifié en dernier par jojolapine le 08 sept. 2006, 16:51, modifié 1 fois.
Mammouth du PHP | 1776 Messages

08 sept. 2006, 16:51 

class parser { 

	var $filename; 
	function parser($filename) 
	{ 
		$this->filename=$filename; 
	} 
	
	function get_nom_commune() 
	{ 
		$file=fopen($this->filename,"r"); 
		if (!$file) 
		{ 
			echo "<p>Impossible d'ouvrir le fichier</p>.\n"; 
			exit; 
		} 
		$rechercher = 0; 
		while (!feof ($file)) 
		{ 
			$ligne=fgetss($file,4096); 
			if (strstr($ligne,'Nom de la commune') <> "") 
				$rechercher = 1; 
			// Si on trouve donneesprincipales, on peut commencer la recherche des elements 
			if ($rechercher == 1) 
			{ 
				echo $lnb,' '; 
				if (strstr($ligne,'Nom de la commune')) 
					$nomcommune = substr(strstr($ligne,'Nom de la commune'),17);
				if (strstr($ligne,'Région')) 
					$region = substr(strstr($ligne,'Région'),6);
				if (strstr($ligne,'Population')) 
					$population = substr(strstr($ligne,'Population'),10);
				if (strstr($ligne,'Altitude'))
					$altitude = substr(strstr($ligne,'Altitude'),8);
				$rechercher = 0; // On arrete la recherche apres cette ligne 
				echo $region.$population; 
			}
		}
	}
}
Mammouth du PHP | 19672 Messages

08 sept. 2006, 16:52

lol, vous êtes pas rapides les gars :langue:

Il y en a pas un autre qui veut mettre sa version des fois ? :roll:
Codez en pensant que celui qui maintiendra votre code est un psychopathe qui connait votre adresse :axe:
ViPHP
ViPHP | 3607 Messages

08 sept. 2006, 16:52

à trois dessus on a plus de chance qu'il adopte l'une ou l'autre mise en page :lol:
Mammouth du PHP | 1776 Messages

08 sept. 2006, 16:55

lol, vous êtes pas rapides les gars :langue:

Il y en a pas un autre qui veut mettre sa version des fois ? :roll:
^_^
Mammouth du PHP | 19672 Messages

08 sept. 2006, 16:58

Allez, pour faire bonne mesure, j'en remets une couche, la même en encore mieux et avec une vision à plus long terme : 
<?php
/**
 * Inscrire ici la description...
 *
 */
class parser
{
    /**
     * Inscrire ici la description...
     *
     * @var unknown_type
     */
    var $filename;

    /**
     * Inscrire ici la description...
     *
     * @param  String  $filename
     * @return parser
     * @access public
     */
    function parser($filename)
    {
        $this->filename=$filename;
    }

    /**
     * Inscrire ici la description...
     *
     * @access public
     */
    function get_nom_commune()
    {
        $file=fopen($this->filename,"r");
        if (!$file)
        {
            echo "<p>Impossible d'ouvrir le fichier</p>.\n"; 
            exit; 
        }
        $rechercher = 0;
        while (!feof ($file)) 
        {
            $ligne=fgetss($file,4096);

            if ( strstr($ligne,'Nom de la commune') <> "")
            {
                $rechercher = 1;
            }
            // Si on trouve donneesprincipales, on peut commencer la recherche des elements
            if ( $rechercher == 1 ) 
            {
                echo $lnb,' ';
                if ( strstr($ligne,'Nom de la commune'))
                {
                    $nomcommune = substr(strstr($ligne,'Nom de la commune'),17);
                }
                if ( strstr($ligne,'Région'))
                {
                    $region = substr(strstr($ligne,'Région'),6);
                }
                if ( strstr($ligne,'Population'))
                {
                    $population = substr(strstr($ligne,'Population'),10);
                }
                if ( strstr($ligne,'Altitude'))
                {
                    $altitude = substr(strstr($ligne,'Altitude'),8);
                }
                $rechercher = 0; // On arrete la recherche apres cette ligne
                echo $region.$population;
            }
        }
    }
}
?>
Et là, on peut même sortir une documentation avec phpDocumentor ;)
Codez en pensant que celui qui maintiendra votre code est un psychopathe qui connait votre adresse :axe:
Mammouth du PHP | 1776 Messages

08 sept. 2006, 17:02

lol Cyrano :langue:
Moi je vais abdiquer, car je dois aller prendre mon train, donc bonne soirée et bon code ^^
   Répondre
11 messages   •   Page 1 sur 1
   Outils de sujet