Generating an MVC3 RadioButton list in a loop statement

Question

A collegaue of mine created a model and here it is.

Model

[Serializable]
public class ModifyCollegeListModel
{
    public List

        

Answer 1

I would do two things.

First up, I would remove the filtering logic from the view. What I mean is this part:

Model.CollegeList.Where(m => m.CategoryId == 3).OrderBy(m => m.SchoolName).ToList()

That sort of logic belongs in a service. Also it will make the view much cleaner.

Secondly, I think you'll need to use a for-loop so MVC binds everything back how you want:

for (int i = 0; i < Model.CollegeList.Count; i++) {
    
        @CollegeList[i].SchoolName
        @CollegeList[i].StateName
        @Html.RadioButtonFor(x => x.CollegeList[i].SchoolId, (int)BrightHorizons.CC.BusinessLogic.CollegeListApplicationStatusEnum.DidNotApply)
        @Html.RadioButtonFor(x => x.CollegeList[i].SchoolId, (int)BrightHorizons.CC.BusinessLogic.CollegeListApplicationStatusEnum.Accepted)
        @Html.RadioButtonFor(x => x.CollegeList[i].SchoolId, (int)BrightHorizons.CC.BusinessLogic.CollegeListApplicationStatusEnum.NotAccepted)
    
}

You'll notice after using the for-loop, that the radiobutton names and ID's also contain their index in the CollegeList. For example: