Persisting Data from a GridView

Question

0.00/5 (No votes)

See more:

.NET2.0

How do you upload the data From GridView and store in database if we have some controls like DropDownLists,TextBoxes,RadioButtons included in it?

Here Is Code
When A of Button Click to upload the data to Database from GridView

C#

protected void Button2_Click(object sender, EventArgs e)
    {
        int i;
        for (i = 0; i < gridview2.Rows.Count; i++)
        {
            SqlCommand cmd = new SqlCommand("insert into rep values('"+gridview2.Rows[i].Cells[0].Text+"','" + gridview2.Rows[i].Cells[1].Text +"','" + gridview2.Rows[i].Cells[2].Text+ "','"+ gridview2.Rows[i].Cells[3].Text+ "')", con);

            con.Open();
            cmd.ExecuteNonQuery();
            con.Close();

        }

Here + gridview2.Rows[i].Cells[2].Text+ is a dropdownlist

and

+gridview2.Rows[i].Cells[3].Text+ is a TextBoxes

Please post the answers.

Posted 13-Jul-10 5:11am

Harshavardhan39

Updated 13-Jul-10 5:17am

TheyCallMeMrJames

v3

Add a Solution

3 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)

TheyCallMeMrJames · Answer 1 · 2010-07-13T05:23:00

con doesn't appear to be in this scope, unless you're using it at a form level. Which is not really good practice.

I would recommend building up objects to store the values, then passing that object collection to a method that takes care of writing the data out.

Build up your command objects in a list. Open the DB, then execute the commands, then close the DB.

If this data is critical to be stored atomically you should wrap the operation in a transaction.

You should also use a try...catch block.

Alternatively, consider a DAL framework which takes care of most of this for you.

Cheers.

Sandeep Mewara · Answer 2 · 2010-07-13T05:27:00

Well, for learning stuff (as beginners), in such cases, you shouldn't try to extract the values directly while formming query.

Instead, break it into two steps. First one, get all the values in a local variable or so and then in second step, use those variables in your query.

As you learn more, you will come to know about parameterized queries and object models that will help more.

jasonHall · Answer 3 · 2010-07-13T06:27:00

Building on what Mr James and Sandeep said, I noticed that this is a web application. You should also take into consideration protecting your database against SQL injections and discontinue the use of embedded SQL statements in your code. Instead, build a stored procedure and call that in your SQLCommand, passing in your data values via SQL Parameters.

Persisting Data from a GridView

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0