Найти сумму элементов которые расположены между максимальным и минимальным элементами массива.Массив задан Рандомом

@ivladden · Регистрация: 16.02.2013

Author24 — интернет-сервис помощи студентам

Найти сумму элементов которые расположены между максимальным и минимальным элементами массива.Массив задан Рандомом.
Не пойму в чем дело.Остановился на последнем шаге,тоесть - на сумме.Сумма не получается как ни крути.Помогите исправить,добрые прогеры

А,да кстати сумму оно выводит на место 5-го элемента.)))
Вот код

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace laba2_mass
{
    class Program
    {
        public const int n = 6;
 
        public static void Main()
        {
            int[] a = new int[n];
            int i, idx1 = 0, idx2 = 0, s = 0;
 
            Console.WriteLine("Массив");
 
            Random rand = new Random();
            for (i = 0; i < n; i++)
            {
                a[i] = rand.Next(-20, 20);
 
                Console.Write(a[i] + " ");
            }
 
            Console.WriteLine();
            Console.WriteLine();
 
 
            //----- Максимальный элемент -----//
 
            for (i = 0; i < n; i++)
            {
                if (a[i] > 0 && a[i] > a[idx1])
                    idx1 = i;
            }
 
            Console.WriteLine("Максимальный элемент = " + a[idx1]);
 
            //------------------------------//
 
 
            //----- Минимальный элемент -----//
 
            for (i = 0; i < n; i++)
            {
                if (a[i] < 0 && a[i] < a[idx2])
                    idx2 = i;
            }
 
            Console.WriteLine("Минимальный элемент = " + a[idx2]);
            
            //--------------------------------//
 
 
            //----- Сумма элементов -----//
 
            for (i = 0; i < n; i++)
            {
                if (idx1 > idx2 && (i-1) < idx1)
                {
                    s += a[i];
                }
                if (idx1 < idx2 && (i-1) < idx2)
                {
                    s += a[i + 1];
                }
            }
 
            //----------------------------------//
 
            Console.WriteLine("Сумма элементов = " + s);
 
            for (i = 0; i < n; i++)
            {
                a[4] = s;
 
                Console.Write(a[i] + " ");
            }
 
            Console.WriteLine();
 
            Console.ReadKey();
        }
    }
}

@stibish · 24.02.2013, 22:49

C#

 //----- Сумма элементов -----//
 
            for (i = 0; i < n; i++)
            {
               
                    s += a[i];
              
             }

Думаю так должен быть

@Дмитрий3241 · 24.02.2013, 22:56

C#

    static void Main( string[] args )
    {
        const int N = 6;
        int[] arr = new int[6];
        for ( int i = 0; i < N; i++ ) 
            arr[i] = new Random( DateTime.Now.Millisecond ).Next( 0, 11 );
 
        int iMin = 0, iMax = 0, tmp = int.MinValue;
        for ( int i = 0; i < N; i++ )
        {
            if ( arr[i] > tmp )
            {
                iMax = i;
                tmp = arr[i];
            }
        }
 
        tmp = int.MaxValue;
        for ( int i = 0; i < N; i++ )
        {
            if ( arr[i] < tmp )
            {
                iMin = i;
                tmp = arr[i];
            }
        }
 
        tmp = 0;
        for ( int i = Math.Min( iMax, iMin ); i < Math.Max( iMax, iMin ); i++ )
            tmp += arr[i];
 
        arr[5] = tmp;
 
        Console.ReadKey( true );
    }

@darkExpressI2P · 24.02.2013, 22:58

C#

using System;
 
class Program
{
    public static void Main()
    {
        int[] array = new int[10];
        int maximum = 0, minimum = 0;
        Random rnd = new Random();
 
        Console.WriteLine("Вывод случайного массива:");
        for (int i = 0; i < array.GetLength(0); i++)
        {
            array[i] = rnd.Next(-20, 20);
            Console.Write(array[i] + "\t");
            if (array[i] > maximum)
                maximum = array[i];
            else if (array[i] < minimum)
                minimum = array[i];
        }
        Console.WriteLine("Максимальный элемент массива:");
        Console.WriteLine(maximum);
        Console.WriteLine("Минимальный элемент массива");
        Console.WriteLine(minimum);
        Console.WriteLine("Сумма максимального элемента массива и минимального:");
        int sum = maximum + minimum;
        Console.WriteLine(sum);
 
 
        Console.ReadKey();
    }
}

Добавлено через 1 минуту
Вероятность того, что у тебя будут все положительные числа при значении rnd.Next(-20, 20); очень мала.

@Дмитрий3241 · 24.02.2013, 23:01

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

Console.WriteLine("Сумма максимального элемента массива и минимального:");

В задаче нужно сумму элементов между максимальным и минимальным.

@darkExpressI2P · 24.02.2013, 23:03

Дмитрий3241, извеняюсь, не внимательность...

@Zheka13 · 24.02.2013, 23:08

C#

if (idx1 > idx2)
            {
 
                for (i = idx2; i <= idx1; i++)
                    s += a[i];
            }
            else
            {
 
                for (i = idx1; i <= idx2; i++)
                    s += a[i];
            }

Вот сумма
Это включая сами мин и макс

@Дмитрий3241 · 24.02.2013, 23:22

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

Вот сумма
Это включая сами мин и макс

Тогда хотя бы вот так: ( красивее выглядит )

C#

1 2	for ( i = Math.Min( idx1, idx2 ); i <= Math.Max( idx1, idx2 ); i++ ) s += a[i]

@Vitall · 25.02.2013, 00:01

Дмитрий3241,
Ваш код уже на начальном этапе дает сбой заполная массив одними и теме же значениями ,и возвращает всегда ноль

Добавлено через 25 минут
Мой алгоритм тоже не идеален ,но хоть что то

C#

        int[] array = new int[10];
        Random rand = new Random(DateTime.Now.Millisecond);
        Console.WriteLine("Sequence is ");
        for (int i = 0; i < 10; i++)
        {
            array[i] = rand.Next(10);
            Console.Write(array[i]+" ");
        }
 
        
        int max =  array.ToList().IndexOf( array.Max() );
        int min =  array.ToList().IndexOf( array.Min() );
        int start,end;
        if(min<=max)
        {
            start = min;
            end   = max ;
        }
        else 
        {
           end   = min;
           start = max ;
        }
        Console.WriteLine("\nSum is : "+array.Where((a, g) => 
            start < g && g < end).Sum());
 
 
        Console.ReadLine();

@ivladden · 25.02.2013, 00:10 **[ТС]**

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

C#

 //----- Сумма элементов -----//
 
            for (i = 0; i < n; i++)
            {
               
                    s += a[i];
              
             }

Думаю так должен быть

Так оно находит сумму всех элементов

@Zheka13 · 25.02.2013, 00:18

ivladden
Если еще неразобрались в ответах

то,а если разобрались то сообщение лишнее)
В ващем коде просто это заменить на

C#

for (i = 0; i < n; i++)
            {
                if (idx1 > idx2 && (i-1) < idx1)
                {
                    s += a[i];
                }
                if (idx1 < idx2 && (i-1) < idx2)
                {
                    s += a[i + 1];
                }
            }

это

C#

1 2	for ( i = Math.Min( idx1, idx2 ); i <= Math.Max( idx1, idx2 ); i++ ) s += a[i];

@ivladden · 25.02.2013, 00:27 **[ТС]**

Vitall, код не работает

выдает просто пустю консоль....

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

Дмитрий3241,
Ваш код уже на начальном этапе дает сбой заполная массив одними и теме же значениями ,и возвращает всегда ноль

Добавлено через 25 минут
Мой алгоритм тоже не идеален ,но хоть что то

C#

        int[] array = new int[10];
        Random rand = new Random(DateTime.Now.Millisecond);
        Console.WriteLine("Sequence is ");
        for (int i = 0; i < 10; i++)
        {
            array[i] = rand.Next(10);
            Console.Write(array[i]+" ");
        }
 
        
        int max =  array.ToList().IndexOf( array.Max() );
        int min =  array.ToList().IndexOf( array.Min() );
        int start,end;
        if(min<=max)
        {
            start = min;
            end   = max ;
        }
        else 
        {
           end   = min;
           start = max ;
        }
        Console.WriteLine("\nSum is : "+array.Where((a, g) => 
            start < g && g < end).Sum());
 
 
        Console.ReadLine();

Добавлено через 3 минуты
Zheka13,пытался,к сожалению выдает пустую консоль

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

ivladden
Если еще неразобрались в ответах

то,а если разобрались то сообщение лишнее)
В ващем коде просто это заменить на

C#

for (i = 0; i < n; i++)
            {
                if (idx1 > idx2 && (i-1) < idx1)
                {
                    s += a[i];
                }
                if (idx1 < idx2 && (i-1) < idx2)
                {
                    s += a[i + 1];
                }
            }

это

C#

1 2	for ( i = Math.Min( idx1, idx2 ); i <= Math.Max( idx1, idx2 ); i++ ) s += a[i];

@Zheka13 · 25.02.2013, 00:32

Только опять проверил,у меня работает.

@ivladden · 25.02.2013, 00:35 **[ТС]**

И тут я понял что просто Visual Studio почему-то выдавало всегда пустую консоль.При любых расскладах.

@Дмитрий3241 · 25.02.2013, 00:37

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

Дмитрий3241,
Ваш код уже на начальном этапе дает сбой заполняя массив одними и теме же значениями ,и возвращает всегда ноль

Ноль он кстати не возвращает, а вот насчет одинаковых чисел это да не подумал, исправил, проверил теперь все работает:

C#

static void Main( string[] args )
    {
        const int N = 6;
        int[] arr = new int[6];
        for ( int i = 0; i < N; i++ )
            arr[i] = new Random( DateTime.Now.Millisecond + i ).Next( 0, 11 );
 
        int iMin = 0, iMax = 0, tmp = int.MinValue;
        for ( int i = 0; i < N; i++ )
        {
            if ( arr[i] > tmp )
            {
                iMax = i;
                tmp = arr[i];
            }
        }
 
        tmp = int.MaxValue;
        for ( int i = 0; i < N; i++ )
        {
            if ( arr[i] < tmp )
            {
                iMin = i;
                tmp = arr[i];
            }
        }
 
        tmp = 0;
        for ( int i = Math.Min( iMax, iMin ) + 1; i < Math.Max( iMax, iMin ); i++ )
            tmp += arr[i];
 
        foreach ( int i in arr )
            Console.Write( i + " " );
        Console.WriteLine( "Сумма: " + tmp );
 
        Console.ReadKey( true );
    }

@ivladden · 25.02.2013, 00:44 **[ТС]**

Zheka13, я перезапустил Visual Studio,все работает,но не так..(

@Zheka13 · 25.02.2013, 00:48

Видимо вам нужно чтобы между ними ,но иих самих не считала,тогда так.

C#

1 2	for (i = Math.Min(idx1, idx2)+1; i < Math.Max(idx1, idx2); i++) s += a[i];

@vladimir8686 · 25.02.2013, 00:54

C#

using System;
 
namespace ElementsSummary
{
    class Program
    {
        static Random r = new Random();
        int[] mass = new int[10];
        
        void MainMeth()
        {
            int max = 0, min = 0, sum = 0;
            int temp = 0, temp2 = 0;
            
            for (int i = 0; i < mass.Length; i++) 
            {
                mass[i] = r.Next(0,20);
            }
            Console.Write("Array: ");
            foreach(int i in mass)
                Console.Write(i + " ");
            
            for (int i = 0; i < mass.Length; i++) 
            {
                if(mass[i] > max)
                {
                    max = mass[i];
                    temp = i;
                }
                if(i == 0)
                    min = mass[i];
                
                if(mass[i] < min)
                {
                    min = mass[i];
                    temp2 = i;
                }   
            }
            Console.WriteLine();
            Console.WriteLine();
            Console.WriteLine("Item {0} has max value with {1}", temp+1, max);
            Console.WriteLine("Item {0} has max value with {1}", temp2+1, min);
            
            if(temp2 < temp)
            {
                for (int i = temp2; i < temp-1; i++) 
                 {
                    sum+=mass[i+1];
                 }  
                 Console.WriteLine("Elements' sum: " + sum);
            }
            else
            {
                for (int i = temp; i < temp2-1; i++) 
                 {
                    sum+=mass[i+1];
                 }  
                 Console.WriteLine("Elements' sum: " + sum);
            }
            mass[4] = sum;
            Console.WriteLine();
            Console.Write("New Array: ");
            foreach(int i in mass)
                Console.Write(i + " ");
        }
        public static void Main(string[] args)
        {
            Program pr = new Program();
            pr.MainMeth();
            Console.ReadLine();
        }
    }
}

Комментить не стал, и так все кристально ясно.

@ivladden · 25.02.2013, 00:56 **[ТС]**

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

Видимо вам нужно чтобы между ними ,но иих самих не считала,тогда так.

C#

1 2	for (i = Math.Min(idx1, idx2)+1; i < Math.Max(idx1, idx2); i++) s += a[i];

Огромное,спасибо))
Вот этот тот самый лакомый кусочек кода,которого мне не хватало.

@ivladden · 25.02.2013, 01:00 **[ТС]**

Вот выкладываю весь код

C#

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
 
namespace laba2_mass
{
    class Program
    {
        public const int n = 6;
 
        public static void Main()
        {
            int[] a = new int[n];
            int i, idx1 = 0, idx2 = 0, s = 0;
 
            Console.WriteLine("Массив");
 
            Random rand = new Random();
            for (i = 0; i < n; i++)
            {
                a[i] = rand.Next(-20, 20);
 
                Console.Write(a[i] + " ");
            }
 
            Console.WriteLine();
            Console.WriteLine();
 
 
            //----- Максимальный элемент -----//
 
            for (i = 0; i < n; i++)
            {
                if (a[i] > 0 && a[i] > a[idx1])
                    idx1 = i;
            }
 
            Console.WriteLine("Максимальный элемент = " + a[idx1]);
 
            //------------------------------//
 
 
            //----- Минимальный элемент -----//
 
            for (i = 0; i < n; i++)
            {
                if (a[i] < 0 && a[i] < a[idx2])
                    idx2 = i;
            }
 
            Console.WriteLine("Минимальный элемент = " + a[idx2]);
            
            //--------------------------------//
 
 
            //----- Сумма элементов -----//
 
            for (i = Math.Min(idx1, idx2) + 1; i < Math.Max(idx1, idx2); i++)
                s += a[i];
 
            //----------------------------------//
 
            Console.WriteLine("Сумма элементов = " + s);
 
            for (i = 0; i < n; i++)
            {
                a[4] = s;
 
                Console.Write(a[i] + " ");
            }
 
            Console.WriteLine();
 
            Console.ReadKey();
        }
    }
}

@ivladden 18 / 7 / 0 Регистрация: 16.02.2013 Сообщений: 101
	25.02.2013, 00:35 [ТС]	14
	И тут я понял что просто Visual Studio почему-то выдавало всегда пустую консоль.При любых расскладах. 0

@Дмитрий3241 660 / 530 / 137 Регистрация: 07.07.2011 Сообщений: 1,232
	24.02.2013, 23:01	5
	Сообщение от darkExpressI2P Console.WriteLine("Сумма максимального элемента массива и минимального:"); В задаче нужно сумму элементов между максимальным и минимальным. 0

@darkExpressI2P 20 / 17 / 0 Регистрация: 15.08.2012 Сообщений: 122
	24.02.2013, 23:03	6
	Дмитрий3241, извеняюсь, не внимательность... 0

@Zheka13 32 / 32 / 12 Регистрация: 18.03.2011 Сообщений: 102
	25.02.2013, 00:32	13
	Только опять проверил,у меня работает. 1

@ivladden 18 / 7 / 0 Регистрация: 16.02.2013 Сообщений: 101
	25.02.2013, 00:44 [ТС]	16
	Zheka13, я перезапустил Visual Studio,все работает,но не так..( Миниатюры 0