C言語自作関数の練習をしています

Question

パスワードの入力判定について自作関数を作っています

ルール
①八文字以上②大文字、小文字、記号、数字が入っていないといけない→全て当てはまるとOKが出る

実行するとどの値を入力してもOKしか出てきません
考えてみましたが何がいけないのかよくわかりません。。

#include<stdio.h>

void check(char MOJI[100])
{
    int i=0,oomoji,komoji,suuji,kigou;
    while(MOJI[i]!='\0')     //何文字であるか判定//
    {
        i++;
    }
    for(i=0;i;i++)            //文字の種類を判定//
    {
        if(MOJI[i]>='A'||MOJI[i]<='Z') oomoji++;
        else  if(MOJI[i]>='a'||MOJI[i]<='z') komoji++;
        else  if(MOJI[i]>='0'||MOJI[i]<='9') suuji++;
        else  kigou++;
    }
    
    
    //OKかNGかの判定//
    if((oomoji>0)||(komoji>0)||(suuji>0)||(kigou>0)||(i<=8))
    {
        printf("OK");
    }
    else
    {
        printf("NG\n\nREASON:");
        if(i<=7)      printf("word count do not  have enough\n");
        if(oomoji==0) printf("oomoji is not inputed\n");
        if(komoji==0) printf("komoji is not inputed\n");
        if(suuji==0)  printf("suuzi is not inputed\n");
        if(kigou==0)  printf("kigou is not inputed\n");
    }
}


//main関数の始まり//
int main()
{
 char moji[100];
 printf("please input your password\n\n");
 printf("password:");
 scanf("%s",moji);
 check(moji);
}

```

Answer 1

色々とありますが、

while(MOJI[i]!='\0') //何文字であるか判定//

このループで、何文字か調べてるようですが、次の for()で、i=0 としているので、i の値は、 0 にされます。　(文字数を保持する変数を用意すべきでしょう)

for(i=0;i;i++)            //文字の種類を判定//

for() の文法を確認してください。
最初のi=0 は、初期化、次の i は、ループの条件。ここが、true(≠0)の間、ループします。 最初に i は、 0になっているため、1回もループしません。

if((oomoji>0)||(komoji>0)||(suuji>0)||(kigou>0)||(i<=8))

全て || なので、どれか一つ trueならば、if()は、真となります。 その前の for()が一度もループしていないので、i=0 従って、常に "OK" となるでしょう。 && かと。

なお、すでに指摘があるように、 oomoji,komoji,suuji,kigou の変数は、式化されていないため、値は不定です。 0に初期化する必要があります。

また、指摘がありますが、ctype.h をインクルードすることで、文字の判定が行えます。 (isdigit() : 数字の判定、 isuppper() : 大文字(英字)、 islower(): 小文字(英字) 等) 　strlen()(string.h) で、文字長さ。
等々。

Answer 2

回答ではないですが・・・
文字の判定は、ctype.hをインクルードして判定しましょう。

Answer 3

何文字であるか？
→ strlen関数を使いましょう。
【追記】８文字以上あるかのチェックがありませんよ。

oomoji,komoji,suuji,kigouが初期化されないままインクリメントされています。
→forループの前に0で初期化しましょう。
全てOKとなる原因はこれだと思います。

Answer 4

int i=0,oomoji,komoji,suuji,kigou;

ローカル変数というのはデタラメの値が入ってます。
なので、使う前にはきちんと初期化（なんかの値を入れておく）しないといけません。

for(i=0;i;i++)            //文字の種類を判定//

これ、おかしいです
どういう動作になるか考えてみましょう

Comments

[y_waiwai]

回答に追記

[退会済みユーザー]

iがゼロからiまでということではないんですか？
そう書きたい場合どうすればいいのでしょう？？
勉強不足ですいません。。

[y_waiwai]

iにゼロいれたら、iはゼロになりますよ。

[y_waiwai]

同じ変数でやろうとするからおかしくなります
別の変数でやりましょう