遍历 char* 并将每个 char 与另一个 char 进行比较

Question

我不精通 c，我遇到了问题

1. 逐个字符迭代 char*
2. 正确比较单个字符与另一个字符

给定一个像“abcda”这样的字符串，我想计算“a”的数量并返回计数

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

    int main(int argc, char** argv){
        char* string_arg;
        int counter = 0;
        if(argc == 2){
            for(string_arg = argv[1]; *string_arg != '\0'; string_arg++){
                printf(string_arg);
                printf("\n");
                /*given abcda, this prints
                abcda                    a
                bcda                     b
                cda        but i want    c
                da                       d 
                a                        a */

                if(strcmp(string_arg, "a") == 0){ //syntax + logical error
                    counter++;
                }
            }
         printf(counter);
         }
         else{
             printf("error");
         }
         return(0);
    }

我也不应该使用 strlen()

如何正确地一次比较一个字符？

Answer 1

if (strcmp(string_arg, "a") == 0) { 
     counter++;
}

对strcmp 的调用不适用于您的情况，因为它比较字符串。使用此语句，您将从string_arg 指向的元素开始的字符串与字符串"a" 进行比较，而不是字符常量'a'。请注意，"a" 等于 'a'+ '\0'。

相反，您需要将*string_arg 与'a' 进行比较：

if (*string_array == 'a') { 
     counter++;
}

puts(string_arg); 打印一个字符串。那不是你想要的。您只想打印一个字符。使用printf("%c", *string_arg); 代替打印字符。

请注意，printf(string_arg); 之类的内容很危险。始终使用格式说明符：printf("%c", *string_arg);。原因在以下链接下解释：

Why is printf with a single argument (without conversion specifiers) deprecated?

---

这就是你想要的：

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

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

    char* string_arg;
    int counter = 0;

    if (argc == 2){

        for (string_arg = argv[1]; *string_arg != '\0'; string_arg++) {

            printf("%c", *string_arg);
            printf("\n");

            if (*string_arg == 'a') { 
                counter++;
            }
        }

        printf("%d times character 'a' encountered.", counter);
     }
     else {
        printf("Error: No second argument at the program invocation!");
     }

     return 0;
}

Answer 2

• arg 未声明。好像应该是argc。
• printf(string_arg); 很危险，因为string_arg 是用户输入，可以包含任意字符串，其中可能包括%。
• strcmp() 如果用于比较字符串。您可以简单地使用== 来比较字符。
• printf(counter); 也是错误的。
• 缺少main 函数末尾的}。

示例修复：

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

int main(int argc, char** argv){
    char* string_arg;
    int counter = 0;
    if(argc == 2){
        for(string_arg = argv[1]; *string_arg != '\0'; string_arg++){
            puts(string_arg);
            /*given abcda, this prints
            abcda                    a
            bcda                     b
            cda        but i want    c
            da                       d 
            a                        a */

            if(*string_arg == 'a'){
                counter++;
            }
        }
        printf("%d", counter);
    }
    else{
        printf("error");
    }
    return(0);
}