Discussion :

[Merillym]

Bonjour,

J'ai un léger souci avec la fonction pow de la lib math.h :

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
#include <stdio.h>
#include <windows.h>
#include <math.h>
 
unsigned int ConvertOneIntCharToItsValue(char c)
{
  return c - '0';
}
 
unsigned int ConvertCharNumberStringToInteger(char s[])
{
  size_t count = 0;
  int nbr_pow = strlen(s)-1;
  unsigned int Res = 0;
 
  while (count < strlen(s))
    Res += ConvertOneIntCharToItsValue(s[count++])*((int)pow(10, nbr_pow--));
 
  return (Res);
}
int main ()
{
    printf("%d\n", ConvertCharNumberStringToInteger("12000"));
    return(0);
}

Le but est de convertir un string de type char, ne contenant que des chiffres au nombre que forment l'ensemble des chiffres.
Seulement, pour toutes les puissances paires, je n'obtiens pas le bon résultat.

Par exemple, pour pow(10, 2), il m'affiche 99 au lieu de 100.

J'ai testé en dehors de la fonction, j'obtiens bien 100, mais dans ma boucle while, j'obtiens toujours -1 par rapport au résulat attendu pour les puissances de 2, soit 99 au lieu de 100, 9999 au lieu de 10000 etc.

Sauriez-vous pourquoi ?

Merci par avance.

[Pouet_forever]

La fonction pow renvoie un double. Un double est un type flottant approximatif, ce qui fait que tu peux avoir des nombres arrondis.

Enfin tu emploies plein de calculs inutiles (calcul de la longueur de la chaîne, utilisation de pow, beaucoup de variables).
Ce ne serait pas plus simple de prendre ton nombre, lui ajouter s[count++] et ensuite le multiplier par 10 ?

J'ai modifié ton programme, ta fonction tient en quelques lignes et une seule variable

[Merillym]

Salut,

Merci pour ta réponse.

Code :

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

1
2
3
4
5
6
7
8
9
double my_pow(unsigned int a, unsigned int b)
{
  size_t count = 0;
  double res = 1;
 
  while (count < b--)
   res *= a;
  return (res);
}

En utilisant ma fonction, mon code ci-dessus marche sans souci.

Concernant la complexité inutile du code, je veux bien le croire, je débute et j'ai encore du mal à aller à l'essentiel.

[Pouet_forever]

Tu fonctionnes avec des unsigned int donc pourquoi utiliser des double dans ta fonction my_pow ? Des simples unsigned int suffisent

Je t'explique rapidos le principe :

Tu as une chaîne de caractère "1234" que tu vas parcourir dans le sens de lecture (de gauche à droite).
Donc tu sais que tu vas toujours commencer avec les nombres "de poids fort" et finir par les nombres "de poids faible".
En partant de ça c'est très simple !
Tu initialises un nombre à 0.
Tu lis ton premier caractère (auquel tu enlèves '0') et tu l'ajoutes à ton nombre. (tu as 1)
Ensuite, tu as un autre nombre -> Tu multiplies ton nombre par 10 (donc tu as 10)
Tu ajoutes ce nombre -> 12
Tu multiplies par 10 -> 120
Tu ajoutes 3 -> 123
Tu multiplies par 10 -> 1230
Tu ajoutes 4 -> 1234
Ta chaîne est finie, on arrête là et on renvoie le nombre.

Et voilà tu t'es passé des appels à pow, strlen et ut as économisé des variables

[Merillym]

Ah oué, vu comme ça, c'est encore plus simple

Je suis bien d'accord qu'actuellement, mon plus gros souci, c'est d'être le plus concis possible. J'ai tendance à me compliquer inutilement.

Merci