如何在 C# 中将数据库值存储在字典中

Question

当我收到一个国家名称时，我想在我的数据库中搜索一个国家名称列表并获取与国家名称相关联的 id。目前我有；

    public static int GetCountryId(string countryName)
    {
        int countryId = 0;
        if (!string.IsNullOrEmpty(countryName))
        {
            var listOfCountries = GetCountries();
            var match = listOfCountries.FirstOrDefault(item => (item.Name).Contains(countryName));
            if (match != null)
            {
                countryId = match.Id;
            }
        }
        return countryId;
    }

    private static List<Country> GetCountries()
    {
        string query = $"SELECT Id, Name FROM Countries";
        List<Country> cases = Common.GetCollection<Country>(Properties.Settings.Default.DbConnectionString, query);
        return cases;
    }

但是，如您所见，每次我想获取国家名称列表时都必须查询数据库。我想拥有它，以便将此列表存储在字典中，而我可以访问字典。

有谁知道我可以如何改进我的代码，这样我就不必每次都访问数据库了？

Answer 1

像这样修改你的方法：

private static List<Country> countries;

private static List<Country> GetCountries()
{
    if (countries == null || countries.Count == 0)
    {
        string query = $"SELECT Id, Name FROM Countries";
        countries = Common.GetCollection<Country>(Properties.Settings.Default.DbConnectionString, query);
    }

    return countries;
}

Answer 2

当你的程序第一次启动时，如何创建一个静态构造函数来填充一个国家 ID 的字典，这样你只需要查询一次数据库。那么调用 GetCountryId 就可以使用那个字典了吗？

private static Dictionary<string, int> CountryIds;

public static NameOfYourClass(){
    CountryIds = new Dictionary<string, int>();
    string query = $"SELECT Id, Name FROM Countries";
    List<Country> cases = Common.GetCollection<Country>(Properties.Settings.Default.DbConnectionString, query);
    foreach (country Country in cases)
    {
        CountryIDs.Add(Country.Name, Country.Id);
    }        
}

public static int GetCountryId(string countryName)
{
    if(!CountryIds.Contains(countryName) return 0;
    return CountryIds[countryName];
}

Answer 3

让我们为此做一个延迟加载！

private Dictionary<string, Country> _countryNames = null;

public Dictionary<string, Country> CountryNames
{
    get
    {
         if(_countryNames == null)
         {
             _countryNames = new Dictionary<int, Country>();
             foreach(var country in GetCountries())
             {
                 _countryNames.Add(country.Name, country)
             }
         }
         return _countryNames;
    }
}

public static int GetCountryId(string countryName)
{
    Country result;
    CountryNames.TryGetValue(countryName, out result);
    if (result == null) return 0;
    return result.Id;
}

private static IEnumerable<Country> GetCountries()
{
    string query = "SELECT Id, Name FROM Countries";
    return Common.GetCollection<Country>(Properties.Settings.Default.DbConnectionString, query);
}

但通常让数据库做这件事会更好：根据需要在那里运行查询，将过滤器字符串传递给数据库。不幸的是，Common.GetCollection<T>() 对我们隐藏了这种能力。 query 变量应如下所示：

string query = "SELECT Id, Name FROM Countries WHERE Name = @CountryName";

但从这里的问题中不清楚如何提供@CountryName 参数值。您应该不 做的是使用字符串替换或插值将值直接包含在查询字符串中。那将是非常糟糕；它创建了一种称为 SQL 注入的严重形式的安全问题。

Answer 4

你可能有一个这样的公共字典：

public static Dictionary<int, string> countries = new Dictionary<int, string>();

如果之前没有填充，该方法将填充字典；

private static void GetCountries()
    {
        if(countries.Count == 0)
        {
              string query = $"SELECT Id, Name FROM Countries";
              countries = Common.GetCollection<Country>(Properties.Settings.Default.DbConnectionString, query)
              .ToDictionary(x => x.Id, x=> x.Name);
        }
    }


public static int GetCountryId(string countryName)
{
    return countries.Contains(countryName) CountryIds[countryName] : 0;
}