I'm looking for thoughts on how should I use Session in an ASP.NET MVC application? Especially when using masterpages and tryin to just get the data to the masterpage without bypassing the controller. This question started off by me asking a lot of little questions, but then I managed to mould it into a solution which as yet I have not implemented but one which is somewhat workable. Would appreciate any feedback.
My proposed solution aka 'what i am about to implement unless someone says stop!'
I have my model classes inheriting from ModelBase -- which contains the information needed by the masterpage (there is only one view per page) for certain things it displays in the masthead or footer as well as configuration driven settings based upon who is logged in.
My best solution is as follows - shown here for a 'products page':
Assumption: I have at some point already stuck certain data in session - for instance perhaps a
partnerId
which came in through a gateway page, or acurrentLoggedInUserEmail
property or a fully blown object.I have a
ModelBase
class from which every model - such asProductModel
inheritsI have a
MySiteControllerBase
class (inherits from Controller) - which is subclassed byProductController
.In my action method in
ProductController
I create the model for the product view with'new ProductModel()'
. This model class itself knows nothing about session or how to populateModelBase
. It essentially doesn't even know aboutModelBase
- it just inherits from it. My chained constructor does nothing (because I don't want to pass itSession
).I override
View(...)
inMySiteControllerBase
for all the overloads that take a model parameter. I check to see if that parameter is of typeModelBase
and if it is I populate the properties such aspartnerid
andcurrentLoggedInuserEmail
. Fortunately because I'm inside a class that inherits fromController
I have direct access toSession
so i can pull them straight out of there.
This method means that the properties on ModelBase
are automatically populated just by me doing 'return View(model)'
. However there is an obvious issue if the model for ProductModel
needs to access anything defined on ModelBase
. It's going to get null because it isn't populated yet.
This issue can be solved by passing in Session
to new ProductModel(session)
which would in turn pass it up the constructor chain to new ModelBase(session)
. I really dont like that solution though becasue I like to think of a model as a pretty dumb data structure that shouldn't know about any external data constructs at all. Another solution might be to just wing it, and if i ever find that ProductController
needs to consume anything defined in ModelBase
that I just create a method MySiteControllerBase.UpdateModelBase(productModel, session)
to explicitly populate it inside ProductController
. I hope thats clear!
Other questions that come to mind are :
Although there is nothing, in principle, wrong with using Session in ASP.NET MVC applications (well, at least nothing more wrong than using it in other ASP.NET applications...), I tend to feel it should be a last resort, when other things don't work.
Although your question is, generally, very well-written, you don't go into any great detail on what you propose to store in Session. The two examples I found in your question are:
The user's e-mail address is already available from forms authentication, if you are using that, and can be added to other ASP.NET membership providers which don't already support it. It's not clear what partnerid actually is, but I'm skeptical that the Session is the only possible place to store it.
On the other hand, it's entirely possible that you need to store stuff you haven't told us about which would really only fit in the session.
So before you go too far down this road, make sure that other solutions are not already available for the data you need to store.
If you love us? You can donate to us via Paypal or buy me a coffee so we can maintain and grow! Thank you!
Donate Us With