Question

我想将iienumerable集成到我的代码中，以便它只读取，但我不知道在哪里可以实现它。到目前为止，我还没有在互联网上找到任何可以解决我问题的解决方案。截至目前，我只使用List并希望在我的代码中集成ienumerable。此外，如果您发现错误的编码习惯，请随时提出建议。

以下是代码：

public static List<GuitarItems> GetGuitarItems(string itemCategory)
{
    List<GuitarItems> list = new List<GuitarItems>();
    string query = string.Format("SELECT * FROM guitarItems WHERE brand LIKE @brand");

    try
    {
        conn1.Open();
        command1.CommandText = query;
        command1.Parameters.Add(new SqlParameter("brand", itemCategory));
        SqlDataReader reader = command1.ExecuteReader();

        while (reader.Read())
        {
            int id = reader.GetInt32(0);
            string type = reader.GetString(1);
            string brand = reader.GetString(2);
            string model = reader.GetString(3);
            double price = reader.GetDouble(4);
            string itemimage1 = reader.GetString(5);
            string itemimage2 = reader.GetString(6);
            string description = reader.GetString(7);
            string necktype = reader.GetString(8);
            string body = reader.GetString(9);
            string fretboard = reader.GetString(10);
            string fret = reader.GetString(11);
            string bridge = reader.GetString(12);
            string neckpickup = reader.GetString(13);
            string bridgepickup = reader.GetString(14);
            string hardwarecolor = reader.GetString(15);

            GuitarItems gItems = new GuitarItems(id, type, brand, model, price, itemimage1, itemimage2, description, necktype, body,
                fretboard, fret, bridge, neckpickup, bridgepickup, hardwarecolor);
            list.Add(gItems);
        }
    }
    finally
    {
        conn1.Close();
        command1.Parameters.Clear();
    }

    return list;
}

然后这是另一个代码：

private void FillPage()
{
    List<GuitarItems> itemList = new List<GuitarItems>();
    List<string> itemListPage = new List<string>();

    itemList = ConnectionClassGuitarItems.GetGuitarItems(brandType);

    StringBuilder sb = new StringBuilder();

    foreach (GuitarItems gList in itemList)
    {
        itemListPage.Add("GuitarItemsIbanezDetails" + (x + 1) + ".aspx");

        sb.Append(
                string.Format(
                    @"
                    <div class='one-two'>
                        <a href='{3}' runat="'server'"><img runat="'server'" src='{0}'/></a>
                        <div class='content'>
                            <div id='label'>{1} {2}</div>
                        </div>

                </div>", gList.ItemImage1, gList.Brand, gList.Model, itemListPage[x]));

        x++;

    }


    lblOutput.Text = sb.ToString();

}

Answer 1

从技术上讲，List<T> 是IEnumerable<T>，但我理解你的意思。如果您返回List<T>，那么该列表可能会被修改。你想要返回一些可以在没有任何人修改其内容的情况下传递的内容。

你可以做两件事来实现这个目标。第一个可能是你所需要的。

首先，改变这个

public static List<GuitarItems> GetGuitarItems(string itemCategory)

到此：

public static IReadOnlyList<GuitarItems> GetGuitarItems(string itemCategory)

如果你这样做而没有别的，那么函数的返回值将被转换为IReadOnlyList<T>。该界面不允许修改列表。（它不会阻止改变列表中项目的属性 - 这是一个完全独立的事情。）

List<T>可以投放为IReadOnlyList<T>，因此您无需更改方法中的任何其他内容。您可以像现在一样创建List<GuitarItems>，但只需更改函数的返回类型即可将其作为IReadOnlyList<GuitarItems>返回。

这可能会完成你所需要的。但是，如果您特别关注修改该列表内容的其他内容，您可以更进一步。

例如，即使从集合返回的项目被转换为IReadOnlyList<GuitarItems>，有人仍然可以这样做：

var readonlyList = GetGuitarItems("category"); // returns IReadOnlyList<GuitarItems>
var list = readonlyList as List<GuitarItems>;
list.Clear(); // Oh, no - they've modified the list anyway!

这将起作用，因为从函数返回的项实际上是List。没有人应该尝试将其作为其他东西 - 他们应该使用你返回的类型。但由于一些奇怪的原因，有人可以这样做。

为了进一步预防，您可以更改函数的返回值：

return list;

对此：

return list.AsReadOnly();

结果是，现在您没有将List<GuitarItems>广告投放回IReadOnlyList<GuitarItems>。 AsReadOnly()创建了一个新集合ReadOnlyCollection<GuitarItems>，并将 <{em> <{em> <\ n> {现在不仅返回的类型表明该列表无法修改，但它实际上无法修改，因为它是一个只读集合。

查看它的一个好方法是方法的返回值应该表明您希望消费者使用它做什么。如果您希望他们收到他们不应修改的列表，请返回IReadOnlyList<GuitarItems>。

相关问题是您是应该返回IReadOnlyList<T>还是IEnumerable<T>。 IReadOnlyList<T>更通用，也不代表可以更改的列表。您会更频繁地看到它的使用。不同之处在于IEnumerable<T>可以表示执行的查询。如果要枚举的项不止一次枚举，那么可能会导致多次执行查询。

在上面的示例中，如果您将IEnumerable<T>更改为List<GuitarItems>，那么您仍然真正返回IEnumerable<GuitarItems>，因此多次枚举该列表不会重复执行查询。但是现在有List<GuitarItems>的消费者无法知道，所以他们会想要做这样的事情：

IEnumerable<GuitarItems>

确保查询执行一次并放入一个列表，他们可以根据需要枚举多次。

返回var items = GetGuitarItems("category").ToList();让他们知道他们拥有的内容绝对是一个列表，而不是每次枚举时都会被查询的内容。

如果使用Resharper，这一点就很明显了。如果您有一个IReadOnlyList<GuitarItems>，并且您不止一次枚举它，它会警告您。该项可能真的是一个列表或数组，但你不能知道。

Answer 2

你想要达到什么目标？

List<T>它已经是IEnumerable<T>，因为它实现了IEnumerable<T>接口，因此您可以将方法参数中的List<T>视为IEnumerable<T>。

如何返回只读IEnumerable <t>而不是List <t>

2 个答案: