ログイン機能が作れない

Question

###前提・実現したいこと
ログイン機能
入力されたデータをファイルから読み取ったデータと同じかチェックする

###発生している問題
複数ユーザーのときできない

###該当のソースコード

unsigned int Login(const char *username, const char *password) {
	FILE *FilePointer;
	char username2[USERNAME_MAX], password2[PASSWORD_MAX];
	short count = 0, EqualCount = 0, count2 = 0, equal = 0;

	// Check username
	if (fopen_s(&FilePointer, "files\system\usernames", "r") != 0) {
		Error("Cannot open file.");
		_getch();
		exit(EXIT_FAILURE);
	}
	while (fgets(username2, USERNAME_MAX, FilePointer) != NULL) {
		count++;
		if (!strcmp(username, username2)) {
			EqualCount++;
			break;
		}
	}
	if (EqualCount == 0)return 1;
	fclose(FilePointer);

	// Check password
	if (fopen_s(&FilePointer, "files\system\passwords", "r") != 0) {
		Error("Cannot open file.");
		_getch();
		exit(EXIT_FAILURE);
	}
	while (fgets(password2, PASSWORD_MAX, FilePointer) != NULL) {
		count2++;
		if ((count == count2) && (!strcmp(password, password2)))equal = 1;
	}
	fclose(FilePointer);

	if (equal) {
		strcpy_s(Username, USERNAME_MAX, username);
		return 0;
	}
	else return 1;
}

###補足情報(言語/FW/ツール等のバージョンなど)
VisualStudio2017
よろしくお願いいたします。

Answer 1

標準入力からの入力のときに改行文字を取り除いていたのに、ファイルからの入力のときには改行文字を取り除いておらず、それを比較したのが原因でした。

if(username2[strlen(username2) - 1] == '\n')
   username2[strlen(username2) - 1] = '\0';

を書き足して解決しました。

わざわざソースコードまで書いていただき本当にありがとうございました。

Comments

[LouiS0616]

『関数を二回目以降に呼び出したとき、予期せぬ結果になる』だったのですね。
ファイル入力が絡んでくる処理は、サンプルデータがあるかどうかで解決のしやすさが断然違うので...
何らかの方法でダミーデータ自体を配布するか、生成するプログラムがあるといいですね。

[bob2017]

そうではなく、ファイルにユーザー名とパスワードが書いてあって、main関数でユーザー名とパスワードを標準入力から入力し、それをlogin関数に渡し、0が返されたら成功として続行し、1が返されたら失敗としてエラーを出してもう一度入力させる設計になっています。
複数ユーザーのときできないというのは、ファイルに複数のユーザー名とパスワードが書いてあるときに、なぜかログインができないということです。これの原因が、入力の最後の改行文字を取り除いたのにもかかわらず、ファイルのほうは取り除いておらず、この二つを比較したことです。
改行文字の差でstrcmp関数は等しくないと処理したようです。
説明不足で本当にすみませんでした。

Answer 2

複数ユーザーのときできない

質問がいくらでも多重解釈出来ますが、次のうちのどれかでしょうか。

• 結果をグローバル変数で受け取りたい

グローバル変数Usernameで結果を受け取るという仮定によるもの。
ユーザ名を複数保存したいなら、これを配列にするしかないでしょう。
しかし、どうせloginに返り値を設定したのですから、グローバル変数は必要ないのでは。

char login_users[LOGIN_USER_MAX][USERNAME_MAX] = {};
int uc = 0;
while (uc < LOGIN_USER_MAX) {
    char input_username[USERNAME_MAX] = {};
    char input_password[PASSWORD_MAX] = {};

    // 入力を受け付ける

    if (!login(username, password)) {
         strcpy_s(login_users[uc++], USERNAME_MAX, username);
    }

    if (入力を辞める) break;    
}

• 多重ログインを出来るように関数を書き換えたい

やめといた方がいいでしょう。複数回関数を呼び出す設計にしましょう。

• 関数を二回目以降に呼び出したとき、予期せぬ結果になる

テストデータがないのでなんとも言えません。再現できるデータを用意してください。

さて、どれでしょうかね。

---

とりあえず、関数loginはもうちょっと整理できます。

typedef struct {
    char username_[USERNAME_MAX];
    char password_[PASSWORD_MAX];
} LOGIN_INFO;

int login_info_equals(const LOGIN_INFO *info1, const LOGIN_INFO *info2) {
    if (strcmp(info1->username_, info2->username_) return -1;
    if (strcmp(info1->password_, info2->password_) return -1;
    return 0;
}

int login(const LOGIN_INFO *info) {
    FILE *fp_username, *fp_password;

    // Open files
    if (fopen_s(&fp_username, "files\system\usernames", "r") != 0) {
        Error("Cannot open file.");
        system("PAUSE");
        exit(EXIT_FAILURE);
    }
    if (fopen_s(&fp_password, "files\system\passwords", "r") != 0) {
        Error("Cannot open file.");
        fclose(fp_username);
        system("PAUSE");
        exit(EXIT_FAILURE);
    }

    // Check info
    LOGIN_INFO tmp_info;
    while (fgets(tmp_info.username_, USERNAME_MAX, fp_username) != NULL) 
        && fgets(tmp_info.password_, PASSWORD_MAX, fp_password) != NULL) {

        if (!login_info_equals(info, &tmp_info)) {
            fclose(fp_username);
            fclose(fp_password);
            return 0;
        }
    }

    // Cannot find
    fclose(fp_username);
    fclose(fp_password);
    return -1;
}

デバッグはしてないです、どこかしらミスはあるかと思います。