error: conflicting types for 'impairr'

Version imprimable

1 pièce(s) jointe(s)

bonjour , j'ai un problème avec un code sources j'ai cette erreur en intitulé ,quels conflit de types ???

pourriez vous m'aider please ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
#include <stdbool.h>
#include <stdio.h>
bool pairr(int n){
    if(n==0){
        printf("true");
        return true ;
    }
    return impairr(n);
}
bool impairr(int n){
if(n==0){
        printf("false");
        return false ;
    }
    return pairr(n);
}

Pièce jointe 420291

18/10/2018, 14h08
Médinoc

Si tu actives les warnings de compilation, tu auras l'explication sous la forme d'un warning:

Citation:

Envoyé par gcc

Implicit declaration of function 'impairr'

Citation:

Envoyé par Microsoft Visual C++

'impairr': Undeclared, assuming extern returning 'int'.

Tu dois déclarer ta fonction avant de l'utiliser.
De plus, le code actuel semble présenter une récursion infinie si on lui passe n'importe quelle valeur non-nulle.

Les fonctions doivent normalement exister dans le code qui précède une fonction appelante. Comme tu fais une fonction qui en appelle une autre et inversement, tu as le problème de la poule et de l'oeuf.

Si tu veux pouvoir appeler une fonction dont le code ne figure pas avant le code de la fonction appelante (par exemple : le code est après dans le même fichier ou ailleurs dans une autre unité de compilation), le fichier compilé doit avoir au moins accès à la déclaration de son prototype avant que l'appel puisse être fait.

Ainsi, dans ton cas, tu pourrais faire :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 #include <stdbool.h> #include <stdio.h> /* prototypes */ bool pairr(int n); bool impairr(int n); /* implémentation */ bool pairr(int n){ /* code appelant impairr() */ } bool impairr(int n){ /* code appelant pairr() */ }
Note que je ne pense pas que les fonctions fassent ce que leur nom suggère, en l'état de ton code, par exemple en appelant

Code:

pairr(1);

tes fonctions vont s'appeler l'une l'autre en une boucle infinie, en épuisant toute la mémoire de la pile d’exécution.

Citation:
Envoyé par alex5956
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 bool pairr(int n){ if(n==0){ printf("true"); return true ; } return impairr(n); } bool impairr(int n){ if(n==0){ printf("false"); return false ; } return pairr(n); }
Bonjour
-Eks- a donné la bonne explication. On déclare ses fonctions avant de les appeler.

Maintenant moi je m'interroge sur ta logique. Tu as une fonction appelée "pair()" qui teste si son argument vaut 0. Si c'est le cas, elle renvoie "vrai". Sinon elle appelle la fonction "impair()" qui, elle, teste si l'argument vaut 0 et si c'est le cas elle renvoie "faux" sinon elle appelle "pair()". :weird:
C'est... éblouissant !!! Bon déjà parce que "0 ou pas 0" ne signifie pas (chez-moi) "pair ou impair" :? mais surtout parce que déjà l'argument il vaut soit 0 soit pas 0 donc s'il vaut 0 c'est un cas sinon c'est l'autre cas et pas besoin d'appeler une autre fonction pour le vérifier. Mais surtout, je m'imagine ce qui se passera si tu appelles pair(5). Et tu devrais essayer aussi de te représenter le déroulement des opérations...:lahola:

Généralement, on ne fait pas deux fonctions pour vérifier une chose et son contraire. On n'en fait qu'une seule (fonction généralement et traditionnellement nommée "is_quelquechose()") et ensuite, si on veut vérifier la chose on appelle if (is_quelquechose() == true) et si on veut vérifier son contraire on appelle if (is_quelquechose() != true). Et surtout on évite d'invoquer la récursivité pour ça d'autant plus quand c'est une récursivité non maitrisée.

18/10/2018, 20h53
alex5956

oui y a une erreur je me disais bien c 'est le prof qui a écrit ça dans le td ils s'est trompé .
par contre ===> pour Microsoft Visual C++

Citation:

'impairr': Undeclared, assuming extern returning 'int'.

pour quoi il veut que je la mette en extern int

et quels rapport au final avec "un conflit de type" ?

encore merci les gars vous me sauvez la vie !XD j'avais déclaré les proto masi dans un header pour mettre toutes les fonctions dans le main
18/10/2018, 21h05
Bousk

Citation:

Envoyé par alex5956

pour quoi il veut que je la mette en extern int

Il veut rien du tout, il te prévient qu'il ne connait pas ce type et qu'il suppose donc que c'est une fonction qui retourne un int.

Citation:

Envoyé par alex5956

et quels rapport au final avec "un conflit de type" ?

La fonction supposée est sencée retourner un int et non un bool, il y a bien un conflit de type.
19/10/2018, 08h45
Bktero

Citation:

Envoyé par alex5956

'impairr': Undeclared, assuming extern returning 'int'.

Il faut comprendre : "impairr n'est pas déclaré, on suppose donc que c'est une fonction externe (== qui est dans un autre fichier), on suppose qu'elle renvoie 'int'. Toi, tu as lu "assuming returning extern int", ce qui n'est pas la même chose ;)
19/10/2018, 09h35
Sve@r

Citation:

Envoyé par alex5956

pour quoi il veut que je la mette en extern int

Le compilateur considère toute fonction non déclarée comme int par défaut et c'est ce qu'il te dit ("assuming" signifie "en supposant"). Ce n'est donc pas un ordre qu'il te donne mais une action qu'il exécute et dont il t'informe.

Citation:

Envoyé par alex5956

j'avais déclaré les proto mais dans un header pour mettre toutes les fonctions dans le main

Rien compris. Je comprends les mots, je vois un peu le sujet de ce dont il s'agit toutefois mettre la conjonction "mais" indiquant une opposition entre la première partie de la phrase qui parle de prototypes et sa seconde partie qui parle de headers (alors que les deux vont ensembles) tout en justifiant cette action par une explication qui n'a rien à voir (mettre toutes les fonctions dans le main) rend ta phrase incompréhensible.
24/10/2018, 21h29
alex5956

ok donc seulement MSVC m'aurait indiqué que la fonction est par défaut reconnu comme retournant un type int alors que de tte façon gcc ne me l'aurait jamais indiqué.

je disais que j'avais mis mes proto dans mesFonctions.h et que je l'avais seulement inclut dans le main
24/10/2018, 22h56
Sve@r

Citation:

Envoyé par alex5956

ok donc seulement MSVC m'aurait indiqué que la fonction est par défaut reconnu comme retournant un type int alors que de tte façon gcc ne me l'aurait jamais indiqué.

gcc te le dit si tu le lui demandes avec l'option -Wimplicit-function-declaration (option incluse dans -Wall)

Citation:

Envoyé par alex5956

je disais que j'avais mis mes proto dans mesFonctions.h et que je l'avais seulement inclut dans le main

C'est exactement comme ça qu'il faut faire (le prototype seulement, pas le code !!!)
25/10/2018, 10h07
alex5956

oui mais j'ai pas mis les proto dans le fichier.c des fonctions recursives