Discussion :

[Daborem]

Bonjour à tous !

Je début en C, en autodidacte, mais je souvent confronté à des soucis que je tente de régler.
Aujourd'hui, il y en a un pour lequel je ne trouve pas de solution, et je pense que j'aurais besoin de quelques explications !

L'exercice que j'ai demande de refaire la fonction atoi. Pour cela, après avoir compter le nombre de caractères de la chaine, je veux échanger, si la valeur est correcte (donc un chiffre, ou un moins au tout début) l'afficher dans une autre chaine de caractère.
J'avais ceci :

Code :

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

1
2
if (*str + lenght >= 48 && *str + lenght <= 57)
*second_str + second_lenght = *str + lenght;

mais j'ai une erreur qui me dit

error: expression is not assignable
*second_str + second_lenght = *str + lenght;

Je débute et ce que je fais est peut être totalement ridicule mais ... Voilà ^^
Du coup : Comment puis-je faire pour que "12AZ" dans *str soit basculé en "12" dans *str_second ?

Merci de votre aide !

[stendhal666]

*second_str + second_lenght = *str + lenght;

Mettons que *second_str == 48 et second_length = 2, cela équivaut à écrire:

Code :

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

50 = *str + length

Tu comprends pourquoi il y a un problème d'assignation, non?

Tu pourrais écrire:

Code :

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

 *(second_str + second_length) = *(str + length)

Cela étant, je te confirme que ton implémentation de atoi a l'air un peu portnawak...

L'esprit général de ce genre de fonction est:

Code :

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

1
2
3
4
5
6
int resultat = 0;
tant que *str est un chiffre
  resultat *= 10
  resultat += *str - '0'
  str++;
retourne resultat

[foetus]

En anglais c'est length avec un th

L'esprit général de ce genre de fonction est:

Code :

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

1
2
3
4
5
6
int resultat = 0;
tant que *str est un chiffre
  resultat *= 10
  resultat += *str - '0'
  str++;
retourne resultat

Pas d'accord

• Ce n'est pas un atoi mais une recopie "Comment puis-je faire pour que "12AZ" dans *str soit basculé en "12" dans *str_second?"
• Il faut allouer le résultat (ou str_second). Par conséquent, il faut compter, dans un premier temps, le nombre de chiffres consécutifs pour, dans un deuxième temps, allouer correctement et ensuite faire la recopie.
• Il ne faut pas travailler directement avec la source (str dans ton exemple)

[Sve@r]

Bonjour

• Il faut allouer le résultat (ou str_second). Par conséquent, il faut compter, dans un premier temps, le nombre de chiffres consécutifs pour, dans un deuxième temps, allouer correctement et ensuite faire la recopie.

Eventuellement, puisqu'on est sûr que "12" rentrera (fatalement !!!) dans "12AZ" on peut alors allouer directement la taille de "12AZ". Certes ça gaspillera parfois un peu de place mais ça évite une première recherche parfois inutile. L'un dans l'autre...

[foetus]

ça évite une première recherche parfois inutile. L'un dans l'autre...

Oui mais avec le nombre de caractères connu, on peut directement utiliser strncpy ou memcpy

[edgarjacobs]

Hello,

Une autre idée: pourquoi ne pas parcourir la chaine tant que l'on a un chiffre, ensuite, lorsque on est sur la caractère 'fautif' (même le \0), le copier dans un char, mettre \0 dans cette position, convertir, et remettre le caractère fautif à sa place, plutot que jouer avec une copie ?

A+

Edgar;

[Sve@r]

...ensuite, lorsque on est sur la caractère 'fautif' (même le \0), le copier dans un char, mettre \0 dans cette position, convertir, et remettre le caractère fautif à sa place...

Ben pas obligé. La fonction qui se charge de la conversion pourra elle-même détecter et s'arrêter sur le caractère 'fautif'. Inutile alors de le remplacer par un '\0' préalable...