Discussion :

[CrH4A4cK]

Bonjour

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

Code :

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

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 :

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

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.

[foetus]

Si ton programme plante tu as un pointeur foireux

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

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

É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

[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...

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

[foetus]

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

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

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

Voici ta fonction:

Code :

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

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

Parce que tu reçois du main une chaîne de caractères
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 .
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?

[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.

[CrH4A4cK]

Salut

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).

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 :

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

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)
     ^

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 !

[foetus]

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 par exemple "5841125147742584477145"

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

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).

[CrH4A4cK]

Hi,

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

[CrH4A4cK]

Salut

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

Code :

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

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 :

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

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?

[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 C :

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

int conversion_F_to_C(int fahrenheit)

[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é!