Getting Different Values

Peter Dzuya

Hi Experts. I am still learning C#. I have two forms. The first one holds a list of Customers. The second one is supposed to edit what is listed in the first form. The User first highlights the customer he/she wants to edit from the customer grid listing form, then clicks the edit button which displays items in the text boxes within the edit form.

The details shown from what is highlighted in the list is different from the edit form.

the code for editing in the listing form is as follows.

private void barButtonEdit_ItemClick(object sender, DevExpress.XtraBars.ItemClickEventArgs e) {

try {

var customerEdit = CustomerListing.GetFocusedRow() as Customer;

if (customerEdit == null) return;

MessageBox.Show(Convert.ToString(customerEdit.Contact.ContactName));

var editCustomer = new frmEditCustomer(customerEdit.ContactId);

editCustomer.ShowDialog();

} catch (Exception Errhand) {

MessageBox.Show(Errhand.Message);

}

and the code in my constructors within the Edit Customer form is as follows

private readonly ICustomerService _customerService;

private readonly Customer _customer;

private readonly IFinancialMonthService _financialMonthService;

private readonly FinancialYear _financialYear;

public frmEditCustomer() {

InitializeComponent();

_customerService = UnityConfig.GetContainer().Resolve();

_financialMonthService = UnityConfig.GetContainer().Resolve();

_customer = new Customer() {

Active = true,

Accountclosed = false,

Datecreated = DateTime.Now,

LastEditedDate = DateTime.Now,

CompanyBranchId = CurrentUserInfo.User.CompanyBranch.CompanyId,

CheckerUserId = CurrentUserInfo.User.UserId,

LastEditorUserId = CurrentUserInfo.User.UserId

};

}

public frmEditCustomer(int customerId) :this(){

this._customer = _customerService.GetById(customerId);

MessageBox.Show(Convert.ToString(_customer.Contact.ContactName));

contactNameTextEdit.Text = _customer.Contact.ContactName;

remarksMemoEdit.Text = _customer.Remarks;

creditDaysCalcEdit.Text = Convert.ToString(_customer.CreditDays);

creditLimitCalcEdit.Text = Convert.ToString(_customer.CreditLimit);

}

the results from the two messageboxes bear different Customers.

So I wonder where I went wrong. Please help. Thanks.

6 Replies

Know the answer? Post it — somebody with the same question will find it here.

Govinda Rajulu Yemineni11y ago
Accepted answer
Hi,

did you keep public frmEditCustomer(int customerId) construtor, if yes remove or comment it
because of that you are getting Errors

MakGarm.Common.Entities.Core - what is this
Govinda Rajulu Yemineni11y ago
Thanks for Appreciation
Peter Dzuya11y ago
Thank you so much Govinda. Yes I did it and it worked. "MakGarm.Common.Entities.Core" is a namespace specifying the path of the entities. Thanks Once more.
Peter Dzuya11y ago
Thank you so much Govinda. Yes I did it and it worked. "MakGarm.Common.Entities.Core" is a namespace specifying the path of the entities. Thanks Once more.
Peter Dzuya11y ago
Thank you Govinda Rajulu,

I have tried your code as shown here below.

private void barButtonEdit_ItemClick(object sender, DevExpress.XtraBars.ItemClickEventArgs e) {
try {

DataRow focusedDataRow = CustomerListing.GetFocusedDataRow();
Customer customer = new Customer();
customer.ContactId = Convert.ToInt32(focusedDataRow["ContactId"]);
customer.Contact.ContactName = focusedDataRow["ContactName"].ToString();
customer.Remarks = focusedDataRow["Remarks"].ToString();
customer.CreditDays = Convert.ToInt32(focusedDataRow["CreditDays"]);
customer.CreditLimit = Convert.ToInt32(focusedDataRow["CreditLimit"]);

frmEditCustomer frm = new frmEditCustomer(customer);
frm.ShowDialog();

} catch (Exception Errhand) {
MessageBox.Show(Errhand.Message);
}
}

But I am getting an exception here. It Reads as Follows.
"Error 1 The best overloaded method match for 'MakGarm.Start.Customers.frmEditCustomer.frmEditCustomer(int)' has some invalid arguments"

and the second exception is.

"Error 2 Argument 1: cannot convert from 'MakGarm.Common.Entities.Core.Customer' to 'int'"
Govinda Rajulu Yemineni11y ago
Hi Peter,

There are so many methods to pass data between forms in Windows application in that some of them are
Using constructor
   Using objects
   Using properties
   Using delegates
here i following constructor method

private void barButtonEdit_ItemClick(object sender, DevExpress.XtraBars.ItemClickEventArgs e)
{

try {

   DataRow focusedDataRow = gridView1.GetFocusedDataRow();

   Customer customer=new Customer ();
   customer.customerId=focusedDataRow["customerId"].ToString();
customer.ContactId=focusedDataRow["ContactId"].ToString();
   customer.ContactName=focusedDataRow["ContactName"].ToString();
customer.Remarks=focusedDataRow["Remarks"].ToString();
   customer.CreditDays=focusedDataRow["CreditDays"].ToString();
   customer.CreditLimit=focusedDataRow["CreditLimit"].ToString();

   frmEditCustomer frm=new frmEditCustomer( customer);
   frm.ShowDialog();

} catch (Exception Errhand) {
MessageBox.Show(Errhand.Message);
}
}

code in the constructor within the Edit Customer form is as follows

public frmEditCustomer(Customer customer)
{

contactNameTextEdit.Text = customer.ContactName;
remarksMemoEdit.Text = customer.Remarks;
creditDaysCalcEdit.Text = Convert.ToString(customer.CreditDays);
creditLimitCalcEdit.Text = Convert.ToString(customer.CreditLimit);
}

If you find anything helpful please accept my answer.