C: Vérification de l'argument de ligne de commande est entier ou pas?

Signature de isdigit

int isdigit(int c);

Signature de atoi

int atoi(const char *nptr);

Je voulais juste vérifier si l'argument de ligne de commande qui s'est passé était un entier ou pas.Voici le Code C:

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

int main(int argc, char *argv[])
{
    if (argc == 1)
        return -1;

    printf ("Hai, you have executed the program : %s\n", argv[0]);
    if (isdigit(atoi(argv[1])))
        printf ("%s is a number\n", argv[1]);
    else
        printf ("%s is not a number\n", argv[1]);
    return 0;
}

Mais la sortie n'est pas comme prévu, lorsque je suis de passage un nombre valide:

$ ./a.out 123
Hai, you have executed the program : ./a.out
123 is not a number
$ ./a.out add
Hai, you have executed the program : ./a.out
add is not a number

Je ne pouvais pas comprendre l'erreur.

Plus évident et simple, thread-safe exemple vous pouvez trouver ici "Comment vérifier si une chaîne est le nombre?"

OriginalL'auteur 0aslam0 | 2015-03-25

c command-line

12

Lorsque vous vous référez à argv[1], il se réfère à un tableau de caractères contenant la valeur 123. isdigit fonction est définie pour une seule entrée de caractères.

Afin de gérer cette situation, il est préférable de définir une fonction, comme suit:
```
bool isNumber(char number[])
{
    int i = 0;

    //checking for negative numbers
    if (number[0] == '-')
        i = 1;
    for (; number[i] != 0; i++)
    {
        //if (number[i] > '9' || number[i] < '0')
        if (!isdigit(number[i]))
            return false;
    }
    return true;
}
```
Bonne alternative..
Vous pouvez utiliser appel isdigit ici pour le faire paraître mieux.
Ok qui peut également être fait. À modifier mon code.
Comment voulez-vous modifier cela, cochez la case pour les nombres négatifs?
Vérifier si le premier caractère est égal à zéro ou non. Je vais modifier mon code pour nombre négatif.

OriginalL'auteur niyasc
12
```
if (isdigit(atoi(argv[1]))) 
```
sera:
```
if (isdigit(atoi("123")))
```
:
```
if (isdigit(123))
```
:
```
if ( 0 )
```
depuis 123 représente le caractère ASCII '{'.

OK. isdigit(x) ; ici x peut être de 0, 1, 2,...9 seulement?
Non, seulement pour '0', '1', etc. La valeur de l'entier sont 48 - 57.
C'est un nouveau morceau de l'info.

OriginalL'auteur R Sahu

Je pensais que je voudrais ajouter quelque chose pour les réponses déjà ici. En plus de vérifier les nombres en base 10, j'ai pensé qu'il serait utile de vérifier et autoriser les nombres hexadécimaux. J'ai aussi permettre à des nombres négatifs.

J'ai aussi ajouté quelques petites choses à vérifier mauvaise entrée (par exemple pointeur null, lettres à l'intérieur d'une chaîne de caractères représentant un nombre décimal, ou invalide lettres à l'intérieur d'une chaîne de caractères représentant un nombre hexadécimal).

Noter que j'utilise le to_lower(char c) fonction afin de s'assurer que les lettres représentant hexadécimal sera en bas de casse, juste pour des raisons de commodité.

Je retourne 1 (vrai) si la chaîne est un nombre valide, 0 si elle ne l'est pas. Si c'est un nombre valide, je stocke la base à l'intérieur du paramètre de base.

//Return 1 if str is a number, 0 otherwise.
//If str is a number, store the base (10 or 16) in param base.
static int is_number(char *str, int *base)
{
//Check for null pointer.
if (str == NULL)
return 0;
int i;
int len = strlen(str);
//Single character case.
if (len == 1)
{
*base = 10;
return isdigit(str[0]);
}
//Hexadecimal? At this point, we know length is at least 2.
if ((str[0] == '0') && (str[1] == 'x'))
{
//Check that every character is a digit or a,b,c,d,e, or f.
for (i = 2; i < len; i++)
{
char c = str[i];
c = to_lower(c);
if (!(
(c >= '0' && c <= '9') || 
(c >= 'a' && c <= 'f')))
return 0;
}
*base = 16;
}
//It's decimal.
else
{
i = 0;
//Accept signs.
if (str[0] == '-' || str[0] == '+')
i = 1;
//Check that every character is a digit.
for (; i < len; i++)
{
if (!isdigit(str[i]))
return 0;
}
*base = 10;
}
return 1;
}

J'ai utilisé cette fonction comme ceci:

int base, num;
if (is_number(str, &base)
num = strtol(str, NULL, base);

OriginalL'auteur mgarey

0

appel isdigit() fonction vérifie pour un chiffre ('0' à '9'), qui dépend bien sûr de valeurs ASCII. Maintenant, la valeur de retour à partir de votre atoi ne relève pas de valeur ASCII entre '0' à '9'. donc, c'est montrer qu'il n'est pas un nombre.

OriginalL'auteur Chintan Patel
0

Je ne sais pas ce isdigit exactement, mais en raison de nom, je pense qu'il devrait prendre un char argument, vérifier le char étant un chiffre, est-il?

Je voudrais écrire comme ceci: (omis, la fonction shell, juste montrer le code de base)
```
char* p = argv[1];
while (*p != '\0')
{
if (*p<'0' || *p>'9')
{
printf("%s is not a number", argv[1]);
return 0;
}
p++;
}
printf("%s is a number", argv[1]);
return 0;
```
btw, vous n'avez pas besoin de savoir ce que le nombre est, droite? Alors pourquoi utiliser atoi? Mieux de ne pas abuser de la fonction
Quand j'ai regardé la signature de isdigit , j'ai vu qu'il accepte une entrée entière. J'ai donc voulu la convertir l'argument de type entier (si l'argument est en effet un entier)et ensuite faire le chèque. Mais l'idée a été entaché d'irrégularités. En outre atoi n'a pas de contrôle d'erreur.

OriginalL'auteur Cherry

-1

//Since There is an implicit conversion from const char* to std::string
//You May use this simplified version of the check_string instead
bool isNumeric(const string str) 
{
//loop Through each character in the string
for(char x:  str)
if(!isdigit(x)) //Check if a single character "x" its a digit
return false;  //if its not return false 
return true; //else return true
}

C'est du C, pas du C++

OriginalL'auteur Yosephhhhhh

Vous devez vous connecter pour publier un commentaire.