问大家一个c语言问题，判断字符出现是否唯一？我的代码哪里错了，为什么没有输出结果。

Question

#include <stdio.h>
#include<string.h>
int main(void)
{
    char s[50];
    for(int i=0;i<50;i++)
    {
        scanf("%c",&s[i]);
    }
    for(int i=0;i<strlen(s);i++)
    {
        for(int j=i+1;j<strlen(s);j++)
        {
            if(s[i]==s[j])
            return 1;
        }
    }
    return 0;
}

Answer 1

引自免费微信小程序：皆我百晓生

从你提供的代码来看，主要存在两个问题导致程序未能正确输出结果。下面将详细解析并提供修正后的代码。

首先，你的代码在读取字符串时存在逻辑错误。你使用了一个循环读取字符，但这会导致无限循环，因为没有设置退出条件。正确的做法是使用scanf的字符串读取功能，即scanf("%s", s);，这会一次性读取整个字符串到s数组中。

其次，你的代码中缺少打印输出结果的部分。你需要在main函数的结尾添加代码来根据返回值打印相应的信息。

基于上述分析，修正后的代码如下所示：

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

// 函数声明，用于检查字符串中的字符是否唯一
int is_unique(const char *str);

int main(void) {
    char s[50];
    // 读取一行字符串，注意留出空间给'\0'
    scanf("%49s", s);
    
    // 调用is_unique函数检查字符串，并根据结果打印
    if (is_unique(s)) {
        printf("1\n"); // 字符不唯一
    } else {
        printf("0\n"); // 所有字符唯一
    }
    
    return 0;
}

// 检查字符串中所有字符是否唯一的函数实现
int is_unique(const char *str) {
    int len = strlen(str);
    for (int i = 0; i < len; i++) {
        for (int j = i + 1; j < len; j++) {
            if (str[i] == str[j]) {
                return 1; // 发现重复字符，返回1
            }
        }
    }
    return 0; // 所有字符均唯一，返回0
}

这段代码首先定义了一个is_unique函数，专门用来检查字符串中的字符是否唯一。然后，在main函数中，我们使用scanf读取字符串，并调用is_unique函数进行检查，最后根据返回值打印相应的结果。

这样修改后，你的程序应该可以正确地判断字符串中的字符是否唯一，并输出相应的结果了。