ASP.NET les valeurs par défaut de MVC QueryString remplacent les valeurs fournies?

Un de mes collègues a trouvé un lien qui indique que c'est par conception et il semble que l'auteur de cet article ait soulevé un problème avec l'équipe MVC en disant que c'était un changement par rapport aux versions précédentes. La réponse d'eux était ci-dessous (pour "page" vous pouvez lire " x " pour l'avoir lié à la question ci-dessus):

C'est par conception. Le routage ne fonctionne pas concerne lui - même avec la chaîne de requête les valeurs; il se préoccupe uniquement avec valeurs de RouteData. Vous devriez au lieu de cela, supprimez l'entrée pour " page" dans le dictionnaire Defaults, et dans soit la méthode d'action elle-même, soit un filtre définit la valeur par défaut pour "page" si elle n'a pas déjà été définie.

Nous espérons à l'avenir avoir un moyen plus facile de marquer un paramètre comme provenant explicitement de RouteData, le chaîne de requête ou un formulaire. Jusqu'à ce qui est mis en œuvre la solution ci-dessus devrait travail. S'il vous plaît laissez-nous savoir si c' ne fait pas!

Il semble donc que ce comportement soit "correct", cependant, il est si orthogonal au principe de moindre étonnement {[9] } Que Je ne peux toujours pas le croire.

Edit # 1: notez que le post détaille une méthode pour fournir des valeurs par défaut, mais cela ne fonctionne plus car la propriété ActionMethod qu'il utilise pour accéder au MethodInfo a été supprimée dans la dernière version de ASP.NET MVC. Je travaille actuellement sur une alternative et le posterai une fois terminé.

Edit # 2: j'ai mis à jour l'idée dans le post lié à travailler avec la version Preview 5 de ASP.NET MVC et je crois qu'il devrait également fonctionner avec la version bêta bien que je ne puisse pas le garantir car nous n'avons pas encore déménagé à cette version. C'est si simple que je viens de le poster en ligne ici.

Tout d'abord, il y a l'attribut par défaut (nous ne pouvons pas utiliser le. net existant DefaultValueAttribute car il doit hériter de CustomModelBinderAttribute):

[AttributeUsage(AttributeTargets.Parameter)]
public sealed class DefaultAttribute : CustomModelBinderAttribute
{
    private readonly object value;

    public DefaultAttribute(object value)
    {
        this.value = value;
    }

    public DefaultAttribute(string value, Type conversionType)
    {
        this.value = Convert.ChangeType(value, conversionType);
    }

    public override IModelBinder GetBinder()
    {
        return new DefaultValueModelBinder(this.value);
    }
}

Le classeur personnalisé:

public sealed class DefaultValueModelBinder : IModelBinder
{
    private readonly object value;

    public DefaultValueModelBinder(object value)
    {
        this.value = value;
    }

    public ModelBinderResult BindModel(ModelBindingContext bindingContext)
    {
        var request = bindingContext.HttpContext.Request;
        var queryValue = request .QueryString[bindingContext.ModelName];
        return string.IsNullOrEmpty(queryValue) 
            ? new ModelBinderResult(this.value) 
            : new DefaultModelBinder().BindModel(bindingContext);
    }
}

Et puis vous pouvez simplement l'appliquer aux paramètres de méthode qui entrent dans la chaîne de requête, par exemple

public ActionResult Foo([Default(1)] int x)
{
    // implementation
}

Fonctionne comme un charme!

Je pense que la raison pour laquelle les paramètres querystring ne remplacent pas les valeurs par défaut est d'arrêter les gens de pirater l'url.

Quelqu'un pourrait utiliser une url dont querystring incluait un contrôleur, une action ou d'autres valeurs par défaut que vous ne vouliez pas modifier.

J'ai traité ce problème en faisant ce que @ Dale-Ragan a suggéré et en le traitant dans la méthode d'action. Fonctionne pour moi.

Je pensais que le point avec le routage dans MVC est de se débarrasser des querystrings. Comme ceci:

routes.MapRoute(
    "test",
    "Test/Foo/{x}",
    new { controller = "Test", action = "Foo", x = 1 });