is using private shared objects/variables on class level harmful ?

Posted by haansi on Stack Overflow See other posts from Stack Overflow or by haansi
Published on 2010-05-28T07:18:17Z Indexed on 2010/05/28 7:21 UTC
Read the original article Hit count: 214

Hello,

Thanks for your attention and time. I need your opinion on an basic architectural issue please.

In page behind classes I am using a private and shared object and variables (list or just client or simplay int id) to temporary hold data coming from database or class library. This object is used temporarily to catch data and than to return, pass to some function or binding a control.

1st: Can this approach harm any way ? I couldn't analyze it but a thought was using such shared variables may replace data in it when multiple users may be sending request at a time?

2nd: Please comment also on using such variables in BLL (to hold data coming from DAL/database). In this example every time new object of BLL class will be made.

Here is sample code:

public class ClientManager

{

    Client objclient = new Client();  //Used in 1st and 2nd method

    List<Client> clientlist = new List<Client>();// used in 3rd and 4th method

    ClientRepository objclientRep = new ClientRepository();



    public List<Client> GetClients()

    {

        return clientlist = objclientRep.GetClients();

    }

    public List<Client> SearchClients(string Keyword)

    {

        return clientlist = objclientRep.SearchClients(Keyword);

    }



    public Client GetaClient(int ClientId)

    {

        return objclient = objclientRep.GetaClient(ClientId);

    }



    public Client GetClientDetailForConfirmOrder(int UserId)

    {

        return objclientRep.GetClientDetailForConfirmOrder(UserId);

    }



}

I am really thankful to you for sparing time and paying kind attention.

© Stack Overflow or respective owner

Related posts about c#

Related posts about ASP.NET