How to validate multiple textboxes in c# windows Application?

Question

0.00/5 (No votes)

See more:

I have 12 textboxes in my registration form out of which i want to validate only 7 textboxes. for this i have tried this code.

C#

public void ValidateFields()
        {

            var controls = new[] { txt_PartyName, txt_ContactPerson, txt_mobile1, txt_add, txt_city, txt_state, txt_PinCode };
            foreach (var control in controls.Where(e => String.IsNullOrWhiteSpace(e.Text)))
            {
                errorProvider1.SetError(control, "Please fill the required field");
            }
        }

But this code doesn't work as expected. I have call this function on my Save button click event but it is not working well. If actually i left all the field blank then also it saves data to the database i.e not required.
So plz provide me assistance to resolve this issue.
Thanks in advance

Posted 14-Sep-14 20:37pm

Member 10276989

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)

Sinisa Hajnal · Answer 1 · 2014-09-14T20:43:00

Change public void to public boolean, after the error is detected return false, if it goes all the way through return true.

Save depending on your validation result.

If (ValidateFields) {
// save to database
}

New version for validate function only IF ALL NEED TO VALIDATED:

C#

public void ValidateFields()
        {

            var controls = new[] { txt_PartyName, txt_ContactPerson, txt_mobile1, txt_add, txt_city, txt_state, txt_PinCode };

boolean isValid = true;
            foreach (var control in controls.Where(e => String.IsNullOrWhiteSpace(e.Text)))
            {
                errorProvider1.SetError(control, "Please fill the required field");
isValid = false;
            }

return isValid;
        }

If this helps, please take time to accept the solution so others may find it. Thank you.

Prasad Avunoori · Answer 2 · 2014-09-14T20:47:00

Solution 2

Put all required Textboxes in a GroupBox.

Then validate as below.

C#

private void button1_Click(object sender, EventArgs e)
        {
            foreach (Control control in groupBox1.Controls)
            {
                string controlType = control.GetType().ToString();
                if (controlType == "System.Windows.Forms.TextBox")
                {
                    TextBox txtBox = (TextBox)control;
                    if (string.IsNullOrEmpty(txtBox.Text))
                    {
                        MessageBox.Show(txtBox.Name+ " Can not be empty");
                    }
                }

            }
        }

Posted 14-Sep-14 20:47pm

Prasad Avunoori

Comments

Sinisa Hajnal 15-Sep-14 2:53am

How this helps?

george4986 15-Sep-14 2:57am

instead of adding new group box and checking 'groupBox1.Controls' why not using 'this.Controls'?

Prasad Avunoori 15-Sep-14 3:10am

If you use this.Control again you need to look for those 8 Text boxes names which is cumbersome.

In future if he wants to apply validation to few more controls he can just drag them to this Group Control.

george4986 15-Sep-14 3:19am

okay thats nice to iterate less but what if the client dont need a group box in the Form.thanks for the reply.

Pravuprasad · Answer 3 · 2014-09-14T23:27:00

Solution 3

see this[^]

Posted 14-Sep-14 23:27pm

Pravuprasad