1

我有两个自定义类。

第一类包含基本数据：

public class Request
{
    public Request(int lineID, string partNo, int qty, int reasonID, int typeID)
    {
        LineID = lineID;
        PartNo = partNo;
        Qty = qty;
        ReasonID = reasonID;
        TypeID = typeID;
    }
    public int LineID { get; private set; }
    public string PartNo { get; private set; }
    public int Qty { get; internal set; }
    public int ReasonID { get; private set; }
    public int TypeID { get; private set; }
}

第二个类包含这些请求对象的列表，其签名如下：

public class Requests : IEnumerable<Request>
{
    private List<Request> list;

    public Requests()
    {
        list = new List<Request>();
    }

    public int Add(Request item)
    {
        if (item != null)
        {
            foreach (var x in list.Where(r => 
              (r.LineID == item.LineID) &&
              (r.PartNo == item.PartNo) &&
              (r.ReasonID == item.ReasonID) &&
              (r.TypeID == item.TypeID)))
            {
                x.Qty += item.Qty;
                return list.IndexOf(x);
            }
            list.Add(item);
            return list.Count - 1;
        }
        return -1;
    }

    // other code 
}

我正在测试我的代码并添加项目正在将新项目放入列表中，但是查找重复项的 LINQ 查询不起作用。

如果将 2 个相同的项目添加到列表中，我希望我的代码足够聪明，可以简单地更新数量，但它似乎不起作用。

有人可以告诉我 LINQ 查询有什么问题吗？

4

3 回答 3

3

有人可以告诉我 LINQ 查询有什么问题吗？

理论上看起来还可以。我认为我们需要了解有关您的数据的更多信息，以便能够找出为什么它没有按您的预期工作。LineID, PartNo,ReasonID和的组合是否能TypeID唯一区分一个项目？由于PartNo是字符串，值是否不区分大小写（您的比较区分大小写）？

如果将 2 个相同的项目添加到列表中，我希望我的代码足够聪明，可以简单地更新数量，但它似乎不起作用。

为此，我建议采用不同的方法。考虑覆盖Equals()您的Request类型。然后您的Add方法可以检查列表是否已经包含该项目，如果是则增加数量，如果不是则添加它：

var idx = list.IndexOf(item);
if(idx != -1)
{
    list[idx].Qty += item.Qty;
}
else
{
    list.Add(item);
}
于 2012-06-14T16:21:32.460 回答
2

制作Request工具IEquatable<Request>，因为这是IndexOf使用：

public bool Equals(Request other) {
    return other != null && (this.LineID == other.LineID) && (this.PartNo == other.PartNo) && (this.ReasonID == other.ReasonID) && (this.TypeID == other.TypeID);
}

然后：

public int Add(Request item) {
    if (item != null)
    {
        int ind = list.IndexOf(item);

        if (ind == -1)
        {
            list.Add(item);
            return list.Count - 1;
        }
        else
        {
            list[ind].Qty += item.Qty;
            return ind;
        }                
    }
    return -1;
}
于 2012-06-14T16:14:28.610 回答
2

您可以通过具体化 LINQ 查询来修改该方法。例如：

public int Add(Request item) {
    if (item != null) {
      foreach (var x in list.Where(r => 
        (r.LineID == item.LineID) &&
        (r.PartNo == item.PartNo) &&
        (r.ReasonID == item.ReasonID) &&
        (r.TypeID == item.TypeID)
        ).ToList()) {
        x.Qty += item.Qty;
        return list.IndexOf(x);
      }
      list.Add(item);
      return list.Count - 1;
    }
    return -1;
  }

但是，因为您的请求必须是唯一的，所以您可以使用它

public int Add(Request item)
{
    if (item != null)
    {
        var req = list.SingleOrDefault(r =>
                              (r.LineID == item.LineID) &&
                              (r.PartNo == item.PartNo) &&
                              (r.ReasonID == item.ReasonID) &&
                              (r.TypeID == item.TypeID)
            );
        if(req!=null)
        {
            req.Qty += item.Qty;
            return list.IndexOf(req);
        }
        list.Add(item);
        return list.Count - 1;
    }
    return -1;
}
于 2012-06-14T16:36:15.807 回答