одномерный массив с++

[Olchik]

помогите пожайлуста найти здесь ошибку...программа вроде написана правильно,и ошибки не выдаёт,но она не работает,не пойму что в ней не так
условие задачи:найти среднее арифметическое двух наименьших элементов массива

#include<iostream>
#include<math.h>
#include<conio.h>
using namespace std;
int main()
{
   int a[5]={5,13,9,7,11},i,min_1,min_2,sr_ar,i_min_1,i_min_2;
   int min=a[0];
   for(i=0;i<5;i++)
      if(a<min)
      {
         min_1=a;
         i_min_1=i;
      }
      cout<<"min_1="<<min_1<<endl;
      cout<<"i_min_1="<<i_min_1<<endl;
      min=a[0];
      for(i=0;i<5;i++)
         if(a<min && i!=i_min_1)
         {
            min_2=a;
            i_min_2=i;
         }
         cout<<"min_2="<<min_2<<endl;
         cout<<"i_min_2="<<i_min_2<<endl;
         sr_ar=(min-1+min-2)/2;
         cout<<"sr_ar="<<sr_ar<<endl;
         getch ();
         return 0;
}

[InfStudent]

Может все таки так:
#include<iostream>
#include<math.h>
#include<conio.h>
using namespace std;
int main()
{
   int a[5]={5,13,9,7,11},i,min_1,min_2,sr_ar,i_min_1,i_min_2;
   int min=a[0];
   for(i=0;i<5;i++)
      if(a<min)
      {
         min_1=a;
         i_min_1=i;
      }
      cout<<"min_1="<<min_1<<endl;
      cout<<"i_min_1="<<i_min_1<<endl;
      min=a[0];
      for(i=0;i<5;i++)
         if(a<min && i!=i_min_1)
         {
            min_2=a;
            i_min_2=i;
         }
         cout<<"min_2="<<min_2<<endl;
         cout<<"i_min_2="<<i_min_2<<endl;
         sr_ar=(min-1+min-2)/2;
         cout<<"sr_ar="<<sr_ar<<endl;
         getch ();
         return 0;
}

[Olchik]

что-то я не пойму что изменилось??
ваша програма так же как и моя комп мусор выдаёт=(
я тут чуть-чуть не правильно написала (квадратные скобки почему-то не ставятся) вот:

Код: [Выделить]

#include<iostream>
#include<math.h>
#include<conio.h>
using namespace std;
int main()
{
int a[5]={5,13,9,7,11},i,min_1,min_2,sr_ar,i_min_1,i_min_2;
int min=a[0];
for(i=0;i<5;i++)
if(a[i]<min)
{
min_1=a[i];
i_min_1=i;
}
cout<<"min_1="<<min_1<<endl;
cout<<"i_min_1="<<i_min_1<<endl;
min=a[0];
for(i=0;i<5;i++)
if(a[i]<min && i!=i_min_1)
{
min_2=a[i];
i_min_2=i;
}
cout<<"min_2="<<min_2<<endl;
cout<<"i_min_2="<<i_min_2<<endl;
sr_ar=(min_1+min_2)/2;
cout<<"sr_ar="<<sr_ar<<endl;
getch ();
return 0;
}
[/size]

[Данила]

ну для начала мне не нравится,что sr_ar вы делаете интовой.... не факт же, что сумма 2ух меньших поделится без остатка...

[Данила]

во вторых вот тут

Код: [Выделить]

int min=a[0];
for(i=0;i<5;i++)
if(a[i]<min)
{
min_1=a[i];
i_min_1=i;
}
вы считаете,что полюбому найдется число меньше 1го элемента массива...а если первый наименьший? тогда у вас переменные не инициализированы - отсюда мусор...надо как то так

Код: [Выделить]

int min=a[0];
for(i=0;i<5;i++)
if(a[i]<min)
{
min_1=a[i];
i_min_1=i;
}
else
                {
                min_1=a[0];
                i_min_1=0;
}


поиск второго тоже косячен....вы берете за минимум 1ый элемент,который уже является минимумом...и там опять таки получается каша....

я бы на вашем месте отсортировал массив и взял 1ые два члена и все....быстрее,проще и изящнее,чем в лоб

[Данила]

Код: [Выделить]

#include<iostream>
using namespace std;

int main(int argc, char *argv[])
{
  int a[5]={5,13,9,7,11},i,j,tmp;
  float sr_ar;
  for (j=0;j<5;j++)
    for (i=j;i<4;i++) {
        if (a[i]>a[i+1]) {
                tmp=a[i];
                a[i]=a[i+1];
                a[i+1]=tmp;
        }
    }    
  cout<<"first minimal element="<<a[1]<<endl;
  cout<<"second minimal element= "<<a[2]<<endl;
  cout<<"middle ariphmetics= "<<(a[1]+a[2])/2<<endl;
  system("PAUSE");
  return 0;
}
вот вся ваша программа...без кучи переменных, циклов и условий...))

кстати,у вас же не сказанно в задание,что массив известной размерности,может стоит делать для общего случая?

[Olchik]

ну,sr_ar там в принципе значения не имеет,у меня главная проблема с минимумами

в вашей программе я половину не поняла,к сожалению...и что такое tmp?
в вашей выходит 1мин=7,а 2мин= 9.....а 5 вроде меньше 7 и 9...всё-равно что то не то:)

[Olchik]

для общего случая конечно можно,но мне просто так более понятно

[Данила]

да,пардон,при выводе не те индексы указал

Код: [Выделить]

#include<iostream>
using namespace std;

int main(int argc, char *argv[])
{
  int a[5]={5,13,9,7,11},i,j,tmp;
  float sr_ar;
  for (j=0;j<5;j++)
    for (i=j;i<4;i++) {
        if (a[i]>a[i+1]) {
                tmp=a[i];
                a[i]=a[i+1];
                a[i+1]=tmp;
        }
    }   
  cout<<"first minimal element="<<a[0]<<endl;
  cout<<"second minimal element= "<<a[1]<<endl;
  cout<<"middle ariphmetics= "<<(a[1]+a[0])/2<<endl;
  system("PAUSE");
  return 0;
}

а что именно не понятно? сортировка пузырьком по-моему основа основ программирования...tmp - просто временная переменная,которую используем ,когда меняем 2 соседних элемента,что бы не потерять одно из них

[InfStudent]

А что именно Вам не понятно с пузырком:
Для массива размером 15
for (i=0;i<=14;i++)
 if (A>A[i+1])
   {
A=t
A=A[i+1]
A[i+1]=t
   }
    

[Данила]

эм....пузырек с 1им циклом это что то новое) так он пройдет массив 1ин раз,поменяет местами пару элементов  и все)

[Olchik]

капец,пузырьки какие-то,как это всё сложно
я только с циклами разобралась,а тут опять что-то новое
не понятно,потому,что я в програмировании особо не шарю

Данила,конкретно для этих чисел ваша программа работает правильно,но если поменять например первый элемент массива 5 на 50,то снова фигня....мин_1=13,мин_2=7

[Данила]

fixed

Код: [Выделить]

  #include<iostream>

using namespace std;

int main(int argc, char *argv[])



{
  int a[5]={5,13,9,7,11},i,j,tmp;
  float sr_ar;
  for (j=0;j<4;j++)
    for (i=0;i<5-j;i++) {
        if (a[i]>a[i+1]) {
                tmp=a[i];
                a[i]=a[i+1];
                a[i+1]=tmp;
        }
    }   
  cout<<"first minimal element="<<a[0]<<endl;
  cout<<"second minimal element= "<<a[1]<<endl;
  cout<<"middle ariphmetics= "<<(a[0]+a[1])/2<<endl;
  system("PAUSE");
  return 0;
}

[InfStudent]

А да там вложенный цикл нужен))