Creating a database class in MVC

Question

I want to expand on the question below:

How to instantiate a DbContext in EF Core

I've done the above in my controller, but am now trying to create DataAccess.cs where I want all my database methods/queries to happen. For example, I have the following so far:

Controller:

public class HomeController : Controller
    {
        DataAccess da = new DataAccess() { };

        private readonly DatabaseContext db;

        public HomeController(DatabaseContext reg)
        {
            db = reg;
        } 

    [HttpPost, ValidateAntiForgeryToken]
    public IActionResult Register(User model)
    {       
        model.WindowsAuth = User.Identity.Name;

        if (ModelState.IsValid) 
        {
            da.Registration(model, db);                
            return RedirectToAction("Index");
        }

In DataAccess.cs

 public class DataAccess
 {        
    public void Registration(User model, DatabaseContext db)
    {
        User newUser = new User();
        db.Users.Add(model);
        db.SaveChanges();
    }
 }

Database context

public class DatabaseContext :DbContext
{


    public DatabaseContext(DbContextOptions<DatabaseContext> options)
               : base(options)
    {
    }

    public DbSet<User> Users { get; set; }

}

The above works, but I want to add more methods into DataAccess.cs. Will I need to pass my database as a parameter from the controller everytime? The db variable comes out as null when I try to instantiate the context class in my data access class. I am new to MVC, so any further reading or sources related to this would be appreciated.

could you please show your DataAccess class?

er-sho
– er-sho

2019-02-20 06:05:33 +00:00
Commented Feb 20, 2019 at 6:05 — er-sho
– er-sho, Commented Feb 20, 2019 at 6:05
It is there. I've added the context class now though.

buffalol
– buffalol

2019-02-20 06:10:08 +00:00
Commented Feb 20, 2019 at 6:10 — buffalol
– buffalol, Commented Feb 20, 2019 at 6:10

Slava Utesinov · Accepted Answer · 2019-02-20 06:15:31Z

1

With the help of DI you should inject DataAccess to controller and DatabaseContext to DataAccess. Also don't forget to register DataAccess at DI container:

public class HomeController : Controller
{
    private readonly DataAccess da;// = new DataAccess() { };
    //private readonly DatabaseContext db;   
    public HomeController(DataAccess da)
    {
        this.da = da;
    } 
}

public class DataAccess
{
    private readonly DatabaseContext db;     
    public DataAccess(DatabaseContext db)
    {
        this.db = db;
    }

    public void Registration(User model/*, DatabaseContext db*/)
    {
        User newUser = new User();
        db.Users.Add(model);
        db.SaveChanges();
    }
}

edited Feb 20, 2019 at 6:15

answered Feb 20, 2019 at 6:10

Slava Utesinov

13.5k2 gold badges21 silver badges26 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

buffalol Over a year ago

Thank you. This worked, had to figure out how to register DataAccess in my DI container, but finally got it.

buffalol Over a year ago

services.AddScoped<Data.DataAccess>(); in startup.cs ConfigureServices() - for anyone in future with a similar problem

Collectives™ on Stack Overflow

Creating a database class in MVC

1 Answer 1

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related