Как закрыть соединение с БД SQLite?

Question

[Евгений Иванов]

Вот рабочий код

// Метод возвращает объект SqliteDataReader, содержащие данные, требуемые в запросе.
public static SqliteDataReader GetSqliteDataReader(string pathToBase, string sqlCommandText)
{
SqliteConnection connection = new SqliteConnection("Data Source="+pathToBase);
connection.Open();
SqliteCommand sqliteCommand = new SqliteCommand();
sqliteCommand.Connection = connection;
sqliteCommand.CommandText = sqlCommandText;
SqliteDataReader sqliteDataReader = sqliteCommand.ExecuteReader();
//connection.Close(); // Тут нельзя закрыть т.к. получим пустой sqliteDataReader
return sqliteDataReader;
//connection.Close(); // Тут не получится закрыть т.к. выше return - уже вышли из метода.
}

Пишу в Unity.
Насколько я знаю, C# сам удаляет мусор (там мы не удаляем объекты).
Возможно C# сам закрывает соединение? А как в Unity?
Как закрыть соединение с БД SQLite и нужно ли это делать?

PS
"Unity это C#?" - в плане того, что C# .NET (весь фреймворк) я могу использовать в Unity и это всё будет работать, скажем под андройд (где нет .NET)?

Comments

[сергей кузьмин]

а вы возвращайте не SqliteDataReader
а данные и проблем будет меньше

[alexalexes]

C таким подходом вам нужно connection где-то в this оставлять, чтобы вовремя закрыть соединение потом.

Answer 1

[Василий Банников]

Вычитай данные из SqliteDataReader и проблем с закрытием не будет.

Если не закроешь, то, конечно, рано или поздно сработает сборщик мусора и финализатор, но этот момент заранее не известен и пока он не произойдёт - ты не сможешь открыть базу второй раз.

Либо заверни его в какой-нибудь свой класс, который можно будет задиспозить потом

Comments

[Евгений Иванов]

Василий Банников

Вычитай данные из SqliteDataReader и проблем с закрытием не будет.

Не понял. Вычитать данные - это как?

задиспозить

Вот так пойдет?

// Класс  - осуществляющий соединение с БД.
public class GetSqliteDataReaderInstance : IDisposable
{
SqliteDataReader sqliteDataReader;
public SqliteDataReader _sqliteDataReader{ get{return sqliteDataReader;} set{sqliteDataReader=value;} }

public GetSqliteDataReaderInstance(string pathToBase, string sqlCommandText)
{
SqliteConnection connection = new SqliteConnection("Data Source="+pathToBase);
connection.Open();
SqliteCommand sqliteCommand = new SqliteCommand();
sqliteCommand.Connection = connection;
sqliteCommand.CommandText = sqlCommandText;
SqliteDataReader sqliteDataReader = sqliteCommand.ExecuteReader();
this.sqliteDataReader = sqliteDataReader;
}


// Хз как это работает и что должно быть внутри, но интерфейс IDisposable обязывает меня реализовать метод.
void IDisposable.Dispose()
{
GC.SuppressFinalize(this);
}
}



// Статический класс - обертка.
public static class SQLiteData
{
public static SqliteDataReader GetSqliteDataReader(string pathToBase, string sqlCommandText)
{
// Директива using должна гарантировано и безопасно уничтожить экземпляр класса, а вместе с ним и подключение к юазе т.к. оно в нем.
using(GetSqliteDataReaderInstance getSqliteDataReaderInstance = new GetSqliteDataReaderInstance(pathToBase, sqlCommandText))
{
return getSqliteDataReaderInstance._sqliteDataReader;
}
}
}

[Василий Банников]

Евгений Иванов, обхртка фигня.
Покажи как ты уже планируешь использовать SqlDataReader

[Евгений Иванов]

Василий Банников,

string sqlCommandText = "SELECT * FROM table1 WHERE field1='test'  LIMIT 1";
SqliteDataReader sqliteDataReader = SQLiteData.GetSqliteDataReader(pathToBase, sqlCommandText); // Получили из класса выше.
i++; arrayOfIntParameters[i] = Convert.ToInt32(sqliteDataReader["id"]);
s++; arrayOfStringParameters[s] = sqliteDataReader["text1"].ToString();

Пояснения этой каши.
1) В программе мне надо многократно обращаться к БД. Я не хочу дублировать код - открытия и настройки соединения. Поэтому я вынес это в класс GetSqliteDataReader.

2) Я возвращаю SqliteDataReader т.к. этот объект содержит всю выборку из БД.
Обычно идет цикл while (sqliteDataReader.Read()) в котором идет выборка данных.
Но у меня отдельный метод, а значит мне нужно не просто выбрать данные, но вернуть их из метода.
С учетом того, что данные в БД разного типа - int, string - у меня нет единого объекта для их хранения или такой объект будет сложным (в PHP это был (и есть) бы массив - он там динамический и не типизированный). Так зачем что то придумывать, если уже есть SqliteDataReader?

i++; arrayOfIntParameters[i] = Convert.ToInt32(sqliteDataReader["id"]);
Это стоит переписать. Кроме i++ я так и не разобрался как автоматически конвертировать данные sqliteDataReader в string или int (там должен быть свой метод).

[Василий Банников]

Евгений Иванов,
1. у тебя же не бесконечное количество разных таблиц, зачем тебе такая динамика?
2. DataReader - это достаточно низкоуровневая абстракция и так её использовать не стоит.
3. Лучше возьми Dapper в таком случае

Answer 2

[freeExec]

sqliteConnection.Close()