C#中读取已关闭MySqlDataReader时出现无效操作异常求助

Hey Kevin, let's break down why you're seeing this error and how to fix it.

The Root Cause #

Looking at your ExecuteReader method, you're wrapping MySqlCommand in a using block. When the using block finishes executing, it automatically disposes of the MySqlCommand instance. Here's the problem: the MySqlDataReader returned by command.ExecuteReader() is tightly linked to that command. When the command gets disposed, the reader is closed immediately. So by the time your duplicateNameCheck method tries to read from it, the reader is already shut down—hence the error.

Solution 1: Remove the using Block for MySqlCommand (Manual Cleanup) #

If you need to return the reader to your calling method, don't wrap the command in a using block. Instead, let the calling code handle cleaning up the command and reader once they're done:

public static MySqlDataReader ExecuteReader(string sqlQuery, MySqlConnection connection) {
    MySqlCommand command = new MySqlCommand(sqlQuery, connection);
    return command.ExecuteReader();
}

Then in your duplicateNameCheck method, make sure to dispose of all resources properly:

using (MySqlConnection conn = GetSqlConnection()) {
    conn.Open();
    MySqlCommand cmd = null;
    using (MySqlDataReader reader = ExecuteReader("Your SQL Query", conn)) {
        cmd = reader.Command; // Track the command to dispose later
        while (reader.Read()) {
            // Your read logic here
        }
    }
    cmd?.Dispose(); // Clean up the command
}

Solution 2: Use a Callback to Read Within the using Block (Recommended) #

A cleaner approach is to handle the reader processing inside the scope where the command is still active. You can use an action callback to pass your read logic into the ExecuteReader method:

public static void ExecuteReader(string sqlQuery, MySqlConnection connection, Action<MySqlDataReader> readAction) {
    using (MySqlCommand command = new MySqlCommand(sqlQuery, connection)) {
        using (MySqlDataReader reader = command.ExecuteReader()) {
            // Run the read logic while both command and reader are active
            readAction(reader);
        }
    }
}

Then call it like this in your duplicateNameCheck method:

using (MySqlConnection conn = GetSqlConnection()) {
    conn.Open();
    ExecuteReader("SELECT * FROM YourTable WHERE Name = @Name", conn, reader => {
        while (reader.Read()) {
            // Process your data here—no closed reader issues!
            string name = reader.GetString("Name");
            // ... rest of your logic
        }
    });
}

This way, the using blocks ensure the command and reader are disposed automatically right after you finish reading, and you never try to access a closed reader.

Quick Side Note #

Don't forget that your GetSqlConnection method returns an unopened connection—always call conn.Open() before executing commands, and wrap the connection in a using block to ensure it gets closed properly when you're done.

内容的提问来源于stack exchange，提问作者kevin