在简单的 WinForms 应用程序中管理数据访问

Question

我有一个使用 SQLite 的简单 WinForms 数据输入应用程序。它将始终是单用户应用程序，并且始终具有本地数据库。我有多个选项卡，其中 UserControls 作为选项卡的内容。每次选择选项卡时，都会初始化相应的 UserControl，并删除旧的 UserControl（使用 TabPage.Controls.Remove）。

每个 UserControl 都会初始化一个通用 DataAccess 对象，该对象包装所有数据库内容，并且可以与任何选项卡内容一起重用。问题是我在选项卡（UserControl）的生命周期内有一个打开的 SQLiteConnection。我在其他地方读到这不是一个好的做法。我不想在设计上过度使用复杂的数据层和业务对象层，部分是因为我不知道如何做到这一点，部分是因为我认为这个应用程序没有必要。

我基本上在内存中保留相同的连接、适配器、DataTable、SqlCommand 等对象，然后使用不同的 sql 查询参数重用它们，并使用其他方法（如 RowCount）获取缓存的数据。我在使用 LoadData 方法时遇到问题，因为它没有清除 DataTable 中以前的查询结果，所以我一开始就手动执行此操作。

我尝试找出一种将“using”与 SQLiteConnection 和其他对象一起使用的方法，但随后我必须重做整个 DataLoad 内容或类似的内容来处理 RowCount 等简单的内容。所以我只是在寻找有关这种数据访问方法的建议和评论。

下面是我的 DataAccess 类。

public class DataAccess
{
    private SQLiteConnection connection = new SQLiteConnection(Global.DbConnectionString);
    private DataTable dataTable = new DataTable();
    private SQLiteDataAdapter dataAdapter = new SQLiteDataAdapter();
    private SQLiteCommandBuilder commandBuilder = new SQLiteCommandBuilder();
    private SQLiteCommand command = new SQLiteCommand();
    private BindingSource bindingSource = new BindingSource();


    public DataAccess()
    {
        dataAdapter.SelectCommand = command;
        commandBuilder.DataAdapter = dataAdapter;
        bindingSource.DataSource = dataTable;
    }

    ~DataAccess()
    {
        connection.Dispose();
    }

    public BindingSource BindingSource
    {
        get { return bindingSource; }
    }


    ///*
    public void LoadData(string sql, Dictionary<string, string> parameters)
    {
        try
        {
            dataTable.Clear();
            command.Connection = connection;

            // Ignore sql parameter if we already have CommandText. This assumes sql never changes per instance
            if (command.CommandText == null)
                command.CommandText = sql;

            foreach (KeyValuePair<string, string> parameter in parameters)
            {
                if (command.Parameters.Contains(parameter.Key))
                    command.Parameters[parameter.Key].Value = parameter.Value;
                else
                {
                    command.Parameters.Add(new SQLiteParameter(parameter.Key));
                    command.Parameters[parameter.Key].Value = parameter.Value;
                }
            }
            dataAdapter.Fill(dataTable);

        }
        catch (SqlException)
        {
            MessageBox.Show("Data Problem, need to display what's wrong later");
        }

    }//*/

    public int RowCount()
    {
        return dataTable.Rows.Count;
    }

    public string GetFieldValue(int row_index, string column_name)
    {
        return dataTable.Rows[row_index][column_name].ToString();
    }

    public void Save()
    {
        dataAdapter.Update(dataTable);
    }

    public void NewRow(Dictionary<string, string> fields)
    {
       DataRow dataRow = dataTable.NewRow();

        foreach (KeyValuePair<string, string> field in fields)
            dataRow[field.Key] = field.Value;

        dataTable.Rows.Add(dataRow);
    }
}

Answer 1

如果您想做得很好，您应该创建一个数据访问层，该层将公开获取数据并修改数据的方法。该层将在必要时打开连接，然后关闭它。您可以在其顶部添加一个缓存层。你的 GUI 只会使用较低层的数据对象。

这不是一个小的重写，所以如果你当前的解决方案有效，并且你不想投入太多精力，那么就这样保留它，这也没有那么糟糕。如果它是一个简单的程序，那么这个简单的解决方案就可以了。