StructureMap - Architecture Service/Repository - Exceptions non interceptées en service et renvoyées à l'application

J'ai une table de compte avec une contrainte unique sur l'adresse mail.StructureMap - Architecture Service/Repository - Exceptions non interceptées en service et renvoyées à l'application

Mon code traverse plusieurs couches Controller -> Service -> Dépôt -> Nhibernate

A l'intérieur du dépôt, une erreur est renvoyée lorsque la clé unique est violée. Je veux attraper cette erreur dans ma couche de service, donc j'ai mis un essai de l'attraper. Malheureusement, l'erreur est transmise directement à MVC après la couche de service.

Voici mes exemples de code.

D'abord le contrôleur:

NewAccountRequest request = new NewAccountRequest() 
       { 
        EmailAddress = model.EmailAddress, 
        FirstName = model.FirstName, 
        LastName = model.LastName, 
        Password = model.Password 
       }; 

       try 
       { 

        accountService.RegisterAccount(request); 

        //send the user off to recieve their activation e-mail 
        return RedirectToAction("SendActivation", new { id = model.EmailAddress }); 
       } 
       catch (EmailAlreadyRegisteredException) 
       { 
        ModelState.AddModelError("EmailAddress", String.Format("The email address {0} is already registered. <a href=\"/SignIn\">Sign In</a>", model.EmailAddress)); 
       }

Ensuite, la méthode que je fais appel au service ressemble à ceci:

public void RegisterAccount(NewAccountRequest request) 
    { 
     Account account = new Account() 
     { 
      EmailAddress = request.EmailAddress, 
      Password = request.Password 
     }; 

     Profile profile = new Profile() 
     { 
      EmailAddress = request.EmailAddress, 
      Name = new Profile.ProfileName() { FirstName = request.FirstName, LastName = request.LastName } 
     }; 

     try 
     { 
      accountRepository.SaveWithDependence<Profile>(account, profile); 
     } 
     catch (System.Data.SqlClient.SqlException ex) 
     { 
      if (ex.Message.ToUpper().Contains("VIOLATION OF UNIQUE KEY")) 
       throw new EmailAlreadyRegisteredException(); 
      else 
       throw; 
     } 
     catch (Exception ex) 
     { 
      if (ex.Message.ToUpper().Contains("VIOLATION OF UNIQUE KEY")) 
       throw new EmailAlreadyRegisteredException(); 
      else 
       throw; 
     } 
    }

Les erreurs de méthode NHibernate enregistrer et au lieu d'être pris au piège par la répar- attraper dans la couche de service, il est transmis directement à l'application MVC. J'ai le sentiment que cela peut être dû à la façon dont j'ai mis en place structuremap. J'ai toutes les déclarations dans mon application MVC. Voilà à quoi ça ressemble:

 //repository registration 
     For(typeof(MySite.Data.IRepository<>)).Singleton().Use(typeof(MySite.Infrastructure.Repository<>)); 

     //services 
     For<MySite.Services.IEmailService>().Singleton().Use<MySite.Services.Impl.BasicEmailService>() 
      .Ctor<string>("activationUrlFormat").Is(ConfigurationHelper.UrlFormats.ActivationLink)   //{0} is the email address, {1} is the HMAC 
      .Ctor<string>("passwordResetUrlFormat").Is(ConfigurationHelper.UrlFormats.ResetPasswordLink); //{0} is the email address, {1} is the HMAC 
     For<MySite.Services.IAccountService>().Singleton().Use<MySite.Services.Impl.AccountService>() 
      .Ctor<int>("activationLinkLifeSpan").Is(ConfigurationHelper.Defaults.ActivationLinkLifespan)   //In hours 
      .Ctor<int>("passwordResetLinkLifeSpan").Is(ConfigurationHelper.Defaults.ResetPasswordLinkLifespan);  //In hours 
     For<MySite.Services.IProfileService>().Singleton().Use<MySite.Services.Impl.ProfileService>();

Mon constructeur de la classe de service ressemble à ceci:

IRepository<Account> accountRepository; 
    IRepository<Profile> profileRepository; 
    IEmailService emailService; 
    int activationLinkLifeSpan; 
    int passwordResetLinkLifeSpan; 

    public AccountService(IRepository<Account> accountRepository, IRepository<Profile> profileRepository, IEmailService emailService, 
     int activationLinkLifeSpan, int passwordResetLinkLifeSpan) 
    { 
     this.accountRepository = accountRepository; 
     this.profileRepository = profileRepository; 
     this.emailService = emailService; 
     this.activationLinkLifeSpan = activationLinkLifeSpan; 
     this.passwordResetLinkLifeSpan = passwordResetLinkLifeSpan; 
    }

tout fonctionne, sauf l'erreur try-catch. Des idées comment je configure ceci pour que le service attrape les erreurs de dépôt? Gardez à l'esprit qu'étant donné que mon référentiel est générique, il est utilisé dans plusieurs services - dans l'exemple de code ci-dessus, il est utilisé dans le service de compte et le service de profil.

EDIT: Ajout du message d'erreur/pile trace

Violation of UNIQUE KEY constraint 'U_Email'. Cannot insert duplicate key in object 'dbo.WebAccounts'. 
The statement has been terminated. 

Description: An unhandled exception occurred during the execution of the current web request. Please review the stack trace for more information about the error and where it originated in the code. 

Exception Details: System.Data.SqlClient.SqlException: Violation of UNIQUE KEY constraint 'U_Email'. Cannot insert duplicate key in object 'dbo.WebAccounts'. 
The statement has been terminated. 

Source Error: 


Line 81:    using (ITransaction tx = Session.BeginTransaction()) 
Line 82:    { 
Line 83:     Session.SaveOrUpdate(entity); 
Line 84:     Session.SaveOrUpdate(dependant); 
Line 85:     tx.Commit(); 

Source File: C:\Users\Josh\Documents\Visual Studio 2010\Projects\MySite.Web\MySite.Infrastructure\Repository.cs Line: 83 

Stack Trace: 


[SqlException (0x80131904): Violation of UNIQUE KEY constraint 'U_Email'. Cannot insert duplicate key in object 'dbo.WebAccounts'. 
The statement has been terminated.] 
    System.Data.SqlClient.SqlConnection.OnError(SqlException exception, Boolean breakConnection) +2030802 
    System.Data.SqlClient.SqlInternalConnection.OnError(SqlException exception, Boolean breakConnection) +5009584 
    System.Data.SqlClient.TdsParser.ThrowExceptionAndWarning() +234 
    System.Data.SqlClient.TdsParser.Run(RunBehavior runBehavior, SqlCommand cmdHandler, SqlDataReader dataStream, BulkCopySimpleResultSet bulkCopyHandler, TdsParserStateObject stateObj) +2275 
    System.Data.SqlClient.SqlDataReader.ConsumeMetaData() +33 
    System.Data.SqlClient.SqlDataReader.get_MetaData() +86 
    System.Data.SqlClient.SqlCommand.FinishExecuteReader(SqlDataReader ds, RunBehavior runBehavior, String resetOptionsString) +311 
    System.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean async) +987 
    System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method, DbAsyncResult result) +162 
    System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method) +32 
    System.Data.SqlClient.SqlCommand.ExecuteReader(CommandBehavior behavior, String method) +141 
    System.Data.SqlClient.SqlCommand.ExecuteDbDataReader(CommandBehavior behavior) +12 
    System.Data.Common.DbCommand.System.Data.IDbCommand.ExecuteReader() +12 
    NHibernate.AdoNet.AbstractBatcher.ExecuteReader(IDbCommand cmd) +278 
    NHibernate.Id.InsertSelectDelegate.ExecuteAndExtract(IDbCommand insert, ISessionImplementor session) +52 
    NHibernate.Id.Insert.AbstractReturningDelegate.PerformInsert(SqlCommandInfo insertSQL, ISessionImplementor session, IBinder binder) +83 

[GenericADOException: could not insert: [MySite.Core.Model.Account][SQL: INSERT INTO WebAccounts (EmailAddress, IsActivated, Password) VALUES (?, ?, ?); select SCOPE_IDENTITY()]] 
    NHibernate.Id.Insert.AbstractReturningDelegate.PerformInsert(SqlCommandInfo insertSQL, ISessionImplementor session, IBinder binder) +226 
    NHibernate.Persister.Entity.AbstractEntityPersister.Insert(Object[] fields, Boolean[] notNull, SqlCommandInfo sql, Object obj, ISessionImplementor session) +204 
    NHibernate.Persister.Entity.AbstractEntityPersister.Insert(Object[] fields, Object obj, ISessionImplementor session) +184 
    NHibernate.Action.EntityIdentityInsertAction.Execute() +150 
    NHibernate.Engine.ActionQueue.Execute(IExecutable executable) +117 
    NHibernate.Event.Default.AbstractSaveEventListener.PerformSaveOrReplicate(Object entity, EntityKey key, IEntityPersister persister, Boolean useIdentityColumn, Object anything, IEventSource source, Boolean requiresImmediateIdAccess) +502 
    NHibernate.Event.Default.AbstractSaveEventListener.PerformSave(Object entity, Object id, IEntityPersister persister, Boolean useIdentityColumn, Object anything, IEventSource source, Boolean requiresImmediateIdAccess) +323 
    NHibernate.Event.Default.AbstractSaveEventListener.SaveWithGeneratedId(Object entity, String entityName, Object anything, IEventSource source, Boolean requiresImmediateIdAccess) +130 
    NHibernate.Event.Default.DefaultSaveOrUpdateEventListener.SaveWithGeneratedOrRequestedId(SaveOrUpdateEvent event) +27 
    NHibernate.Event.Default.DefaultSaveOrUpdateEventListener.EntityIsTransient(SaveOrUpdateEvent event) +63 
    NHibernate.Event.Default.DefaultSaveOrUpdateEventListener.PerformSaveOrUpdate(SaveOrUpdateEvent event) +89 
    NHibernate.Event.Default.DefaultSaveOrUpdateEventListener.OnSaveOrUpdate(SaveOrUpdateEvent event) +191 
    NHibernate.Impl.SessionImpl.FireSaveOrUpdate(SaveOrUpdateEvent event) +260 
    NHibernate.Impl.SessionImpl.SaveOrUpdate(Object obj) +256 
    MySite.Infrastructure.Repository`1.SaveWithDependence(T entity, K dependant) in C:\Users\Josh\Documents\Visual Studio 2010\Projects\MySite.Web\MySite.Infrastructure\Repository.cs:83 
    MySite.Services.Impl.AccountService.RegisterAccount(NewAccountRequest request) in C:\Users\Josh\Documents\Visual Studio 2010\Projects\MySite.Web\MySite.Services\Impl\AccountService.cs:50 
    MySite.Web.Controllers.RegisterController.Index(NewUserRegistrationModel model) in C:\Users\Josh\Documents\Visual Studio 2010\Projects\MySite.Web\MySite.Web\Controllers\RegisterController.cs:66 
    lambda_method(Closure , ControllerBase , Object[]) +108 
    System.Web.Mvc.ActionMethodDispatcher.Execute(ControllerBase controller, Object[] parameters) +51 
    System.Web.Mvc.ReflectedActionDescriptor.Execute(ControllerContext controllerContext, IDictionary`2 parameters) +409 
    System.Web.Mvc.ControllerActionInvoker.InvokeActionMethod(ControllerContext controllerContext, ActionDescriptor actionDescriptor, IDictionary`2 parameters) +52 
    System.Web.Mvc.<>c__DisplayClassd.<InvokeActionMethodWithFilters>b__a() +127 
    System.Web.Mvc.ControllerActionInvoker.InvokeActionMethodFilter(IActionFilter filter, ActionExecutingContext preContext, Func`1 continuation) +436 
    System.Web.Mvc.<>c__DisplayClassf.<InvokeActionMethodWithFilters>b__c() +61 
    System.Web.Mvc.ControllerActionInvoker.InvokeActionMethodFilter(IActionFilter filter, ActionExecutingContext preContext, Func`1 continuation) +436 
    System.Web.Mvc.<>c__DisplayClassf.<InvokeActionMethodWithFilters>b__c() +61 
    System.Web.Mvc.ControllerActionInvoker.InvokeActionMethodFilter(IActionFilter filter, ActionExecutingContext preContext, Func`1 continuation) +436 
    System.Web.Mvc.<>c__DisplayClassf.<InvokeActionMethodWithFilters>b__c() +61 
    System.Web.Mvc.ControllerActionInvoker.InvokeActionMethodFilter(IActionFilter filter, ActionExecutingContext preContext, Func`1 continuation) +436 
    System.Web.Mvc.<>c__DisplayClassf.<InvokeActionMethodWithFilters>b__c() +61 
    System.Web.Mvc.ControllerActionInvoker.InvokeActionMethodWithFilters(ControllerContext controllerContext, IList`1 filters, ActionDescriptor actionDescriptor, IDictionary`2 parameters) +305 
    System.Web.Mvc.ControllerActionInvoker.InvokeAction(ControllerContext controllerContext, String actionName) +830 
    System.Web.Mvc.Controller.ExecuteCore() +136 
    System.Web.Mvc.ControllerBase.Execute(RequestContext requestContext) +111 
    System.Web.Mvc.ControllerBase.System.Web.Mvc.IController.Execute(RequestContext requestContext) +39 
    System.Web.Mvc.<>c__DisplayClass8.<BeginProcessRequest>b__4() +65 
    System.Web.Mvc.Async.<>c__DisplayClass1.<MakeVoidDelegate>b__0() +44 
    System.Web.Mvc.Async.<>c__DisplayClass8`1.<BeginSynchronous>b__7(IAsyncResult _) +42 
    System.Web.Mvc.Async.WrappedAsyncResult`1.End() +141 
    System.Web.Mvc.Async.AsyncResultWrapper.End(IAsyncResult asyncResult, Object tag) +54 
    System.Web.Mvc.Async.AsyncResultWrapper.End(IAsyncResult asyncResult, Object tag) +40 
    System.Web.Mvc.MvcHandler.EndProcessRequest(IAsyncResult asyncResult) +52 
    System.Web.Mvc.MvcHandler.System.Web.IHttpAsyncHandler.EndProcessRequest(IAsyncResult result) +38 
    System.Web.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute() +8841105 
    System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously) +184

Source

2010-11-02 Josh

Je ne peux pas imaginer ce rien avoir à faire avec StructureMap. Si vos dépendances sont correctement résolues par le conteneur, le conteneur fait son travail (et vous l'avez configuré correctement).

Avez-vous parcouru le code pour vous assurer (par exemple) que vous saisissez et relancez correctement la bonne exception dans le service?

Définissez des points d'arrêt et vérifiez si vous avez raison sur ce qui se passe réellement.

Source

2010-11-02 20:22:08

Je l'ai fait et il saute la prise tout à fait. Les erreurs sont transmises du référentiel à l'application MVC. – Josh

Cela n'a aucun sens. Vous avez également des points d'arrêt dans la capture GENERALE (catch (Exception ex))? Il semble que SQLException soit l'exception interne, l'exception externe étant GenericADOException, ce qui signifie que votre capture spécifique ne sera pas touchée. Juste pour confirmer: vous définissez un point d'arrêt sur "accountRepository".SaveWithDependence (compte, profil); "et puis appuyez sur F10 pour faire un pas? –

J'ai essayé de mettre un point d'arrêt dans la capture générale et a trouvé mon problème.L'exception étant levée est une exception NHibernate.Exception.SqlException, avec le InnerException étant le SqlException avec le texte comme ci-dessus Alors que le serveur signalait une exception SqlException à l'écran (et mon gestionnaire), l'erreur racine était l'exception NHibernate, qui venait juste d'être réécrite et d'une manière ou d'une autre à l'exception sql. Si cela est normal, ou si c'est parce que mon code Web et ma couche de service n'ont aucune connaissance de nhibernate – Josh

Je n'ai pas encore assez de points pour ajouter des commentaires à votre question, alors je les mettrai ici.

Une trace de pile et le message d'exception exact seraient utiles. Je ne vois un point dans votre code qui pourrait être la cause de vos exceptions à-est relancée en est:

if (ex.Message.ToUpper().Contains("VIOLATION OF UNIQUE KEY"))

Le message d'exception contiennent effectivement ce texte?

Source

2010-11-02 20:25:30 pitx3

Je place des points d'arrêt là et ce code n'est jamais touché Il passe directement du référentiel à mon code de gestion des exceptions sur mon site web MVC , contournant complètement le plateau-catch – Josh

Et oui, l'erreur lancée a ce texte, je vais l'ajouter ci-dessus. – Josh

StructureMap - Architecture Service/Repository - Exceptions non interceptées en service et renvoyées à l'application

Répondre

Questions connexes