Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Центр помощи > [C++] Задание с одномерным массивом


Автор: Asakurbi4 25.1.2012, 23:49
Задание: объявляем одномерный массив целого типа, размера не менее 15 этот массив необходимо заполнить с помощью интегратора случайных чисел. информацию сортируют методом пузырька, элементы с четными индексами отсортированный по убыванию, с нечетными по возрастанию.

Подскажите где? и какая ошибка?  Препод говорит, что я сортировку не доделал...

Код

#include <iostream>
#include <iomanip>
#include <ctime>
 
int main()
{
        setlocale(LC_ALL, "Russian");   //  ставится какой-то язык
 
        int N; // переменная N - целая
        std::cout << "Введите кол-во элементов массива: ";
        std::cin >> N;
        int *MAS=new int [N];// Массив размера N
        std::cout << "Первоначальный массив: " << std::endl;
        for (int i=0; i<N; i++) // заполняем случайнми числами,
      {          
 MAS[i]=rand() % 20;\\массив I рандомно заполняется целыми числами
        std::cout << std::setw(3) << MAS[i]; // выводим на экран массив
        }
        std::cout << std::endl;
 
        int tmp;
        
    for (int i=0; i < N; i++) // фильтруем массив методом пузыркя, сравнивая элемент с предыдущем и если меньше меняем местами
        for (int j=N-1; j > i; j-- )    
        if ( MAS[j-1] < MAS[j] )
                {
                            tmp=MAS[j-1]; MAS[j-1]=MAS[j]; MAS[j]=tmp;
                }
  
                std::cout << "Отсортированный массив: " << std::endl; // и выводим его на экран.
                for (int i=0; i<N; i++)
                        std::cout << std::setw(3) << MAS[i];
 
        std::cout << std::endl;
        delete []MAS;
    system("pause");
    return 0;
}


Автор: t_gran 26.1.2012, 05:24
Цитата

Подскажите где? и какая ошибка?

Ошибок нет

Цитата

Препод говорит, что я сортировку не доделал...

Не повезло Вам с препадом smile

Автор: Asakurbi4 26.1.2012, 09:26
То есть условие полностью выполнено?

Автор: t_gran 26.1.2012, 09:52
Asakurbi4, пардон, действительно ведь ваш алгоритм не отвечает требованию. Просто задание до конца не прочитал и начал искать баг в самой сортировке, и естественно я его там не нашёл. Вы сортируете свой массив по убыванию. Это действительно только часть, т.к. по убыванию у вас должны быть только по чётным индексам. По нечётным должно быть возрастание.

Автор: Asakurbi4 26.1.2012, 10:09
А не могли бы вы помочь это сделать пожалуйста!!!

Автор: t_gran 26.1.2012, 10:16
Asakurbi4, ну что-то вроде этого, но только лучше направление сортировки поменять, а то выходит путаница и когда чётное количество элементов, то чётные сортируются по убывание, а когда нечётное количество элементов, то на возрастание. Поэтому, лучше поменяйте направление.
Код

#include <iostream>
#include <iomanip>
#include <cstdlib>
#include <ctime>

template <typename T>
void Swap(T& a, T& b)
{
   T buff = a;
   a = b;
   b = buff;
}

int main()
{
   srand(time(NULL));
   setlocale(LC_ALL, "Russian");   //  ставится какой-то язык

   int N; // переменная N - целая
   std::cout << "Введите кол-во элементов массива: ";
   std::cin >> N;
   int* MAS = new int [N]; // Массив размера N
   std::cout << "Первоначальный массив: " << std::endl;
   for (int i = 0; i < N; i++) // заполняем случайнми числами,
   {
      MAS[i] = rand()%89 + 10;
      //массив I рандомно заполняется целыми числами
      std::cout << std::setw(3) << MAS[i]; // выводим на экран массив
   }
   std::cout << std::endl;

   for (int i = 0; i < N; i += 2)
   {
      for (int j = N-1; j > i; j -= 2)
      {
         if (MAS[j-2] < MAS[j])
         {
            Swap(MAS[j-2], MAS[j]);
         }
      }
   }

   for (int i = 1; i < N; i += 2)
   {
      for (int j = N-2; j > i; j -= 2)
      {
         if (MAS[j-2] > MAS[j])
         {
            Swap(MAS[j-2], MAS[j]);
         }
      }
   }
   
   std::cout << "Отсортированный массив: " << std::endl; // и выводим его на экран.
   for (int i = 0; i < N; i++)
   {
      std::cout << std::setw(3) << MAS[i];
   }

   std::cout << std::endl;
   delete [] MAS;
   system("pause");
   return 0;
}


Добавлено через 8 минут и 27 секунд
Т.е. как-то так:
Код

#include <iostream>
#include <iomanip>
#include <cstdlib>
#include <ctime>

template <typename T>
void Swap(T& a, T& b)
{
   T buff = a;
   a = b;
   b = buff;
}

int main()
{
   srand(time(NULL));
   setlocale(LC_ALL, "Russian");   //  ставится какой-то язык

   int N; // переменная N - целая
   std::cout << "Введите кол-во элементов массива: ";
   std::cin >> N;
   int* MAS = new int [N]; // Массив размера N
   std::cout << "Первоначальный массив: " << std::endl;
   for (int i = 0; i < N; i++) // заполняем случайнми числами,
   {
      MAS[i] = rand()%89 + 10;
      //массив I рандомно заполняется целыми числами
      std::cout << std::setw(3) << MAS[i]; // выводим на экран массив
   }
   std::cout << std::endl;

   for (int i = N-2; i > 0 ; i -= 2)
   {
      for (int j = 0; j < i; j += 2)
      {
         if (MAS[j] > MAS[j+2])
         {
            Swap(MAS[j], MAS[j+2]);
         }
      }
   }

   for (int i = N-1; i > 0 ; i -= 2)
   {
      for (int j = 1; j < i; j += 2)
      {
         if (MAS[j] < MAS[j+2])
         {
            Swap(MAS[j], MAS[j+2]);
         }
      }
   }

   std::cout << "Отсортированный массив: " << std::endl; // и выводим его на экран.
   for (int i = 0; i < N; i++)
   {
      std::cout << std::setw(3) << MAS[i];
   }

   std::cout << std::endl;
   delete []MAS;
   system("pause");
   return 0;
}

Автор: Asakurbi4 26.1.2012, 10:25
Огромное вам спасибо!!!

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)