[C++] Fonctions

Fermé
Oni - 14 nov. 2010 à 15:48
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 - 14 nov. 2010 à 16:35
Bonjour, Une question me torture l'esprit :

voici un exemple de fontion basique :

    int carPresent(char tab, int nb, char unCar)
    {
        const char MAX = 10;
        char tab[MAX];
        int nb, i;
        char unCar;
        bool present;

        i = 1;
        present = false;

        while ((i <= nb) && (present == false))
        {
            present = (unCar == tab[i]);
            i++;
        }

        return present;
    }


Cette fonction a pour but de déterminer si un caractère est présent ou non dans une chaine de caractères (entré au préalable dans la fonction main via un cin.getline), en l'occurence, si unCar est dans tab ou non (je présice aussi que dans le main, nb = strlen(chaine1), chaine1 correspondant à tab dans l'appel de la fonction)

Lors de la compilation, l'erreur suivante survient : "Warning : shadows a parameter"

Apres quelques recherches, on me dit que c'est parce que les variables d'une fonction n'ont pas le droit de de porter le même nom que les paramètres de cette même fonction.
En changeant les noms de variables ou de paramètres, ça compile, mais le programme ne marche pas comme il faut : c'est normal, les paramètres que je manipule, je ne sais même pas si c'est eux ou pas qui sont manipulés dans la fonction !

Ma question est : comment faire pour que l'ordi sache quels paramètres je veux manipuler sans leur donner le même nom que les variables ?
En fait ça me paraissait logique que en donnant les mêmes noms de variables qu'aux paramètres, les variables que je manipulent correspondent à ces paramètres, donc comment faire si je change les noms pour que l'ordi sache de quoi je parle?

J'éspère que je me suis fait comprends, j'ai eu du mal à tourner mes phrases^^

Je vous remercie d'avance,

Oni.

3 réponses

Arya Dröttningu Messages postés 581 Date d'inscription mardi 12 janvier 2010 Statut Membre Dernière intervention 3 janvier 2019 157
14 nov. 2010 à 15:52
Bonjour,
J'ai pas tout tout compris......
Mais en regardant le code j'ai vu ça const char MAX = 10; ce ne serait pas plutôt int à la place de char ?
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844
14 nov. 2010 à 16:02
Tu utilises mal le principe des fonctions.

Le prototype est : int carPresent(char tab, int nb, char unCar)
Cela veut dire que lorsque tu appelleras la fonction, il y aura les variables : char tab, int nb et char unCar qui seront créées au sein de ta fonction.
En aucun cas, il faut redéfinir ces variables.

Pire encore, tu redéfinis : char tab[MAX]; alors que dans la fonction c'est char tab...

Pour corriger, il faut changer le prototype en : int carPresent(char *tab, int nb, char unCar) et enlever dans le corps de la fonction les déclarations des variables tab, nb et unCar.

Dernier point : bool present; et c'est la valeur que tu renvoies. Il faut donc que la fonction soit bool carPresent ou alors bool present; doit devenir int present;

Cordialement,
0
@Arya : Ah oui, erreur d'inattention ^^

@fiddy : J'ai pas tout compris ^^ pourquoi dans mon prototype char *tab, pourquoi l'étoile ? Ce ne serait pas un pointeur par hasard ? Je ne les ai pas encore vu et je ne pense pas que j'y aurais droit (cet exercice est un exo d'entrainement à mes partiels qui approchent).

Mais plus bizarre encore : si j'enlève les déclarations des variables tab, nb et unCar, comment ma fonction pourrait-elle marcher ? Je peux manipuler directement les paramètres?

Et pourquoi changer le type bool en int a "present" ? C'est bien une valeure vraie ou fausse que je veux retourner non ?

Cordialement.
0
fiddy Messages postés 11069 Date d'inscription samedi 5 mai 2007 Statut Contributeur Dernière intervention 23 avril 2022 1 844
14 nov. 2010 à 16:35
Tu peux mettre char tab[] à la place de char *tab ou même char tab[MAX]
C'est la même chose. En C, on ne peut pas envoyer le tableau en argument, on n'envoie en fait qu'un pointeur (mais t'es pas censé le savoir encore ;-)))).
Mais en aucun cas tu peux mettre char tab dans ton prototype.

j'enlève les déclarations des variables tab, nb et unCar, comment ma fonction pourrait-elle marcher ?
Car c'est comme ça que ça marche ^^. Tu déclares déjà tes variables dans ton prototype, il ne faut donc pas les redéclarer.

Et pourquoi changer le type bool en int a "present" ? C'est bien une valeure vraie ou fausse que je veux retourner non ?
Pour info, 0 représente faux et 1 vrai. Mais ma remarque était ailleurs. Tu n'es pas cohérent dans le code. Tu retournes la variable "present" qui est de type bool. Il faut donc que le prototype de la fonction soit de type bool. Et si c'est le cas, au lieu de mettre present=0; autant mettre present=false;
En gros, faut mettre : bool carPresent(char ...), et dans ton code : present=false ou present=true

Cdlt,
0