[問題] 有關於 return local variable

看板C_and_CPP作者gowrite (gowrite)時間8年前 (2017/12/16 01:05)推噓9(9推 0噓 15→)

留言24則, 7人參與討論串1/1

開發平台(Platform): (Ex: Win10, Linux, ...) GNU/Linux Ubuntu 編譯器(Ex: GCC, clang, VC++...)+目標環境(跟開發平台不同的話需列出) Vim 額外使用到的函數庫(Library Used): (Ex: OpenGL, ...) n/a 問題(Question)：最近在練習資料結構，突然想到有關於在函式中回傳區域變數的問題不知道該怎解決，所以想問一下實際上遇到這種問題的最好的解法餵入的資料(Input)：程式碼如下預期的正確結果(Expected Output)：輸出沒問題錯誤結果(Wrong Output)：沒錯誤程式碼(Code)：(請善用置底文網頁, 記得排版) /*List 結構類型*/ typedef struct node { struct node* next; char* id; }List; /*全域變數 head*/ List head = {NULL , "head"}; /*經由 get_last_node() 函式，可以回傳 linked list 的最後一個 node 的指標*/ List* get_last_node(void){ if( head.next == NULL ){ return &head; } List* current = head.next; while( current != NULL ){ if( current->next == NULL ){ return current; //我的問題就是這一行 Question，問題在下方補充 } current = current->next; } } void add_new_node(char* id){ List* last_node = get_last_node(); //....接著去 malloc 新空間跟新增 node 等等的動作 } void main(){ List n1 = {NULL , "n1"}; head.next = &n1; List n2 = {NULL , "n2"}; n1.next = &n2; add_new_node("another new node"); /*基本上這個程式，寫到這邊沒有發生什麼錯誤*/ /*那這邊要取得最後 node 的指標是為了更後面要實做 add_new_node() 的函式*/ } 補充說明(Supplement)：就是 current 區域變數是在 get_last_node() 函式裡面經由一開始取得 head.next 之後，在由 while loop 來得到最後一個 node 的指標可是因為回傳的是 current 區域變數，回傳的時候，理論上 get_last_node() 在記憶體的空間應該已經被釋放掉了，所以我的 main 函式可以收到正確的 current 內容，應該是運氣好記憶體內容還沒被覆蓋掉; 我看了一些 c 語言的教學，都說回傳區域變數是不好的習慣但是在這個 case，不回傳區域變數的解決方式似乎只有兩種第一種：就開一個全域變數（可是全域變數會造成命名衝突的可能）第二種：不回傳指標，所以後續的所有動作都在同一個函式裡面做完， current 取得指標之後，就在 get_last_node() 同一個函式裡面完成新增 node 的動作包括 List* new_node = malloc 新空間，賦值，等等還有把 current->next = new_node 這樣可是這樣似乎又沒辦法遵循『一個函式只做一件事情』的好習慣請問大家如此應該要怎辦呢？？還是我的觀念哪裡有錯誤，希望大家可以給我指導跟解惑，謝謝。 -- ※ 發信站: 批踢踢實業坊(ptt.cc), 來自: 123.195.54.122 ※ 文章網址: https://www.ptt.cc/bbs/C_and_CPP/M.1513357506.A.7AD.html ※ 編輯: gowrite (123.195.54.122), 12/16/2017 01:08:57 ※ 編輯: gowrite (123.195.54.122), 12/16/2017 01:09:35

→

moebear

12/16 01:23, 8年前 , 1^F

12/16 01:23, 1^F

你好，請問你的意思是說，例如： List* last_node; get_last_node( &last_node ) 是這個意思嗎？？ ※ 編輯: gowrite (123.195.54.122), 12/16/2017 01:48:12

推

LPH66

12/16 01:55, 8年前 , 2^F

12/16 01:55, 2^F

→

LPH66

12/16 01:55, 8年前 , 3^F