Question

我是C语言的新手，想编写一个简单的程序，用正好两个命令行参数来调用（第一个是程序的名称，第二个是字符串）。我已经验证了参数的 number ，现在想验证输入中仅包含数字。

#include <stdio.h>
#include <string.h>
#include <ctype.h>
#include <cs50.h>

int main(int argc, string argv[])
{
    if (argc == 2)
        {
            for (int i = 0, n = strlen(argv[1]); i < n; i++)
            {
                if isdigit(argv[1][i])
                {
                    printf("Success!\n");
                }
                else
                    {
                    printf("Error. Second argument can be numbers only\n");
                    return 1;
                    }    
            }
    else
        {
            printf("Error. Second argument can be numbers only\n");
            return 1;
        }
}

虽然上面的代码没有引发错误，但是它不能正确地验证输入，我也不知道为什么。有人可以指出我正确的方向吗？

非常感谢，一切顺利

Answer 1

if (argc != 2)

必须

if (argc == 2)

做

if isdigit(argv[1][i])
{
  printf("Success!\n");
}

如果数字有4位数字，您将打印4次“成功”，最好这样做：

for (int i = 0, n = strlen(argv[1]); i < n; i++)
{
   if (!isdigit(argv[1][i])
   {
     printf("Error. Second argument can be numbers only\n");
     return 1;
   }    
 }
 printf("Success!\n");
 return 0;

示例：

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

int main(int argc, char * argv[])
{
  if (argc == 2)
  {
    int i, n;

    for (i = 0, n = strlen(argv[1]); i < n; i++)
    {
      if (!isdigit(argv[1][i]))
      {
        printf("Error. Second argument can be numbers only\n");
        return 1;
      }
    }
    printf("Success!\n");
  }
  else {
    puts("argument is missing");
    return 1;
  }

  return 0;
}

编译和执行：

pi@raspberrypi:/tmp $ gcc -pedantic -Wextra c.c
pi@raspberrypi:/tmp $ ./a.out 
argument is missing
pi@raspberrypi:/tmp $ ./a.out az
Error. Second argument can be numbers only
pi@raspberrypi:/tmp $ ./a.out 12
Success!
pi@raspberrypi:/tmp $ ./a.out 12a
Error. Second argument can be numbers only
pi@raspberrypi:/tmp $

在valgrind下执行：

pi@raspberrypi:/tmp $ valgrind ./a.out 123
==2051== Memcheck, a memory error detector
==2051== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==2051== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==2051== Command: ./a.out 123
==2051== 
Success!
==2051== 
==2051== HEAP SUMMARY:
==2051==     in use at exit: 0 bytes in 0 blocks
==2051==   total heap usage: 1 allocs, 1 frees, 1,024 bytes allocated
==2051== 
==2051== All heap blocks were freed -- no leaks are possible
==2051== 
==2051== For counts of detected and suppressed errors, rerun with: -v
==2051== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 6 from 3)

Answer 2

虽然您可以依靠循环检查每个isdigit()中的字符，但是如果您的目的是使用参数表示的数字值，则也可以使用strtol/strtoul将值转换为signed/unsigned的价值，并依靠可靠的错误报告准确地告诉您转换期间发生了什么以及参数的什么部分（如果有）由非数字组成。

strtoX函数家族将指向数字字符串的指针和endptr的地址作为参数（以及整数转换的基数）。该函数尝试将数字转换为更新endptr的数字，以指向成功转换最后一位数字之后的第一个字符。这样，您就可以确定参数是否由前导数字组成，是否在尝试转换的类型的范围内，以及在转换的最后一位数字之后是否存在任何其他字符。

一个简短的示例说明：

#include <stdio.h>
#include <stdlib.h>     /* for strtol */
#include <errno.h>      /* errno - for strtol validation */

int main (int argc, char **argv) {

    char *endptr;   /* end pointer for strtol */
    long l;

    if (argc < 2) { /* validate at least one argument provided */
        fprintf (stderr, "error: insufficient arguments provided.\n"
                         "usage: %s number\n", argv[0]);
        return 1;
    }

    errno = 0;                          /* zero errno before call */
    l = strtol (argv[1], &endptr, 0);   /* attempt converstion to long */

    if (endptr == argv[1]) {    /* check if digits were converted */
        fprintf (stderr, "error: no digits converted in '%s'.\n", endptr);
        return 1;
    }
    else if (errno) {           /* check for error in conversion */
        perror ("strtol-argv[1]");
        return 1;
    }

    printf ("number provided: %ld\n", l);   /* output number */

    if (*endptr)    /* endptr not pointing to `'\0'`, chars remain */
        printf ("\nadditional characters following number: '%s'.\n", endptr);

    return 0;
}

使用/输出示例

$ ./bin/strtolarg 123456
number provided: 123456

$ ./bin/strtolarg -123456
number provided: -123456

$ ./bin/strtolarg "val=123456"
error: no digits converted in 'val=123456'.

$ ./bin/strtolarg "123456=val"
number provided: 123456

additional characters following number: '=val'.

仔细研究一下，如果您有任何疑问，请告诉我。

检查命令行输入是否为数字

2 个答案: