この例では、temp変数またはインラインを使用しますか？

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が追加されやすくなります。

最初は、第1のオーバーseconfの唯一の利点は、それことである

Answer 2

私はそれらの間の任意の実際のパフォーマンスの違いがあると思い、また正しさの違いはありません。

私は最初にデバッグするのが簡単だと思います。なぜなら、行の追加とliの内容の表示を簡単にやめることができるからです。

Answer 3

私はと思っていますが、両方とも実際には同じバイトコードにコンパイルされます。しかし、私は通常、新しい変数を使用するときに導入する傾向があります。またはコードを明確にするとき。この場合、文は非常に複雑ではなく、すぐに目に見える形で表示されるため、一時変数を使用することには実質的な利点はありません。

これは単なる推測です。リフレクターなどで確認してください。

Answer 4

（...が、一部が読み困難かもしれより高い高さ）一部が読みやすくを見つけることが下部幅を有しています一度だけ使用される変数を削除します。リファクタリングでは何かが望ましいことがあります。

一方、このような中間変数を使用すると、関連する手順を少しずつ分けることで、コードを少しだけ読みやすくすることができます。

Answer 5

私はもともと私のサンプルデータでうまく動作したら、メソッド1を使ってそれを書き、デバッグし、次にメソッド2を使ってデバッグします。

Answer 6

私はVB.NETコードを書くことができないので、LINQ（とC＃を使用することもできます）。

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 7

私は両方の例が同等であると思いますが、私は後者を好んでいます。