Question

I have the following code:

if (Session["CurrentUrl"] != null & Session["CurrentHost"] != null)

I use this in many places. I am wondering if anyone can think of a way to optimize the C# code just to make things slight more clear.

Answer 1

This seems like a prime candidate for an extension method:

public static class SessionExtensions
{
    public static bool HasHostAndUrl(this HttpSessionState session)
    {
        return session["CurrentUrl"] != null && session["CurrentHost"] != null;
    }
}

And then:

if (Session.HasHostAndUrl()) { /* ... */ }