C++ 構造体の二次元配列に値を入れ昇順に並び替えたら値が0になる

Question

c++で構造体の二次元配列に値を入れ昇順に並び替えたら値が0になる

ポインタを使って構造体にアクセスして初期値をいれたあと、その合計値の高い順に並び変えたのですが何故か2次元配列の中の値が最初の数字以外全部0になります。

初期値を入れた後に再度値が入っているかを確認した際に値が入っていたので並び替えのときに二次元配列がおかしくなっているような気がします。
それか、ただ単に値は入っているけど表示の仕方が誤っているかです。
わからないので教えてください。

コード
header

#pragma once
#include <iostream>
using namespace std;
//メイン関数
void HowToStruct_EX02();

//初期値を入れる関数
void SetInit(struct empl* pe, int array[5][6]);
//値を表示する関数
void ShowStruct(struct empl* p, int x, int y);
//並び替え関数
void HowToStruct_Sort2(struct empl* p, int n);

コード
Cpp

//残業時間
struct  zangyo {
	int hours[5][6];
	float ave;
};

//残業社員を求める------
struct empl {
	string name;
	string section;
	zangyo times;
};


void HowToStruct_EX02() {
	
       //月の残業時間
	int hours[5][6] = {
	{23,89,50,10,30,75},
	{10,50,90,62,20,83},
	{40,55,36,44,80,15},
	{90,60,86,78,26,49},
	{58,81,76,11,66,99}
	};

	float average[5] = { 0 };

	//残業
	struct zangyo times[5] = {
		{hours[0][0],average[0]},
		{hours[1][0],average[1]},
		{hours[2][0],average[2]},
		{hours[3][0],average[3]},
		{hours[4][0],average[4]}
	};

	//社員
	struct empl zangyo[5] = {
		//名前//部署//月の残業
		{"阿部","人事",times[0]},
		{"加藤","経理",times[1]},
		{"須藤","総務",times[2]},
		{"西川","技術",times[3]},
		{"田島","営業",times[4]},
	};

	struct empl* pe;
	pe = zangyo;

	cout << "氏名|" << "部門|" << "1月|" << "2月|" << "3月|"
		<< "4月|" << "5月|" << "6月" << endl;

	SetInit(pe, hours);
	ShowStruct(pe, 6, 5);

	cout << "残業が多い順に並べると" << endl;

	HowToStruct_Sort2(pe, 5);

	ShowStruct(pe, 6, 5);
}

//構造体の初期化
void SetInit(struct empl* pe,int array[5][6]) {
	for (int i = 0; i < 5; i++)
	{
		int sum = 0;
		for (int j = 0; j < 6; j++)
		{
			pe->times.hours[i][j] = array[i][j];
			sum += pe->times.hours[i][j];
		}
		pe->times.ave = sum;
		pe++;
	}
}

//現在の構造体の中身を表示
void ShowStruct(struct empl* pe,int x,int y) {

	for (int i = 0; i < y; i++)
	{
		//残業の合計を保存するための変数
		cout << pe->name << "|";
		cout << pe->section << "|";
		for (int j = 0; j < x; j++)
		{
			cout << pe->times.hours[i][j] << " |";
		}
		cout << pe->times.ave;

		cout << endl;
		pe++;
	}
}

//残業が多い順に組み替える  -----------------------------★★★★★ここがおかしい気がします★★★★★
void HowToStruct_Sort2(struct empl* pe, int n) {

	struct empl tmp;

	for (int i = 0; i < 5; i++)
	{
		for (int j = 0; j < 5; j++)
		{
			if ((pe + i)->times.ave >(pe + j)->times.ave) {
				tmp = *(pe + i);
				*(pe + i) = *(pe + j);
				*(pe + j) = tmp;
			}
		}
	}
}

Answer 1

初期化時zangyo[i](i=0～4)に対する残業時間はzangyo[i].times.hours[i][j](j=0～5)に格納しています。
ソート後は並び順が変わるのでzangyo[i].times.hours[i]には期待する値が入っていません。

---

各個人は自分の残業時間だけ保持すればいいので二次元にする必要はありません。

//残業時間
struct  zangyo {
-    int hours[5][6];
+    int hours[6];
    float ave;
};

//構造体の初期化
void SetInit(struct empl* pe, int array[5][6]) {
    for (int i = 0; i < 5; i++)
    {
        int sum = 0;
        for (int j = 0; j < 6; j++)
        {
-            pe->times.hours[i][j] = array[i][j];
+            pe->times.hours[j] = array[i][j];
-            sum += pe->times.hours[i][j];
+            sum += pe->times.hours[j];
        }
        pe->times.ave = sum;
        pe++;
    }
}
//現在の構造体の中身を表示
void ShowStruct(struct empl* pe, int x, int y) {

    for (int i = 0; i < y; i++)
    {
        //残業の合計を保存するための変数
        cout << pe->name << "|";
        cout << pe->section << "|";
        for (int j = 0; j < x; j++)
        {
-            cout << pe->times.hours[i][j] << " |";
+            cout << pe->times.hours[j] << " |";
        }
        cout << pe->times.ave;

        cout << endl;
        pe++;
    }
}

---

このあたりも不要ですね。

-    float average[5] = { 0 };
    //残業
-    struct zangyo times[5] = {
-        {hours[0][0],average[0]},
-        {hours[1][0],average[1]},
-        {hours[2][0],average[2]},
-        {hours[3][0],average[3]},
-        {hours[4][0],average[4]}
-    };
    //社員
    struct empl zangyo[5] = {
        //名前//部署//月の残業
-        {"阿部","人事",times[0]},
+        {"阿部","人事",},
-        {"加藤","経理",times[1]},
+        {"加藤","経理",},
-        {"須藤","総務",times[2]},
+        {"須藤","総務",},
-        {"西川","技術",times[3]},
+        {"西川","技術",},
-        {"田島","営業",times[4]},
+        {"田島","営業",},
    };

Comments

[BRAK]

あ！たしかにそうですね。timesのhoursの中に配列があるから二次元配列じゃなくて全然よかったです。
ありがとうございます！

[BRAK]

あと、型名と変数名をよくよく見たら同じだったので気をつけます。

[BRAK]

あ、もう直で代入できるのか

[SHOMI]

struct zangyo times[5] = {～};
の箇所、times[0~4].hours[0]にhours[0~4][0]、hours[0~4][1]にave[0~4]が代入されてしまっています。
その後のSetInit()で正しく設定し直しているので動いてはいますが…

[BRAK]

色々こんがらがって、そもそもfloat average[5] = {0};にも何も入れてなかったですね。