Discussion :

[troumad]

Bonjour

Je viens de récupérer http://pocket.free.fr/download/wav2bin.zip sur http://pocket.free.fr/html/soft/soft_f.html#
J'essaie de le compiler sous Linux avec gcc et je tombe sur l'erreur suivante :

Code :

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

1
2
3
4
/home/bs/tmp/ccFaKabU.o: In function `main':wav2bin.c:(.text+0x29bc): undefined reference to `strupr'
:wav2bin.c:(.text+0x2ac1): undefined reference to `strupr'
:wav2bin.c:(.text+0x2bed): undefined reference to `strupr'
:wav2bin.c:(.text+0x2cec): undefined reference to `strupr'

Voici le code incriminé :

Code :

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

        (void) strupr (argS[1]) ;

C'est quoi ça ?

nb : il existe aussi une version MAC osX. Est-ce que je ne ferais pas mieux de prendre cette version ?

[DaZumba]

C'est une fonction non-standard qui passe une chaine de caractere en majuscules. Ca se re-code en 30 secondes...

[troumad]

toupper et tolower est-ce standard ?

Code :

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

1
2
3
4
5
6
7
8
void strupr(char * chaine)
{
 while(*chaine)
 {
  if (*chaine<='z' && *chaine>='a') *chaine+='A'-'a';
  chaine++;
 }
}

[PRomu@ld]

http://www.linux-kheops.com/doc/man/manfr/man-html-0.9/man3/toupper.3.html

Oui c'est ANSI

[DaZumba]

Fait attention, la fonction strupr a pour prototype (d'après Google):

Code :

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

1
2
 
char *strupr(char *s1);

Elle modifie s1 et renvoie cette adresse.
Donc:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
#include <ctype.h>
 
char *strupr(char *s1)
{
   char *p = s1;
   while(*p)
   {
       toupper(*p);
       p++;
   }
   return s1;
}

[troumad]

Si c n'est ni un caractère non-signé, ni EOF, le comportement de ces fonctions est imprévisible.

Ceci me fait un peu peur... Je n'ai pas encore testé le programme ! Et j'ai peur que je veuille déjà l'utiliser en dehors du cadre prévu !

[troumad]

Fait attention, la fonction strupr a pour prototype (d'après Google):

Code :

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

1
2
 
char *strupr(char *s1);

Pourtant elle est utilisée comme ça par le programme :

Code :

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

(void) strupr (argS[1]) ;

Il y a un problème dans l'air !

[DaZumba]

Pourtant elle est utilisée comme ça par le programme :

Code :

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

(void) strupr (argS[1]) ;

Il y a un problème dans l'air !

Non, c'est normal. En mettant (void) devant la fonction, le programmeur signale au compilateur qu'il sait que la fonction renvoie quelque chose mais qu'il a décidé d'ignorer cette valeur (certains paranos mettent cela devant printf()!). C'est un style de codage.

[troumad]

OK !
Donc, la procédure modifie la chaîne et renvoie l'adresse qu'on lui a donnée... Intérêt de ce renvoi ?
Autant renvoyer 0 ou 1 (en int ou char) pour indiquer le bon ou mauvais fonctionnement de la procédure !

[DaZumba]

Donc, la procédure modifie la chaîne et renvoie l'adresse qu'on lui a donnée... Intérêt de ce renvoi ?

Renvoyer l'adresse de l'argument est pratique pour enchaîner les appels sans variable intermédiaire explicite. Du genre:

Code :

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

1
2
 
printf("%s\n", strupr(s));

Ce n'est pas très propre, mais c'est utilisé souvent.

[troumad]

OK !