Перенести нулевые элементы в начало массива

@Komo4ek · Регистрация: 08.05.2016

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

Доброго времени. Ниже дан фрагмент программы, который должен сортировать массив следующим образом: сначала располагаются все элементы, равные нулю, а потом — все остальные. Пример: исходный массив - 10203, отсортированный должен выглядеть так - 00123. Но алгоритм не работает.

C++ (Qt)

// Сортировка массива
void assort(int M[100], int k)
{
 int i,j;
 int p;
 j=0;
 for (i=1; i<=k; i++)
 {
  if (M[i]==0)
  {
    p = M[i];
    M[i] = M[j];
    M[j] = p;
    j=j+1;
  }
 }
 cout << "Assorted array: ";
 for (i=1; i<=k; i++)
  cout << M[i] << " ";
}

@Banonanelly · 29.06.2016, 08:13

А почему нумерацию с 1 начинаете?

C++

1	for (i = 0; i < k; i++)

Добавлено через 12 минут
И вообще, вот правильный алгоритм

C++

int i1=0, i2=n-1;
 
while (i1<i2)
{
    while (i1 < i2 && !a[i1])
         i1++;
    while (i2 > i1 && a[i2])
        i2--;
    if (i1 < i2)
    {
        int tmp=a[i1];
        a[i1]=a[i2];
        a[i2]=tmp;
    }
    i1++;
    i2--;
}

@zss · 29.06.2016, 08:35

Banonanelly, Ваш алгоритм нарушит порядок расположения ненулевых элементов (их сортировать не надо).

C++

#include <iostream>
using namespace std;
 
void assort(int* M, int k)
{
    int* t=new int[k];
    int j=0;
    for (int i=0; i<k; i++)
        if (M[i]==0)
            t[j++]=0;
    for (int i=0; i<k; i++)
        if (M[i]!=0)
            t[j++]=M[i];
    for (int i=0; i<k; i++)
        M[i]=t[i];
    delete[] t;
}
int main()
{
    int x[5]={1,0,99,0,15};
    assort(x,5);
    cout << "rezult array: ";
    for (int i=0; i<5; i++)
        cout << x[i] << " ";
    system("pause");
    return 0;
}

@Komo4ek · 30.06.2016, 05:45 **[ТС]**

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

C++

Ваш алгоритм работает, но в моей программе (когда я его перенес и адаптировал) он выдаёт следующее (см. скрин 1). Причем это 1049 выдает при любом массиве любой размерности вместо последнего 0 или первого числа<>0 в отсортированном массиве в зависимости от того, с чего начинаем. На 2 и 3 скрине должны быть одинаковые результаты, но это не так.
Может, проблема в другой части программы? Вот код всей программы, хотя я ошибок не нашел.

C++ (Qt)

//---------------------------------------------------------------------------
 
#include <vcl.h>
#pragma hdrstop
 
//---------------------------------------------------------------------------
 
#include <Windows.h>
#include <iostream.h>
using namespace std;
//---------------------------------------------------------------------------
#include <stdio.h>
 
#pragma argsused
 
// Ввод массива
void input(int M[100], int k)
{
 int i;
 for (i=1; i<=k; i++)
 {
  cout << "Enter " << i <<" element: ";
  cin >> M[i];
 }
 cout << "Entered array: ";
 for (i=1; i<=k; i++)
  cout << M[i] << " ";
}
 
// Минимальный элемент массива
float min_n(int M[100], int k)
{
 int i;
 int minx;
 minx=100000;
 for (i=1;i<=k;i++)
 {
  if (M[i]<minx)
  minx=M[i];
 }
 return minx;
}
 
// Сумма элементов, расположенных между первым и последним положительными элементами
float sum_fl(int M[100], int k, bool &f1)
{
 int i,p,q,d;
 int s;
 p=0;
 s=0;
 for (i=1;i<=k;i++)
 {
  if (M[i]>0)
  {
   p=p+1;
   if (p==1) q=i;
   d=i;
  }
 }
 if (p<2) f1=false;
 else
 {
  f1=true;
  for (i=q+1; i<=d-1; i++)
   s=s+M[i];
 }
 return s;
}
 
// Сортировка массива
void assort(int* M, int k)
{
    int* t=new int[k];
    int j=0;
    for (int i=0; i<k; i++)
        if (M[i]==0)
            t[j++]=0;
    for (int i=0; i<k; i++)
        if (M[i]!=0)
            t[j++]=M[i];
    for (int i=0; i<k; i++)
        M[i]=t[i];
    delete[] t;
 
    /* for (int i=0; i<k; i++)
     if (M[i]==1049)
      M[i]=0; */
      
    cout << "rezult array: ";
    for (int i=0; i<k; i++)
        cout << M[i] << " ";
}
 
// Основная программа
int main()
{
 /* SetConsoleCP(1251);
 SetConsoleOutputCP(1251); */
 int A[100];
 int n;
 bool f;
 do
 {
  cout << "Razmer massiva (n>0): ";
  cin >> n;
 }
 while (n<=0);
 input(A,n);
 cout << "\n";
 cout << "Minimalnii element: ";
 cout << min_n(A,n);
 cout << "\n";
 cout << "Summa mejdu poloj. elementami: ";
 sum_fl(A,n,f);
 if (f==false)
 {
 cout << "--- \n";
 cout << "(V massive menshe 2 polojitel'nih elementov)";
 }
 else cout << sum_fl(A,n,f);
 cout << "\n";
 assort(A,n);
  cout << "\n";
 system("pause");
 return 0;
 
}

@zss · 30.06.2016, 06:41

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

for (i=1; i<=k; i++)

Индексы в массиве начинаются с нуля и заканчиваются k-1 - м элементом

C++

1	for (i=0; i<=k-1; i++)

@Komo4ek · 30.06.2016, 07:01 **[ТС]**

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

Индексы в массиве начинаются с нуля и заканчиваются k-1 - м элементом

Заработало, спасибо, но почему это так важно? Почему от такой мелочи зависит так много?

@zss · 30.06.2016, 07:06

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

@Komo4ek 0 / 0 / 0 Регистрация: 08.05.2016 Сообщений: 19
	30.06.2016, 07:01 [ТС]	6
	Сообщение от zss Индексы в массиве начинаются с нуля и заканчиваются k-1 - м элементом Заработало, спасибо, но почему это так важно? Почему от такой мелочи зависит так много? 0

@zss Модератор 13507 / 10757 / 6412 Регистрация: 18.12.2011 Сообщений: 28,718
	30.06.2016, 07:06	7
	Это не мелочь. Нулевой элемент остается неинициализированным, а обращаясь к k-ому элементу Вы выходите за границы массива с непредсказуемыми последствиями. 0