Позвольте мне немного очистить ваш код:
public class EventInfo
{
public int OPR { get; set; }
public int EVENT_ID { get; set; }
public string SUBSITE { get; set; }
... some other properties that you might want to use
public static IEnumerable<EventInfo> Select()
{
var helper = new Dbhelper();
using (var cmd = helper.GetSqlStringCommond("SELECT * FROM WS_EVENTINFO"))
using (var reader = helper.ExecuteReader(cmd))
{
while (reader.Read())
{
yield return new EventInfo
{
OPR = reader.GetInt32(reader.GetOrdinal("OPR")),
EVENT_ID = reader.GetInt32(reader.GetOrdinal("EVENT_ID")),
SUBSITE = reader.GetString(reader.GetOrdinal("SUBSITE"))
}
}
}
}
}
, затем в действии контроллера:
public ActionResult List()
{
var model = EventInfo.Select().ToList();
return View(model);
}
и, наконец, в представлении:
<%@ Page
Language="C#"
Inherits="System.Web.Mvc.ViewPage<IEnumerable<EventInfo>>" %>
<% foreach (var item in Model) { %>
<tr>
<td>
<%= Html.ActionLink(item.TITLE, "Detail", new { id = item.EVENT_ID }) %>
</td>
...
Следующее улучшение, которое должно быть сделано в этом, состоит в том, чтобы перенести доступ к данным (статический метод Select) в отдельный репозиторий и заставить контроллер использовать этот репозиторий вместо прямого вызова метода Select для запроса к базе данных.