在此示例中使用临时变量还是内联变量？

Question

我正在加载 ASP.NET 下拉列表。

这样做有什么好处吗::

    Private Sub LoadSeasonsListbox(ByVal seasons As List(Of Season))
      Dim li As ListItem
      For Each s As Season In seasons
        li = New ListItem(s.SeasonDescription, s.SeasonCodeID)
        frm.SeasonsList.Items.Add(li)
      Next
    End Sub

比这样做:

Private Sub LoadSeasonsListbox(ByVal seasons As List(Of Season))
    For Each s As Season In seasons
        frm.SeasonsList.Items.Add(New ListItem(s.SeasonDescription, s.SeasonCodeID))
    Next
End Sub

Answer 1

调试时，第一个可以更轻松地检查所添加的 ListItem。

第一个宽度也较低，有些人可能会发现更容易阅读（但高度较高，有些人可能会发现更难阅读......）

Answer 2

我认为它们之间没有任何真正的性能差异，也没有正确性差异。

我认为第一个更容易调试，因为您可以轻松地停在 Add 行并查看 li 的内容。

Answer 3

我认为¹两者实际上都会编译为相同的字节码。 然而，我通常只在使用新变量时才倾向于引入它们。 或者当它使代码更清晰时。 在这种情况下，语句并不是非常复杂，并且可以立即看出您正在做什么，因此使用临时变量可能没有真正的好处。

¹这只是一个猜测。 使用 Reflector 或类似工具进行验证。

Answer 4

我最初会使用方法 1 来编写它并进行调试，然后在它与示例数据配合良好时重构为方法 2。

Answer 5

您还可以使用 LINQ（和 C#，因为我无法编写 VB.NET 代码；）来完成此操作。

private void LoadSeasonsListbox(IEnumerable<Season> seasons)
{
    frm.SeasonsList.Items.AddRange(seasons
        .Select(s => new ListItem(s.SeasonDescription, s.SeasonCodeID))
        .ToArray());
}

我更喜欢这个解决方案，因为我讨厌这个循环只是复制或翻译对象 - 它们使代码变得非常混乱。 我什至会考虑编写一个扩展方法。

private void LoadSeasonsListbox(IEnumerable<Season> seasons)
{
    frm.SeasonsList.Items.AddRange(
        seasons.ToListItems(s => s.SeasonDescription, s => s.SeasonCodeID));
}

Answer 6

与第一个相比，第二个的唯一优点是它消除了仅使用一次的变量。 重构中的某些东西有时是可取的。

另一方面，使用这样的中间变量可以通过稍微分解所涉及的步骤来使代码更具可读性。

Answer 7

我认为这两个例子是等效的，但我更喜欢后者。