Question

这段代码似乎对我不起作用。它输出的数字17显然是错误的。如果遇到5,15,25,50等数字，计数器应该会上升。

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

int main(void) {
    int i;
    int counter; 
    char num[4322];

    for (i = 1; i < sizeof(num); i++){
        num[i] = i;
        if ( strstr(&num[i], "5")){
            counter = counter + 1;
        }
    } 

    printf("%d", counter);
    return 0;
}

Answer 1

您没有将数字转换为字符串，并且您将不是以空字符结尾的字符串传递给strstr()。您也忘了初始化counter。

您应该逐个检查数字。试试这个：

#include <stdio.h>

int main(void) {
    int max = 4322;
    int target = 5;
    int i, cur, counter = 0;
    for (i = 1; i <= max; i++) {
        for (cur = i; cur > 0; cur /= 10) {
            if (cur % 10 == target) counter++;
        }
    }
    printf("%d\n", counter);
    return 0;
}

Answer 2

此代码中存在许多问题。

char num[4322];

这是一个包含4322个字符的数组。你可能不想那样，我不确定它为什么会在这里。

num[i] = i;

这是无效的，因为您无法将大号分配给char。这不会将数字转换为字符串，这会将数字转换为不同的char。因此5不会变为"5"，但会变为\x05。

strstr(&num[i], "5")

这将搜索"5"，但由于数组不是以空值终止的，因此它不正确。另请注意，您正在存储

int counter;

请注意counter未初始化。这是一个错误。

这是一个类似的程序，它是正确的（但本身并不“有效”）：

#include <stdio.h>
#include <string.h>
int main() {
    int counter = 0; // initialize to 0
    for (int i = 0; i < 4322; i++) {
        // Put the number in a string
        char buf[10];
        snprintf(buf, sizeof(buf), "%d", i);
        // Count 5s
        for (char *p = buf; *p; p++) {
            if (*p == '5') {
                counter++;
            }
        }
    }
    printf("%d", counter);
}

请注意@ MikeCAT的答案会跳过int - ＆gt;字符串转换，这有点不必要。

查找数字5出现在0 - 4322之间的次数

2 个答案: