Discussion :

[cristelle1986]

bonjour, j'ai ceci comme code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
function readDataForwards($bdd){
    $sql='SELECT numero, nom_client,date_offre FROM _1offre_de_prix  ORDER BY numero DESC ';
try{
    $stmt=$bdd->prepare($sql,array(PDO::ATTR_CURSOR,PDO::CURSOR_SCROLL));
    $stmt->execute();
    while($row=$stmt->fetch(PDO::FETCH_NUM, PDO::FETCH_ORI_NEXT)){
        $data="- \n Numero offre:\n <font color='#ff1493'>".$row[0]."</font>,\n Nom client:\n <font color='blue'>".$row[1]."</font>,\n Date offre:\n <font color='red'>".$row[2]."</font>. <br>";
        print $data;
    }
    $stmt=null;
    }catch(PDOException $e){
    print $e->getMessage();
}
}
readDataForwards($bdd);

celui-ci est inclu dans une div dans ma page documents.php, mais lorsque je veux remettre la même fonction (avec une modification dans le SELECT pour avoir une autre table) il me le refuse en disant que la fonction existe déjà dans cette page...

Cannot redeclare readDataForwards() (previously declared in C:\wamp\www\Jung-Distribution\affichage_Script_documents_jung.php:18) in C:\wamp\www\Jung-Distribution\affichage_Script_BLF_documents_jung.php on line 24
Call Stack

Comment puis-je faire?
merci de vos réponses

[Celira]

En donnant un autre nom à ta 2e fonction ?

[Bovino]

celui-ci est inclu dans une div dans ma page documents.php

Non, ça n'a strictement rien à voir. PHP ne sait même pas ce qu'est une div.

Formulé correctement, ça donnerait :
"Je mélange tout dans ma page (affichage, traitements, fonctions) et à un endroit, dans de l'affichage HTML, j'essaye de redéfinir une fonction qui existe peut-être déjà dans un autre débarras de mon code".

Tout cela pour te dire qu'il est essentiel d'organiser son code pour le rendre plus facile à lire, plus facile à déboguer et à maintenir.
Si tu faisais de la sorte, tu ne te retrouverais surement pas à en être à ne pas savoir si une fonction a déjà été définie ou non.

Bref, dans ton cas, le mieux serait de réécrire ton code (mais mon petit doigt me dit que tu ne voudras pas...) mais au pire, tu peux utiliser function_exists().

[cristelle1986]

"Je mélange tout dans ma page (affichage, traitements, fonctions) et à un endroit, dans de l'affichage HTML, j'essaye de redéfinir une fonction qui existe peut-être déjà dans un autre débarras de mon code".

Tout cela pour te dire qu'il est essentiel d'organiser son code pour le rendre plus facile à lire, plus facile à déboguer et à maintenir.
Si tu faisais de la sorte, tu ne te retrouverais surement pas à en être à ne pas savoir si une fonction a déjà été définie ou non.

alors non justement mon code est bien clair, dans ma page documents.php je n'ai que 2 div, car le lien est doc.php?page=documents.php
donc je n'ai qu'une <div> avec un include ''; qui est a donc la même fonction readDataForwards!

et ensuite, j'ai une autre <div>avec un autre include ''; avec un readDataForwards mais qui appel une autre table...d'où l'erreur

[Bovino]

...d'où l'erreur

Bizarre, moi j'aurais plutôt conclu avec "CQFD"...
Ce que tu tentes d'expliquer démontre bien que ton code est un bazar sans nom...
Plusieurs fonctions ayant le même nom à des endroits différents ; définir plusieurs fonctions pour faire la même chose à un paramètre près ; une fonction qui allie requête SQL et affichage à même la page...

[Celira]

Tu as donc 2 fonctions différentes qui portent le même nom, qui sont définies dans 2 scripts différents, qui sont eux-même inclus dans un même 3e script.

C'est exactement comme si tu avais une casserole sur une étagère de ton placard et une autre casserole sur une autre étagère du même placard et que tu demandais à un ami qui n'a jamais mis les pieds dans ta cuisine "Prends la casserole dans le placard". Tu vas obtenir comme réponse "Laquelle ?".

Si tes fonctions font des choses différentes, il faut leur donner un nom différent.

[cristelle1986]

bon ok je vais essayer d'être plus clair alors, car je veux bien lui donner un autre nom mais là il va falloir m'aiguiller vu que la fonction est: readDataForwards () et donc je pense que l'on ne peut pas la changer...

voilà mon documents ou je veux afficher:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
<?php
session_start();
?>
<p>page de documents.php</p>
 
 
<div>
<h3>Affichage des 5 dernières offres créé.</h3>
    <?php
    include 'affichage_Script_documents_jung.php';
 
    ?>
 
</div>
<div>
    <h3>Affichage des 5 derniers BLF créé.</h3>
    <?php
    include 'affichage_Script_BLF_documents_jung.php';
    ?>
 
</div>

affichage_Script_documents_jung.php:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
<?php
include 'bdd.php';
 
function readDataForwards($bdd){
    $sql='SELECT numero, nom_client,date_offre FROM _1offre_de_prix  ORDER BY numero DESC ';
try{
    $stmt=$bdd->prepare($sql,array(PDO::ATTR_CURSOR,PDO::CURSOR_SCROLL));
    $stmt->execute();
    while($row=$stmt->fetch(PDO::FETCH_NUM, PDO::FETCH_ORI_NEXT)){
        $data="- \n Numero offre:\n <font color='#ff1493'>".$row[0]."</font>,\n Nom client:\n <font color='blue'>".$row[1]."</font>,\n Date offre:\n <font color='red'>".$row[2]."</font>. <br>";
        print $data;
    }
    $stmt=null;
    }catch(PDOException $e){
    print $e->getMessage();
}
}
readDataForwards($bdd);
?>

affichage_Script_BLF_documents_jung.php:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<?php
include 'bdd.php';
 
 
function readDataForwards($bdd){
    $sql='SELECT numero, date, nom  FROM _2blf  ORDER BY numero DESC ';
    try{
        $stmt=$bdd->prepare($sql,array(PDO::ATTR_CURSOR,PDO::CURSOR_SCROLL));
        $stmt->execute();
        while($row=$stmt->fetch(PDO::FETCH_NUM, PDO::FETCH_ORI_NEXT)){
            $data="- \n Numero BLF:\n <font color='#ff1493'>".$row[0]."</font>,\n Nom client:\n <font color='blue'>".$row[1]."</font>,\n Date BLF:\n <font color='red'>".$row[2]."</font>. <br>";
            print $data;
        }
        $stmt=null;
    }catch(PDOException $e){
        print $e->getMessage();
    }
}
readDataForwards($bdd);
?>

alors je pensais faire ceci dans le même fichier Script php pour éviter d'en avoir 2 et utiliser qu'une seule fois cette fonction readDataForwards() ==>

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
<?php
include 'bdd.php';
 
function readDataForwards($bdd){
    $sql='SELECT *  FROM _2blf, _1offre  ORDER BY numero DESC ';
    try{
        $stmt=$bdd->prepare($sql,array(PDO::ATTR_CURSOR,PDO::CURSOR_SCROLL));
        $stmt->execute();
        while($row=$stmt->fetch(PDO::FETCH_NUM, PDO::FETCH_ORI_NEXT)){
            $data="- \n Numero BLF:\n <font color='#ff1493'>".$row[0]."</font>,\n Nom client:\n <font color='blue'>".$row[1]."</font>,\n Date BLF:\n <font color='red'>".$row[2]."</font>. <br>";
            print $data;
        }
        $stmt=null;
    }catch(PDOException $e){
        print $e->getMessage();
    }
}
readDataForwards($bdd);
?>

le seul soucis c'est pour le print $data;, vu que je veux dans la premier <div> de documents.php que ce ne soit que les offres qui s'y affiche (les 5 premières) et dans l'autre <div> que ce soit les BLF qui s'y affiche (les 5 premiers également)...

donc je ne vois pas comment le faire en un seul fichier

[Celira]

Je ne vois pas pourquoi tu ne pourrais pas changer le nom de la fonction. Il faut juste bien faire attention à modifier également l'appel à la fonction.
Par exemple pour ton script affichage_Script_BLF_documents_jung.php, tu peux appeler ta fonction readForwardBLF :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<?php
include 'bdd.php';
 
 
function readDataForwardsBLF($bdd){
    $sql='SELECT numero, date, nom  FROM _2blf  ORDER BY numero DESC ';
    try{
        $stmt=$bdd->prepare($sql,array(PDO::ATTR_CURSOR,PDO::CURSOR_SCROLL));
        $stmt->execute();
        while($row=$stmt->fetch(PDO::FETCH_NUM, PDO::FETCH_ORI_NEXT)){
            $data="- \n Numero BLF:\n <font color='#ff1493'>".$row[0]."</font>,\n Nom client:\n <font color='blue'>".$row[1]."</font>,\n Date BLF:\n <font color='red'>".$row[2]."</font>. <br>";
            print $data;
        }
        $stmt=null;
    }catch(PDOException $e){
        print $e->getMessage();
    }
}
readDataForwardsBLF($bdd);
?>

[cristelle1986]

ah génial merci je ne savais pas que je pouvais la renommer je pensais que c'était impossible....
ca fonctionne nikel...

[Celira]

Tant que ce n'est pas un code qui est généré par autre chose (auquel cas ta modif risque d'être écrasée par la prochaine génération) ou un morceau d'un framework (auquel cas ça peut problème avec les montées de version du framework), tu peux faire ce que tu veux avec ton code.

Si tu as envie d'appeler ta fonction laMereMichelleChercheSonChat ou toto_fait_du_velo ou même f, tu peux le faire.