Поиск

[shtandra]

Доброго времени суток.
Как мне написать программу чтобы не создавать класса Accessor, а сделать все его обязаности исполнялись в HomeController.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.Mvc;
using SearchPanelDemo.DAL;
using SearchPanelDemo.Models;
using System.Web.Helpers;
using System.Web.WebPages;
 
namespace SearchPanelDemo.Controllers
{
    public class HomeController : Controller
    {
        //
        // GET: /Home/
        public ActionResult Index(string zipfilter, string cityfilter, string statefilter,
            string latitudefilter, string longitudefilter)
        {
            ViewBag.Title = "US Postal Codes";
            List<PostalCodeModel> viewModel = Accessor.FetchPostalCodes().ToList();
 
            if (!string.IsNullOrWhiteSpace(zipfilter))
                viewModel = viewModel.Where(p => p.ZipCode.StartsWith(zipfilter)).ToList();
 
            if (!string.IsNullOrWhiteSpace(cityfilter))
                viewModel = viewModel.Where(p => p.City.ToUpper().StartsWith(cityfilter.ToUpper())).ToList();
 
            if (!string.IsNullOrWhiteSpace(statefilter))
                viewModel = viewModel.Where(p => p.State.ToUpper().StartsWith(statefilter.ToUpper())).ToList();
 
            if (!string.IsNullOrWhiteSpace(latitudefilter))
                viewModel = viewModel.Where(p => p.Latitude.Contains(latitudefilter)).ToList();
 
            if (!string.IsNullOrWhiteSpace(longitudefilter))
                viewModel = viewModel.Where(p => p.Longitude.Contains(longitudefilter)).ToList();
 
            return View(viewModel);
        }
 
    }
}

Вот код код класса Accessor

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using SearchPanelDemo.Models;

namespace SearchPanelDemo.DAL
{
    public class Accessor
    {
        public static IEnumerable<PostalCodeModel> FetchPostalCodes()
        {
            PostalCodesEntities context = new PostalCodesEntities();

            return (from a in context.ZIP_Codes join b in context.States on a.State_Code 
                    equals b.State_Code orderby b.State_Code ascending, a.City, a.ZIP_Code1 ascending select new PostalCodeModel
                    {
                          ZipCode = a.ZIP_Code1,
                          City = a.City,
                          State = b.State_Abbreviation,
                          Latitude = a.Latitude,
                          Longitude = a.Longitude
                    }).ToList<PostalCodeModel>();

        }
    }
}

[Stilet]

            
 List<PostalCodeModel> viewModel = 
from a in context.ZIP_Codes join b in context.States on a.State_Code 
                    equals b.State_Code orderby b.State_Code ascending, a.City, a.ZIP_Code1 ascending select new PostalCodeModel
                    {
                          ZipCode = a.ZIP_Code1,
                          City = a.City,
                          State = b.State_Abbreviation,
                          Latitude = a.Latitude,
                          Longitude = a.Longitude
                    }).ToList<PostalCodeModel>()

Не?

[phomm]

Ну, а что мешает просто выдрать метод из класса аксессора и вставить в класс контроллера, вызов соответственно укоротиться на "Accessor." и вообще весь класс аксессора не понадобится.
А класс нужен для простого разделения ответственности (хоть и несёт всего лишь 1 статический публичный метод) - т.к. не контроллер суть "носитель"/контейнер данных.

[despercool]

Типо так наверно...

using System;
using System.Collections.Generic;
using System.Linq;
using System.Web;
using System.Web.Mvc;
using SearchPanelDemo.DAL;
using SearchPanelDemo.Models;
using System.Web.Helpers;
using System.Web.WebPages;
 
namespace SearchPanelDemo.Controllers
{
    public class HomeController : Controller
    {
        //
        // GET: /Home/
        public ActionResult Index(string zipfilter, string cityfilter, string statefilter,
            string latitudefilter, string longitudefilter)
        {
            ViewBag.Title = "US Postal Codes";
            List<PostalCodeModel> viewModel = FetchPostalCodes().ToList();
 
            if (!string.IsNullOrWhiteSpace(zipfilter))
                viewModel = viewModel.Where(p => p.ZipCode.StartsWith(zipfilter)).ToList();
 
            if (!string.IsNullOrWhiteSpace(cityfilter))
                viewModel = viewModel.Where(p => p.City.ToUpper().StartsWith(cityfilter.ToUpper())).ToList();
 
            if (!string.IsNullOrWhiteSpace(statefilter))
                viewModel = viewModel.Where(p => p.State.ToUpper().StartsWith(statefilter.ToUpper())).ToList();
 
            if (!string.IsNullOrWhiteSpace(latitudefilter))
                viewModel = viewModel.Where(p => p.Latitude.Contains(latitudefilter)).ToList();
 
            if (!string.IsNullOrWhiteSpace(longitudefilter))
                viewModel = viewModel.Where(p => p.Longitude.Contains(longitudefilter)).ToList();
 
            return View(viewModel);
        }

        public static IEnumerable<PostalCodeModel> FetchPostalCodes()
        {
            PostalCodesEntities context = new PostalCodesEntities();

            return (from a in context.ZIP_Codes join b in context.States on a.State_Code 
                    equals b.State_Code orderby b.State_Code ascending, a.City, a.ZIP_Code1 ascending select new PostalCodeModel
                    {
                          ZipCode = a.ZIP_Code1,
                          City = a.City,
                          State = b.State_Abbreviation,
                          Latitude = a.Latitude,
                          Longitude = a.Longitude
                    }).ToList<PostalCodeModel>();

        } 

    }
}