開發平台(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 這樣
可是這樣似乎又沒辦法遵循『一個函式只做一件事情』的好習慣
請問大家如此應該要怎辦呢??
還是我的觀念哪裡有錯誤,
希望大家可以給我指導跟解惑,
謝謝。