2011-07-25 5 views
1

私はMVC3 Razorコードをいくつか持っています。私はRazorにあまり精通していませんが、コードが何をしているのか理解できます。私はこのコードをきれいにしたいと思います。何かできることはありますか、私は可能な限り最高のものですか?個人的にMVC3コードをクリーンアップしようとしています

@{ var i=1; foreach (var topic in @Model.Topic) 
    { 
    <option value="@topic.RowKey">@(i++). @topic.Description</option> 
    } 
} 
+1

個人的には、@casperOneが示唆したものよりも読みやすいコードを見つけました。なぜなら、私はそれが何をしているのか考える必要はないからです。 –

答えて

1

反復しながら、私はそうのように、シーケンス内の項目と項目の指標の選択をマージしたい:

@{ foreach (var topic in @Model.Topic. 
     Select((t, i) => new { Topic = t, Index = i + 1})) 
    { 
    <option value="@topic.RowKey">@(topic.Index). 
     @topic.Topic.Description</option> 
    } 
} 

この方法で、あなたは分離はありませんトピックからロジックを索引付けすると、すべてが匿名型内にきちんと含まれています。

0

選択リストのオプションのように見えます。その場合は、ループしたり文字列を書き込むのではなく、組み込みの@Html.SelectListビットを使用したい場合があります。

関連する問題