Populate form after data is saved

Question

4.00/5 (2 votes)

See more:

I have a form that a user fills out and clicks submit. Once the data is saved and the user is redirected back to the home screen. I am trying to get the data that the user just saved to populate the form when the user goes back to the form they just filled out. Is there a way of doing this?

C#

SqlConnection con3 = new SqlConnection(System.Configuration.ConfigurationManager.ConnectionStrings["HotConnectionString"].ConnectionString);
            con3.Open();

            SqlCommand scmd3 = new SqlCommand("Select INST_ID, FT_UNDERGR, FT_GRAD, FTE_UNDERG, FTE_GRAD, NON_CREDIT, TOTAL_FTE, FCFTUHC, FCFTPBHC, FCPTUHC, FCPTPBHC, NCHC from Table33 where INST_ID = '" + TextBoxINST_ID.Text + "'", con3);
            SqlDataReader dr3 = scmd3.ExecuteReader();
            if (dr3.Read())

            TextBoxFTUG.Text = dr3["FT_UNDERGR"].ToString();
            TextBoxFTG.Text = dr3["FT_GRAD"].ToString();
            TextBoxTHUGDR.Text = dr3["FTE_UNDERG"].ToString();
            TextBoxTHGDR.Text = dr3["FTE_GRAD"].ToString();
            TextBoxNCCDR.Text = dr3["NON_CREDIT"].ToString();
            TextBoxTCNC.Text = dr3["TOTAL_FTE"].ToString();
            TextBoxTNFUG.Text = dr3["FCFTUHC"].ToString();
            TextBoxTNFG.Text = dr3["FCFTPBHC"].ToString();
            TextBoxTNCPUG.Text = dr3["FCPTUHC"].ToString();
            TextBoxTNCPG.Text = dr3["FCPTPBHC"].ToString();
            TextBoxTNNCC.Text = dr3["NCHC"].ToString();
            con3.Close();
            dr3.Close();

Posted 11-Jan-14 9:57am

Computer Wiz99

Updated 11-Jan-14 11:33am

v2

Add a Solution

2 solutions

Solution 1

Loads of ways... Probably what you want to do is save the data and restore it in the page load event, but exactly how you do that is up to you.
If you log your users in, then you might want to save the data in your database and restore forum there. Alternatives are cookies (stored on the client, so can be preserved from session to session, provided the user uses the same PC) or in the Session collection (on the server, so it's lost when the browser page closes, or twenty minutes, whichever happens first)

Have a think about exactly what you want to do, and then Google can help you with more details.

Posted 11-Jan-14 10:26am

OriginalGriff

Comments

Computer Wiz99 11-Jan-14 16:47pm

OriginalGriff, I currently have a code that populates other textboxes on the form with data from the database but when I try to do the same to the other textboxes I get an error. This is my error: "Invalid attempt to read when no data is present.". I do understand why I get this error but, How can I stop the error if the there is no data until the user enters it? When the page loads is when the error comes for the first time.

OriginalGriff 11-Jan-14 17:03pm

Depends on your code: have you checked the fields you are fetching from the DB?

Computer Wiz99 11-Jan-14 17:31pm

Yes I have. They are all correct. Here is the code I just uploaded.

OriginalGriff 11-Jan-14 17:46pm

Um.
Did you check your TextBoxINST_ID.Text?
Because on page load, it's going to be blank... And that won't return anything, or will return the wrong info...
BTW: for your own sake, don't do SQL like that! Never concatenate strings to form an SQL statement, particularly not with a text box content, and very definitely not with a web application! Google "Bobby Tables" and don't assume it's a joke: SQL Injection can easily damage or destroy your database from the other side of the world. And probably from your login screen as well...

Use parameterized queries... Or make sure your SQL server backups are seriously up to date, because you are going to need them...

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)

Mike Meinz · Accepted Answer · 2014-01-11T11:43:00

Add { and } around the code block that should be executed only when data is read. See the example below. The way your code is written would cause the assignment statements starting with TextBoxFTG.Text = dr3["FT_GRAD"].ToString(); to be executed whether there was data or not.

I added an if statement at the start of the code to check to be sure that there is data in TextBoxINST_ID.Text. This will handle the case when TextBoxINST_ID.Text does not yet contain a value. Also, see comment below about using SQLParameter class.

C#

if (TextBoxINST_ID.Text.Trim().Length > 0)
{
SqlConnection con3 = new SqlConnection(System.Configuration.ConfigurationManager.ConnectionStrings["HotConnectionString"].ConnectionString);
            con3.Open();
 
            SqlCommand scmd3 = new SqlCommand("Select INST_ID, FT_UNDERGR, FT_GRAD, FTE_UNDERG, FTE_GRAD, NON_CREDIT, TOTAL_FTE, FCFTUHC, FCFTPBHC, FCPTUHC, FCPTPBHC, NCHC from Table33 where INST_ID = '" + TextBoxINST_ID.Text + "'", con3);
            SqlDataReader dr3 = scmd3.ExecuteReader();
            if (dr3.Read())
            {
            TextBoxFTUG.Text = dr3["FT_UNDERGR"].ToString();
            TextBoxFTG.Text = dr3["FT_GRAD"].ToString();
            TextBoxTHUGDR.Text = dr3["FTE_UNDERG"].ToString();
            TextBoxTHGDR.Text = dr3["FTE_GRAD"].ToString();
            TextBoxNCCDR.Text = dr3["NON_CREDIT"].ToString();
            TextBoxTCNC.Text = dr3["TOTAL_FTE"].ToString();
            TextBoxTNFUG.Text = dr3["FCFTUHC"].ToString();
            TextBoxTNFG.Text = dr3["FCFTPBHC"].ToString();
            TextBoxTNCPUG.Text = dr3["FCPTUHC"].ToString();
            TextBoxTNCPG.Text = dr3["FCPTPBHC"].ToString();
            TextBoxTNNCC.Text = dr3["NCHC"].ToString();
            }
            con3.Close();
            dr3.Close();
}

Also, it would be a best practice to use a parameterized query rather than '" + TextBoxINST_ID.Text + "'" in you SELECT statement. This prevents SQL Injection Attacks and provides better performance. See Microsoft help file documentation for SQLParameter Class[^]