views:

419

answers:

2

I have .aspx page that page inserts the data to the database on a button click. But when i press the button it is going right. i m getting the Successfully message as " successfully inserted data". In this situation if i press "F5" or Refresh the page it is firing the button click event. Why it should be ? How to avoid this condition ?

+1  A: 

When the user clicks F5 (or uses a toolbar button to refresh the page) it will cause a new request, identical to the previous one, to be sent to the server. The Button.Click event will be raised again, but you have a few ways to protect yourself against inserting the data twice.

The best way, IMHO, is to use the Post/Redirect/Get pattern. In your code, right after the point where the data is saved, do a 302 redirect to a confirmation page:

protected void btnSaveStuff_Click(object sender, EventArgs e)
{
    SaveStuffToDatabase();
    Response.Redirect("confirmation.aspx");
}

When using the pattern, the POST to the original page will not end up in the browser history, and refreshing the result page will cause the final GET to be repeated, which should be safe.

Jørn Schou-Rode
I read the article. but How to use GET/POST in master detail scenario then?
Lalit
I am not sure what you mean by "master detail scenario". As shown in my code sample, you simply need to do a `Response.Redirect`, which will "cancel" then `POST` and make the browser to a fresh `GET`.
Jørn Schou-Rode
ok i'll try this, thankx for reply
Lalit
A: 

The refresh will re-submit the form you posted last time when you clicked the button.

Usually, when you refresh a page you think of GETing the page again, or doing an HTTP GET, but since the last thing you did was a POST (when you clicked the submit button) the browser will perform the post again to attempt to invoke the same response.

I suggest using the Post/Redirect/Get pattern as suggested by Jorn Schou-Rode.

This article also seems relevant. http://aspalliance.com/687_Preventing_Duplicate_Record_Insertion_on_Page_Refresh

TJB
I don't think TJB the second link that you provided is use full. I needs too much execution right? really code is good but not proper way. See If i have 100 pages so according to this article I need to code on each page with respective fields. Which is wrong. Is any other way ??please
Lalit