string or binary data would be truncated

I have a table User_ExtraInfo.
 TableNow I got an erro: string or binary data would be truncated in the code. All parameters come from textboxs or a dropdown menu.
Please help me, many thanks.
 protected void btnSave_Click(object sender, EventArgs e)
        {
            MembershipUser user = Membership.GetUser(userName);
            
            user.Email = TextEmail.Text;
            Membership.UpdateUser(user);
            SqlConnection con = new SqlConnection(ConfigurationManager.ConnectionStrings["MembershipDB"].ConnectionString);
            con.Open();
            SqlCommand cmd = new SqlCommand();
            cmd.Connection = con;
            string UserID = user.ProviderUserKey.ToString();
            cmd.Parameters.Add("@UserID", SqlDbType.VarChar).Value = UserID;

            cmd.Parameters.Add("FirstName", SqlDbType.NVarChar).Value =TextFirstName.Text;
            cmd.Parameters.Add("LastName", SqlDbType.NVarChar).Value = TextLastName.Text;

            cmd.Parameters.Add("Degree", SqlDbType.NVarChar).Value = TextDegree.Text;
            cmd.Parameters.Add("Organization", SqlDbType.NVarChar).Value = DropDownList1.SelectedValue;
            cmd.Parameters.Add("Phone", SqlDbType.NChar).Value = TextPhone.Text;
            cmd.Parameters.Add("Ext", SqlDbType.NVarChar).Value = TextExt.Text;
            cmd.Parameters.Add("last4SSN", SqlDbType.NChar).Value = TextSSN.Text;
            cmd.CommandText = "update User_ExtraInfo SET FirstName=@FirstName, LastName=@LastName,Degree=@Degree,Organization=@Organization,Phone=@Phone,Ext=@Ext,last4SSN=@last4SSN where UserID =@UserID";

            cmd.ExecuteNonQuery(); // here exception was thrown
            con.Close();


            Response.Redirect("ManageUsers.aspx");
        }

Open in new window

zhshqzycAsked:
Who is Participating?

[Product update] Infrastructure Analysis Tool is now available with Business Accounts.Learn More

x
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

Scott CraigWebmasterCommented:
This generally means that one of your field lengths will exceed what your database is set to.  Have you done any pre-check on the data to ensure that each field is no longer than the field length set in your database?
0
Luis PérezSoftware Architect in .NetCommented:
It's a good practice to set the MaxLength property of the TextBox controls to the maximum capacity of each field. For example, the TextFirstName TextBox must have a MaxLength of 50; the TextDegree TextBox must have a MaxLength of 10, and so on. With this simple and effortless approach you ensure that none of your text fields exceeds your database capacity, so you will not have the "string or binary data would be truncated" error.

Hope that helps.
0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
käµfm³d 👽Commented:
With this simple and effortless approach you ensure that none of your text fields exceeds your database capacity, so you will not have the "string or binary data would be truncated" error.
Of course you should always validate your data before sending it to your DB since you can't always guarantee that a user will be submitting requests to your site from within a browser  ; )
0
OWASP Proactive Controls

Learn the most important control and control categories that every architect and developer should include in their projects.

Jesus RodriguezIT ManagerCommented:
I recomended to you that as everyone said validate the data of each text bx more or les like this

iif(TextPhone.lenght>0,left( TextPhone.Text,10),"")
where 10 will mean the amount of character that support each field in your database.
0
zhshqzycAuthor Commented:
I have a dropdown select value, how to set up its length?
0
käµfm³d 👽Commented:
I think you should spend some time reading up on n-tier architecture. You are heading down the road of brittle design by mixing your UI logic and your data access logic. By creating layers to your application, you make it easier to handle and reuse things like validation and DB access logic.

As to your most recent question, as I evoked earlier, validate the length of the selected item in your code--don't focus on validating it in the UI. Validate the items before you send them to the DB. If the values don't pass validation, let your user know with the appropriate error message.
0
Jesus RodriguezIT ManagerCommented:
iif(DropDownList1.SelectedIndex<>-1,left(DropDownList1.SelectedValue.toString,10),0)
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
C#

From novice to tech pro — start learning today.