Быстрая сортировка. Что не так?

@c0cu4 · Регистрация: 06.12.2012

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

Всем привет. Задание - написать квиксорт с уменьшенной глубиной рекурсии, то есть "сначала сортировать более короткий кусок, а затем вместо повторного вызова переходить к началу процедуры с новыми значениями указателя на массив и его длины"

C++

void f9(double *a, int n)
{
  double x, tmp, *b = a;
  int i, j, length = n;
 
  while (length > 1)
    {
      x = b[length/2];
 
      for (i = 0; i < length; i++)
        if (b[i] >= x)
          break;
 
      for (j = length - 1; j > i ; j--)
        if (b[j] <= x)
          break;
 
      while (i <= j)                                 //начинается обмен
        {
          tmp = b[j];
          b[j] = b[i];
          b[i] = tmp;
          i++;
          j--;
      
          for (; i < length - 1; i++)
            if (b[i] >= x)
          break;
 
          for (; j >= i ; j--)
            if (b[j] <= x)
          break;
        }                                            //заканчивается обмен 
 
      if (length - i > j)                            //дальше находим меньшую часть
        {                                            //и вызываем для неё еще раз
          f9(b, j);                                  //наибольшая, по идее, продолжается
          b = b + i;                                 //в цикле while (length > 1)
          length = length - i;
        }
      else
    {
      if (length > i)
            f9(b + i, length - i);
          length = j;
        }
    }
}

Всё никак не могу понять, что не так. Буду благодарен за помощь.

@petyahohlov · 22.04.2014, 13:20

В цикле while( i <=j ) ты два breakа не включил в for

Добавлено через 2 минуты
У тебя какая то странная функция. У меня без всякой рекурсии

@c0cu4 · 23.04.2014, 10:31 **[ТС]**

Спасибо, сейчас посмотрю.
А можешь скинуть вариант без рекурсии, если он зарыт не где-нибудь глубоко?

Так это я криво напечатал вроде только. Там же for, потом в нём if, а break в if'e, то бишь к for относится

Добавлено через 19 часов 48 минут
Если сюда вдруг кто заглянет, то выкладываю рабочий вариант. Ошибка была в том, что длина левого подмассива j + 1, а не j (37 и 45 строки). И эту идиотскую ошибку я пытался найти полтора дня

Спасибо всем откликнувшимся, а именно одному человеку.

C++

void f9(double *a, int n)
{
  double x, tmp, *b = a;
  int i, j, length = n;
 
  while (length > 1)
    {
      x = b[length/2];
 
      for (i = 0; i < length; i++)
        if (b[i] >= x)
          break;
 
      for (j = length - 1; j > i ; j--)
        if (b[j] <= x)
          break;
 
      while (i <= j)                                 //начинается обмен
        {
          tmp = b[j];
          b[j] = b[i];
          b[i] = tmp;
          i++;
          j--;
      
          for (; i < length - 1; i++)
            if (b[i] >= x)
          break;
 
          for (; j >= i ; j--)
            if (b[j] <= x)
          break;
        }                                            //заканчивается обмен 
 
      if (length - i > j)                            //дальше находим меньшую часть
        {                                            //и вызываем для неё еще раз
          f9(b, j + 1);                              //наибольшая, по идее, продолжается
          b = b + i;                                 //в цикле while (length > 1)
          length = length - i;
        }
      else
    {
      if (length > i)
            f9(b + i, length - i);
          length = j + 1;
        }
    }
}

@petyahohlov · 24.04.2014, 01:15

C++

void QuickSort( Tstudent st[], int n)   // Tstudent st[] - передаём структуру которую сортируем
{                                       // n -  количество элементов
    struct stack
    {
        int l;
        int r;
    }stack[20];
 
    int i, j, left, right, x, s = 0;
    Tstudent t;
 
    stack[s].l = 0, stack[s].r = n -1;
    while(s != -1)
    {
        left = stack[s].l;
        right = stack[s].r;
        s--;
        while(left < right)
        {
            i = left;
            j = right;
            x = st[(left + right)/2].oaip;
            while(i <= j)
            {
                while(st[i].oaip < x) i++;
                while(st[j] > x) j--;
                if(i <= j)
                {
                    t = st[i]; 
                    st[i] = st[j];
                    st[j] = t;
                    i++;
                    j--;
                }
            }
            if((j - left) < (right - i))
            {
                if(i < right)
                {
                    s++;
                    stack[s].l = i;
                    stack[s].r = right;
                }
                right = j;
            }
            else
            {
                if(left < j)
                {
                    s++;
                    stack[s].l = left;
                    stack[s].r = j;
                }
                left = i;
            }
        }
    }
}

@petyahohlov 27 / 22 / 13 Регистрация: 31.07.2013 Сообщений: 121
	22.04.2014, 13:20	2
	В цикле while( i <=j ) ты два breakа не включил в for Добавлено через 2 минуты У тебя какая то странная функция. У меня без всякой рекурсии 0

	24.04.2014, 01:15