在 C# 的 while 循环中使用 IF 条件。假值仍在循环

Question

我不知道为什么我的 while 循环中的 else 值仍在循环，无论结果是登录还是无效。警报对话框始终弹出。

这些是我的代码。

MySqlConnection conn = new MySqlConnection();    
string query = "server=sample.com;port=3306;database=sample;user id=sample;password=sample";
conn.ConnectionString = query;

MySqlCommand cmd = new MySqlCommand("select * from wp_users", conn);
try
{
    conn.Open();


    MySqlDataReader myReader = cmd.ExecuteReader();

    string user1 = "";
    string pass1 = "";
    //bool stopLoop = false; // stop looping for false value;
    while (myReader.Read())
    {

        user1 = myReader[1].ToString(); //datacolumn -> user_login
        pass1 = myReader[2].ToString(); //datacolumn -> user_pass

        if ((user1 == txtUsername.Text) && (pass1 == txtPassword.Text))
        {

            Intent myIntent;
            myIntent = new Intent(Activity, typeof(index));

            string a = user1;

            myIntent.PutExtra("myItem", a);
            StartActivity(myIntent);
        }
        else
        {
            Android.Support.V7.App.AlertDialog.Builder alert = new Android.Support.V7.App.AlertDialog.Builder(Activity);
            alert.SetMessage("Invalid username or password");
            alert.SetPositiveButton("Ok", (senderAlert, args) =>
            {
                alert.Dispose();
            });
            alert.Show();
        }
    }
    myReader.Close();
}
catch (MySqlException ex)
{
    Android.Support.V7.App.AlertDialog.Builder except = new Android.Support.V7.App.AlertDialog.Builder(Activity);
    except.SetTitle("Please report this to our website(error server timeout)");
    except.SetMessage(ex.ToString());
    except.SetPositiveButton("Ok", (senderAlert, args) =>
    {
        except.Dispose();
    });
    except.Show();
}
finally
{
    conn.Close();
}

Answer 1

您应该在if 中使用String.Equals(user1 ,txtUsername.Text.ToString ()) 和String.Equals(pass1 ,txtPassword.Text.ToString ()) 而不是user1 == txtUsername.Text 和pass1 == txtPassword.Text。进行正确的字符串比较

您应该将else 部分移到循环之外。如果用户名和密码正确，则在 if 条件中添加 break; 作为最后一条语句。

您可以维护一个布尔变量，比如isLogin=false; 当登录成功时将此变量设置为 true。循环后，检查标志并决定是否显示登录错误对话框。

Answer 2

在您的问题中，您永远不会说停止读取数据读取或在找到所需记录时使用中断。

                MySqlDataReader myReader = cmd.ExecuteReader();

                string user1 = "";
                string pass1 = "";
                bool stopLoop = false;
                while (stopLoop == false && myReader.Read())
                {

                    user1 = myReader[1].ToString(); //datacolumn -> user_login
                    pass1 = myReader[2].ToString(); //datacolumn -> user_pass

                    if ((user1 == txtUsername.Text) && (pass1 == txtPassword.Text))
                    {
                        stopLoop = true;

                        //rest of the true if code

                        // If you didnt use a stopLoop flag you could have inserted here a break;
                    }
                    else
                    {
                        // your else code
                    }
                }
                myReader.Close();

您也可以查看我对问题 cmets 的建议，以尝试使其变得更好，因为我认为没有必要 ;-)

Answer 3

问题是你的逻辑有点缺陷，让我用一些东西来演示一下。假设您有 user1、user2、user3 等，对 10.. 说。

你有

while reading through list
  if entered user = user item from list
     do happy stuff
  else 
     do unhappy stuff
end while.

所以我输入 user3

if user3 = user1 .. nope say unhappy stuff
if user3 = user2 .. nope say unhappy stuff
if user3 = user3 .. yes do happy stuff
if user4 = user3 .. nope say unhappy stuff
....

如果你有 2000 个用户，你就会有 1999 件不开心的事。

你需要一个像这样工作的循环

item found = false
while reading through list
  if entered user = user from list
   item found = true
    do happy stuff
    break; // because theres no point checking another 1999 !
end while

if item found = false do unhappy stuff

Answer 4

与您的问题相关的代码有 2 个问题。

首先，登录成功时不要中断循环。这可以使用break; 关键字轻松完成。

所以你会得到：

    [...]
    myIntent.PutExtra("myItem", a);
    StartActivity(myIntent);
    break;                               // <----
}
else [...]

---

第二个问题是，在每个循环中，如果登录不正确，您还要处理这种情况。这意味着如果数据与数据库返回的第一个用户的数据不匹配，代码将立即声明登录失败。

您需要跟踪输入是否与任何用户匹配，并且只有当它不匹配任何用户时（因此在 while 循环之后）您才能处理登录失败。

---

至于一些建议：

• 如果这将是一个有实际用户的应用程序，请不要将密码作为纯文本存储在数据库中，对它们进行哈希处理并比较哈希值。请参阅 this post 了解如何对密码进行哈希处理。

• 不要从数据库中获取整个用户表并在循环中检查每个用户，而是使用查询立即检查您正在寻找的用户。数据库针对搜索进行了优化，可以让您更快地编写代码。

  类似：

      using (var cmd = new MySqlCommand("select user_login from wp_users where user_login = @user and user_pass = @pass", conn))
      {
          conn.Open();
  
          // Set parameters
          cmd.Parameters.AddWithValue("@user", user);
          cmd.Parameters.AddWithValue("@pass", pass);
  
          // Get result
          using (var reader = cmd.ExecuteReader(CommandBehavior.SingleRow))
          {
              // If ANY row, there is a user
              if (reader.Read())
              {
                  // Login succesfull
              } 
              else
              {
                  // Login failed
              }
          }
      }