0

私はこの問題に遭遇しました：

エラー:この接続に関連付けられている開いている DataReader が既に存在します。これを最初に閉じる必要があります。

私のコードを見てください：

 Dim sqlQuery As String = "SELECT * FROM users"
    Dim myAdapter As New MySqlDataAdapter

    If txtUsername.Text = String.Empty And txtPassword.Text = String.Empty Then
        MsgBox("Enter username and password", MsgBoxStyle.Exclamation, "Tea Sparkle POS")
    Else
        Dim sqlquerry = "Select * From users where username = '" + txtUsername.Text + "' And password= '" + txtPassword.Text + "'"
        Dim myCommand As New MySqlCommand()
        myCommand.Connection = SQLConnection
        myCommand.CommandText = sqlquerry
        'Starting The Query
        myAdapter.SelectCommand = myCommand
        Dim mydata As MySqlDataReader
        mydata = myCommand.ExecuteReader()
        'To check the Username and password and to validate the login a
        If mydata.HasRows = 0 Then
            MsgBox("Invalid Login")
            txtPassword.Clear()
            txtUsername.Clear()
        Else
            Dim authorityid = 0
            While mydata.Read()
                authorityid = mydata.GetInt32("authorityid")
            End While
            MsgBox("Welcome " + txtUsername.Text + "!")
            If authorityid = 1 Then
                MainForm.Show()
            Else
                MainForm.Show()
            End If
            Me.Hide()
        End If
    End If


    Private Sub Login_Load(sender As System.Object, e As System.EventArgs) Handles MyBase.Load

    SQLConnection.ConnectionString = ServerString

    Try
        If SQLConnection.State = ConnectionState.Closed Then
            SQLConnection.Open()
        Else
            SQLConnection.Close()
        End If
    Catch ex As Exception
        MsgBox(ex.ToString)
    End Try

End Sub

このエラーは次の行にあります。

 mydata = myCommand.ExecuteReader()

これの何が問題なのですか？どんな助けでも本当に感謝しています。

4

1 に答える 1

3

これの何が問題なのですか？

さて、既存の接続を再利用しているようです:

myCommand.Connection = SQLConnection

そうしないでください。データベースとの通信が必要になるたびに新しい接続を作成し、完了したら閉じます。Usingステートメントを使用して、例外がスローされた場合でも接続が確実に閉じられるようにします。

さらに、Usingコマンドにはステートメントを使用し、リーダーには別のステートメントを使用します。これらはすべて、閉じる必要があるリソースです。

ああ、UI スレッドでこれを行っているようにも見えますが、データベースへのアクセスが進行している間は UI が応答しなくなるため、これは悪い考えです。

于 2013-02-05T19:28:17.750 に答える