Thread: isdigit()

What's wrong with the concept:
It seems like a better idea to call isdigit from your original input function, rather than trying to do input and checking in the same function.

Not strictly wrong, but completely meaningless:
To continue the loop is implied at the end of the loop, and there is nothing to "skip to" at beyond your continue, so you can remove that.

Possibly wrong:
If you have not yet set digit before calling the function, it is incorrect to call isdigit() on the uninitialized value of digit.

Some things that are DIRECTLY wrong in
Using the %s format to read a single character will not work - that format reads a string consisting of zero or more characters, terminating with a zero at the end of the string. You are supplying scanf with a single char - the only valid string that could possibly be stored in one char is an empty string - whch is not a digit. I suggest you use %c instead.

You also do not supply the address of digit, to scanf.

Also beware of the newline - you may want to put a space in your format string.

--
Mats

if you want to check a char printf or scan do not use %s but %c. %s is for strings(char*),pointers to char.
If you change %s to %c you check only one char.
TO check a string use char* type and check every char of the string.a char* is an array of chars.
Use strlen(mystring) to get the length of your string and then
use for to check every char of your string like

Code:

char userinput[100];
scanf("%s",userinput);
len=strlen(userinput);

for(int i=0;i<len;i++)
{
if(isdigit(userinput[i])==true)
...
else
...
}

your function could be like this:

Code:

int is_digit(char* digit)
    {
         int len=strlen(digit);
         for(int i=0;i<len;i++)
        {
             if(isdigit(digit[i])==false)
             {
                   return -1;
              }
        }
    
     return atoi(digit);
    }

the function get a char*(a string) and checks every char if is digit. If all the chars are digits will return the atoi of the digit(ascii to int value). if found a char that is not digit will return -1 which will mean for your main programm error(a char of the user's input is not digit).

From your main program if you get -1 calling is_digit just ask him for input again.It's good practice not to scanf from is_digit because this function sounds only checking and not for gettting user's input

in your main program to get 6 times user's input:

Code:

int res=0;
char user[100];

for(int i=1;i<=6;i++)
{
printf("for i=%d\n",i);
scanf("%s",user);
res=is_digit(user);

if(res==-1)
{
printf("error on input\n");
}
else
{
printf("Ok value is:%d\n",res);
}

}

Code:

                        if (i=1)
                        {
                        //scan for number, send to is_digit


                        }
                        else if (i=2)
                        {
                            //scan for number, send to is_digit
                        }
                        else if (i=3)
                        {
                            //scan for number, send to is_digit
                        }
                        else if (i=4)
                        {
                           //scan for number, send to is_digit
                        }
                        else if (i=5)
                        {
                            //scan for number, send to is_digit
                        }
                        else if (i=6)
                        {
                            //scan for number, send to is_digit
                        }

1. You have mixed up comparison and assignment operator: compare if equal is ==, make it equal is =.
2. The point of a loop here is that you should be able to walk through your string [which is an array] and pass the current position to isdigit. You should not have a for-loop and then do if(i == x) for every value of the loop - that's completely pointless.

--
Mats

My example works only for integers >= 0. if the input number is negative the is_digit will return -1 because of the "-".

It depencne what u want to do and what the program expect from user, u can use convertion functions.

atoi() wiil convert an char* to int - > example atoi("123abcf") will return 123
atoi("absd1234") will return 0

I never tested but i think that atoi will return a negative int if you write something like this atoi("-100")

So, u can use the concertions and then u can check if the (converted) input is that that your program expects

strtol give even more feedback
if you pass "123abc" to it, it returns 123 and sets the end pointer to point to "abc",

so checking that something left in the input buffer after converted number you can be sure that the whole buffer is correct or not