遍历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

• arg未声明。似乎应该是argc
• printf（string_arg）很危险，因为stringarg是一个用户输入，它可以包含任意字符串，其中可能包括%
• strcmp（）如果用于比较字符串。您可以简单地使用==来比较字符
• printf（计数器）也是错误的
• }主function末尾的缺失

示例修复:

#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);
}

Answer 2

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

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

相反，您需要将*string_arg与'a':

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

put（string_arg）；打印一个字符串。这不是您想要的。您只想打印一个字符。请使用printf（"%c"，*string_arg）；来打印一个字符。

注意，类似于< code > printf(string _ arg)；是危险的。始终使用格式说明符:< code>printf("%c "，* string _ arg)；。原因原因在以下链接中有解释:

为什么不推荐使用带有单个参数（不带转换说明符）的printf？

这将是你想要的：

#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;
}