How do I get this right?

Question

2.14/5 (3 votes)

See more:

The method below is to assign elements of x to some textboxes.

        string[] x = new string[] { "NAME", "SEX", "AGE", "SCHOOL"};
        public void ClearControls(TextBox[] A)
        {
            for (int32 i = 0; i < A.Length; i++)
            {
                A[i].Text = x[i];
            }
        }

The implementation of the method is as sampled below but error line would appear under the ClearControls with hint message: No overload for method ‘ClearControls’ takes 4 arguments.
        
        private void btnNext_Click(object sender, EventArgs e)
        {
            ClearControls(txtNAME,txtSEX,txtAGE,txtSCHOOL);

         }
What do I do?

Posted 5-May-14 10:03am

Salisu Shaibu

Add a Solution

3 solutions

Solution 1

Hello Salisu Shaibu,

Note this two lines particularly:

C#

public void ClearControls(TextBox[] A)

...and...

C#

ClearControls(txtNAME,txtSEX,txtAGE,txtSCHOOL);

Something seems wrong?

JAFC

Posted 5-May-14 10:15am

José Amílcar Casimiro

Solution 3

Other solutions do what you want. But what you want is not quite good. As far as I understand, you want to initialize some text boxes with some values. I advice you to do that one by one. set each textbox text with its initial value as in the following.

C#

void ResetControls() {
  this.txtNAME.Text="[NAME]";
  this.txtSEX.Text="[SEX]";
  // other initializations
}

better, declare some constant strings for initial values and use those strings in reset method. Therefore, if you need to check a control against its initial value, you can use that constant.

Posted 5-May-14 10:31am

Vedat Ozan Oner

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)

Matt T Heffron · Accepted Answer · 2014-05-05T10:19:00

Solution 2

You've declared ClearControls to take an array as the parameter but you are passing the 4 individual TextBox items.
Change the declaration to:

C#

public void ClearControls(params TextBox[] A)

so that the arguments are collected into an array automatically.

Of course, if you call this with more than 4 TextBox arguments then the indexing into x will fail as out of bounds.

Posted 5-May-14 10:19am

Matt T Heffron

Comments

Sergey Alexandrovich Kryukov 5-May-14 16:23pm

Sure, a 5.
—SA

Matt T Heffron 5-May-14 16:48pm

Thanks

Salisu Shaibu 5-May-14 17:06pm

10Q man;
Perfect!!!

Salisu Shaibu 5-May-14 17:46pm

I tried to use:
public void ClearControls(params TextBox[] A, params String[] B)
so that "NAME", "SEX", ... can be added on implementation but error line hints that "parameter array must be last parameter in a formal parameter list".
What am I supposed to do please?

Matt T Heffron 5-May-14 18:08pm

That you can't do!
You can have ONLY ONE params parameter that automatically assembles the "tail" of the argument list into an array and it must be the last parameter.
For what you now appear to want you should just do all of the assignments explicitly as in Solution 3.
If you insist on passing the strings and controls to a method then you must build the array for either the strings or the controls and pass that as an array parameter AHEAD of the params parameter of the other type:
public void ClearControls(string[] values, params TextBox[] textBoxes)
used as:
string[] v = new string[] { "NAME", "SEX", "AGE", "SCHOOL"};
ClearControls(v, txtNAME,txtSEX,txtAGE,txtSCHOOL);
OR
public void ClearControls(TextBox[] textBoxes, params string[] values)
used as:
TextBox[] tb = new TextBox[] { txtNAME,txtSEX,txtAGE,txtSCHOOL };
ClearControls(tb, "NAME", "SEX", "AGE", "SCHOOL");

Salisu Shaibu 5-May-14 18:25pm

I got ur comment on the latest post Matt, I was only trying to be stubborn but now I know there is no other way.
Thank you.