Question

i am using 3 tier architecture in my winform application so i have static class which handle the operation of equipment

 public static class Equipments 
{


    public static void AddEquipment(string name, decimal dimLength)
    {

            DBClassesDataContext db = new DBClassesDataContext();

            Equipment equipment = new Equipment();
            equipment.Name = name;
            equipment.DimLength = dimLength;

            db.Equipments.InsertOnSubmit(equipment);
            db.SubmitChanges();

    }



  public static void UpdateEquipment(int equipmentID, string name, decimal dimLength)
        {
                         DBClassesDataContext db = new DBClassesDataContext();

                Equipment oldEquipment;
                oldEquipment = db.Equipments.Where("EquipmentID = @0",equipmentID).SingleOrDefault();
                oldEquipment.Name = name;
                oldEquipment.DimLength = dimLength;

                db.SubmitChanges();}

so my questions are :

1. Do i need to create instance of DBClassesDataContext in each method ?
   because when i done global static DBClassesDataContext it didn’t work correctly.
2. Is there any better way to handle DBClassesDataContext instead to create it each time inside the method (like create new DBClassesDataContext each time i run a method from this class)

Thanks

Answer 1

Do i need to create instance of DBClassesDataContext in each method?

You should do, absolutely – just like you should normally create a new SqlConnection each time you want to access the database in non-LINQ code. In general, avoid global state – it’s almost always a bad idea.

There is any better way to handle DBClassesDataContext instead to create it each time inside the method

No – that’s exactly the right approach. Why would you try to avoid just creating it each time?