c - 简单的字符串计数器程序调试(指针)

Question

我是 C 编程和指针的新手。我制作了一个简单的程序，我可以在其中读取字符串，该程序会告诉您有多少个字符以及有多少个字母出现了多少次。不知何故，我的输出不正确。我认为这可能是我的指针和取消引用问题。

这是我的主要内容:

extern int* count (char* str);

int main(int argc, char* argv[])
{
  int numOfChars =0;
  int numOfUniqueChars = 0;
  char str[80];
  int *counts;

  strcpy(str, argv[1]);
  printf("counting number of characters for \"%s\"..\n", str);
  printf("\n");
 
  counts = count(str);
  
  int j;
  for (j =0; j<sizeof(counts); j++)
  {
    if(counts[j])
      printf("character %c", *str);
      printf("appeared %d times\n", counts[j]);
      numOfChars++;
      numOfUniqueChars++;
  }

  printf("\"%s\" has a total of %d character(s)\n", str, numOfChars);

  printf(wow %d different ascii character(s) much unique so skill\n", numOfUniqueChars);
}

这是我的计数函数:

int* count(char* str)
{
  int* asctb = malloc(256);
  int numOfChars =0;
  int i;
  int c;
    for(i = 0; i<strlen(str); i++)
       c = str[i];
       asctb[c]++;

  numOfChars += strlen(str);

  return asctb;
}

当我编译并运行它时，结果如下:

./countingCharacter doge

counting number of characters for "doge"...

appeared 0 times

appeared 0 times

appeared 0 times

appeared 0 times

"doge" has a total of 4 character(s)

wow 4 different ascii character(s) much unique so skill

但是，我希望我的结果是这样的:

Character d appeared 1 times

Character e appeared 1 times

Character g appeared 1 times

Character o appeared 1 times

"doge" has a total of 4 character(s)

wow 4 different ascii character(s) much unique so skill

任何帮助将不胜感激。

提前致谢!

编辑:

我在主函数中为 for 循环添加了大括号。

现在我得到这个结果:

./countingCharacter doge

character @ appeared 7912 times

character d appeared 1 times

character e appeared 1 times

character g appeared 1 times

character o appeared 1 times

为什么我会在开头看到“@”？

Answer 1

正如 @kaylum 所说，一个特别大的问题是大括号的使用。如果控制流语句中不使用大括号(if、for、while 等)，则仅计算下一行作为该声明的一部分。因此，该部分:

if (counts[j])
    printf("character %c", *str);
    printf("appeared %d times\n", counts[j]);
    /* ... */

...如果counts[j] != 0，只会执行第一个printf，但会无条件执行后面的语句。

您对 malloc 的使用也不正确。 malloc(256) 只会分配 256 字节；一个 int 通常是 4 个字节，但是根据编译器和机器的不同而有所不同。因此，当 malloc 任何类型的数组时，最好使用以下技术:

type *array = malloc(element_count * sizeof(type));

就您而言，这将是:

int *asctb = malloc(256 * sizeof(int));

这可确保您有空间计算 char 的所有可能值。此外，您还必须更改迭代 counts 的方式，因为 sizeof (counts) 不能准确表示数组的大小(很可能是 4或 8，具体取决于您的系统)。

变量numOfChars不会按照您期望的方式运行。在我看来，您试图在两个函数之间共享它，但由于它的声明方式，这种情况不会发生。为了提供对变量的全局访问，需要在任何函数之外的全局范围中声明它。

此外，该行:

printf("character %c ", *str);

...既不跟踪您已打印的字符，也不跟踪您应该打印的字符，而只是重复打印第一个字符。 *str 应该是 (char)j，因为您要打印 ASCII 值。

我想应该可以了。