Trying to pass an array into a method

[357mag]

I have a program I did in Java and I'm trying to do it in C#. Visual Studio is flagging a lot at the bottom. Where I declare and define the method at the bottom. I want my method to receive the numbers array and then multiply each element by 2 and then send that result back into the main program.

But I almost think I got the code in the wrong spot, I don't know.

 private void buttonShowModifiedValues_Click(object sender, EventArgs e)
        {
            int[] numbers = { 2, 4, 5, 6, 8 };
            int[] newArray;

            newArray = modifyNumbers(numbers);
        }

        private void FormPassArrayToMethod_Load(object sender, EventArgs e)
        {
            int[] numbers = { 2, 4, 5, 6, 8 };

            for (int i = 0; i < numbers.Length; i++)
                textBoxOriginalArrayValues.Text = textBoxOriginalArrayValues.Text + numbers[i] + " ";
        }
        public static int[] modifyNumbers(int[] modifiedNumbers)

            for(int i = 0; i<modifiedNumbers.Length; i++)
                modifiedNumbers[i] *= 2;
            
            return modifiedNumbers;

 

[357mag]

I got rid of all the red, squiggly lines. I put in braces so that problem looks solved. But when I click on the Show Modified Values button, my program doesn't show anything in the textbox. I'll keep digging around.

 

[357mag]

Okay I found the problem. I was missing a loop to print the new array in the textbox. I don't quite understand why I needed to add braces around my method definition though.

 

[Skydiver]

C# is not Python which uses indents to denote scopes. C# (and Java) need braces around method bodies (unless you use expression bodied methods).

 

[cjard]

I don't quite understand why I needed to add braces around my method definition though.

C# language spec says so. That's fairly inviolable I'm afraid. Methods typically look like:

returntype MethodName(argtype arg1, argtype arg2..){
  multi;
  line;
  body;
}

returntype MethodName(argtype arg1, argtype arg2..) => singleline body;

Note the uppercase letter at the start of the name