关于Linq：如何修改此C#代码，以便Visual Studio识别出我不是白痴？

How can I modify this C# code so that Visual Studio recognizes that I'm not an idiot?

我有三条线

1
2
3
4
5
6
7
8
9
10

int selectedOrgId;

foreach (Organization o in PD.orgs)
if (o.orgname == selectedOrgName)
selectedOrgId = o.orgid;

PD.cats.InsertOnSubmit(new Category {
orgid = selectedOrgId,
catname = newCatName
});

在我的程序环境中，中间一行(循环)被保证为selectedOrgId设置一个值。但是，Visual Studio正在标记最后一行，因为

Use of unassigned local variable 'selectedOrgId'

除了

1
2
3
4
5
6
7
8
9
10

int selectedOrgId = 69;

foreach (Organization o in PD.orgs)
if (o.orgname == selectedOrgName)
selectedOrgId = o.orgid;

PD.cats.InsertOnSubmit(new Category {
orgid = selectedOrgId,
catname = newCatName
});

？？？？？

虽然它是有效的，但它似乎是一个不雅的解决方案，因为它涉及一个神奇的数字。我想知道解决这个问题的正确方式。

编辑：

看看这里的讨论，我应该指定数据库中只有这样的orgid。我的foreach声明应该写得像

1
2
3
4
5
6
7
8

foreach (Organization o in PD.orgs)
{
if (o.orgname == selectedOrgName)
{
selectedOrgId = o.orgid;
break;
}
}

谢谢你给我介绍了一些更好地完成这整件事的方法！

相关讨论

您似乎在迭代一个集合，试图根据组织名称找到一个匹配的值。您可以使用Linq的SingleOrDefault()查找(最多)一个匹配项：

1	var selectedOrg = PD.orgs.SingleOrDefault(o => o.orgname == selectedOrgName);

然后，只有在找到值时才调用InsertOnSubmit()(否则，selectedorg将为空)

1 2	if (selectedOrg != null) PD.cats.InsertOnSubmit(new Category { orgid = selectedOrg.orgid, catname = newCatName });

不。

如果条件o.orgname == selectedOrgName不满足PD.orgs中的任何值，会发生什么？那么，selectedOrgId将保持未初始化状态。

但是，根据您的方法，以下代码可能更"优雅"：

1 2	int selectedOrgId = PD.orgs.Single(o => o.orgname == selectedOrgName).orgid; PD.cats.InsertOnSubmit(new Category { orgid = selectedOrgId, catname = newCatName });

请注意，您的代码将把selectedOrgId设置为它的最后一个实例，而我的代码将假定只有一个实例存在。

相关讨论

这不是唯一的问题。你知道你会找到一个或多个匹配项，编译器不会。

但"或更多"也是一个问题。代码只是不清楚你想要什么，这是根本原因。你有一个隐含的"最后一个胜利"策略。

当您使用一个更接近需求的解决方案时，编译器问题会消失，不会有任何黑客攻击。

没有LINQ：

1
2
3
4
5
6
7
8
9
10
11
12
13

// int selectedOrgId;

foreach (Organization o in PD.orgs)
if (o.orgname == selectedOrgName)
{
PD.cats.InsertOnSubmit(new Category {
orgid = o.orgid,
catname = newCatName
});
return; // or break;
}
// shouldn't get here
throw new ...

和LINQ

1
2
3
4
5

Organization org = PD.orgs.Single(o => o.orgname == selectedOrgName);
PD.cats.InsertOnSubmit(new Category {
orgid = selectedOrgId,
catname = newCatName
});

Single(Predicate)方法与您的问题非常匹配。它将在结果集具有时引发！＝1个元素。

相关讨论

编辑：op更改了问题，指定只找到一个项目，不需要多个解决方案，这里是选项1a。

方案1a-单线直线法

这提供了一个单行的LINQ查询，它将过滤掉不必要的组织，获取单个项目，并选择一个新的类别对象作为插入的参数。如果找不到单个项，则此方法将引发异常，但也就是说，根据您的问题应该发生什么。

1
2
3
4
5

PD.cats.InsertOnSubmit(
PD.orgs.Where(o=>o.orgname==selectedOrgName)
.Single()
.Select(o=>new Category { orgid = o.orgId, catname = newCatName })
);

选项1b-迭代筛选列表并执行工作

这里的所有其他答案都建议使用LINQ，并假设只会找到一个记录。为什么不做循环中的所有事情，并使用LINQ过滤结果呢？

1
2
3
4

foreach (Organization o in PD.orgs.Where(o=>o.orgname==selectedOrgName))
{
PD.cats.InsertOnSubmit(new Category { orgid = o.orgId, catname = newCatName });
}

这里的好处是没有if语句，它可以处理单个或多个案例。有一种方法可以在一行中进行此操作，并删除explicit for each和use list.for each(请参见比较)：

1 2	PD.orgs.Where(o=>o.orgname==selectedOrgName).ToList() .ForEach(o=>PD.cats.InsertOnSubmit(new Category { orgid = o.orgId, catname = newCatName }));

选项2-使用异常

这将使代码意图非常清楚，并让Visual Studio知道您已经处理好了这一点。我们的想法是让您的代码非常接近现在的状态：

1
2
3
4
5
6

int selectedOrgId;
foreach (Organization o in PD.orgs)
{
if (o.orgname == selectedOrgName)
selectedOrgId = o.orgid;
}

但是，此时我建议您使用一个例外，例如：

1 2	if(selectedOrgId == 0) throw new InvalidDataException("Selected Org Id cannot be 0"); PD.cats.InsertOnSubmit(new Category { orgid = selectedOrgId, catname = newCatName });

相关讨论

您可以使用int?：

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15

int? selectedOrgId = null;

foreach (Organization o in PD.orgs)
{
if (o.orgname == selectedOrgName)
{
selectedOrgId = o.orgid;
}
}

PD.cats.InsertOnSubmit(new Category
{
orgid = selectedOrgId,
catname = newCatName
});

相关讨论

"int"的意思是什么？当你可以简单地给变量赋值0时，给它赋值为空？此外，category.orgid不能为空。
@Mehrzadchehraz是因为您希望能够分辨出ID为0和未找到ID之间的区别。手术从来没有显示什么是奥吉德。
@Mehrzad 0可能是int的有效值。而在这种情况下，null表示一个"空值"，这是完全不同的。
category.orgid不可为空，因为他没有提供更多信息，您不能假定它可以为空和/或空有效或零无效。
在这种情况下，空值就像一个标志，表示没有匹配的orgName。
尚不清楚他的对象是否具有可为空的int属性，因此对于类别orgid，需要执行selectedorgid.getValueOrDefault()。我觉得这更混淆了代码。
这里的大括号与原始问题不同。PD.cats.InsertOnSubmit发生在前臂外侧。例如，尝试运行：foreach (var i in Enumerable.Range(1, 5)) if (true) Console.WriteLine(i); Console.WriteLine("finally here");。
@JasonWilczak即使他的代码不可以为空，他也可以这样做，如果他想指出一个值找不到的话。不知道空性周围的大惊小怪是怎么回事。
@这里是一个关于nullables用法的很好的讨论。基本的想法是，有时您无法帮助它(来自数据库的数据)，但如果可以，可以使用基本类型。
@JasonWilczak这绝对是一个很好的讨论，但它也是一个有味道的问题。在这种情况下，我认为使用nullable没有什么错，尽管使用linq的其他答案和这个解决方案一样好(甚至更好)。这仍然是可能的。
@尤瓦利茨查科夫，你的假设被证明是错误的。实际上，在OP告诉我们在找不到指定的"组织"时预期的行为之前，无法回答这个问题。
@是的，我在做一个假设。你似乎真的被那个事实困扰了，不知道为什么。如果这不可能或不符合作战需求，那么他可以放弃它，为他的问题提供了许多很好的答案。
@尤瓦利茨查科夫，至少你可以在答案中包含这个假设。事实上，每个人都在做假设并给出答案。是的，希望你能用这些答案。我放手：D。
我已经编辑了我的帖子，以明确我在找一张唱片
这样，从循环中取出该值并消除了原始问题的警告，因此，删除下拉投票并将其标记为正确的答案。
@哇，帽子太多了。
我在喊…………

下面的代码与您的代码所做的完全相同，但如果您没有匹配的组织(您认为不可能发生)，则会引发异常：

1
2
3
4

PD.cats.InsertOnSubmit(new Category {
orgid = PD.orgs.Last(o => o.orgname == selectedOrgName).orgid,
catname = newCatName
});

使用linq选择带selectedOrgName的组织，得到orgid的组织：

1
2
3
4

PD.cats.InsertOnSubmit(new Category {
orgid = PD.orgs.First(o => o.orgname == selectedOrgName).orgid,
catname = newCatName
});

假设selectedOrgName总是在PD.orgs中，我是根据变量名进行假设的；但是，如果情况并非总是这样，您可以执行以下操作：

1
2
3
4
5
6
7
8
9

var selectedOrg = PD.orgs.FirstOrDefault(o => o.orgname == selectedOrgName);

if (selectedOrg != null)
{
PD.cats.InsertOnSubmit(new Category {
orgid = selectedOrg.orgid,
catname = newCatName
});
}