MVC шаблоны и практика

Question

Я бы попросил также о лучших методах и шаблонах для asp.net MVC, используя этот пример:

У меня есть проект. Это его архитектура:

1. Модели
2. Controler
3. Просмотров
4. Инструменты

В модели папки у меня есть каждый ViewModel в отдельном поле класса. Все объявления таблиц (объекты) я помещаю в один файл (AccountModels.cs или ForumModels.cs). У меня есть отдельный файл для контекста EF (MyAppContext.cs).

Контролер - здесь у меня есть только классы-контроллеры. Но, может быть, часть кода будет лучше пример и предложение, что я могу улучшить:

private AppContext db = new AppContext(); 

[HttpPost] 
[Authorize] 
public ActionResult AddGun(GunModel model) 
{ 
    if (ModelState.IsValid) 
    { 
     Gun gunToAdd = new Gun 
     { 
      Tilte = model.Tilte, 
      AuthorID = UserTools.getUser(User.Identity.Name).UserId, 
      AddDate = DateTime.UtcNow, 
      Content = model.Content, 
      CategoryID = model.CategoryID, 
      CategoryName = GunsTools.getCategoryName(model.CategoryID) 

     }; 

     db.Guns.Add(gunToAdd); 
     db.SaveChanges(); 

     return RedirectToAction("Details", new { ID = gunToAdd.ID }); 

    } 

return RedirectToAction("Index"); 

} 

Это часть контроллера с AddGun действий. Другие ActionResults похожи - Дженерали я использую лямбда-выражение на моем контексте дб, чтобы получить значения и т.д.

Просмотров - Просмотров в отдельной папке, для Частичный вид I установить специальный префикс (например - _NavigationPartial.cshtml или _CalculatorPartial.cshtml). Есть ли что-то для улучшения здесь? Конечно, Views использует ViewModels, а не модели.

И не менее - Инструменты. Я поставил здесь несколько классов и методов, чтобы предотвратить повторение кода. Вот несколько методов, которые witch возвращает из базы данных некоторые объекты или просто строки, такие как GetUser (..) или GetCategoryName (..). Это хорошая практика?

Я думаю, что многие молодые разработчики MVC имеют одинаковую архитектуру проекта (я лично видел это в нескольких компаниях), и многие из них, возможно, также хотят улучшить что-то в своих проектах, чтобы стать лучшими программистами.

С уважением

Answer 1

Как отмечалось в предыдущих комментариях, это трудно ответить на такой широкий вопрос, но я сделаю пару замечаний

---

• Это не хорошая идея, чтобы использовать свой базы данных непосредственно в вашем контроллере, поскольку это тесно связывает ваш контроллер с базой данных и может привести к тому, что данные будут отображаться на вашем представлении, которые на самом деле там не принадлежат.

---

Вместо этого вы должны иметь отдельный слой данных, который абстрагируется от деталей, что такое юридическое лицо Пушки (с точки зрения базы данных) и то, что пистолет Модель (с точки зрения вида)

случайно выбранный, но хороший SO вопрос об этом можно найти here

---

• Вы могли бы рассмотреть с помощью Dependency INJ прогиб (DI) для контроллера

---

так, что у вас есть конструктор, который выглядит следующим образом:

readonly AppContext _db; 
public GunController(AppContext db) 
{ 
    if (db==null) {throw new ArgumentNullException("db is null");} 
    _db=db; 
} 

Вам нужно использовать DI контейнер для этого вверх. Случайно выбранный (но на самом деле хорошо) статью можно найти here

---

Прежде всего, продолжают задавать вопросы, но старайтесь держать их более конкретными т.е. о конкретных областях вашего приложения - и в кратчайшие сроки вы будете иметь действительно хорошая картина того, что есть и не является хорошей практикой!