как использовать массив пользовательского ввода в сортировке кучи

Я новичок, так что извините, если вопрос глупый. Я пытаюсь закодировать сортировку кучи, которая принимает входные данные от пользователя в массиве и сортирует кучи, но я не могу найти способ реализовать массив пользовательского ввода в моем коде. в конце, в основной функции, я должен использовать массив arr[]={}, но мне нужно либо определить элементы, либо количество элементов, поэтому он создает нули, если я определяю массив как arr[10]={} вот код:

// Heap Sort in C++

#include <iostream>
using namespace std;

void heapify(int arr[], int n, int i) {
// Find largest among root, left child and right child
int largest = i;
int left = 2 * i + 1;
int right = 2 * i + 2;

if (left < n && arr[left] > arr[largest])
    largest = left;

if (right < n && arr[right] > arr[largest])
    largest = right;

// Swap and continue heapifying if root is not largest
if (largest != i) {
    swap(arr[i], arr[largest]);
    heapify(arr, n, largest);
}
}

// main function to do heap sort
void heapSort(int arr[], int n) {
// Build max heap
for (int i = n / 2 - 1; i >= 0; i--)
    heapify(arr, n, i);

// Heap sort
for (int i = n - 1; i >= 0; i--) {
    swap(arr[0], arr[i]);

    // Heapify root element to get highest element at root again
    heapify(arr, i, 0);
}
}

// Print an array
void printArray(int arr[], int n) {
for (int i = 0; i < n; ++i)
    cout << arr[i] << " ";
cout << "\n";
}

// Driver code
int main() {

int arr[10] = { };
cout << "Enter the numbers : " << endl;
for (int x = 1; x <= 5; x++)
{
    cin >> arr[x];
}
int n = sizeof(arr) / sizeof(arr[0]);
heapSort(arr, n);

cout << "Sorted array is \n";
printArray(arr, n);
}

вот результат: выходное изображение кода


person Usama Tariq    schedule 22.05.2020    source источник
comment
Я не уверен, в чем твоя проблема. Если вы хотите отсортировать только первые 5 элементов массива, просто передайте x вместо n (т. е. 5). Подпрограмме все равно, лежат ли за теми, которые вы используете, еще несколько запасных элементов. Тем не менее, да, следуйте совету DXZ и используйте вектор. Кроме того, инициализация массива по умолчанию с помощью {} не нужна: используемые вами элементы все равно перезаписываются, неиспользуемые - ну, не используются...   -  person Peter - Reinstate Monica    schedule 22.05.2020


Ответы (1)


Вы можете рассмотреть std::vector и передать его по ссылке на ваш heapify(...) для экономии памяти.

person DXZ    schedule 22.05.2020