Question

出于好奇，我想知道如何最好地实现可用于避免CA1006警告的类

CA1006：Microsoft.Design：考虑一种设计，其中'IReader.Query（String，String）'不嵌套泛型类型'IList（Of IDictionary（Of String，Object））'。

这是返回泛型类型

的方法

public virtual IList<IDictionary<string, object>> Query(
    string fullFileName, 
    string sheetName)
{
    using (var connection = new OdbcConnection(
        this.GetOdbcConnectionString(fullFileName)))
    {
        connection.Open();
        return connection
            .Query(string.Format(
                CultureInfo.InvariantCulture,
                SystemResources.ExcelReader_Query_select_top_128___from__0_,
                sheetName))
            .Cast<IDictionary<string, object>>()
            .ToList();
    }
}

像

这样的东西

SourceData<T, U> Query(string fullFileName, string sheetName)
SourceData Query(string fullFileName, string sheetName)

编辑：

按照Marc的建议，我在这个类中封装了嵌套泛型

public class QueryRow : List<KeyValuePair<string, object>>
{
    protected internal QueryRow(IEnumerable<KeyValuePair<string, object>> dictionary)
    {
        this.AddRange(dictionary.Select(kvp => kvp));
    }
}

Answer 1

首先，请注意它是一个设计指南，而不是编译器错误。这里有一个有效的方法是：忽略它。

另一个可能是 - 封装它;即返回List<QueryRow>，其中QueryRow是带有索引器的IDictionary<string,object>上的浅包装器，即

public class QueryRow {
    private readonly IDictionary<string,object> values;
    internal QueryRow(IDictionary<string,object> values) {
        this.values = values;
    }
    public object this[string key] {
        get { return values[key]; }
        set { values[key] = value; }
    }
}

然后，因为这是通过dapper访问的，请填写：

var data = connection.Query(....)
        .Select(x => new QueryRow((IDictionary<string,object>)x).ToList()

另一种选择（我不是非常喜欢），可能是：return DataTable。

输入DataTable后，

_{开始洗手......哇！现在两次}

如何实现通用的IEnumerable或IDictionary来避免CA1006？

1 个答案: