Предварительный порядок обхода дерева работает, но почтовый перевод не работает

Question

У меня есть две функции, которые пересекают дерево в preorder и postorder, каждая из которых вставляет значения узлов в массив и возвращает массив.

Однако моя функция postorder не работает. Я получаю ошибку сегментации при вызове функции.

Когда следующий код компилируется и запускается, но после вызова метода postorder я получаю ошибку сегментации.

Это мой код:

int* preorder_recursive(node *root, int* dataArray)
{

  if (root == NULL)
    return dataArray;

  for (int i = 0; i < 512; i++)
  {
    if (dataArray[i] == INT_MIN)
    {
      dataArray[i] = root->data;
      printf("%d is being inserted to the preorder array at pos %d\n", root->data, i);
      break;
    }
  }

  preorder_recursive(root->left, dataArray);
  preorder_recursive(root->right, dataArray);
}

int* postorder_recursive(node *root, int *dataArray)
{
  if (root == NULL)
    return dataArray;

  postorder_recursive(root->left, dataArray);

  postorder_recursive(root->right, dataArray);

  for (int i = 0; i < 512; i++)
  {
    // any "empty" spots in the array should contain INT_MIN
    if (dataArray[i] == INT_MIN)
    {
      dataArray[i] = root->data;
      printf("%d is being inserted to the postorder array at pos %d\n", root->data, i);
      break;
    }
  }
}

При звонке:

int * complete_pre_b = preorder_recursive(b, pre_order_b);
for(int i = 0; i < 3; i++)
{
    printf("pre b is %d\n", complete_pre_b[i]);
}

int * complete_post_b = postorder_recursive(b, post_order_b);
// here is the last print I see - code get till here fine
for(int i = 0; i < 3; i++)
{
    printf("post b is %d\n", complete_post_b[i]);
}

( Обратите внимание - у меня есть дерево с 3 узлами, поэтому я зацикливаюсь на i от 0 до 3)

В чем может быть проблема? Чем отличается мой пост от предзаказа?

Answer 1

Обратите внимание, что вы делаете: complete_post_b = postorder_recursive(b, post_order_b);, когда complete_post_b определяется в начале main как: int* complete_post_b;

Однако при postorder_recursive вы не не возвращаете никаких данных. поэтому при присвоении complete_post_b оно фактически аннулируется.

Ваш цикл for должен быть следующим:

for(int i = 0; i < 3; i++)
{
   printf("post b is %d\n", post_order_b[i]); // and not complete_post_b 
}

Или вы можете вернуть dataArray и использовать complete_post_b.

Для части интереса: почему это происходит только на postOrder ?

Обратите внимание, что единственный раз, когда вы возвращаете dataArray, это когда узел равен нулю. Я предполагаю, что в этом случае регистр для возвращаемого значения будет содержать массив данных. При наличии рекурсивного вызова функции в конце вашей функции адрес массива данных остается в регистре и передается вперед - но вы не можете рассчитывать на это, и вам действительно нужно вернуть адрес, если вы хотите его использовать