ポインタ・構造体・並び替え

質問

http://www9.plala.or.jp/sgwr-t/c/Q/ens15-3.html
↑のページの問２のプログラムを
#include <stdio.h>

typedef struct data{
int number;
int subject[4];
int sum;
}data_t;

int get_sum(data_t);

int main(void)
{
data_t st[5]={{1001,85,74,63,90},
{1002,78,65,70,62},
{1003,89,92,88,76},
{1004,32,48,66,25},
{1005,92,76,81,98},
};

data_t *p;
data_t q;
int i,j;

for(i=0;i<5;i++)
{
p=&st[i];
p->sum=get_sum(st[i]);
}

for(i=0;i<3;i++)
{
for(j=0;j<3-i;j++)
{
if((p+j)->sum<(p+j+1)->sum)
{
q=*(p+j);
*(p+j)=*(p+j+1);
*(p+j+1)=q;
}
}
}

for ( i = 0; i < 5; i++ ) {
printf( "%4d ", ( p+i )->number );
for ( j = 0; j < 4; j++ )
printf("%4d ", ( p+i )->subject[j] );
printf( " %4d\n", ( p+i )->sum );
}

return 0;
}

int get_sum(data_t data)
{
data_t *p=&data;
int i,sum=0;

for(i=0;i<4;i++)
{
sum+=p->subject[i];
}
return sum;
}
と、作ってみたのですが、最高点の1005が一番上に来るけど、それ以外が正しく表示されません。
どこを直せばいいでしょうか？
よろしくお願いします。

回答No.2

nag0720 · ベストアンサー

＞構造体のsumの部分の値だと思います。

「指している」という表現が誤解を与えてしまったのかな？
pの値が何なのかを聞いたつもりだったんですが。

Ｃ言語ではポインタは非常に重要な概念です。
作ったプログラムを見れば、ポインタを理解してるように思えますが、この返答ではあまり理解していないような。どっちなんだろう？

結論を言ってしまうと、

・・・・・・
for(i=0;i<5;i++)
{
p=&st[i];
p->sum=get_sum(st[i]);
}

p=&st[0];　　　　　　　　　←　追加
for(i=0;i<3;i++)
{
for(j=0;j<3-i;j++)
{
・・・・・・

とすればいいはずです。（実行していないので確実とは言えませんが）

回答No.1

nag0720

for(i=0;i<5;i++)
{
p=&st[i];
p->sum=get_sum(st[i]);
}

これが実行されたあと、ポインタpはどこを指しているか分かりますか？

補足

構造体のsumの部分の値だと思います。