Objectif des méthodes publiques NonAction dans MVC

_{(j'ai restructuré la réponse afin de mieux répondre aux questions dans les commentaires)}

Je pense, l'attribut est ici uniquement pour une meilleure flexibilité. En tant que cadre de designer, une envie de se détendre codage des contraintes à l'extrémité de l'utilisateur autant que possible. L'exigence de ne pas avoir de public non-actions peut sembler bon "en général", mais peut-être trop restrictive pour certains projets. L'ajout de [NonAction] résout leur problème (introduit par leur mauvaise conception du bien) - et, évidemment, vous n'êtes pas obligé d'utiliser l'attribut, donc c'est un win-win à partir d'un cadre de concepteur de point de vue.

Une autre raison peut être l'héritage le plus tôt MVC uniquement les versions méthodes marqué avec [Action] considérées comme des actions. Alors, quand ils ont détendu l'exigence (et toutes les méthodes publiques est devenu traitées comme des actions), ils ont gardé [NonAction] afin que les développeurs ne soit pas trop confus.

---

En général, l'utilisation de NonAction est une mauvaise pratique, exactement pour les raisons que vous avez déclaré. Si quelque chose ne devrait pas être une action, il ne devrait pas être public en premier lieu.

Problème avec le service public de non-méthodes d'action sur le contrôleur n'est qu'ils font les gens tentés d'instancier votre contrôleur et l'appel à la méthode, au lieu de séparer la logique commune:

Comparer

public class MyController : IController
{
    public ActionResult Foo(long orderId)
    {
        var order = new OrdersController().GetOrder(orderId); //GetOrder is public
        ...
    }
}

avec

public class MyController : IController
{
    public ActionResult Foo(long orderId)
    {
        var order = _orderService.GetOrder(orderId);
        ...
    }
}

La première approche conduit à une augmentation du couplage entre les contrôleurs et les non-simple code dans les actions. Code devient difficile à suivre et refactoriser, et la lourdeur de se moquer/test.

Outre l'accroissement de couplage, tout public, non-méthode d'action est un trou de sécurité - si vous avez oublié de le marquer avec [NonAction] (ou, mieux, du changement de public) - parce qu'il est considéré comme normal d'action et peuvent être invoqués à l'extérieur. Je sais que la question d'origine plutôt implique que vous certainement ne jamais oublier de joindre l'attribut si nécessaire, mais c'est aussi un peu important de comprendre ce qui peut arriver si vous voulez 😉 Oh et bien, et que nous sommes sur ce, il me semble que "l'oubli de l'attribut" est théoriquement plus probable, la comparant à "oublier de faire la méthode privée".

---

Parfois les gens disent avoir public non-actions est nécessaire pour les tests unitaires, mais encore une fois, quand quelque chose n'est pas une action, il plus probable peut être isolé dans une catégorie distincte et testés séparément. En outre, même si il n'est pas possible pour quelque raison que ce soit, marquant une méthode public des fins de test uniquement est une mauvaise habitude - à l'aide de internal et InternalsVisibleTo est la méthode recommandée.

Ce qui est bénéfique lorsque l'Url ne sont pas sensibles à la casse. De sorte que, par exemple, si vous avez la demande Accueil/à Propos de cela va HomeController et Sur l'actionainsi que accueil/à Propos de au même contrôleur et la même méthode d'action.

Comme ci-dessous

public class HomeController:Controller
{
....
    public ViewResult About()
    {
        return View();
    }

    public ViewResult aBOut()
    {
        return View();
    }
}

Le cadre ne peut pas déterminer le about la fonction à appeler, et lève l'exception indiquant que l'appel est ambigu.

Bien sûr une façon de résoudre ce problème est de changer le nom de l'action.

Si pour quelque raison vous ne voulez pas changer le nom de l'action, et l'une de ces fonction n'est pas une action, alors vous pouvez décorer cette absence de méthode d'action avec NonAction attribut. Exemple:

[NonAction]
public ActionResult aBOut()
{
   return View();
}

Nous sommes à l'aide des contrôleurs de liaison des pilotes personnalisés ASP pipeline, chaque pilote est responsable pour le rendu d'une section (vue partielle) de la page de résultat. Alors nous sommes en utilisant les méthodes comme:

[NonAction]
publi int GetOrder() 

pour résoudre les sections de commande à la page ou à d'autres pour résoudre l'autorisation pour l'utilisateur courant (par exemple, si la section est modifiable ou tout simplement en lecture seule).

De sorte que vous ne devrait pas limiter vous-même à réfléchir sur le Contrôleur de seulement une manière de traiter les demandes, mais aussi comme un outil pour construire votre personnalisé cadre pour le rendu de la page. De cette façon, nous gardons nos Contrôleurs responsables exactement une tâche et nous sommes séparer de domaine préoccupations.