5

我正在构建一个程序，该程序使用值的 ascii 表随机生成密码，并且每个字符只能包含一个。它会生成一个 8 个字符长的密码。

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

#define SIZE 10

char Charactor(char x);
void Check(char* y);

int main()
{
char string[SIZE];//defines varriables
char* point;
point=&string[SIZE];
srand(time(NULL));
for (int i=0;i<SIZE-1;i++)//empties out the string
{
    string[i]='\0';
}
for (int i=0;i<SIZE-2;i++)//randomizes a char for each space in the string
{
    string[i]=Charactor(*point);
}
Check(point);/checks the string for duplicated values
printf("%s\n",string);//prints string on screen
}

char Charactor(char x)
{
int rnd=0;
rnd=rand()%2;//chooses char or number using ascii
if (rnd==0)
{
    rnd=rand()%10+48;
}
else
{
    rnd=rand()%26+65;
}
x=(char)rnd;
return x;
}

void Check(char* y)
{
int run=0;
for (int i=0; i<SIZE-2;i++)
{
    for (int x=0; x<SIZE-2; x++) 
    {
        if (y[i]==y[x] && run=0)
        {
            run++;
            continue;
        }
        else
        {
            y[i]='\0';
            y[i]=Charactor(*y);
        }
    }
}
return;
}

通过这些更改，代码现在正在运行，我只需要弄清楚如何更改正确的值，所以我没有任何重复。

4

2 回答 2

4

使固定：

char* point =&string[0]; //Make it to point to first element

由于您Charactor(*point);实际上并没有做任何事情，因此您可能会*point使用Check(point);可能从字符串开头开始扫描。

和

 if (y[i]==y[x] && run==0)
                      ^^Use Equality check

您不能将boolean结果修改y[i]==y[x] && run为零。

注意：但是if (y[i]==y[x] && (run=0) )不会抛出此错误。

于 2013-09-06T17:51:17.033 回答
2

您的错误似乎是您错误地将 run=0 设置为

if (y[i]==y[x] && run=0)

这是最有可能让您的编译器感到困惑的部分。不需要和 Y 做任何事情。

修复到：

if (y[i]==y[x] && run==0)
于 2013-09-06T17:53:56.077 回答