Question

I have just recently got involved in a classic ASP.NET project which contains lots of storing and reading values from the session and query strings. This could look something like the following:

Session["someKey"]=someValue;

And somewhere else in the code the value in the session is read. Clearly this violates the DRY principle since you’ll have the literal string key spread out all over the code. One way to avoid this could be to store all keys as constants that could be referenced everywhere there is a need to read and write to the session. But I’m not sure that’s the best way to do it. How would you recommend I best handle this so that I don’t violate the DRY principle?

Answer 1

Create a separate public class where you can define your constants, e.g

public class SessionVars
{
   public const string SOME_KEY = "someKey";
   public const string SOME_OTHER_KEY = "someOtherKey";
}

and then anywhere in your code you can access session variables like this:

Session[SessionVars.SOME_KEY]=someValue;

This way you can get IntelliSence and other bells and whistles.