erreur conflit de type

Version imprimable

bonjour,
lors de la compilation j'ai un problème, je vous donne mon code :

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
27
28
29
30
31
32
33
34
 
int main(int ac, char **av)
{
   char *str;
   str = test_name[av(2)];
}
 
char *test_name(char *s)
{
   int   c;
   int   test;
   char  *str;
 
   test = 0;
   c = 0;
   while (s[c] != '\0')
    {
      if (s[c] = '/')
        test = 1;
      c++;
    }
  if (test = 1)
    {
      c = strlen(s);
      while (s[c] != '/')
        c--;
      while (s[c] != '\0')
        str[test] = s[c];
      c++;
      return (str);                           
} 
else
    return (s);               
}

et quand je compile j'ai un message d'erreur "
- error : conflicting types for 'test_name'
- error : previous implicit declaration of 'test_name' was here

je ne voit pas de quoi ca vient. merci

22/11/2009, 11h43
diogene
Code:

str = test_name[av(2)];

test_name est-il un tableau ? Où est-il défini ? N'y a t-il pas confusion avec la fonction ?

Code:

str = test_name(av(2));

Dans ce cas, il faut que la fonction test_name soit déclarée AVANT, soit en déplaçant la fonction soit en plaçant son prototype avant le main.

La fonction test_name est fausse et va planter ici :
Code:

1 2 3 char *str; .... str[test] = s[c];
str est un pointeur non initialisé et pas un tableau.
22/11/2009, 12h25
dharth

oui test_name est bien une fonction, j'ai bien mit des parenthèse dans mon prog, c'est juste une erreur de recopie^^

non av[c] est un parametre donné en entrée de programme, ce paramétre est un nom de fichier, et ma fonction sert a tester si le av[c] est un nom de fichier ou un lien.
22/11/2009, 13h11
Pouet_forever

Comme le dit diogene il faut que ta fonction soit déclarée avant sont appel.
Sinon tu devrais avoir 2 warning :
*) Implicit declaration of function XXX
*) Conflicting types for XXX
Où XXX est le nom de ta fonction.

Par défaut si la fonction n'est pas déclarée avant son appel le type de retour est de type int, et comme toi tu veux retourner un char* il y a un problème ;)
Donc soit tu mets le prototype de ta fonction au début de ton fichier, soit tu mets ta fonction avant son appel (donc avant le main dans ton cas).

Attention aussi, tu confonds = et ==, le premier est fait pour l'affectation et le second pour la comparaison ;)

Et puis même remarque que diogene sur le fait que tu utilises un pointeur non initialisé.
De plus ce pointeur n'est «vivant» que dans ta fonction donc dès que tu sera sorti de ta fonction il sera supprimé. Il faut que tu utilises l'allocation dynamique ;)
22/11/2009, 13h53
dharth

c'est bon, j'ai fait un fichier h ou j'ai prototypé mes fonctions et ca marche.
merci