Pourquoi puis-je obtenir ce résultat inattendu avec atoi() en C?

atoi() convertit une chaîne représentant un nombre entier en sa valeur. Il ne sera pas convertir des caractères arbitraires dans leur valeur décimale. Par exemple:

int main(void)
{
    const char *string="12345";

    printf("The value of %s is %d\n", string, atoi(string));

    return 0;
}

Il n'y a rien dans la bibliothèque standard C qui va convertir "Une" à 65 ou "Z" à 90, vous devez écrire vous-même, en particulier pour tout ce charset vous attend à l'entrée.

Maintenant que vous savez ce atoi() n', veuillez ne pas l'utiliser de traiter avec entrée numérique dans tout ce que vous venez avec. Vous devriez vraiment faire face avec la contribution de ne pas être ce que vous attendez. Hmm, ce qui se passe quand je rentre de 65 ans au lieu d'Un? Les enseignants de l'amour de casser des choses.

atoi() ne fait pas de vérification d'erreur que ce soit, ce qui rend tout fier de convertir arbitraire d'entrée fragile, au mieux. Au lieu de cela, utiliser strtol() POSIX (centrée sur l'exemple):

#include <stdio.h>
#include <stdlib.h>
#include <errno.h>

int main(void)
{
    static const char *input ="123abc";
    char *garbage = NULL;
    long value = 0;

    errno = 0;

    value = strtol(input, &garbage, 0);

    switch (errno) {
        case ERANGE:
            printf("The data could not be represented.\n");
            return 1;
        //host-specific (GNU/Linux in my case)
        case EINVAL:
            printf("Unsupported base /radix.\n");
            return 1;
    }

    printf("The value is %ld, leftover garbage in the string is %s\n",
           //Again, host-specific, avoid trying to print NULL.
           value, garbage == NULL ? "N/A" : garbage);

    return 0;
}

Lorsqu'il est exécuté, ce qui donne:

La valeur est de 123, des restes de déchets dans
la chaîne abc

Si vous n'avez pas de soins sur l'enregistrement /l'examen de la poubelle, vous pouvez définir le deuxième argument de NULL. Il n'est pas nécessaire de free(garbage). A noter également, si vous passez 0 en tant que troisième argument, il est supposé que l'entrée est la valeur souhaitée d'un décimal, hexadécimal ou octal représentation. Si vous avez besoin d'une base de 10, de 10 ce sera un échec si l'entrée n'est pas comme vous l'attendez.

Vous pouvez également vérifier la valeur de retour pour le maximum et le minimum de la valeur d'un long int peut gérer. Toutefois, si un est retourné pour indiquer une erreur, errno est réglé. Un exercice pour le lecteur est de changer *input de 123abc à abc123.

Il est important de vérifier le retour, que votre exemple montre ce qui se passe si vous ne le faites pas. AbcDeFg n'est pas une représentation de chaîne de caractères en un nombre entier, et vous avez besoin de votre fonction.

Pour votre mise en œuvre, la plupart des conseils de base, je peux vous donner une série de commutateurs, quelque chose comme:

//signed, since a return value of 0 is acceptable (NULL), -1
//means failure
int ascii_to_ascii_val(const char *in)
{
    switch(in) {
        //64 other cases before 'A'
        case 'A':
           return 65;
        //keep going from here
        default:
            return -1; //failure

}

.. alors juste l'exécuter dans une boucle.

Ou, de pré-remplir un dictionnaire d'une fonction de recherche pourrait champ d'application (mieux). Vous n'auriez pas besoin de tables de hachage, juste une touche -> valeur magasin puisque vous savez ce qu'il va contenir à l'avance, où les caractères ASCII standard sont les touches, et leurs identificateurs correspondants sont des valeurs.

OriginalL'auteur Tim Post

Lire atoi() comme un i (ASCII en entier).

atoi() convertit une chaîne de caractères représentant un nombre décimal à l'entier.

char s[] = "42";
int num = atoi(s); //The value of num is 42.

OriginalL'auteur Deep