ヒープソートがわかりません

以下のようなヒープソートのプログラムを作ったのですが、どうしても動きません。

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

#define NS 3000000 /* 入力の最大数 */
#define WC 64 /* 入力の１行の文字数 */

void down(int from, int to);
void heapsort(int s[NS+1][WC], int n);


void down(int from, int to){
int i, j;
int val;
int s[NS+1][WC];

i = from;
val = s[from][WC];

while(i <= to/2){
j = i*2;
if(j+1 <= to && s[j] > s[j+1])
j++;
if(val <= s[j][WC])
break;
s[i][WC] = s[j][WC];
i = j;
}

s[i][WC] = val;
}

void heapsort(int s[NS+1][WC], int n) {
int i;
int tmp;
for(i = n/2; i >= 1; i--)
down(i, n);
for(i = n; i >=2; i--){
tmp = s[1][WC];
s[1][WC] = s[i][WC];
s[i][WC] = tmp;
down(1, i-1);
}
}

main(int argc, char *argv[]) {
FILE *fp;
char s[NS][WC], buff[WC];
int i=0,n=0;

if ((argc==1) || (argc>=3)) {
printf("Input filename\n");
exit(1);
}
fp=fopen(argv[1],"r");


while(fgets(buff, sizeof(buff),fp)!=NULL){
sscanf(buff,"%s",&s[i]);
i=n;
}

printf("File reading completed.\n");

heapsort(s[NS+1][WC],n);

for (i=0; i<n; i++) {
if (strcmp(A1[i],A2[i])!=0) {
printf("Results are incorrect!\n");
exit(1);
}
}

printf("Results are correct.\n");

fclose(fp);
exit(0);
}

No.1 ベストアンサー 回答者： chie65536 回答日時： 2007/11/07 14:48

ヒープソートとか以前の問題。

>main(int argc, char *argv[]) {
で
>char s[NS][WC], buff[WC];
って定義して
>heapsort(s[NS+1][WC],n);
と呼ぶと「２次元配列sの(NS+1,WC)番目の要素と、n」を引数に渡す事になる。
つまり、これは
main(int argc, char *argv[]) {
で
char c;
って定義して
heapsort(c,n);
と呼んでいるのと同じ。

「１番目の引数にcharを渡してる」って事が理解出来て無い。

自分が
heapsort('a',n);
って書いてるのと同じだって事が理解出来ないうちは「出直しておいで」としか言えない。

しかも
>char s[NS][WC]
って定義してるなら、１次元目は「０～NS-1」、２次元目は「０～WC-1」までしか存在しない。

NSが3000000で、WCが40なら、sの添字は、s[0][0]からs[2999999][39]の範囲しか取れない。

>s[NS+1][WC]
にアクセスすると、その実体は「配列の外」になるので、最悪の場合「メモリアクセス違反で例外を起こす」事になる。

>void heapsort(int s[NS+1][WC], int n) {
しかも、呼ばれてる関数の方は、引数の定義が違うし。

まずは「配列の引数渡し」を理解してから。

それが理解できたら、その次に
>while(fgets(buff, sizeof(buff),fp)!=NULL){
>sscanf(buff,"%s",&s[i]);
>i=n;
>}
のループの中で「iとnの値がどうなってるか？」を考えよう。

ぶっちゃけ、ファイルから配列に読み込むのさえ、マトモに書けてない。

そんなレベルで「ヒープソートが動かない」って言うのは、自分の習得レベルが理解出来て無いって意味で、情けないと思う。まるで生まれたてのＬｖ１の勇者が、素手でＬｖ３００のラスボスに戦いを挑んでるみたい。