c - 计算给定字符串中有多少由空格分隔的整数值

Question

我想做一个函数，给定包含由空格分隔的（无符号）整数值的字符串，给我字符串中值的数量：

int conta_coords(char *args) {
  char *pal;
  int k=0;
  pal = strtok (args," ");
  while (pal != NULL)
  {
      k++;
      pal =strtok (NULL," ");
  }
  return k;
}

这个函数不会给我正确的数字。谁能帮我？

Answer 1

像这样的东西：

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

bool is_stringified_num(const char *str, const size_t size)
{
    int n = 0;
    while(n < size && isdigit(str[n++]));

    return n == size;
}

int str_numbers_count(char *str)
{
    int numbers_count = 0;
    char *prev_pos = str;
    char *pos = strchr(str, ' ');

    while(pos)
    {
        if (is_stringified_num(prev_pos, pos - prev_pos)) ++numbers_count ;
        prev_pos = ++pos;
        pos = strchr(pos, ' ');
    }

    if (is_stringified_num(prev_pos, strlen(str) - (prev_pos - str))) ++numbers_count ;

    return numbers_count ;
}

int main(int argc, char *argv[])
{
    char str[] = "1 23 456 7890 12345";
    printf("%s contains %d numbers\n", str, str_numbers_count(str));

    return 0;
}

Ideone 编译良好并输出：

1 23 456 7890 12345 contains 5 numbers

Answer 2

它以什么方式不会给你正确的号码？这是嵌入到 SSCCE 中的代码（简短、独立、正确的示例）。

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

extern int conta_coords(char *str);

int conta_coords(char *args) {
  char *pal;
  int k=0;
  pal = strtok (args," ");
  while (pal != NULL)
  {
  k++;
      pal =strtok (NULL," ");
  }
  return k;
}

int main(void)
{
    char data[] = "1 23 456 7890 12345";
    printf("Data: %s\n", data);
    printf("Number: %d\n", conta_coords(data));
    printf("Data split: %s\n", data);
    return 0;
}

输出：

$ ./cntnum
Data: 1 23 456 7890 12345
Number: 5
Data split: 1
$ 

这在我看来是正确的。但请注意，原始字符串已被切成碎片。此外，如果我传递了一个只读字符串（字符串文字），我可能会得到不同的结果，因为strtok()修改了它所处理的数据，但字符串文字并不总是可修改的（你可能会因为尝试而得到核心转储修改它）。例如：

    printf("Number: %d\n", conta_coords("   1     23    45   67    99    "));

这给了我一个“总线错误”（如果它们没有被禁用，它会给出一个核心转储）。

这是一个替代实现，它通过根本不修改搜索字符串来处理常量字符串，使用被低估的 C89 标准函数strspn()和strcspn()：

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

extern int conta_coords(const char *str);

int conta_coords(const char *str)
{
    const char  digits[] = "0123456789";
    const char *ptr = str;
    int   k = 0;
    int   n = strcspn(ptr, digits);
    while (ptr[n] != '\0')
    {
        ptr += n;
        n = strspn(ptr, digits);
        if (n > 0)
            k++;
        ptr += n;
        n = strcspn(ptr, digits);
    }
    return k;
}

int main(void)
{
    char data[] = "1 23 456 7890 12345";
    printf("Data: %s\n", data);
    printf("Number: %d\n", conta_coords(data));
    printf("Data unsplit: %s\n", data);
    printf("Number: %d\n", conta_coords("   1     23    45   67    99    "));
    return 0;
}

输出：

Data: 1 23 456 7890 12345
Number: 5
Data unsplit: 1 23 456 7890 12345
Number: 5

请注意，对此的一个合理批评是它不要求整数用空格分隔（因此更准确的描述是“计算一个或多个连续数字的序列数（由一个或多个非数字分隔）出现在给定的字符串中'）。但是原始代码也可以基于类似的理由受到批评：它计算给定字符串中出现的连续非空白序列的数量，由一个或多个空白分隔。您可以改进实现，但要注意如何处理错误格式的数据并报告问题。