Problème sur petite fonction (conversion fahr / C°)

Version imprimable

Bonjour

En soit j'ai déjà réussi mon petit exercice si je ne dit pas de betise (voici mon code) :
Code:

1 2 3 4 5 6 7 8 int.....conversion_F_C(int fahrenheit) { int...celsius; celsius = 0.5 * (fahrenheit - 32); return (celsius); }
Mon probleme survient lorsque j'essai de rentrer les arguments avec "int ac, char **av".
j'ai recodé ma fonction mais lorsque je l'execute, j'ai des segmentation fault ( généralement je comprend pourquoi du coup je peux les corriger) ou sinon j'ai bien mon opération qui se fait mais que sur un charactere et non sur la chaine de charactere.
Je ne sais pas si je suis tres claire, mais je pense que cela sera plus claire apres avoir vue mon code :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 int.....conversion_F_c(char *f) { int...f1; int...celsius; celsius = 0; f1 = *f; celsius = 0.5 * (f1 - 32); return (celsius); } int main(int ac, char **av) { printf("%d\n", conversion_F_c(av[1]); printf("%d\n", conversion_F_C(50)); //pour verifier le resultat return (0); }
Merci d'avance.

09/11/2015, 12h39
foetus

Si ton programme plante tu as un pointeur foireux :mrgreen:

Ton pointeur n'est pas utile, tout comme ta variable f1 :mrgreen:

Si tu passes en paramètre un char, soit ta température sera comprise en -128 et 127 [ce qui est correct en température terrestre], ~~soit tu dois retirer '\0'~~ (<- *).

Édit: Ils sont intentionnels tes points :koi: :koi:

Édit 1: En réalité, le paramètre char* correspond à un pointeur sur une chaîne de caractères et non pas un pointeur sur un char.

*: c'est débile de convertir une température entre 0 et 9 :wow:
09/11/2015, 12h57
CrH4A4cK

Salut

Donc le probleme vient de mon pointeur sur f?
Pourquoi mon pointeur sur f n'est pas utilie? Je comprend que ma variable f1 ne sert a rien mais pour le pointeur...

Citation:

soit tu doit retirer '\0'

je ne comprend pas pourquoi.
Puis si je veux retirer le '\0', je parcourt ma chaine de caractere et je décrémente de 1?

oui les petits points sont intentionnel, ils apparaissent lorsque j'utilise les tabulations, c'est un truc que j'ai configuré dans mon .vimrc
09/11/2015, 13h51
foetus

Je n'avais pas scrollé pour voir le main: :mrgreen:

Citation:

Envoyé par CrH4A4cK

Donc le probleme vient de mon pointeur sur f?

Et pourtant tu ne fais que le déréférencer pour prendre sa valeur. Donc si ton programme plante à cet endroit, c'est que ton pointeur pointe sur rien :mrgreen:

Citation:

Envoyé par CrH4A4cK

Je comprend que ma variable f1 ne sert a rien mais pour le pointeur...

Voici ta fonction: :mrgreen:

Code:

#define CONVERT_FAHRENHEIT_TO_CELSIUS(DEGREE) ((DEGREE - 32) / 2)

Parce que tu reçois du main une chaîne de caractères :mrgreen:
Donc pas 50, mais "50"

Et si je te disais de retirer le caractère '\0', c'est parce que je pensais que tu pouvais recevoir un caractère :mrgreen:.
Donc pas 0, 1, 2, 3, ... mais '0', '1', '2, '3',...

As-tu vérifié le paramètre ac pour être sûr que av[1] existe?
09/11/2015, 14h23
Médinoc

Pour commencer, c'est tout le code censé convertir une chaîne de caractères en nombre qui manque.
Utiliser la fonction strtol() pour cela.

Salut

Citation:

As-tu vérifié le paramètre ac pour être sûr que av[1] existe?

je l'avais fait mais j'ai effacé la ligne quand je vous ai envoyé le code ( j'ai laissé que l'essentiel selon moi).

Citation:

Pour commencer, c'est tout le code censé convertir une chaîne de caractères en nombre qui manque.
Utiliser la fonction strtol() pour cela.

Grâce a strtol() mon programme fonctionne presque "parfaitement".
Il me reste un warning et une note que je n'arrive pas a eliminer.
Code:

1 2 3 4 5 6 7 conversion_F_C.c: In function ‘main’: conversion_F_C.c:68:31: warning: passing argument 1 of ‘conversion_F_to_C’ from incompatible pointer type [-Wincompatible-pointer-types] my_putnbr(conversion_F_to_C(&result)); ^ conversion_F_C.c:50:5: note: expected ‘char *’ but argument is of type ‘int *’ int conversion_F_to_C(char *f) ^
Citation:

Si tu passes en paramètre un char, soit ta température sera comprise en -128 et 127

Comment je pourrais faire pour que la température soit comprise entre un intervalle plus grand?
j'ai essayé de changer le type du parametre en gardant la meme structure de code mais ca ne fonctionne pas.
un petit indice?

atoi() est similaire a strtol()?

Merci !

09/11/2015, 19h56
foetus

Citation:

Envoyé par CrH4A4cK

Comment je pourrais faire pour que la température soit comprise entre un intervalle plus grand?

Avec une chaîne de caractère ton nombre est infini :mrgreen: par exemple "5841125147742584477145"

Ton type int est un peu grand, mais il fait le job.

Citation:

Envoyé par CrH4A4cK

atoi() est similaire a strtol()?

atoi est plus ou moins dépréciée et elle retourne un int (le i de atoi) alors que strtol retourne un long (le l de strtol).
09/11/2015, 23h04
CrH4A4cK

Hi,

Citation:

Avec une chaîne de caractère ton nombre est infini par exemple "5841125147742584477145"

C'est ce que j'avais cru comprendre mais lorsque je fais ./a.out "128" ca me fait n'importe quoi (il me renvoi -80 au lieu de 48 environ) alors que ma fonction "test" me renvoi le bon resultat.

Merci

Salut

Voici mon code apres l'aide que vous m'avez apporté :

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
 
int.....conversion_F_to_C(char *f)
{
    int...celsius;
 
    celsius = 0;
    celsius = 0.5 * (*f - 32);
    return (celsius);
}
 
  int.....main(int ac, char **av)
  {
    int...result;
    char..*fin = NULL;
    result = 0;
 
    if (ac != 2)
      return (0);
    result = strtol(av[1], &fin, 10);
    my_putnbr(conversion_F_to_C(&result));
    my_putchar('\n');
    my_putnbr(conversion_F_C(128));
    my_putchar('\n');
    return (0);
}

Il me reste quelques petits problèmes :
Comme je l'ai dit plus haut, si mon argument av[1] est supérieure a 127 le resultat est faux alors que normalement je suis sencé pouvoir mettre n'importe quel nombre, non?

j'ai aussi ces deux erreures je ne comprend pas du tout comment les corriger :

Code:

1
2
3
4
5
6
conversion_F_C.c: In function ‘main’:
conversion_F_C.c:68:31: warning: passing argument 1 of ‘conversion_F_to_C’ from incompatible pointer type [-Wincompatible-pointer-types]
   my_putnbr(conversion_F_to_C(&result));
                               ^
conversion_F_C.c:50:5: note: expected ‘char *’ but argument is of type ‘int *’
 int conversion_F_to_C(char *f)

un indice?

12/11/2015, 16h50
Médinoc

Dès la première ligne, l'erreur est apparente.
Ta fonction de conversion entre °C et °F doit travailler exclusivement avec des nombres, et non pas des chaînes de caractères.

Code:

int conversion_F_to_C(int fahrenheit)
12/11/2015, 17h32
CrH4A4cK

Merci beaucoup!!
j'étais persuadé que cela 'allait pas fonctionné si je changé le type.
j'avais déjà essayé certaine modification mais cela avait foiré!