Произведение между максимальным и минимальным значениями массива

[Maxim93m]

Добрый вечер!
Подскажите,пожалуйста, почему не считает произведение между максимальным и минимальным значениями массива.

#include <stdio.h>
#include <conio.h>


int main ()
{
 const int n = 5;
 float mass [n] = {3, 2, 1, 4, -5};
 float sum,sum1;
 int indx;
 float pr;


   	 for(indx=0,sum=0;indx<n;indx++){
   if (mass[indx]<0) { sum+=mass[indx];}}
   	 for(indx=0,sum1=0;indx<n;indx++){ sum1+=mass[indx];}
       for(indx=0;indx<n-1;indx++){
  int imin=indx;
  		 for (int j=indx+1; j<n; j++)
   if( mass[j]<mass[imin]) imin =j;
   float a=mass[indx];
   mass[indx]=mass[imin];
   mass[imin]=a;
  }


      for(indx=1,pr=1;indx<n-1;indx++){ pr*=mass[indx];}

  printf("\nsumma otric = %0.2f\n", sum);
  printf("summa vseh = %0.2f\n", sum1);
  printf("\nV porjadke vozrast:\n");
  for(indx=0;indx<n;indx++){ printf("%0.2f\n", mass[indx]);}
  printf("proizved = %0.2f\n",pr);
 }

[Lardes]

А так разве не вариант?

    pr = mass[0] * mass[n - 1];

вместо

      for(indx=1,pr=1;indx<n-1;indx++){ pr*=mass[indx];}

Или я что-то не так понимаю?

[8Observer8]

На C++11 эта программа будет короче, проще и надёжнее

Произведение элементов массива между максимальным и минимальным:

#include <iostream>     // std::cout
#include <algorithm>    // std::min_element std::max_element std::accumulate

int main( )
{
    // Определяем массив
    int myints[] = { 3, 2, 1, 4, -5 };
    const size_t n = sizeof( myints ) / sizeof( myints[0] );

    // Копируем массив в контейнер
    std::vector<int> vec( myints, myints + n );

    // Максимальный элемент
    auto itMax = std::max_element( vec.cbegin(), vec.cend() );

    // Минимальный элемент
    auto itMin = std::min_element( vec.cbegin(), vec.cend() );

    // Вычисляем индексы максимального и минимального элементов
    int indexMax = std::distance( vec.cbegin(), itMax );
    int indexMin = std::distance( vec.cbegin(), itMin );

    // Вычисляем произведение
    int product;
    if ( indexMax < indexMin ) { // Проверка очерёдности max-min
        product = std::accumulate( itMax,
                                   itMin,
                                   1,
                                   [] ( int a, int b ) { return a * b; } );
        std::cout << "Product is " << product << "\n\n";
    } else {
        std::cerr << "Error: the max element must be first." << "\n\n";
    }

    return 0;
}

[Maxim93m]

Спасибо всем! Я попробую.

[Maxim93m]

Решил задачу. Всем огромное спасибо!