Question

I am using ASP.NET/C#.

I am currently displaying the Logged In User’s First Name and Middle Name by making use of Session.

Here is the code

HTML:

<li><a href="#"><%=Session["UserFName"]%>&nbsp;<%=Session["UserMName"]%>&#9660;</a>

Code Behind:

//Call the function which returns User's Name
var userName = userBLL.GetUserName(HttpContext.Current.User.Identity.Name);
if (userName != null)
{
Session["UserFName"] = userName.FirstName;
Session["UserMName"] = userName.MiddleName; 
}

Here is the GetUserName function

public Users_AdditionalInfo GetUserName(string userCode)
{
using (var db = new DataClasses1DataContext())
{
var userName = db.Users_AdditionalInfos.Where(user => user.UserCode.Equals(userCode)).SingleOrDefault();
return userName; 
}

}

Is using Session variables to do what I am trying to do is all right?

What are some other ways that many of you do to achieve this?

Any suggestions are welcome.

Answer 1

You just mention ASP.NET, not MVC. Are you using MVC or not?

If not, why don’t you put a Literal control inside the <li> and populate that on page-load

In your aspx something like this:

<li><a href ="#"><asp:Literal ID="lt_name" EnableViewState="false" 
 runat="server" Text=""></asp:Literal></a> 

In page load you do:

var userName = userBLL.GetUserName(HttpContext.Current.User.Identity.Name);  

lt_name.text = userName.Firstname + " " + username.Lastname;