C＃ユーザー名がヌルでないか、空でないか、または既に取得されていないかを確認してください。

Question

私が言ったように、ユーザー名がヌルでないか、または空であることをレジスタでチェックします。

​​

私は、コードを修正する助けてください、問題がどこかにここにある：

static void Register() { 
     Console.WriteLine("What will be your username?"); 
     while (string.IsNullOrEmpty(name) && IsUsernameAlreadyTaken(name)==false) 
     { 
      name = Console.ReadLine(); 
      if (IsUsernameAlreadyTaken(name)) 
      { 
       Console.WriteLine("That's already taken"); 
      } 
      if (string.IsNullOrEmpty(name)) 
      { 
       Console.WriteLine("Why?"); 
      } 
     } 
} 

今

私はプログラムにすでに名前を付けた場合：

static bool IsUsernameAlreadyTaken(string name)//Username Checker 
    { 
     SqlDataAdapter sda = new SqlDataAdapter("SELECT count(*) FROM Login WHERE Nev='" + name + "'", Con); 
     sda.Fill(dt); 
     if (dt.Rows[0][0].ToString() == "1") return true; 
     else return false; 
    } 

またはこのwhileループで

データベースに存在していて、それは既に取られているとは言えません。

Answer 1

問題はここにある：それはnullの場合nameがnullまたは

空であることを保証するために

1. チェック：

   while (string.IsNullOrEmpty(name) && IsUsernameAlreadyTaken(name)==false) 
   

   あなたが行うコードを求めている何がこれですまたは空の場合は、すでに使用されているかどうかを確認してください。

明らかに、このロジックには深刻な欠陥があります。

あなたはこれを意味しましたか？

while (string.IsNullOrEmpty(name) || IsUsernameAlreadyTaken(name)==false)