ヒープソートで比較回数、交換回数を表示させたいが、出来ない

Question

ヒープソートソートで比較回数比較回数と交換交換回数回数を表示表示させたいのですが、実行実行した所した所、、どちらとも0となります。どこが違うのか違うのか教えて教えてほしいほしい。

#include <stdlib.h>
#include <time.h>
 
#define MAX_ELEMENTS    10000 /* 最大のデータ数 */  
#define OFFSET          100   /* データ数の増分値 */  
#define MAX_LINES       10    /* 関数の最大行数 */  
#define MAX_HEAP        1000
  
int  a[MAX_ELEMENTS] ;        /* 探索するデータ領域 */  
  
int  comp, swap      ;        /* 比較回数、交換回数を格納する変数 */  
  
void downheap(int from,int to);  
void init_step(void);  
void print_step(int);  
void print_header(char *);  
int sorted(int);  
  
int main(void){  
  int i;  
  int n; /* データ数 */  
  
  srandom(time(0));           /* 乱数の種を初期設定する */  
  
 /* ランダム入力の場合 */  
  print_header("random");  
  for (n = OFFSET; n <= MAX_ELEMENTS; n += OFFSET) {  
    init_step();               /* comp, swap の初期化 */  
    for (i = 0; i < n; i++) {  
      a[i] = random() % n ;    /* 配列要素に乱数値を設定する */  
    }  
  
    downheap(i,n);         /* ヒープソートを実行 */  
 
    /* ここに降順のチェックをいれる */
    
    /* ここまでに降順のチェックをいれる */
 
 
    print_step(n);             /* 比較回数、交換回数の表示 */  
  }  
  
 /* 昇順にソートされた入力の場合 */  
  print_header("ascending order");  
  for (n = OFFSET; n <= MAX_ELEMENTS; n += OFFSET) {  
    init_step();               /* comp, swap の初期化 */  
    for (i = 0; i < n; i++) {  
      a[i] = i ;               /* 配列要素に昇順のデータ値を設定する */  
    }  
  
    downheap(i,n);         /* ヒープソートを実行 */  
 
    /* ここに降順のチェックをいれる */
    
    /* ここまでに降順のチェックをいれる */
 
    print_step(n);             /* 比較回数、交換回数の表示 */  
  }  
  
 /* 降順にソートされた入力の場合 */  
  print_header("descending order");  
  for (n = OFFSET; n <= MAX_ELEMENTS; n += OFFSET) {  
    init_step();               /* comp, swap の初期化 */  
    for (i = 0; i < n; i++) {  
      a[i] = n - i ;           /* 配列要素に降順のデータ値を設定する */  
    }  
  
    downheap(i,n);         /*  ヒープソートを実行 */  
 
    /* ここに降順のチェックをいれる */
    
    /* ここまでに降順のチェックをいれる */
 
    print_step(n);             /* 比較回数、交換回数の表示 */  
  }  
  
  return 0;  
}  
  
void init_step(void){  
  swap = 0; comp = 0;  
}  
  
void print_header(char *s) {  
  printf("%s\n   n, 比較回数, 交換回数, チェック",s);  
  printf("\n");  
}  
  
void print_step(int n){  
  printf("%4d, %8d, %8d", n, comp, swap);  
  
  if (sorted(n)) { /* 配列が昇順に整列されているかチェック */  
    printf(", sorted\n");  
  } else {  
    printf(", unsorted\n");  
  }  
}  
  
int sorted(int n) {  
  int i;  
  for (i=0; i < n-1; i++)  
    if (a[i] > a[i+1]) return 0;  
  return 1;  
}  
  
int n = 0;
void downheap(int from,int to)
{
    int i, j;
    int val;
    
    val = a[from];
    i = from;
    while (i <= to/2){
        j = i*2;
        comp++;
        if (j+1 <= to && a[j] > a[j+1])
        j++;
        
     
        if(val <= a[j])
        break;
        

        a[i] = a[j];
        i = j;
        
    }
    a[i] = val;
    
}
    void heapsort()
    {
        int i;
        int tmp;
        
        for (i = n/2; i >= 1; i--)
        downheap(i,n);
        
        for (i = n; i >= 2; i--){
            swap++;
            tmp = a[1];
            a[1] = a[i];
            a[i] = tmp;
            downheap(1, i-1);
            
        }
    }

Comments

[y_waiwai]

そのコードの中で、比較回数、交換回数ってのはどれが表してるんでしょうか

[0623]

comp++;とswap++；です。一番下の所のヒープソートの所に挿入してあります。

[melian]

※ 間違ったコメントのため削除

[y_waiwai]

print_step関数でどういう値が表示されるんでしょうか

[ak.n]

comp++やswap++を一回も通らないからです。

なぜ、通らないかは、ソートの関数を呼び出すところに問題があるでしょうね。

for (i = 0; i < n; i++) {
a[i] = random() % n ; /* 配列要素に乱数値を設定する */
}

downheap(i,n);

これ、i はループを抜けたとき n になっています。
なので、

downheap(n,n);

を実行していることになっていますが、なにか違うのではないでしょうか。

[0623]

どのように修正すればいいでしょうか？

[0623]

print_stepではコメントアウトの通り比較回数と交換回数が表示されます。