Как я могу отсортировать массив двойных указателей на основе значений, на которые они указывают? - PullRequest
2 голосов
/ 20 августа 2008

Я пытаюсь построить функцию в C / C ++ для сортировки массива и замены каждого значения его «счетом» или рангом. Он принимает массив двойных указателей для массива целых и сортирует двойные указатели на основе разыменованных значений целых чисел. Я несколько раз пытался заставить его работать, но не могу его сломать. Еще раз, он должен отсортировать двойные указатели на основе значений, на которые они указывают. Вот что у меня есть:

void SortArray( int ** pArray, int ArrayLength )
{
  int i, j, flag = 1;     // set flag to 1 to begin initial pass
  int * temp;             // holding variable orig with no *
  for(i = 1; (i <= ArrayLength) && flag; i++)
  {
    flag = 0;
    for (j = 0; j < (ArrayLength -1); j++)
    {
        if (*pArray[j+1] > *pArray[j])    // ascending order simply changes to <
        { 
            temp = &pArray[j];            // swap elements
            pArray[j] = &pArray[j+1];
            pArray[j+1] = &temp;
            flag = 1;                     // indicates that a swap occurred.
        }
    }
  }
}

Ответы [ 5 ]

5 голосов
/ 20 августа 2008

Ты рядом. Вы ссылаетесь на адрес элементов массива при замене, что не обязательно. Элементы в массиве являются указателями, и это то, что нужно поменять местами.

См. Ниже:

void SortArray( int ** pArray, int ArrayLength )
{
    int i, j, flag = 1;    // set flag to 1 to begin initial pass
    int * temp;             // holding variable orig with no *
    for(i = ArrayLength - 1; i > 0 && flag; i--)
    {
        flag = 0;
        for (j = 0; j < i; j++)
        {
            if (*pArray[j] > *pArray[j+1])      // ascending order simply changes to <
            { 
                temp = pArray[j];             // swap elements
                pArray[j] = pArray[j+1];
                pArray[j+1] = temp;
                flag = 1;               // indicates that a swap occurred.
            }
        }
    }
}

Кроме того, зацените эту прекрасную запись в блоге о Bubble Sorting на тот случай, если вам интересно (извините, бесстыдная вилка :)). Надеюсь, что это поможет вам с домашним заданием;)


Редактировать: обратите внимание на тонкую «оптимизацию», при которой вы отсчитываете от длины массива и увеличиваете только до «i» во внутреннем цикле. Это избавит вас от ненужного повторного разбора уже отсортированных элементов.

3 голосов
/ 20 августа 2008

Хех, это не домашняя работа.

Если это так, рассмотрите возможность использования STL для управления массивами и сортировки. Его легче разрабатывать и поддерживать, а алгоритм std :: sort асимптотически быстрее, чем пузырьковая сортировка.

2 голосов
/ 20 августа 2008

Вам следует рассмотреть возможность использования std::swap() для обмена. Если это так, назовите его так:

swap( obj1, obj2 );

вместо:

std::swap( obj1, obj2 );

Поскольку первый семантический вызов позволит правильному поиску пространства имен найти правильную перегрузку, если она существует Убедитесь, что у вас есть:

using namespace std;

или

using std::swap;

где-то.

1 голос
/ 20 августа 2008

Хм, у меня мало опыта с STL. Не могли бы вы привести пример?

Эта программа создает вектор целых, сортирует его и отображает результаты.

#include <vector>
#include <algorithm>
#include <iostream>
using namespace std;

int main()
{
    vector<int>; vec;
    vec.push_back(7);
    vec.push_back(5);
    vec.push_back(13);
    sort(vec.begin(), vec.end());

    for (vector<int>::size_type i = 0; i < vec.size(); ++i)
    {
        cout << vec[i] << endl;
    }
}
0 голосов
/ 22 сентября 2008

Чтобы завершить пост Брайана Энсинка, вы найдете STL, полный сюрпризов. Например, алгоритм std :: sort:

#include <iostream>
#include <vector>
#include <algorithm>

void printArray(const std::vector<int *> & p_aInt)
{
   for(std::vector<int *>::size_type i = 0, iMax = p_aInt.size(); i < iMax; ++i)
   {
      std::cout << "i[" << static_cast<int>(i) << "] = " << reinterpret_cast<unsigned     int>(p_aInt[i]) << std::endl ;
   }

   std::cout << std::endl ;
}


int main(int argc, char **argv)
{
   int a = 1 ;
   int b = 2 ;
   int c = 3 ;
   int d = 4 ;
   int e = 5 ;

   std::vector<int *> aInt ;

   // We fill the vector with variables in an unordered way
   aInt.push_back(&c) ;
   aInt.push_back(&b) ;
   aInt.push_back(&e) ;
   aInt.push_back(&d) ;
   aInt.push_back(&a) ;

   printArray(aInt) ; // We see the addresses are NOT ordered
   std::sort(aInt.begin(), aInt.end()) ; // DO THE SORTING
   printArray(aInt) ; // We see the addresses are ORDERED

   return EXIT_SUCCESS;
}

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

i[0] = 3216087168
i[1] = 3216087172
i[2] = 3216087160
i[3] = 3216087164
i[4] = 3216087176

i[0] = 3216087160
i[1] = 3216087164
i[2] = 3216087168
i[3] = 3216087172
i[4] = 3216087176

Заглянуть в заголовок STL http://www.cplusplus.com/reference/algorithm/ Вы найдете много утилит. Обратите внимание, что у вас есть другая реализация контейнеров, которая может вам лучше подойти (std :: list? Std :: map?).

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...