Asp.net - Action Refactor

Question

J'ai ce « problème » que j'ai le code comme ceci dans plusieurs de mes actions de contrôleur:

var users = new List<SelectListItem>(); 
foreach(var user in userRepository.GetAll()) 
{ 
var item = new SelectListItem { Text = user.FriendlyName, Value = user.UserId.ToString() }; 

if (User.Identity.Name == user.UserName) 
    item.Selected = true; 

users.Add(item); 
} 

ViewData["Users"] = users; 

Comment décririez-vous ce factoriser à une solution plus propre? Je veux devenir SEC!

Answer 1

Je créerais que ce soit une méthode d'extension appliquée à List<user> ou quel que soit votre userRepository.GetAll() retourne donc alors dans votre code, vous pouvez remplacer tous ces usages avec juste

ViewData["Users"] = userRepository.GetAll().ToSelectList(); 

Modifier exemple de code : Il y a 2 façons, vous pouvez le faire

public static List<SelectListItem> ToSelectList(this List<Agent> users) 
{ 
    List<SelectListItem> items = new List<int>(); 
    foreach (var user in users) 
    { 
     var item = new SelectListItem { Text = user.FriendlyName, 
           Value = user.UserId.ToString() }; 

     if (User.Identity.Name == user.UserName) 
      item.Selected = true; 

     items.Add(item); 
    } 

    return items; 
} 

et l'utilisation serait comme

userRepository.GetAll().ToSelectList(); 

Ou si vous avez des problèmes avec l'identité étant dans la méthode d'extension ont

public static List<SelectListItem> ToSelectList(this List<Agent> users, 
                 string selectedUserName) 
{ 
    List<SelectListItem> items = new List<int>(); 
    foreach (var user in users) 
    { 
     var item = new SelectListItem { Text = user.FriendlyName, 
           Value = user.UserId.ToString() }; 

     if (user.UserName == selectedUserName) 
      item.Selected = true; 

     items.Add(item); 
    } 

    return items; 
} 

et l'utilisation serait comme

userRepository.GetAll().ToSelectList(User.Identity.Name); 

Answer 2

Vous pouvez placer cette logique dans un BaseController (abstrait) puis en déduire tous vos contrôleurs et les faire appeler une méthode dans le BaseController pour obtenir ces données si nécessaire.

Si vous devez l'inclure dans ALL vos actions de contrôleur, vous pouvez également le placer dans un OnActionExecuting() surchargé dans le BaseController.

Answer 3

Mettez ce code, ou une partie du code dans une classe séparée - appelons-le UserService. Laissez UserService mettre en œuvre IUserService:

public interface IUserService 
{ 
    IEnumerable<SelectListItem> GetUsers(); 
} 

Injecter IUserService dans votre contrôleur par injection Constructor:

public MyController(IUserService userService) 
{ 
    this.userService = userService; 
} 

Utilisez le champ userService dans vos actions de contrôleur pour obtenir les utilisateurs.

public ViewResult DoSomething() 
{ 
    var users = this.userService.GetUsers(); 
    // the rest of the implementation 
}