Question

我正在构建一个程序，该程序使用ascii tabe值随机生成密码，并且只能包含每个char中的一个。它会生成一个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;
}

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

Answer 1

修正：

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的{{1}}结果修改为零。

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

Answer 2

您的错误似乎是您错误地在

中设置了run = 0

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

这是最容易混淆编译器的部分。不需要对Y做任何事情。

修复：

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

使y [i]成为C中的可修改变量

2 个答案: