为什么while循环不能正常工作？

Question

我编写了一些 C 代码来生成一个随机字符并检查该字符是否存在于数组中。如果存在，我想重新生成一个新字符。我使用标志作为指示符并使用do/while 循环来检查此标志，但不幸的是，代码无法按预期工作，并且我得到了一个已存在于数组中的字符。

我需要你的帮助来理解和解决这个问题。

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int main() {
    char c;
    int num;
    int flag = -1;
    char temp[5] = { '3', '2', '5', '9', '1' };
    srand(time(NULL));
    do {
        num = rand() % 5;
        c = num;
        for (int i = 0; i < 5; i++) {
            if (temp[i] == c) {
                flag = 0;
                break;
            } else {
                flag = 1;
            }
        }
    } while (0 == flag);

    printf("number is : %d\n", c);
    return 0;
}

Answer 1

有多个问题：

• 5 和 '5' 不是一回事。 '5' 是一个字符值，而 5 是一个数值。字符值取决于字符编码，很可能是当前系统上的 ASCII，其中'5' 的数值为53 (0x35)。
• C 标准保证'0' 到'9' 的字符数字是连续的，因此您可以通过将'0' 添加到随机数来绘制字符数字。
• 您应该在for 循环之前将flag 设置为1，并且只有在找到字符时才将其清除。
• 请注意，您在0、1、2、3 和4 之间绘制一个字符。您可能希望包含所有字符数字？

这是修改后的版本：

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int main() {
    char temp[] = { '3', '2', '5', '9', '1' };
    const size_t temp_len = sizeof(temp) / sizeof(temp[0]);
    char c;
    int flag;

    srand(time(NULL));

    do {
        // select a random digit character 
        c = '0' + rand() % 10;
        flag = 1;
        for (size_t i = 0; i < temp_len; i++) {
            if (temp[i] == c) {
                flag = 0;
                break;
            }
        }
    } while (flag == 0);

    printf("character is: %c\n", c);
    return 0;
}

请注意，您可以使用memchr() 来搜索字符，并避免使用容易引起混淆的do/while 循环。这是一个更简单的选择：

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

int main() {
    char temp[] = { '3', '2', '5', '9', '1' };
    char c;

    srand(time(NULL));

    for (;;) {
        // select a random digit character 
        c = '0' + rand() % 10;
        if (!memchr(temp, c, sizeof temp))
            break;
    }
    printf("character is: %c\n", c);
    return 0;
}