i am adding rows dynamically in datagridview using datatable but its always showing the same row multiple times in c#

Question

0.00/5 (No votes)

See more:

private void btnAddServices_Click(object sender, EventArgs e)
{
DataTable dt = new DataTable();
DataRow dr;
DataColumn dc;

dc = new DataColumn("Description");
dt.Columns.Add(dc);
dc = new DataColumn("Duration");
dt.Columns.Add(dc);
dc = new DataColumn("Price");
dt.Columns.Add(dc);
foreach (DataGridViewRow row in gvBillGenerate.Rows)
{
dr = dt.NewRow();

dr["Description"] = cmbService.SelectedValue;
dr["Duration"] = txtDuration.Text;
dr["Price"] = txtPrice.Text;

dt.Rows.Add(dr);

}
dt.AcceptChanges();
gvBillGenerate.DataSource = dt;

it will display multiple rows with same data.

Posted 27-Oct-14 18:30pm

nainika_

Add a Solution

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Laiju k · Answer 1 · 2014-10-27T18:35:00

Solution 1

foreach (DataGridViewRow row in gvBillGenerate.Rows)
{
dr = dt.NewRow();

dr["Description"] = cmbService.SelectedValue;
dr["Duration"] = txtDuration.Text;
dr["Price"] = txtPrice.Text;

dt.Rows.Add(dr);

}
dt.AcceptChanges();

you are passing same value.Because only one value can exit on textbox and dropdown at a time.
If you are going to add rows dynamically you should store previous data in ViewState or Session whatever your need is and take data From there and Add new rows.

ViewState["Bill"] = dt;
dt=ViewState["Bill"] as DataTable

Posted 27-Oct-14 18:35pm

Laiju k

Updated 27-Oct-14 18:40pm

v3

Comments

nainika_ 28-Oct-14 0:46am

but i am using windows applications , i can't use viewstate over there.

Laiju k 28-Oct-14 0:50am

remove the foreach part maintain with same datatable no need to initializing it every time I dont Know about Windows.try it

[no name] 28-Oct-14 0:55am

yeah Laiju is right. I agree.

nainika_ 28-Oct-14 0:59am

even i can't use sessions in windows applications, i ll only be possible in asp.net.

Laiju k 28-Oct-14 1:00am

I have changed the comment you try that

Laiju k 28-Oct-14 1:03am

thanks

nainika_ 28-Oct-14 1:02am

when i add first row it will be easily added and when i m trying to add second row ,same row will show two times and so on , this is the error.

Laiju k 28-Oct-14 1:02am

what is your scenario,what are you trying to acheive ?

nainika_ 28-Oct-14 1:04am

i ve already done this, by doing so no row is adding in datagridview

nainika_ 28-Oct-14 1:05am

i want to add rows in datatable that will show in datagridview one by one by selecting fields.

Laiju k 28-Oct-14 2:15am

have you got it else i will modify my solution

nainika_ 28-Oct-14 11:55am

Done.. anyways thanks, i forgot to tell u

Praneet Nadkar · Answer 2 · 2014-10-27T18:41:00

Hi Nainika,

Check these values:

C#

dr["Description"] = cmbService.SelectedValue;
dr["Duration"] = txtDuration.Text;
dr["Price"] = txtPrice.Text;

cmbService will have only one selected value. Check what text you have in these two TextBoxes. I guess the values that you are assigning will remain the same.

Apply a debugger and check them first.

Regards,
Praneet