双方向循環リストでのバブルソート

Question

前提・実現したいこと

双方向循環リストでのバブルソートを行い、表示したい。

発生している問題・エラーメッセージ

ソート部分のプログラムが正しいかよくわからず、またソート後の表示ができない。
おそらくソートを行う関数で不具合が起こっており、無限ループになる。

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

typedef struct node{
        struct node *prev;
        struct node *next;
        int value;
}NODE;

#define DUMMY -1
#define MAX 99
#define NUM 10

NODE* make_list(int num);
NODE* make_cell(int val);
void insert_cell_to_list(NODE *x, NODE *p);
NODE* delete_cell_from_list(NODE *x);
void print_list(NODE *link_list_head);
void free_list(NODE *link_list_head);
void bubble_sort_cell(NODE *link_list_head);

int main(void){
        NODE *list_head;

        srand((unsigned int)time(NULL));


        list_head=make_list(NUM);
        printf("ランダムで作成したリストを表示\n");
        print_list(list_head);
        bubble_sort_cell(list_head);
        printf("バブルソート後\n");
        print_list(list_head);
        printf("\n");

        free_list(list_head);

        return 0;
}

NODE* make_list(int num){
        NODE *list, *cel, *list_head;
        int i;

        list=make_cell(DUMMY);
        list->next=list;
        list->prev=list;
        list_head=list;

        for(i=0; i<num; i++){
                cel=make_cell(rand()%MAX);
                list=list->next;
                insert_cell_to_list(cel, list);
        }
        return (list_head);
}

NODE* make_cell(int val){
        NODE *p;

        p=(NODE*)malloc(sizeof(NODE));

        if (p == NULL) {
                fprintf(stderr, "Memory allocation error!\n");
                exit(-1);
        }
        p->value =val;
        p->next =NULL;
        p->prev =NULL;
        return(p);
}

void insert_cell_to_list(NODE *x, NODE *p){

  x->prev = p;
  x->next = p->next;
  p->next = x;
  p->next->prev = x;
}

NODE* delete_cell_from_list(NODE *x){

  x->prev->next = x->next;
  x->next->prev = x->prev;

  return (x);
}

void print_list(NODE *link_list_head){
        NODE *p;

        for (p = link_list_head->next; p != link_list_head; p = p->next) {
          printf("%d\n", p->value);
        }
}

void free_list(NODE *link_list_head){
        NODE *p, *q;

        p=link_list_head;
        p->prev->next = NULL;
        while (p!=NULL){
                q=p->next;
                free(p);
                p=q;
        }
}


void bubble_sort_cell(NODE *link_list_head){
        NODE *p, *q, *tmp;

        for (p = link_list_head->next; p != link_list_head->prev; p = p->next) {
          for (q = link_list_head->prev; q != p; q = q->prev) {
            if (q->value < q->prev->value) {
              tmp = delete_cell_from_list(q->prev);
              insert_cell_to_list(tmp, q);
            }
          }
        }
}

Answer 1

考えなおしたら解決しました。
回答ありがとうございました。

Comments

[kazuma-s]

どのようにして解決したか、書いてもらえませんか？
bubble_sort_cell の中の delete_celll_from_list と insert_cell_to_list でリストの入れ替えが
あるので、2つの for 文の p = p->next と q = q->prev が正しく機能しないのは分かるんですが。
まさか、value だけを交換するようにしたとかではないでしょうね。

[submaru]

交換をした時にqの更新をしました。あとはfor文の範囲の見直しです。

Answer 2

C言語のコードを書くなら、デバッグできる環境を整えましょう。
Eclipseや、WindowsならVisualStudioなど。
コードの任意の場所で実行を止め、変数のナカミを見ることができます。そこから１行づつ実行して、コードの流れを見れるようになります
そうすれば、アテズッポでコードを書かなくて済むようになります。

Answer 3

全体は見ていませんが、明らかにまずい個所が一つ

void insert_cell_to_list(NODE *x, NODE *p){
/*中略*/
  p->next = x;　　　　　　// ここで p->next が x で上書きされるので
  p->next->prev = x;     // これは x->prev = x と自分を参照している
}

Comments

[submaru]

順番を逆にしたら大丈夫ですかね？