How do I call a 'public' array?

Question

1.00/5 (1 vote)

See more:

How do I define a public method that has an array in it?

I would like to set a method to public so it can be used with multiple button methods but the program says 'not all code paths return a value'. I assume that means I have not changed the 'void' value to the correct type but can anyone tell me what it is?

I have the following code:

C#

        public String bob ()
        {
        string [] averages = new string[2];
        averages[0] = value1.Text;
        averages[1] = value2.Text;

        }

        private void button1_Click(object sender, EventArgs e)
        {
            bob();
            label1.Text = averages[1];

}

Thanks.

[Edit]Code block added[/Edit]

Posted 26-Mar-13 3:39am

Member 9941061

Updated 26-Mar-13 3:40am

Thomas Daniels

v2

Add a Solution

Comments

Sergey Alexandrovich Kryukov 26-Mar-13 13:39pm

You cannot "call" an array. It is not a method, function, procedure, subroutine, property or operator.
—SA

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)

Thomas Daniels · Answer 1 · 2013-03-26T03:41:00

Solution 1

Hi,

Change bob() into this:

C#

public String[] bob ()
       {
       string [] averages = new string[2];
       averages[0] = value1.Text;
       averages[1] = value2.Text;
       return averages;
       }

More about the return keyword:
http://msdn.microsoft.com/en-us/library/1h3swy84.aspx[^]
Hope this helps.

Posted 26-Mar-13 3:41am

Thomas Daniels

Comments

Ian A Davidson 26-Mar-13 12:24pm

+5 for the answer. However, Member9941061 please note 1) you shouldn't need your method to be public, because you're calling it from a private method; 2) see Jeff's solution. Why do you need to construct an array of the values in your text boxes anyway, when you have direct access to them?

Jeff Blankenburg · Answer 2 · 2013-03-26T03:45:00

Solution 2

Why not just do this instead, since it's what you appear to be trying to do?

C#

private void button1_Click(object sender, EventArgs e)
{
   label1.Text = value2.Text;
}

Posted 26-Mar-13 3:45am

Jeff Blankenburg

D.Nikolov · Answer 3 · 2013-03-26T03:48:00

C#

public string[] bob ()
{
    string [] averages = new string[2];
    averages[0] = value1.Text;
    averages[1] = value2.Text;
    return averages;
}

private void button1_Click(object sender, EventArgs e)
{
    string[] bobResult = bob();
    label1.Text = bobResult[1];
}

This is one solution to your question. The method "bob" returns string array. In the button1_Click method the result from "bob" call is assigned to variable with type string[] and name bobResult.