Я перехожу с Java на C# теперь, когда понял, что предпочитаю функции языка C# функциям Java, но у меня есть небольшая проблема. Я считаю, что в MySQL Connector/J и JDBC одно из моих приложений позволяло выполнять несколько PreparedStatement
, в то время как другое было открыто, например, я мог бы выполнить запрос, который возвращает ResultSet
, и пока этот ResultSet
все еще открыт, я мог бы открыть еще один PreparedStatement
и получить еще один ResultSet
, или я мог бы просто выполнить обновление на основе данных, которые я получил от моего первого ResultSet
(т. е. вставить значение соли и обновить столбец пароля с помощью хэша SHA512, когда я понимаю, что строка имеет открытый текстовый пароль в столбец пароля).
Однако с Connector/NET я понял, что всякий раз, когда я пытаюсь это сделать, я получаю эту ошибку: MySql.Data.MySqlClient.MySqlException: There is already an open DataReader associated with this Connection which must be closed first.
Есть ли простой способ исправить эту ошибку, возможно, какие-либо другие реализации моста MySQL для .NET? Я действительно не хочу создавать много подключений к БД в одном приложении, хотя мне может понадобиться создать по одному для каждого потока в моем приложении (как в ThreadLocal). Соединение с БД ThreadLocal поможет, когда я выполняю два запроса одновременно двумя разными методами, но, очевидно, я не могу разделить эти две команды на разные потоки и не хочу создавать лишние потоки.
Кстати, вот сам код. Да, я могу перенести код обновления вниз после того, как закрою читалку, но у меня есть еще много подобных методов, и некоторые из них исправить сложнее, чем этот:
MySqlConnection con = DatabaseConnection.GetConnection();
MySqlCommand cmd = con.CreateCommand();
cmd.CommandText = "SELECT `id`,`password`,`salt`,`pin`,`gender`,`birthday` FROM `accounts` WHERE `name` = '" + AccountName + "'";
MySqlDataReader reader = cmd.ExecuteReader();
if (reader.Read())
{
AccountId = reader.GetInt32(0);
string passhash = !reader.IsDBNull(1) ? reader.GetString(1) : null;
string salt = !reader.IsDBNull(2) ? reader.GetString(2) : null;
m_pin = !reader.IsDBNull(3) ? reader.GetString(3) : null;
Gender = !reader.IsDBNull(4) ? reader.GetByte(4) : WvsCommon.Gender.UNDEFINED;
m_birthday = !reader.IsDBNull(5) ? reader.GetInt32(5) : 0;
if (!HashFunctions.HashEquals(pwd, HashAlgorithms.SHA512, passhash + salt))
{
if (passhash == pwd || salt == null && HashFunctions.HashEquals(pwd, HashAlgorithms.SHA1, passhash))
{
salt = HashFunctions.GenerateSalt();
passhash = HashFunctions.GenerateSaltedSha512Hash(pwd, salt);
MySqlCommand update = con.CreateCommand();
update.CommandText = "UPDATE `accounts` SET `password` = '" + passhash + "', `salt` = '" + salt + "' WHERE `id` = " + AccountId;
update.ExecuteNonQuery();
update.Dispose();
}
}
}
reader.Close();
cmd.Dispose();
Если перемещение кода обновления является единственной или лучшей возможностью, я полагаю, что мне придется обойтись этим, но я хочу сначала получить больше идей о других возможностях, а затем выбрать вариант.