印刷中の無限ループ

Question

先ず、私はC言語の新機能で、Cを使用しなければならないプログラムに投げ込まれました。今では私の問題はポインタと関係していると思いますが、リストを印刷しようとすると（私が正しい順序でプログラムを保存していることを確認するために、私のプログラムが2回目のループの後でクラッシュする）

これは私の挿入機能で、必要最低限​​の時間で挿入します。ここに私の印刷がクラッシュ

void insertProcess(Process& process, Process* &head, Process* curr, Process* prev){ 
curr = head; 
if(head == NULL){ 
    head = &process; 
} 
else{ 
    prev = head; 
    while(process.timeNeeded > curr->timeNeeded){ 
     prev = curr; 
     curr = curr->next; 
    } 
    prev->next = &process; 
    process.next = curr; 
} 
} 

としてそれをテストすることと、リストを印刷する必要があり、私の単純なループである。これは、「アクセスVIOを与える前に数回実行されます読み込み場所 '

while(curr->next != NULL){ 
    printf("%s %i %i %i\n", process.processName, process.arrivalTime,  process.timeNeeded, process.priority); 
    curr = curr->next; 
} 

エラーがこれらのコードセグメントの1つにあると確信していますが、私は助けていただければと思います。

編集：最初のポスターの助けを借りて元の問題を修正しましたが、今は印刷する際に無限ループが発生します。私は順番に挿入しているので、最後の項目の後にNULLポインタを設定しないので、それが本当に確実だろう。それは正しいと思われますか？そうであれば、それを修正する方法はありますか？あなたの機能のおかげで再び

Answer 1

あなたは値でProcessを渡すと、この関数は、関数が終了したときに破壊されるProcessオブジェクトのローカルコピーで作業することを意味します。

あなたのコードはこのローカルコピーにリンクされています。したがって、関数を終了すると、リンクリストは既に破棄されたProcessオブジェクトを使用しています。

もう1つの問題は、挿入ポイントを検索するためにループするときに、挿入されたプロセスが最後のものである場合を考慮していないことです。この場合、ループではcurrがNULLになります。

Answer 2

デバッガの使い方を学ぶ必要があります。 Linuxでは、-g（コンパイラによって生成されたデバッグ情報を取得する）とgccまたはg++に渡される-Wall（すべての警告を取得する）の両方でコンパイルする必要があります。その後、実行可能ファイルにgdbデバッガを使用することができます。