I am using DataClassesDataContext to map all the tables from the db into my asp.net application.
For doing CRUD operations i have made static classes with methods, and inside every method a instantiate DataClassesDataContext.
For instance:
public static class UserQ
{
public static User getUserById(int userId)
{
DataClassesDataContext db = new DataClassesDataContext();
var requestedUser = (from u in db.Users
where u.User_id == userId
select u).First();
if (requestedUser != null)
return (User)requestedUser;
else
return null;
}
}
I aam not sure if this way of doing database operations in a web application is safe? If not, can you suggest please a better pattern?
As DataClassesDataContext implements IDisposable
, you should be wrapping it with a using directive:
using (DataClassesDataContext db = new DataClassesDataContext())
{
...
}
I would be very very careful about using STATIC in web applications. Sometimes the bugs are so subtle that you will spend a lot of time debugging.
I think bnkdev & Oded hit the nail on the head: look at repository pattern & wrap your context call in a using statement...
HTH.
The technical post webpages of this site follow the CC BY-SA 4.0 protocol. If you need to reprint, please indicate the site URL or the original address.Any question please contact:yoyou2525@163.com.