SqlDataReader 总是读为 false

Question

我无法弄清楚为什么这个函数会导致错误，即使它看起来是正确的。

这并没有错。有什么解决办法吗？

连接字符串或 SQL 相关函数没有问题。我的功能有问题。我使用 SQL Server。

ESEMKABank > Employee:

EmployeeId    int IDENTITY(1, 1)
Username      nvarchar(64)
Password      nvarchar(64)
FullName      nvarchar(64)
RoleId        int                     -- <-- FOREIGN KEY

对于 VB.NET：

Imports System.Data.SqlClient

Public Class Form_Login

    'Dimensions
    Dim theConnection As New SqlConnection
    Dim theCommand As New SqlCommand
    Dim theDataReader As SqlDataReader
    Dim theDataAdapter As New SqlDataAdapter

    'Initialization
    Private Sub Form_Login_Load(sender As Object, e As EventArgs) Handles Me.Load
        theConnection.ConnectionString = "SERVER = MORGAN\SQLEXPRESS; DATABASE = ESEMKABank; INTEGRATED SECURITY = TRUE;"

        'Parameterize
        theCommand.Parameters.Add("@Username", SqlDbType.VarChar).Value = Me.TextBox_Username.Text
        theCommand.Parameters.Add("@Password", SqlDbType.VarChar).Value = Me.TextBox_Password.Text

    End Sub

    'Login
    Private Sub Button_Login_Click(sender As Object, e As EventArgs) Handles Button_Login.Click
        If Me.TextBox_Username.Text = "" Or Me.TextBox_Password.Text = "" Then
            Me.PictureBox_Wrong.Visible = True
        Else
            Using theDataReader

                'Declaration
                theCommand.Connection = theConnection
                theCommand.CommandText = "SELECT Username, Password FROM Employee WHERE Username = @Username AND Password = @Password"
                theConnection.Open()
                theDataReader = theCommand.ExecuteReader()

                If theDataReader.Read() Then

                    'Dispose!
                    theCommand.Dispose()
                    theConnection.Close()
                    SqlConnection.ClearPool(theConnection)

                    Me.PictureBox_Wrong.Visible = False
                Else

                    'Dispose!
                    theCommand.Dispose()
                    theConnection.Close()
                    SqlConnection.ClearPool(theConnection)

                    Me.PictureBox_Wrong.Visible = True
                End If
            End Using
        End If

    End Sub

End Class

Answer 1

不要在 Form_Load 中设置参数值。在这一点上，它们只是空字符串。在设置.CommandText 属性之后设置参数值。

当我在这里时，假设用户在第一次尝试时输错了密码。此代码将在第二次尝试时总是失败 - 即使他们已更正密码 - 因为您已释放连接。在 .Net 中，最佳做法是在大多数情况下使用全新的连接对象（也包含在 Using 块中），这样您就可以避免此类问题并利用连接池。

最后，这段代码意味着您以纯文本形式存储密码，而不是使用加盐哈希。这是非常令人难以置信的错误，我什至不会留下显示其他修复的示例代码，因为我不想留下执行用户名/密码检查的示例，让其他人看到不涉及哈希密码。