Storing data into SQL Server database and generating html div dynamically and display sql database content c#

Question

This button will store what I typed in my textbox into my SQL Server database:

string aOption = Request.Form["option"].ToString();
string ATitle = Request.Form["title"].ToString();
string Adesc = Request.Form["desc"].ToString();

con = new SqlConnection(constr);
con.Open();

cmd = new SqlCommand("INSERT INTO Thread (shareORask, Thread_Title, Thread_Description) VALUES (@shareORask, @Thread_Title, @Thread_Description)", con);

cmd.Parameters.AddWithValue("@shareORask", askORshare_Selected);
cmd.Parameters.AddWithValue("@Thread_Title", AS_Title);
cmd.Parameters.AddWithValue("@Thread_Description", AS_desc);

cmd.ExecuteNonQuery();

con.Close();

Next, I wanted to dynamically generate div and display database content. I did it in Page Load.

protected void Page_Load(object sender, EventArgs e)
{
    SqlConnection con = new SqlConnection(constr);
    con.Open();

    SqlCommand cmd = new SqlCommand("Select * from Thread", con);

    SqlDataAdapter da = new SqlDataAdapter(cmd);
    DataSet ds = new DataSet();
    da.Fill(ds);

    reptater.DataSource = ds;
    reptater.DataBind();

    con.Close();
}

<asp:Repeater runat="server" ID="reptater">
    <ItemTemplate>
        <div><%#Eval("Thread_Description") %></div>
    </ItemTemplate>

It works but when I submit my input into the textbox. I'm not able to get data shown in the dynamically created. It only works when i refresh the page.

However, duplicate data will be added into my SQL Server database, how do I solve this?

You should check out Can we stop using AddWithValue() already? and stop using .AddWithValue() - it can lead to unexpected and surprising results... — marc_s
– marc_s, Commented Dec 29, 2020 at 18:16
I would assume that is is a simple matter that right after you run the code stub to add the row, you then simply do a re-data bind on your repeater and your additonal data should display. In other words, you want put a check in your page load if IsPostback = False for the code that loads/binds the repeater. However, that re-bind should not be required unless your add code runs. So I would move out the re-bind/display routine to a seperate sub. That way both on-load (first time - check isPostback = false), and then also in your code stub to add - you call the load/bind routine again. — Albert D. Kallal
– Albert D. Kallal, Commented Dec 29, 2020 at 22:12

Martin Brown · Accepted Answer · 2020-12-29 16:48:51Z

1

I'm assuming that the button code is in a method called something like MyButton_Click(object sender, EventArgs e).

The button click event is a "Control event" and these happen after the Page_Load event in the page life cycle. To insert into the database during the click event and render the div in one page cycle you would need to render the div in one of the later events, probably the Page_PreRender event.

edited Dec 29, 2020 at 16:48

answered Dec 29, 2020 at 16:38

Martin Brown

25.5k16 gold badges88 silver badges134 bronze badges

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

1 Comment

Oddacon Over a year ago

I was going to say something like this. Also depends on if the page IsPostBack. I would add a grid or manage your db call outside of page load and then call the sub in the page_load after postback.

Collectives™ on Stack Overflow

Storing data into SQL Server database and generating html div dynamically and display sql database content c#

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related