根据我遵循的课程连接到数据库

Question

我正在学习一个在线课程，在课程中他们解释了如何从数据库中检索数据。创建连接和命令由DbProviderFactories 类完成。我了解课程中的代码，但是否使用using 进行必要的连接、命令和阅读器？另外，是否有必要进行空检查？代码看起来很杂乱，如果您的数据库中有很多模型（大陆、国家、货币……），则需要大量复制/粘贴，这很糟糕吗？

所以问题真的是，下面的代码是好是坏，还有哪些可以改进的地方？目标是使用 SQLite 作为数据库提供者。这适用于以下方法吗？

public static ObservableCollection<Continent> GetContinents()
{
    var continents = new ObservableCollection<Continent>();
    var provider = ConfigurationManager.ConnectionStrings["DbConnection"].ProviderName;
    var connectionString = ConfigurationManager.ConnectionStrings["DbConnection"].ConnectionString;

    using (var connection = DbProviderFactories.GetFactory(provider).CreateConnection())
    {
        if (connection == null) return null;

        connection.ConnectionString = connectionString;
        connection.Open();

        using (var command = DbProviderFactories.GetFactory(provider).CreateCommand())
        {
            if (command == null) return null;

            command.CommandType = CommandType.Text;
            command.Connection = connection;
            command.CommandText = "SELECT * FROM Continent";

            using (var reader = command.ExecuteReader())
                while (reader.Read())
                    continents.Add(new Continent(reader["Code"].ToString(), reader["EnglishName"].ToString()));
        }
    }

    return continents;
}

Answer 1

使用 using 连接、命令和阅读器有必要吗？

是的。 这里我注释了代码

using (var command = DbProviderFactories.GetFactory(provider).CreateCommand()) // here you've created the command
            {
                if (command == null) return null;

                command.CommandType = CommandType.Text;
                command.Connection = connection;
                command.CommandText = "SELECT * FROM Continent";

                using (var reader = command.ExecuteReader()) //Here you're reading what the command returned.
                    while (reader.Read())
                        continents.Add(new Continent(reader["Code"].ToString(), reader["EnglishName"].ToString()));
            }

另外，是否有必要进行空检查？

它可以返回空数据，所以绝对是的

代码看起来很混乱

这就是程序员的一生。对对象使用循环将节省空间。