[Système] Cannot redeclare mafonction() (previously declared)

Version imprimable

Bonjour,

À l'intérieur d'une classe, j'ai déclaré la fonction suivante :

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
 
/**
 * Trie les enregistrements selon la colonne donnée
 * Paramètres : string $col (Nom de la colonne à trier)
 **/
    function trier($col) {
      //Le paramètre est passé en variable globale pour atteindre la fonction ci-dessous
      $GLOBALS["col"] = $col;
      //Fonction servant à comparer deux véhicules
      function lineicmp($c1, $c2) {
        switch($GLOBALS["col"]) {
          case "id"          : return $c1->getId()       - $c2->getId();
          case "id_emp"      : return $c1->getEmp()      - $c2->getEmp();
          case "ddq"         : return strcasecmp($c1->getDdq()  , $c2->getDdq());
          case "hdq"         : return strcasecmp($c1->getHdq()  , $c2->getHdq());
          case "odt"         : return $c1->getOdt()      - $c2->getOdt();
          case "ort"         : return $c1->getOrt()      - $c2->getOrt();
          case "dfq"         : return strcasecmp($c1->getDfq()  , $c2->getDfq());
          case "hfq"         : return strcasecmp($c1->getHfq()  , $c2->getHfq());
          case "id_route"    : return $c1->getRoute()    - $c2->getRoute();
          case "id_vehicule" : return $c1->getVehicule() - $c2->getVehicule();
        }
      }
      usort($this->c, "lineicmp");
    }

J'ai une autre fonction qui va appeler à deux reprises la fonction trier, ce qui cause me message d'erreur suivant :

Code:

1
2
 
Fatal error: Cannot redeclare lineicmp() (previously declared) in /var/www/html/monrepertoite/class/cartes.php:453) in /var/www/html/monrepertoire/class/cartes.php on line 453

Je suppose que c'est probablement que j'utilise mal la notion de callback. Faudrait-il que je sorte ma fonction lineicmp() de trier()?

15/09/2006, 16h55
berceker united

Je crois que ça arrive parce que tu as du surement faire un copier/Collier de la fonction avec le mot clés devant "function trier" au lieu de trier tous cour.

Voici ma fonction qui appelle trier. Je ne crois pas qu'il y aie une erreur de syntaxe. De toute façon, le problème concerne la fonction lineicmp(). Je ne sais pas trop, je me doute que la solution est probablement très simple mais ce n'est pas ça.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 /** * Renvoie un tableau contenant toutes les cartes débutant un jour donné * Paramètre : string $date **/ function getCarteJour($date) { $cartes = array(); $this->trier("hdq"); $this->trier("ddq"); foreach($this->c as $id => $carte) { if($carte["ddq"] = $date) array_push($cartes, $this->c[$id]); } return $cartes; }

15/09/2006, 17h04
guitou12

Ta fonction lineicmp() est tout simplement déclarée 2 fois et la première fois c'était ici --> cartes.php:453
15/09/2006, 17h14
FrankOVD

La fonction lineicmp() n'apparaît qu à la ligne 453. Nulle part ailleurs, ni dans aucun autre fichier inclus. Et même si je n'appelle qu'une seule fois ma fonction trier le problème reste entier.
15/09/2006, 17h16
FCYPBA

On peut déclarer une fonction dans une fonction et tout ceci dans une classe ???
15/09/2006, 17h20
berceker united

Trier est appelé combien de fois? car la fonction lineicmp() est déclaré à l'intérieur de Trier. Apres elle rappeler dans le unsort. Je crois qu'il y a un os à ce niveau là.
15/09/2006, 17h20
berceker united

Citation:

Envoyé par FCYPBA

On peut déclarer une fonction dans une fonction et tout ceci dans une classe ???

Oui. Et je crois qu'il est possible de déclarer une classe dans une fonction déclarer une classe dans une methode d'une classe.
15/09/2006, 17h25
guitou12

Citation:

Envoyé par berceker united

Oui. Et je crois qu'il est possible de déclarer une classe dans une fonction déclarer une classe dans une methode d'une classe.

Les déclarations de fonctions dans d'autres fonctions ca donne vraiment envie de :vomi: , faut pas s'étonner d'avoir des bugs ensuite :mouarf:
15/09/2006, 17h27
FrankOVD

J'ai déjà utilisé à plusieurs reprises cette façon de faire mais je n'ai malheureusement pas accès à mes sources d'où je suis. Cependant je sais qu'il est possible d'appeler plusieurs fois une fonction structurée comme je tente de la faire pour trier(). Il doit y avoir quelque chose que je fais mal dans la déclaration de cette fonction.
15/09/2006, 17h30
FrankOVD

Citation:

Envoyé par guitou12

Les déclarations de fonctions dans d'autres fonctions ca donne vraiment envie de :vomi: , faut pas s'étonner d'avoir des bugs ensuite :mouarf:

J'ai déjà utilisé des classes avec plusieurs fonctions qui faisaient un travail comme trier() dans la même classe. Si je déclarais ma fonction en dehors, je devrais toutes les baptiser différemment alors qu'elles sont toutes semblables. Je ne déclare jamais de fonction à l'intérieur d'une fonction excepté pour les callback.
15/09/2006, 17h32
FCYPBA

Je pense que le problème est surement lié à l'endroit ou ta fonction est déclarée.

Et je n'arrive pas à comprendre pourquoi tu la déclare à cet endroit précis.

Pierre
15/09/2006, 17h35
berceker united

Ce que je ne comprend pas c'est cette ligne là.

Code:

usort($this->c, "lineicmp");
15/09/2006, 17h36
FCYPBA

Avec ce code, le tableau c de la classe est trié à l'aide la fonction lineicmp

C'est le B-a-Ba dans le traitement des tableaux
15/09/2006, 17h38
FrankOVD
Okay j'explique.

Dans ma fonction trier(), j'appelle la fonction usort() qui nécessite que je lui fournisse le nom de ma fonction qui comparera mes données. Cette dernière fonction n'est pas vue par usort() si elle est à l'extérieur du bloc logique que représente la fonction trier().

J'ai essayé de la mettre à l'extérieur.
Code:

1 2 Warning: usort() [function.usort]: Invalid comparison function. in /var/www/html/cartetemps4/class/cartes.php on line 483
15/09/2006, 17h40
FCYPBA

Okay pour l'emplacement de la déclaration.

Maintenant, peut etre faut-il détruire la fonction de tri après le usort() .

Peut-on détruire une fonction ???
15/09/2006, 17h46
berceker united
Citation:
Envoyé par FrankOVD

Okay j'explique.

Dans ma fonction trier(), j'appelle la fonction usort() qui nécessite que je lui fournisse le nom de ma fonction qui comparera mes données. Cette dernière fonction n'est pas vue par usort() si elle est à l'extérieur du bloc logique que représente la fonction trier().

J'ai essayé de la mettre à l'extérieur.

Code:

1 2 Warning: usort() [function.usort]: Invalid comparison function. in /var/www/html/cartetemps4/class/cartes.php on line 483
Effectivement je comprend pourquoi tu es obligé de le mettre la déclaration d'une fonction à l'intérieur d'une methode. De ce faite il se pourrait que ceci rende les choses incompatible. Je sais que la fonction déclarer dans la methode ne sont visible que pour la méthode. Quoi c'est a vérifier mais il y a pas de raison.
15/09/2006, 19h25
FrankOVD
J'ai essayé de déclarer ma fonction comme privée mais ça n'a pas fonctionné non plus.
Code:

1 2 Parse error: parse error, unexpected T_PRIVATE in /var/www/html/cartetemps4/class/cartes.php on line 453
15/09/2006, 19h27
FrankOVD

Citation:

Envoyé par berceker united

De ce faite il se pourrait que ceci rende les choses incompatible.

Tu peux détailler un petit peu?
15/09/2006, 20h13
FCYPBA

Je pense qu'il veut dire que tu ne pourrais pas utiliser usort au sein d'une méthode d'une classe.

Tiens j'en ai une qui marche ici, déclarée au beau milieu d'une classe. Je l'ai trouvée dans un de mes sources d'une application fonctionnelle et terminée.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
 
function sortdate($col) {
  $GLOBALS["sortcol"] = $col;
  //Compare deux lignes d'un tableau
  function linecmp($line1, $line2) { 
    $a = $line1[$GLOBALS["sortcol"]];
    $b = $line2[$GLOBALS["sortcol"]];
    if($a < $b) return -1;
    elseif($a > $b) return 1;
    else return 0;				
  }
  usort($this->ts, "linecmp");
}

Et celle là c'est celle qui ne marche pas :

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
 
/**
     * Trie les cartes selon la colonne donnée
     * Paramètres : string $col (Nom de la colonne à trier)
     **/
    function trier($col) {
      //Le paramètre est passé en variable globale pour atteindre la fonction ci-dessous
      $GLOBALS["col"] = $col;
      //Fonction servant à comparer deux véhicules
      function lineicmp($c1, $c2) {
        switch($GLOBALS["col"]) {
          case "id"          : return $c1->getId()       - $c2->getId();
          case "id_emp"      : return $c1->getEmp()      - $c2->getEmp();
          case "ddq"         : return strcasecmp($c1->getDdq()  , $c2->getDdq());
          case "hdq"         : return strcasecmp($c1->getHdq()  , $c2->getHdq());
          case "odt"         : return $c1->getOdt()      - $c2->getOdt();
          case "ort"         : return $c1->getOrt()      - $c2->getOrt();
          case "dfq"         : return strcasecmp($c1->getDfq()  , $c2->getDfq());
          case "hfq"         : return strcasecmp($c1->getHfq()  , $c2->getHfq());
          case "id_route"    : return $c1->getRoute()    - $c2->getRoute();
          case "id_vehicule" : return $c1->getVehicule() - $c2->getVehicule();
          default            : return 0
        }
      }
      usort($this->c, "lineicmp");
    }

15/09/2006, 20h31
FCYPBA

Le code est sensiblement similaire, en tout cas il n'y a pas de différence à la déclaration ou a l'utilisation.

C'est donc que le problème vient d'ailleurs :
- La classe ??
- La manière d'utiliser la fonction

Pierre
15/09/2006, 21h01
FrankOVD

J'ai regardé au niveau de la déclaration de la classe et c'est similaire aussi. Il me reste à espérer qu'il y ait un petit quelque chose de spécial dans la façon d'appeler la fonction ce qui sincèrement me surprendrais.

Je vérifie tout de même...
15/09/2006, 21h04
FrankOVD

Hmm, l'appel se fait de la même façon.

Je commence à désespérer.
15/09/2006, 21h38
Djakisback
Salut,
peut-être que tu appelles une seul fois par script ta fonction sortdate().
C'est vrai que c'est bizzare, il me semble aussi que la portée devrait se limiter à la méthode. Dans le pire des cas tu peux la mettre à l'extérieur en static.

[edit] Euh, oublie le static, t'auras pas accès aux membres, je dis nimp :oops:
Pour info ceci ne fonctionne pas chez moi :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 class Test { function truc() { function truc2() { } } } $t = new Test(); $t->truc(); $t->truc();
[/edit]
15/09/2006, 22h29
FrankOVD

Ça dit quoi?
15/09/2006, 22h32
Djakisback

Comme toi au 2e appel de truc().
T'es sur que tu n'appelle pas qu'une seule fois ta fonction dans ton appli qui fonctionne?
15/09/2006, 22h46
FrankOVD

Ouais je ne l,appelle qu'une seule fois, mais dans mon application qui ne fonctionne pas aussi (J'ai commenté le 2e appel) et ça ne fonctionne toujours pas.
18/09/2006, 15h06
FrankOVD

Peut-être que ce pourrait être dû au fait que la version de PHP ici est antérieure à celle avec laquelle j'avais développé mon autre application. Je sais que l'utilisation de la POO en PHP n'existe pas depuis des lustres, peut-être que c'ette possibilité est nouvelle.
22/09/2006, 22h15
FrankOVD

Je fonctionne avec PHP 5.0.4 et Apache 2.0.54 ça devrait marcher vous croyez?
22/09/2006, 23h33
kankrelune

Essaye en utilisant create_function() dans ta methode... .. .

@ tchaOo°

Ah! C'est plutôt intéressant comme alternative.... et je pourrais y intégrer toute une structure switch?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
function lineicmp($c1, $c2) {
        switch($GLOBALS["col"]) {
          case "id"          : return $c1->getId()       - $c2->getId();
          case "id_emp"      : return $c1->getEmp()      - $c2->getEmp();
          case "ddq"         : return strcasecmp($c1->getDdq()  , $c2->getDdq());
          case "hdq"         : return strcasecmp($c1->getHdq()  , $c2->getHdq());
          case "odt"         : return $c1->getOdt()      - $c2->getOdt();
          case "ort"         : return $c1->getOrt()      - $c2->getOrt();
          case "dfq"         : return strcasecmp($c1->getDfq()  , $c2->getDfq());
          case "hfq"         : return strcasecmp($c1->getHfq()  , $c2->getHfq());
          case "id_route"    : return $c1->getRoute()    - $c2->getRoute();
          case "id_vehicule" : return $c1->getVehicule() - $c2->getVehicule();
          default            : return 0;
        }
      }

26/09/2006, 09h40
kankrelune

Citation:

Envoyé par FrankOVD

et je pourrais y intégrer toute une structure switch?

Biensur... c'est chiant à écrire mais c'est faisable... .. .

@ tchaOo°