Оптимизировать код.

[RickeyF]

Напишите метод который в качестве входных параметров принимает массив целых чисел. Результатом работы метода является индекс максимального элемента массива. Вот такая задача, набросал кода, но может можно сделать проще?

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
 
namespace ConsoleApplication1
{
    class PassOb
    {
        public static int MaxInd(int[] nums)
        {
            int max;
            int ind;
 
            max = nums[0];
            ind = 0;
            for (int i = 1; i < nums.Length; i++)
            {
                if (max < nums[i])
                {
                    max = nums[i];
                    ind = i;
                }
            }
 
            return ind;
        }
        static void Main(string[] args)
        {
            int a;
            int i;
            int j = 0;
            //Запрашиваем длину массива
            Console.WriteLine("Введите длину массива");
            j = Convert.ToInt32(Console.ReadLine());
            int[] arr = new int[j];
            //Вводим массив с клавиатуры
            for (i=0; i<arr.Length;i++)
            {
                Console.WriteLine("Введите число "+(i+1));
                arr[i] = Convert.ToInt32(Console.ReadLine());
            }
            //Выводим массив
            Console.WriteLine("Ваш массив\n");
            for (i=0; i<arr.Length;i++)
            {
                Console.Write(arr[i]+" ");
            }
            //Вызываем метод
            a = MaxInd(arr);
            Console.WriteLine("\nИндекс Макс Числа="+a);
            Console.ReadKey();
 
        }
    }
}

[Serge_Bliznykov]

во-первых, при поиске индекса НЕ нужна переменная для хранения самого максимального значения:

        public static int MaxInd(int[] nums)
        {
            int ind = 0;
            for (int i = 1; i < nums.Length; i++)
            {
                if (nums[ind] < nums[i])
                {
                    ind = i;
                }
            }
            return ind;

а во-вторых, так не проще?

int maxIndex = Array.IndexOf(arr, arr.Max());
Console.WriteLine("\nИндекс Макс Числа = [0]",maxIndex);

[RickeyF]

Так а как будет выглядеть полностью проект с этим кодом? Я вставляю а у меня пишет тип не используется arr и тд.

[Serge_Bliznykov]

Сообщение от RickeyF

Регистрация: 19.05.2017
Сообщений: 9
Репутация: 10

Так а как будет выглядеть полностью проект с этим кодом?

так:

using System;
using System.Linq;

namespace ConsoleApplication1
{
    class PassOb
    {
        static void Main(string[] args)
        {
            //Запрашиваем длину массива
            Console.WriteLine("Введите длину массива");
            int n = Convert.ToInt32(Console.ReadLine());
            int[] arr = new int[n];
            //Вводим массив с клавиатуры
            for (int i=0; i<arr.Length;i++)
            {
                Console.WriteLine("Введите число "+(i+1));
                arr[i] = Convert.ToInt32(Console.ReadLine());
            }
            //Выводим массив
            Console.WriteLine("Ваш массив\n");
            for (int i=0; i<arr.Length;i++)
            {
                Console.Write(arr[i]+" ");
            }
            

		int maxIndex = Array.IndexOf(arr, arr.Max());
		Console.WriteLine("\nИндекс Макс Числа = {0}",maxIndex);
			
		Console.ReadKey();
 
        }
    }
}