Using a data access layer calling method on different page wont update database

Question

I have a data access layer to update and trying to call method on different page wont update database

i have tried moving some code around to be outside of the loop and also set a break point but the data doesn't reach the break point

Data access layer code (DAL):

public void editBuyerByID(int bID, string title, string fname, string lname, string email, string pWord, string tel,string pCode, double price, string type)
{
  using (var context = dc)
  {
    Buyer editBuyer = (from b in context.Buyers where b.BuyerID == bID select b).FirstOrDefault();

    if (editBuyer != null)
      {
        editBuyer.BuyerTitle = title;
        editBuyer.BuyerFName = fname;
        editBuyer.BuyerLName = lname;
        editBuyer.BuyerEmail = email;
        editBuyer.BuyerPassword = pWord;
        editBuyer.BuyerTelNo = tel;
        editBuyer.BuyerPostcode = pCode;
        editBuyer.BuyerPriceRange = price;
        editBuyer.BuyerType = type;

        //context.Buyers.Add(editBuyer);
        context.SaveChanges();

      }
    }
}

Account Page where method being called:

   protected void btnUpdate_Click(object sender, EventArgs e)
        {
            try
            {
                aDal.editBuyerByID(Convert.ToInt32(Session["Buyer"].ToString()),
                                            txtTitle.Text, txtFName.Text, txtLName.Text,
                                            txtEmail.Text, txtPassword.Text, txtTelNo.Text, txtPostcode.Text, Convert.ToSingle(txtPrice.Text), txtType.Text);
                string editBuyerScript = "alert(\"Your details have been updated\");";
                ScriptManager.RegisterStartupScript(this, GetType(),
                                      "ServerControlScript", editBuyerScript, true);
            }
            catch
            {
                string invalidScript = "alert(\"All fields must be filled correctly\");";
                ScriptManager.RegisterStartupScript(this, GetType(),
                                      "ServerControlScript", invalidScript, true);
            }



        }

since you are working in C# (an object oriented programming language), you could pass the Buyer details to _editBuyerByID_ method with a _Buyer_ class object instead of a long list of fields! — Rashid Ali, Sep 05 '19 at 19:43
do you encounter with any error, if yes what is the error message? — Rashid Ali, Sep 08 '19 at 20:39
no its just that it doesn't update in the database the message comes up that updated successfully but when checked it hasn't — amb101293, Sep 08 '19 at 22:17
try adding this line of code `context.Entry(editBuyer).State = EntityState.Modified;` before `context.SaveChanges();` in DAL function. — Rashid Ali, Sep 08 '19 at 22:26
How would I do it through a class I have a buyer class but am unsure how to go about it — amb101293, Sep 23 '19 at 16:54
I can write code for you to use Buyer class, but I would have to add an answer for that! — Rashid Ali, Sep 23 '19 at 17:00
It comes up error saying cannot implicitly convert system.data.entitystate to system.data.entity.ebtitystate.. An explicit conversion exists (are you missing a cast) — amb101293, Sep 23 '19 at 17:09
I'm not sure what you mean by that but any help would be appreciated — amb101293, Sep 23 '19 at 17:09
also adding this code doesnt work it does nothing : context.Entry(editBuyer).State = EntityState.Modified; before context.SaveChanges(); — amb101293, Sep 23 '19 at 18:02

score 0 · Answer 1 · answered Sep 24 '19 at 20:06

in your DAL, editBuyerByID method could receive the changed Buyer object instead of a set of fields, in the method just mark the entity as changed and save changes:

public void editBuyerByID(Buyer buyer)
{
  using (var context = dc)
  {
    context.Entry(buyer).State = EntityState.Modified;
    context.SaveChanges();
  }
}

and in the btnUpdate_Click event make the following changes, get the buyer object (assuming you have method to get buyer by ID in the DAL), update fields and send to the editBuyerByID DAL method:

protected void btnUpdate_Click(object sender, EventArgs e)
{
  try
  {
    int buyerId = Convert.ToInt32(Session["Buyer"].ToString());
    Buyer buyer = aDal.getBuyerByID(buyerId);

      if(buyer!=null) 
      {
        buyer.BuyerTitle = txtTitle.Text;
        buyer.BuyerFName = txtFName.Text;
        buyer.BuyerLName = txtLName.Text;
        buyer.BuyerEmail = txtEmail.Text;
        buyer.BuyerPassword = txtPassword.Text;
        buyer.BuyerTelNo = txtTelNo.Text;
        buyer.BuyerPostcode = txtPostcode.Text;
        buyer.BuyerPriceRange = Convert.ToSingle(txtPrice.Text);
        buyer.BuyerType = txtType.Text;

        aDal.editBuyerByID(buyer);
      }

      string editBuyerScript = "alert(\"Your details have been updated\");";
      ScriptManager.RegisterStartupScript(this, GetType(), "ServerControlScript", editBuyerScript, true);
  }
  catch
  {
    string invalidScript = "alert(\"All fields must be filled correctly\");";
    ScriptManager.RegisterStartupScript(this, GetType(), "ServerControlScript", invalidScript, true);
  }
}

i dont have the method to get the buyer id only a list of buyers — amb101293, Oct 02 '19 at 15:41
i have this code in DAL im not sure if its the same thing you mean : public Buyer findBuyerByID(int ID) { using (var context = new DBModel()) { Buyer findBuyer = (from b in context.Buyers where b.BuyerID == ID select b).FirstOrDefault(); return findBuyer; } } — amb101293, Oct 02 '19 at 19:26

Using a data access layer calling method on different page wont update database

1 Answers1