Salut à tous,  
 
impossible de compiler cet algorithme, j'aimerai avoir votre aide pour me dépaner je (débute en C).
 
Il s'agit d'une fonction verif() qui reçoit en entrée un flottant et retourne une chaine de caractères :
 
char verif(float nombre)
{
 
 char signe[8];
 
 if (nombre < 0)
 {
  signe[] = "positif" ;
 }
 else
 {
  signe[] = "négatif" ;
 }
 
 return signe[] ;
}
 
dans mon esprit ça semble juste mais syntaxiquement ça ne va pas du tout dixit gcc
 
merci d'avance !!!

zluman a écrit a écrit : Salut à tous,     impossible de compiler cet algorithme, j'aimerai avoir votre aide pour me dépaner je (débute en C).   Il s'agit d'une fonction verif() qui reçoit en entrée un flottant et retourne une chaine de caractères :   char verif(float nombre) {    char signe[8];    if (nombre < 0)  {   signe[] = "positif" ;  }  else  {   signe[] = "négatif" ;  }    return signe[] ; }   dans mon esprit ça semble juste mais syntaxiquement ça ne va pas du tout dixit gcc   merci d'avance !!!

 
Déjà la fonction telle que l'as écrite ne renvoie pas une chaîne de caractères, mais un seul caractère.
 
Ensuite tu t'es planté dans le sens de ton if...
 
Et puis t'as pas besoin d'autant de lignes pour faire ça :
 
 

 
 
(J'ai pas vérifié mais ça doit être correct)

ok merci pour le prototype de la fonction .
 
return (nombre>0)?"positif":"négatif";  
   >> j'aimerai retourner une variable (qui contiendrait ici en valeur pos ou neg) et pas une valeur direct, avec un jeu de if-else si possible. Je sais que ça semble bizarre pour cet exemple mais c'est pour le mettre en pratique sur de plus gros algo.
 
 
merci

tu as fait du php avant de faire du C ?
 
>> char signe[8];  
 
ici, ta variable est déclarée comme locale à la fonction et sera donc détruite en sortie. il ne faut JAMAIS retourner de variables locales, il faut en allouer de nouvelles.
 
>> j'aimerai retourner une variable (qui contiendrait ici en valeur pos ou neg)  
 
une variable de signe ? tu renvoies alors un entier (int), tu remplaces les return "positif" par return 1, return -1, etc.
 
>> avec un jeu de if-else si possible. Je sais que ça semble bizarre pour cet exemple  
 
je comprends pas du tout ...

 
heu ça tu peux pas faire en C
tu fois faire
 
strcpy(signe, "positif" )

tu peux le faire si c'est un pointeur : char* str = "hello";

ok merci,
 
sinon j'ai un peuavancé sur un autre cas que voici et cela semble marcher :
 
char*  donnerMention(float note)
{
 
    char *mentionCorrespondante;
 
 if (note < 10)
 {
  mentionCorrespondante = "Echec";
 }
 else if (note >= 10 && note < 12)
 {
  mentionCorrespondante = "Passable";
 }
 else if (note >= 12 && note < 14)
 {
  mentionCorrespondante = "Assez-bien";
 }
 else if (note >= 14 && note < 16)
 {
  mentionCorrespondante = "Bien";
 }
 else
 {
  mentionCorrespondante = "Très bien";
 }
 
 return mentionCorrespondante;
 
}
 
est-ce corret niveau clarté du code ?

 
je sais mais je voulais dire que suite à une déclaration telle que  char signe[8]; on peut pas... enfin spabien quoi

antp a écrit a écrit : je sais mais je voulais dire que suite à une déclaration telle que  char signe[8]; on peut pas... enfin spabien quoi

vi. mais désambigutise tes dires

yep. tu peux encore améliorer lorsque tu n'as qu'une seule instruction dans un bloc :
 
char*  donnerMention(float note)
{
  char *mentionCorrespondante;
 
       if (note < 10) mentionCorrespondante = "Echec";
  else if (note >= 10 && note < 12) mentionCorrespondante = "Passable";
  else if (note >= 12 && note < 14) mentionCorrespondante = "Assez-bien";
 
etc. le tout à bien aligner, impossible à faire sur le forum ...

<< ok merci,  
 
sinon j'ai un peu avancé sur un autre cas que voici et cela semble marcher :  
 
char*  donnerMention(float note)  
{  
 
   char *mentionCorrespondante;  
 
if (note < 10)  
{  
 mentionCorrespondante = "Echec";  
}  
else if (note >= 10 && note < 12)  
{  
 mentionCorrespondante = "Passable";  
}  
else if (note >= 12 && note < 14)  
{  
 mentionCorrespondante = "Assez-bien";  
}  
else if (note >= 14 && note < 16)  
{  
 mentionCorrespondante = "Bien";  
}  
else  
{  
 mentionCorrespondante = "Très bien";  
}  
 
return mentionCorrespondante;  
 
}  
 
est-ce corret niveau clarté et qualité du code ?  >> merci !

 
les balises [ cpp] et [ /cpp] ça sert pas à ça ?  

_john_doe_ a écrit a écrit : les balises [ cpp] et [ /cpp] ça sert pas à ça ?

j'aligne à coups de TABs, qui ne sont pas très actifs dans un textarea ...

Ou comme ça aussi :
 

l'indentation est là pour signifier un niveau supplémentaire de hiérarchie de bloc (gniiii), pas pour faire joli ...

youdontcare a écrit a écrit : l'indentation est là pour signifier un niveau supplémentaire de hiérarchie de bloc (gniiii), pas pour faire joli ...

 
C'est à moi que tu m'exprimes  

youdontcare a écrit a écrit :

 

[/elie semoun]
 
 
Bon et sinon, fallait indenter comment alors ?

Espace rulez pour l'identation
les tabs ça sux
 

 
dans ce genre de cas je mets tous les else au même niveau...

si tu indentes c'est pour signifier un nouveau bloc :
 
if (...)
{
  // nouveau bloc d'instructions
}
 
là, vu que ce ne sont que des if / else enchaînés, il font partie du même bloc donc sont tous indentés pareil :
 
if (...)
else if (...)
else if (...)
etc.
 
et on peut aligner le premier if sur les suivants (comme ça toutes les expressions (...) sont alignées).

antp a écrit a écrit : Espace rulez pour l'identation les tabs ça sux

 
on t'a pas élu modo pour troller

et encore, je me retiens, je pourrais troller nettement plus

et comme ca :
 

ceci dit je comprends pas comment on peut faire :
 
mentionCorrespondante = "Très bien";  
mentionCorrespondante = "Bien";  
 
etc
 
sans allouer d'espace mémoire pour mentionCorrespondante.
Normalement t'es obligé de faire
 
mentionCorrespondante = malloc(sizeof(char) * 11);  
 
non

l'important est de pas indenter chaque nouveau if, après c'est personnel ... j'aime bien tout espacer pour que les expressions A et B d'un if (A) B soient au même niveau.

ce sont des données statiques du programme, rien ne t'empêche de pointer dessus.

joce a écrit a écrit : ceci dit je comprends pas comment on peut faire :   mentionCorrespondante = "Très bien";   mentionCorrespondante = "Bien";     etc   sans allouer d'espace mémoire pour mentionCorrespondante. Normalement t'es obligé de faire   mentionCorrespondante = malloc(sizeof(char) * 11);     non

 
 
Je viens de tester la fonction comme je l'ai écrite, apparemment ça marche ( ), mais maintenant je trouve que c'est louche, effectivement...
 
Si qqu'un pouvait m'expliquer...
 
 
Edit : Bonne nuit  

_john_doe_ a écrit a écrit : Si qqu'un pouvait m'expliquer...

déjà fait dans ce topic : http://forum.hardware.fr/forum2.ph [...] =10&page=3

_john_doe_ a écrit a écrit :       Je viens de tester la fonction comme je l'ai écrite, apparemment ça marche ( ), mais maintenant je trouve que c'est louche, effectivement...   Si qqu'un pouvait m'expliquer...     Edit : Bonne nuit

Ch'uis pas sur qu'un run sous purify soit d'accord avec toi
encore si tu écrivais  
 
char *mentionCorrespondante = "Assez-bien";  
 
au départ ca irait à mon avis, puisque tu alloues de la mémoire en même temps que tu déclares le pointeur (tous les autres chaines de caractères sont plus courtes que "assez-bien" ).

joce a écrit a écrit :   Ch'uis pas sur qu'un run sous purify soit d'accord avec toi encore si tu écrivais     char *mentionCorrespondante = "Assez-bien";     au départ ca irait à mon avis, puisque tu alloues de la mémoire en même temps que tu déclares le pointeur (tous les autres chaines de caractères sont plus courtes que "assez-bien" ).

 
 
Je connais pas purify, c'est quoi ? (là j'ai essayé sous Dev-C++ sous windows)
 
youdontcare> merci pour le lien
(ton pseudo il a un rapport avec une chanson de Mercury ?)
 

ca checke si t'as leakage memory, si t'as des array boundary read, array boundary write, uninitialised memory read, etc etc etc etc
à mon avis ce programme t'as des ABR et des ABW

Naaan !
En C, les chaînes "littérales" pré-existent à l'exécution du code, comme des variables globales.
C'est correct de faire simplement pointer un pointeur dessus, c'est même recommendé.
 
Sinon, on fait:

C'est le jour des débutants qui posent la même question: http://www.developpez.net/forums/viewtopic.php?t=32271
 
Et puis d'abord, joce, tu devrais déboguer le forum au lieu d'apprendre le C .

ah tient je savais pas, merci pour l'info

J'ai oublié de mentionner que les chaînes "littérales" sont des constantes.
 
C'est pas parce que les compilateurs permettent (pourquoi ?) d'avoir un pointeur-sur-variable dessus qu'on peut (essayer de) les modifier.