選択ソート うまくできません

Question

struct slist seiseki[7] = {{1, "Ito", {65, 55, 80}, 0},
{2, "Kato", {30, 40, 50}, 0},
{3, "Tanaka",{65, 85,100}, 0},
{4, "Suzuki",{85,90, 65}, 0},
{5, "Yamada",{20, 50, 80}, 0},
{6, "Takada",{90, 30, 40}, 0},
{7, "Sato",{50, 70, 55}, 0}};
この構造体の点数の合計点を上から高い順にソートするプログラムを作っています。

#include<string.h>

struct slist{
    int num; /* 番号 */
    char name[20]; /* 名前 */
    int ten[3]; /* 各科目の点数(3科目） */
    int total; /* 合計点 */
};

int main(void){
    /*変数宣言と配列の初期値設定*/
    int i,j,jmax,totalmax,nummax;
    int N=7,M=3,K=6; /* N:学生数, M:科目数 K:項目数*/
    char a[7];
    char b[20];
    struct slist seiseki[7] = {{1, "mike", {23, 55, 49,81}, 0},
                              {2, "tag", {56, 35, 50,92}, 0},
                              {3, "hiro",{79, 85,32,63}, 0},
                              {4, "kaito",{89,90, 19,92}, 0},
                              {5, "Ymeg",{19, 34,34,37}, 0},
                              {6, "kinto",{39, 11, 68,12}, 0},
                              {7, "eri",{29, 10, 29,27}, 0}};
    char hyodai[6][7]={"No.","Name","Eigo","Sugaku","Buturi","Total"};


    /*表題の表示*/
    printf("------------------------------------------------\n");
    printf(" Original Data\n");
    printf("------------------------------------------------\n");
    for(i=0;i<K;i++){
        printf("%8s",hyodai[i]);
    };
    printf("\n");

	for (i = 0; i < N; i++) {
		printf("%8d%8s", seiseki[i].num, seiseki[i].name);

		for (j = 0; j < M; j++) {
			printf("%8d", seiseki[i].ten[j]);
			seiseki[i].total += seiseki[i].ten[j];
		}
		printf("%8d\n", seiseki[i].total);
	}



	for (i = 0; i < N - 1; i++) {

			totalmax = seiseki[i].total;
			jmax= i;


			/* 最大値を求める*/
			for (j = i + 1; j < N; j++) {

				if (totalmax < seiseki[j].total) {
					jmax = j;
					totalmax = seiseki[j].total;
				}
				nummax = seiseki[jmax].num;
				strcpy(a,seiseki[jmax].name);
			strcpy(b,seiseki[jmax].ten);
			}

			/* 最大値(jmax番目)のデータとi番目のデータを交換*/
			seiseki[jmax].total = seiseki[i].total;
			seiseki[i].total = totalmax;
			seiseki[jmax].num = seiseki[i].num;
			seiseki[i].num = nummax;
			strcpy(seiseki[i].name,seiseki[jmax].name);
			strcpy(a,seiseki[i].name);
			strcpy(seiseki[i].ten,seiseki[jmax].ten);
			strcpy(b,seiseki[i].ten);
	}
		/*並ぎ替え後の合計点の表示 */
		printf("--------------------------\n");
		printf("並び替えデータ\n");
		printf("--------------------------\n");
		for(i=0;i<K;i++){
		        printf("%8s",hyodai[i]);
		    };
		    printf("\n");

		for (j = 0; j < N; j++) {
			printf("%5d%5s%5d%5d\n", seiseki[j].num, seiseki[j].name,seiseki[j].ten, seiseki[j].total);
		}

		return (0);
	}

としましたが実行結果に同じ名前が複数出たり、素点がうまく表示されないなどとうまくいきません。
どのように直せばいいのか教えてください。

Answer 1

tenをstrcpyでコピーしてるのが問題です。
memcpyを使いましょう。

というか、

    char a[7];
    char b[20];

の代わりに

    struct slist tmp;

を用意して

    tmp = seiseki[jmax];

でいいです。

---

追記
表示部分が間違ってますね

printf("%8d%8s%8d%8d%8d%8d\n", seiseki[j].num, seiseki[j].name,seiseki[j].ten[0],seiseki[j].ten[1],seiseki[j].ten[2], seiseki[j].total);

追記その2
aとbのサイズが逆です。

また、strcpyの引数は
strcpy(コピー先,コピー元);
です。
このため

            strcpy(seiseki[i].name,seiseki[jmax].name);
            strcpy(a,seiseki[i].name);

は
[i] ← [jmax]
a ←[i]
なってしまいます。

Comments

[taro2233]

ありがとうございます。
どうしてscrcpyではだめなのでしょうか？

[asm]

strcpyは受け取ったアドレスから'\0'までをコピーするからです

[taro2233]

素点のコピーの時のみmemcpyなのですね。
書き直していますが同じ名前が複数表示されてしまいます。。。