How can I optimise this code?

[VitzzViperzz]

Okay, so I have written this method that receives a number from the user and performs some mathematical operations on it. The only problem is that it requires lots of Console.WriteLine() statements.

Originally I was thinking of using a For loop but that ended badly. Can someone help?

here is my code:

namespace Multiplication
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.WriteLine("Enter a number: ");
            int userInput = Convert.ToInt32(Console.ReadLine());


            multipicationEngine(userInput, userInput);
            Console.ReadKey();
        }


        static void multipicationEngine(int userNumber, int numberToMultiply)
        {
            Console.WriteLine($"{userNumber} x 1 = {numberToMultiply * 1}");
            Console.WriteLine($"{userNumber} x 2 = {numberToMultiply * 2}");
            Console.WriteLine($"{userNumber} x 3 = {numberToMultiply * 3}");
            Console.WriteLine($"{userNumber} x 4 = {numberToMultiply * 4}");
            Console.WriteLine($"{userNumber} x 5 = {numberToMultiply * 5}");
            Console.WriteLine($"{userNumber} x 6 = {numberToMultiply * 6}");
            Console.WriteLine($"{userNumber} x 7 = {numberToMultiply * 7}");
            Console.WriteLine($"{userNumber} x 8 = {numberToMultiply * 8}");
            Console.WriteLine($"{userNumber} x 9 = {numberToMultiply * 9}");
            Console.WriteLine($"{userNumber} x 10 = {numberToMultiply * 10}");
        }
    }
}

Can you see the long list of 'Console.WriteLine'?

 

[JohnH]

You need to use a For loop and interpolate the index variable just like the method parameters that is already in the string. Post your attempt and maybe someone can help with that.

 

[jmcilhinney]

Why are you passing two numbers to that method? What do they represent? Your code doesn't really seem to make sense from that perspective.