なぜCoutはリストアドレスを変更するのですか？

Question

私はその後、私はCOUT、先頭のアドレスが完全に変更されたときに、見つけリストを生成し、

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


/** 
* Definition for singly-linked list. 
* struct ListNode { 
*  int val; 
*  ListNode *next; 
*  ListNode(int x) : val(x), next(NULL) {} 
* }; 
*/ 
struct ListNode{ 
    int val; 
    ListNode *next; 
    ListNode(int x) :val(x), next(NULL){ 
    } 
}; 
class Solution { 
public: 
    ListNode* mergeTwoLists(ListNode* l1, ListNode* l2) { 

    } 
}; 

ListNode* geneate_list(vector<int> data){ 
    ListNode *p1, *p2; 
    if (data.size() <= 0){ 
     p1 = (ListNode*)malloc(sizeof(ListNode)); 
     return p1; 
    } 
    /*ListNode *p1, *p2;*/ 
    ListNode head(data[0]); 
    p2 = &head; 
    for (int i = 1; i < data.size(); i++){ 
     p1 = (ListNode*)malloc(sizeof(ListNode)); 
     p1->val = data[i]; 
     p1->next = NULL; 
     p2->next = p1; 
     p2 = p1; 
    } 
    return &head; 
} 

int main(){ 
    vector<int> data = {1,2,3,4,5,6}; 
    ListNode* head = geneate_list(data); 
    ListNode * g = head; 

    while (head!=NULL){ 
     cout << head->val; 
     head = head->next; 
    } 

    return 0; 
} 

でのvalを印刷したいです。

だから、

後

前 、誰もがこれを説明することができますTHX

Answer 1

generate_list機能はいくつかの理由で間違っている？

1. 定義されていない振る舞いを生じるローカル変数のアドレスを返します。 Google は、詳細についてはローカル変数のアドレスを返します。
2. 空のベクトルをgenerate_listに渡すと、新しく生成されたリストの先頭の内容は初期化されません。しかし、とにかくリストが空であれば、とにかく頭を持つことはできません。 valには何が含まれますか？したがって、空のベクトルにはgenerate_listがNULLを返す必要があります。
3. コンストラクタでオブジェクトを割り当てる場合は、mallocを使用しています。したがって、ListNodeのコンストラクタは呼び出されません（このコンストラクタはここではあまり役に立ちませんが）。

あなたはこのような何かしたい：

ListNode* geneate_list(vector<int> data) { 
    ListNode *head = NULL; 
    ListNode *previous = NULL; 

    for (int i = 0; i < data.size(); i++) { 
    ListNode *p = new ListNode(0); 

    if (i == 0) 
     head = p; 

    p->val = data[i]; 
    p->next = NULL; 

    if (previous) 
     previous->next = p; 

    previous = p; 
    } 

    return head; 
} 

は免責事項：元の関数にできるだけ近いこの多かれ少なかれC関数を。 C++では、これを別の方法で行います。