使用Lambda表达式和linq

Question

因此，我刚刚开始使用linq以及使用lambda表达式。 尝试获取一些我想要的数据时，我遇到了一个小小的麻烦。 此方法应返回从Jira打开或正在进行的所有项目的列表

这是代码

    public static List<string> getOpenIssuesListByProject(string _projectName)
    {
        JiraSoapServiceService jiraSoapService = new JiraSoapServiceService();
        string token = jiraSoapService.login(DEFAULT_UN, DEFAULT_PW);
        string[] keys = { getProjectKey(_projectName) };

        RemoteStatus[] statuses = jiraSoapService.getStatuses(token);
        var desiredStatuses = statuses.Where(x => x.name == "Open" || x.name == "In Progress")
            .Select(x=>x.id);

        RemoteIssue[] AllIssues = jiraSoapService.getIssuesFromTextSearchWithProject(token, keys, "", 99);
        IEnumerable<RemoteIssue> openIssues = AllIssues.Where(x=>
            {
                foreach (var v in desiredStatuses)
                {
                    if (x.status == v)
                        return true;
                    else
                        return false;
                }
                return false;
            });
        return openIssues.Select(x => x.key).ToList();
    }

现在，这仅选择“打开”的问题，似乎跳过了“进行中”的问题。

我的问题：首先，为什么我只收到“公开”问题，其次，还有更好的方法吗？

我首先获取所有状态的原因是问题仅存储了该状态ID，因此我获取了所有状态，获取了与“打开”和“进行中”匹配的ID，然后将这些ID号与问题状态字段匹配。

Answer 1

IEnumerable<RemoteIssue> openIssues = AllIssues.Where(x=>
        {
            foreach (var v in desiredStatuses)
            {
                if (x.status == v)
                    return true;
            }
            return false;
        });

您拥有的代码仅检查第一个状态并返回false。 您需要迭代所有状态并仅在根本不在列表中时返回false。

Answer 2

好吧，你可以改变

IEnumerable<RemoteIssue> openIssues = AllIssues.Where(x=>
{
    foreach (var v in desiredStatuses)
    {
        if (x.status == v)
            return true;
        else
            return false;
    }
    return false;
});

至

IEnumerable<RemoteIssue> openIssues =
      AllIssues.Where(x=> desiredStatuses.Contains(x.status));

至于为什么你没有同时获得两种身份-斯蒂芬回答了。 我上面的代码更改也将解决该问题。

Answer 3

之所以只得到一个状态，是因为您总是在第一次检查后就退出循环。 如果第一个项目不匹配，则不会检查更多项目。 如果您删除else中的收益，它将起作用：

foreach (var v in desiredStatuses) {
  if (x.status == v) {
    return true;
  }
}
return false;

您应该确保实现所需状态的收集，以免您每次使用它时都不会重新运行创建它的查询：

var desiredStatuses =
  statuses
  .Where(x => x.name == "Open" || x.name == "In Progress")
  .Select(x=>x.id)
  .ToList();

如果您只想检查几种状态，则无需使其效率更高。 如果状态很多，您可以将状态实现到HashSet并使用它的Contains方法，该方法比遍历项目快得多。

Answer 4

该代码对我来说似乎正确，尽管有许多方法可以减少编写的代码。

关于：

现在，这仅选择“打开”的问题，似乎跳过了“进行中”的问题。

您能否确定两者都处于期望状态？

我还假设RemoteIssue.status属性确实是指状态的ID，而不是名称，因为这就是您要与之比较的内容？

然后根据马丁·哈里斯（Martin Harris）的答案编写代码：我将使用Contains运算符，而不是您的内部循环...

Answer 5

更改此：

IEnumerable<RemoteIssue> openIssues = AllIssues.Where(x=>
            {
                foreach (var v in desiredStatuses)
                {
                    if (x.status == v)
                        return true;
                    else
                        return false;
                }
                return false;
            });

对此：

IEnumerable<RemoteIssue> openIssues = AllIssues.Where(x=>
            {
                foreach (var v in desiredStatuses)
                {
                    if (x.status == v)
                        return true;
                    //else
                        //return false;
                }
                return false;
            });

Answer 6

其他答案是正确的，但是您可以使用简单的Lambda而不是匿名委托来更加简洁地完成此操作。

IEnumerable<RemoteIssue> openIssues = AllIssues.Where(x=> 
        desiredStatuses.Contains(x.status)

因此，您的整个方法如下所示：

public static List<string> getOpenIssuesListByProject(string _projectName)
{
    JiraSoapServiceService jiraSoapService = new JiraSoapServiceService();
    string token = jiraSoapService.login(DEFAULT_UN, DEFAULT_PW);
    string[] keys = { getProjectKey(_projectName) };

    RemoteStatus[] statuses = jiraSoapService.getStatuses(token);
    var desiredStatuses = statuses.Where(x => x.name == "Open" || x.name == "In Progress")
        .Select(x=>x.id);

    RemoteIssue[] AllIssues = jiraSoapService.getIssuesFromTextSearchWithProject(token, keys, "", 99);
    IEnumerable<RemoteIssue> openIssues = AllIssues.Where(x => desiredStatuses.Contains(x.status));
    return openIssues.Select(x => x.key).ToList();
}

这基本上发出了SQL“ IN”子句的等效项。 因此，您的陈述如下：

SELECT <RemoteIssue> FROM AllIssues AS x WHERE x.status IN <desiredStatuses>