使用 strchr() 计算字符串中字符的出现次数

Question

我的学期快结束了，我正在写一个函数来查找字符串中某个字符的编号，给定老师分配的函数原型。我知道我一定是在做一些愚蠢的事情，但是这段代码要么被锁定，要么在我的函数中无限循环。

这是一项任务，所以我不是在寻找任何人为我做作业，而只是指出我错在哪里以及原因，以便我了解如何解决它。如果您愿意提供任何帮助，我将不胜感激。

这是我写的代码：

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

int charCounter(char* pString, char c);

int main(void)
{
    char* inpString = "Thequickbrownfoxjumpedoverthelazydog.";
    int charToCount;
    int eCount;

    eCount = 0;
    charToCount = 'e';
    eCount = charCounter(inpString, charToCount);
    printf("\nThe letter %c was found %d times.", charToCount, eCount);

    return 0;
} // end main

int charCounter(char* pString, char c)
{
    int count = 0;
    char* pTemp;

    do
    {
        pTemp = strchr(pString, c);
        count++;
    }
    while(pTemp != NULL);

    return count;
} // end countCharacter

Answer 1

您的循环总是从pString 的开头查找，并且总是一遍又一遍地找到第一个“e”。

如果您声明char* pTemp = pString;，那么您可以进行一些不同的迭代（我之前粘贴了错误的版本，抱歉！）：

char* pTemp = pString;

while(pTemp != NULL)                                                                                                    
{                                                                                                                       
    pTemp = strchr(pTemp, c);                                                                                                           
    if( pTemp ) {
        pTemp++;
        count++;
    }                                                                                                
}

这会强制pTemp 指向您刚刚找到的字符之后，然后再查找下一个字符。

这样做会更容易：

char* pTemp = pString;
while( *pTemp )
    if( *pTemp++ == c) count++;

好的，经过考虑，即使您已经完成了这项工作，我还是将内部循环更改为我更满意的形式：

while( (pTemp = strchr(pTemp, c)) != NULL) {                                                                                                                       
   count++;                                                                                                             
   pTemp++;
}

Answer 2

你总是从头开始。难怪你永远不会结束。

char* pTemp; // Init here = pString
do {
    pTemp = strchr(pString, c); // pString? Really? Should be pTemp
    count++;
} while(pTemp != NULL); // pTemp != NULL is verbose for pTemp here

不过，最好避免使用库函数并直接循环所有元素。

只是为了跳上马车：

size_t count(const char* s, char c) {
    size_t r = 0;
    for (; *s; ++s)
        r += *s == c;
    return r;
}

如果你坚持使用strchr()：

size_t count(const char* s, char c) {
    size_t r = 0;
    while ((s = strchr(s, c)))
        ++r;
    return r;
}

Answer 3

strchr 返回一个指针，指向 pString 中字符 c 的第一次出现。

因此，如果您在每个循环中提供指向字符串开头的指针，则 pTemp 将始终具有相同的值，并且如果字符 c 存在，则永远不会为 NULL。这就是为什么你有一个无限循环。

你可能想在这里做一些指针运算来解决你的问题；）

Answer 4

strchr() 总是在同一个地方寻找，它不会在字符串中前进......

试试这个修改traverse使用字符串的长度，和一个简单的char比较：

int i, len = strlen(pString);
count = 0;
for(i=0;i<len;i++)
{
    if(pString[i] == c) count++; //increment count only if c found
}

return count;  

不使用strlen()（地址评论）

i=-1, count = 0;
while(pString[++i])
{
   if(pString[i] == c) count++;
}  
return count;