OOPを使用したC＃シンプルライブラリプロジェクト

Question

ライブラリ用のシンプルなソフトウェアを作りたいと思います。私はちょうどC＃でOOPの概念を把握し始めました。私は教授の一人から、クライアントコードがデータを扱うクラスと直接対話しないように、コードにリポジトリクラスを持たせるのがよいことを教えられましたベース。だから、私はWindowsフォームでログインフォームを登録しようとしていますが、いくつか問題があります。

public User GetByUsernameAndPassword(string username, string password) 
    { 
     using (FileStream fs = new FileStream(filePath, FileMode.OpenOrCreate)) 
     { 
      using (StreamReader sr = new StreamReader(fs)) 
      { 
       while (!sr.EndOfStream) 
       { 
        User item = new User(); 
        item.FirstName = sr.ReadLine(); 
        item.LastName = sr.ReadLine(); 
        item.Username = sr.ReadLine(); 
        item.Password = sr.ReadLine(); 
        if (item.Username == username && item.Password == password) 
        { 
         return item; 
        } 
       } 
       return null; 
      } 
     } 
    } 

私のポイントは、私は、私は、新しいユーザーを登録するとき場合、既に同じユーザ名を持つ別のユーザーが存在するかどうかを確認したいということで、そこにある場合、私はメッセージボックスが同じユーザー名をすることはできませんと言って表示させたいです2回使用されます。どうすればこれを達成できますか？返された結果がnullの場合にメッセージボックスを表示させ、結果がクラスUserからオブジェクトを返すときにユーザーにログインするにはどうすればよいですか？

ありがとうございます。

Answer 1

重複するユーザー名を探すことを考える前に、「データベース」の構造を定義することをお勧めします。分では、これは単純なテキストファイルのように見えますが、複数のユーザーを格納するための規定はありません。

アプリケーションのユーザーカウントが低い場合、テキストファイルは問題ありませんが、フォーマットを定義することが重要です（たとえば、1行に1人のユーザー、プロパティはコンマで区切られます）。この場合、起動時にファイル全体をRAMに読み込んでIEnumerable型に格納することをお勧めします。

あなたのアプリケーションのユーザー数が多い場合は（コンテキストによってはそれがないと思われますが）、適切なデータベースソリューション（おそらくSQLベースのもの）を検討することをお勧めします。テキストファイルはこの時点で非効率になります。

これをソートすると、重複を検出することが簡単になります。あなたはユーザーオブジェクトのリストを持っていた場合たとえば、あなたは可能性：

private List<User> users; 

internal bool DoesUsernameExist(string username) 
{ 
    return (this.users.Where(x => x.Username == username).Count() > 0); 
} 

もう一つの事は注意します。パスワードをプレーンテキストで保存するのは良い考えではないでしょう。 https://msdn.microsoft.com/en-us/library/system.security.cryptography.sha1(v=vs.110).aspxを保護するために、業界標準のハッシュアルゴリズムを使用してパスワードをハッシュしてみることをお勧めします。