当存在ASP.NET用户名时，请更改为用户名（x）

Question

将SQL成员资格提供程序用于ASP.NET成员资格。 我使用first.last作为用户名，该用户名是通过表格中填写的用户详细信息以编程方式创建的。

当用户提交表单时，我希望能够检查用户名是否存在，如果存在，则将其更改为username1，检查username1是否存在，并使其命名为username2（如果存在），依此类推，直到成为唯一的用户名为止。

我不知道如何执行存储过程，所以我试图使用SQLDataReader来检查用户名是否存在。

问题是我的循环。 逻辑基本上是设置一个布尔值并保持循环并将1加到计数器上，直到找不到重复为止。 我已经遍历了很多次，即使将boolean设置为false，它也会不断循环播放。

有什么想法吗？

后面的代码：

protected void Membership_add()
{
    SqlConnection con = new SqlConnection(connectionString);
    string NewUserNameString = FirstName.Text + "." + LastName.Text;

    //Check for duplicate aspnet membership name and add a counter to it if exists
    // Check for valid open database connection before query database
    bool match = true;
    SqlDataReader _SqlDataReader = null;
    string TestNameString = NewUserNameString;
    string selectDupeString = "SELECT UserId FROM aspnet_Users WHERE UserName = '" + TestNameString + "'";
    SqlCommand SQLdatareaderCmd = new SqlCommand(selectDupeString, con);
    int UserNameCounter = 0;
    con.Open();
    while (match = true)
    {
        //Open the connection

        try
        {
            //Read the table
            _SqlDataReader = SQLdatareaderCmd.ExecuteReader();
        }
        catch (Exception ex)
        {
            lblDatareaderEx.Text = "An Exception occurred. " + ex.Message + " " + ex.GetType().ToString();
        }

        if (_SqlDataReader.HasRows)
        {
            //match = true;
            //increase counter by 1 for each record found and change First.Name to First.Namex
            TestNameString = NewUserNameString;
            UserNameCounter = UserNameCounter + 1;
            TestNameString = TestNameString + UserNameCounter.ToString();
            _SqlDataReader.Close();
            _SqlDataReader.Dispose();
            selectDupeString = "SELECT UserId FROM aspnet_Users WHERE UserName = '" + TestNameString + "'";
            SQLdatareaderCmd = new SqlCommand(selectDupeString, con);
        }
        else
        {
            // close sql reader
            _SqlDataReader.Close();
            _SqlDataReader.Dispose();
            //get out of loop
            match = false;
        }
    }
    con.Close();
    con.Dispose();
}

Answer 1

这行：

while (match = true)

做作业。
如果要使代码正常工作，则必须进行比较：

while (match == true)

或者，由于您的变量已经是布尔变量，因此您可以直接使用该变量：

while(match)

Answer 2

目前，您正在设置match而不是比较它的值。

尝试将while (match = true)为while (match == true)

Answer 3

如果将代码分成较小的块，则代码将变得更容易阅读。

private string MembershipAddUser(string firstName, string lastName)
{
    string username = firstName + "." + lastName;

    int i = 0;
    while (UserExists(username))
    {
        i++;
        username = firstName + "." + lastName + i.ToString();
    }

    return username;
}

private bool UserExists(string username)
{
    string sql = "SELECT COUNT(*) FROM dbo.aspnet_Users WHERE UserName = @UserName";

    SqlConnection connection = new SqlConnection(connectionString);

    SqlCommand command = new SqlCommand(sql, connection);
    command.Parameters.AddWithValue("@UserName", username);

    using (connection)
    {
        connection.Open();

        int count = (int) command.ExecuteScalar();
        return (count != 0);
    }
}