Asp.Net WebApi Héritage avec un BaseController

Je suis actuellement des difficultés pour obtenir l'héritage WebApi de travail comme je le voudrais.
Peut-être que vous pouvez me pointer dans la bonne direction?

Mon but est d'avoir un générique WebApi projet, qui peut être réutilisée dans plusieurs autres WebApi projets.

Par exemple: j'ai une réutilisables UserController qui peut gérer la connexion à plusieurs Sites web. Maintenant, je créer un nouveau Site web simple et ajouter le UserController pour le projet nouvellement créé, qui fonctionne simplement.

Mais maintenant l'exigence pour le nouveau Site web et j'ai besoin de surcharger le UserController méthodes pour tenir compte de ces changements, sans modifier le contrôleur générique.

Ce que je veux atteindre peut être vu dans l'extrait de code suivant (bien sûr, cela ne compile pas car C# se plaint pas de méthode adéquate pour la surcharge trouvé.)

//This belongs to solution "GenericWebApi"

public class LoginRequest
{
    public string User { get; set; }
}

public class BaseApiController : ApiController
{
     [Route("login")]
    public virtual HttpResponseMessage Login(LoginRequest request) 
    {
        return new HttpResponseMessage();
    }
}


//This belongs to solution "CarWebsite"

public class CarWebsiteLoginRequest : LoginRequest
{
    public string Car { get; set; }
}

[RoutePrefix("api/users")]
public class CarWebsiteApiController : BaseApiController
{
    public override HttpResponseMessage Login(CarWebsiteLoginRequest request)
    {
        //Do some car specific login stuff
        return base.Login(request);
    }
}

Une solution que j'ai trouvé de travail a été de remplacer le "LoginRequest" et "CarWebsiteLoginRequest" avec "dynamique", mais qui se sent tout simplement mauvais, et le ModelBinder de cours ne sera pas fonctionner comme prévu.

Des idées comment vous résoudre ce problème?

Merci d'avance

OriginalL'auteur coalmee | 2015-01-15

Vous pouvez faire votre BaseApiController générique:

public class BaseApiController<TLoginRequest> : ApiController
where TLoginRequest : LoginRequest
{
    [Route("login")]
    public virtual HttpResponseMessage Login(TLoginRequest request)
    {
        return new HttpResponseMessage();
    }
}

Et faire de votre CarWebsiteApiController utilisation CarWebsiteLoginRequest comme un paramètre générique:

public class CarWebsiteApiController : BaseApiController<CarWebsiteLoginRequest>
{
    public override HttpResponseMessage Login(CarWebsiteLoginRequest request)
    {
        //Do some car specific login stuff
        return base.Login(request);
    }
}

Merci pour vos suggestions, cela semble être une bonne méthode. Mais mon BaseApiController aurait > de 10 à 15 WebApi actions. Donc j'aurais besoin de faire BaseApiController<T1,T2,T3....>. Est-il un autre moyen que vous connaissez?

OriginalL'auteur Sergey Kolodiy

Si vous sortez de la variable et de le lire à l'intérieur de la méthode, il est plus facile de surcharge

    ///<summary>
///Notice name is not Controller.cs  This would expose this API if the name ended in controller
///</summary>
public class GenericControllerApi : ApiController
{
protected JavaScriptSerializer JsonReader = new JavaScriptSerializer();
[HttpPost]
public virtual HttpResponseMessage Login()
{
string JsonRequest = Request.Content.ReadAsStringAsync().Result;
LoginRequest JSONData = null;
try
{
JSONData = (LoginRequest)JsonReader.Deserialize(JsonRequest, typeof(LoginRequest));
}
catch { }
if (JSONData != null)
{
if (CommonUserCalls(JSONData.User))
{
ShowUser UserReturn = new ShowUser();
UserReturn.UserName = "BaseUser";
return Request.CreateResponse(HttpStatusCode.OK, UserReturn);
}
return Request.CreateErrorResponse(HttpStatusCode.Unauthorized, "Invalid User");
}
else
{
return Request.CreateErrorResponse(HttpStatusCode.BadRequest, "ShowUser not provided");
}
}
protected bool CommonUserCalls(string User)
{
if (User == "MyUser")
{
return true;
}
return false;
}
public class LoginRequest
{
public string User { get; set; }
}
protected class ShowUser
{
public string UserName { get; set; }
}
}
public class CarController : GenericControllerApi
{
[HttpPost]
public override HttpResponseMessage Login()
{
string JsonRequest = Request.Content.ReadAsStringAsync().Result;
CarWebsiteLoginRequest JSONData = null;
try
{
JSONData = (CarWebsiteLoginRequest)JsonReader.Deserialize(JsonRequest, typeof(CarWebsiteLoginRequest));
}
catch { }
if (JSONData != null)
{
if (CommonUserCalls(JSONData.User))
{
ShowUser UserReturn = new ShowUser();
UserReturn.UserName = "CarUser";
return Request.CreateResponse(HttpStatusCode.OK, UserReturn);
}
return Request.CreateErrorResponse(HttpStatusCode.Unauthorized, "Invalid User");
}
else
{
return Request.CreateErrorResponse(HttpStatusCode.BadRequest, "CarWebsiteLoginRequest not provided");
}
}
public class CarWebsiteLoginRequest : LoginRequest
{
public string Car { get; set; }
}
}

En supposant que vous êtes la lecture/envoi JSON

Merci pour la suggestion. Cela ne semble pas offrir des avantages sur l'utilisation de "dynamique" (avec JObject par exemple pour la sérialisation). En outre, votre solution ajoute beaucoup de code dupliqué, il semble y avoir aucun réel point pour l'utilisation du contrôleur générique plus.
Ce petit exemple montre comment surcharger la base de l'appel, refactoriser comme nécessaires à l'optimisation. Un médicament générique code doit être placé dans une méthode protégée. J'ai édité pour le prouver.
Je veux aller plus dans le sens de @Sergey approche, de faire usage de la ModelBinder etc. Votre solution ne me laisse pas utiliser le WebApi caractéristiques comme ils ont l'intention d'utiliser (plugable sérialiseurs, la validation du modèle, etc.)

OriginalL'auteur Brian from state farm

Vous devez vous connecter pour publier un commentaire.