印刷値が関数（C++）から返さ

Question

Iは、配列へのポインタを返す関数を有し、そして私はそれを実行すると、アレイ

の各値を印刷すること5,8,9、 〜5 8 9 ^ -5728 ^ 942815029 ^ 6297320 値：959592096私はそれを実行すると、プリント

int* merge_sort(int arr[], int size) { 
    if (size <= 1) { 
     return &arr[0]; 
    } 

    int size1 = size/2; 
    int arr1[size1]; 
    for (int i = 0; i < size1; i++) { 
     arr1[i] = arr[i]; 
    } 

    int size2 = size-size1; 
    int arr2[size2]; 
    for (int i = 0; i < (size2); i++) { 
     arr2[i] = arr[i+size1]; 
    } 

    int p1 = 0; 
    int p2 = 0; 
    int sorted[size]; 

    while (p1 < size1 && p2 < size2) { 
     if (arr1[p1] < arr2[p2]) { 
      sorted[p1+p2] = arr1[p1]; 
      p1++; 
     } else { 
      sorted[p1+p2] = arr2[p2]; 
      p2++; 
     } 
    } 

    while (p1 < size1) { 
     sorted[p1+p2] = arr1[p1]; 
     p1++; 
    } 

    while (p2 < size2) { 
     sorted[p1+p2] = arr2[p2]; 
     p2++; 
    } 

    cerr << "sorted: "; 
    for (int& i : sorted) { 
     cerr << i << ","; 
    } 
    cerr << endl; 

    return &sorted[0]; 
} 

/** 
* Auto-generated code below aims at helping you parse 
* the standard input according to the problem statement. 
**/ 
int main() 
{ 
    int N; 
    int horses[N]; 

    cin >> N; cin.ignore(); 
    for (int i = 0; i < N; i++) { 
     int Pi; 
     cin >> Pi; cin.ignore(); 
     horses[i] = Pi; 
    } 

    int *sorted = merge_sort(horses, N); 

    cerr << "~ " << sorted[0] << " " << sorted[1] << " " << sorted[2] << endl; 

    //cerr << "~~ " << sorted << " " << *sorted << " " << endl; 

    cerr << "^ " << *(sorted + 0) << endl; 
    cerr << "^ " << *(sorted + 1) << endl; 
    cerr << "^ " << *(sorted + 2) << endl; 

    for (int i = 0; i < N; i++) { 
     //cerr << "i " << i << endl; 
     // cerr << "SORTED?? " << sorted[i] << endl; 
     cerr << "value: " << *(sorted + i) << " "; 
     //cerr << "*** " << *(sorted) + i << endl; 

    } 
    cerr << endl; 
} 

、それは

ソートを印刷します値：0値：-157570874

私のforループは値 "5,8,9"を印刷しないのはなぜですか？どうすれば修正できるのですか？

（より詳細なように編集された。また、私は私のマージソートが間違っている実感が、私はちょうどそれは私が^今使用できる何かを返すために取得しようとしている。^）

Answer 1

あなたが戻っていることを確認してください正しいポインタは、（例えば、ポインタがvaluesを指す必要がある場合）：

std::cerr << "\n" << sorted << " "<< &values << "\n"; 

は同じ値を返す必要があります。

003DFD64 003DFD64 

Answer 2

がOKと思われます。あなたのソート機能をチェックしてください

cpp.sh/4rrk

// Example program 
#include <iostream> 
#include <algorithm> 

const size_t N = 3; 

int* sort(int arr[]) { 
    // Sort arr and it should be equal to [5, 7, 8] 
    std::sort(arr, arr+N); 
    return arr; 
} 

int main() { 
    int values[N]; 
    values[0] = 7; 
    values[1] = 5; 
    values[2] = 8; 

    for (size_t i = 0; i < N; ++i) { 
     std::cout << *(values + i) << " "; 
    } 
    std::cout << std::endl << values[0] << " " << values[1] << " " << values[2] << std::endl; 

    int *sorted = sort(values); 

    std::cout << "It is same arrays: " << values << " " << sorted << std::endl; 

    for (size_t i = 0; i < N; ++i) { 
     std::cout << *(sorted + i) << " "; 
    } 
    std::cout << std::endl << sorted[0] << " " << sorted[1] << " " << sorted[2] << std::endl; 

    return 0; 
} 

出力：

7 5 8 
7 5 8 
It is same arrays: 0x7329bfdc1d90 0x7329bfdc1d90 
5 7 8 
5 7 8