ポインタを使って動的配列の管理、表示がしたいです

Question

前提・実現したいこと

ど素人の初歩的な質問ですみません。いくら調べても、よくわからなかったので質問させていただきます。
ユーザーの数、名前、年齢を聞いて、名前の文字数を表示するプログラムを作りたいのですが、思ったように表示されません。
正直言って、まだポインタを完璧に理解できていないので、どう直せばいいのかわかりません…

理想の結果は：
The name Tanaka has length 6
The name Satou has length 5

発生している問題・エラーメッセージ

とくにエラーメッセージなどは、表示されませんが、データの保存がうまくいきません。
一人目の名前はちゃんと記録されるのですが、二人目以降が全部最初の人の名前になってしまいます。

該当のソースコード

#include <iostream>
#include <string> 
using namespace std;

// Person class
class Person
{
private:
	string name;
	int  age;
public:
	Person() {
		name = "";
		age = 0;
	}
	
	Person(string name1, int age1)
	{
		name = name1;
		age = age1;
	}

	string setName(string name1) {
		name = name1;
		return name;
	}

	int setAge(int age1) {
		age = age1;
		return age;
	}

	int getAge() { return age; }
	string getName() { return name; }
};


// Prototypes
int lengthOfName(Person* p);

int main()
{
	const int LENGTH = 30;
	int i, ages;
	char* names = nullptr;
	names = new char[LENGTH];

	Person* pPerson = nullptr; 

	cout << "Please enter number of persons: ";
	cin >> i;

	pPerson = new Person[i];


	for (int index = 0; index < i; index++) {

		cout << "Please enter your name: ";
		cin.get();
		cin.getline(names, LENGTH);


		cout << "Please enter your age: ";
		cin >> ages;

		pPerson[index].setName(names);
		pPerson[index].setAge(ages);


	}
	
	for (int index = 0; index < i; index++) {
		cout << "The name " << pPerson->getName()
			<< " has length " << lengthOfName(pPerson) << endl;

		
	}
	
	delete[]pPerson;

	return 0;
}

//ここはいじらなくて大丈夫です
int lengthOfName(Person* p)
{
	string name = p->getName();
	return name.length();
}

/*実行結果
The name ??? has length 3
The name ??? has length 3
The name ??? has length 3
*/

試したこと

いろんなサイトを調べて、なんとかここまでこぎつけましたが、これ以上はどう直せばいいのかよくわかりません。

追記

いわれた通り、インデックスを加えたらちゃんと名前が表示されるようになりましたが、もう一つの目的である文字数の表示がうまくいきません。

for (int index = 0; index < i; index++) {
		cout << "The name " << pPerson[index].getName()
			<< " has length " << lengthOfName(pPerson) << endl;

現在の実行結果は
The name Tanaka has length 6
The name Satou has length 6
The name Ito has length 6
といった感じです。

これをどうにかして上記の理想の実行結果にしたいです。

lengthOfName(pPerson)

おそらくこの部分がうまく参照できず、一番最初の名前を引っ張ってきているみたいです。
どうすれば解決するのでしょうか？？

Answer 1

値を設定する方のループでは

    for (int index = 0; index < i; index++) {
    //略
        pPerson[index].setName(names);
        pPerson[index].setAge(ages);
    }

なのに、表示では

    for (int index = 0; index < i; index++) {
        cout << "The name " << pPerson->getName()
            << " has length " << lengthOfName(pPerson) << endl;
    }

表示のループ内の処理にindexが含まれていないのですから、ループの意味がないんじゃないか、ぐらいは気がついて欲しいと思います。

---

以下質問追記分について

lengthOfName(pPerson)

おそらくこの部分がうまく参照できず、一番最初の名前を引っ張ってきているみたいです。

ズバリその通り。名前の表示の方も根っこは同じ話なんですけど、なんでそっちは出来てこっちは出来なかったのかしら。

C/C++では、配列はその先頭要素へのポインタで代表されます。配列が単独で記述されれば先頭要素へのポインタと解釈されますし、pがポインタ、iが整数としたとき *(p+i)とp[i]は等価、同じ意味を持つという決まりになっています。
この決まりに沿って読み解けば、lengthOfName()関数に与えられたpPersonは、配列としては最初の要素(つまり一人目)へのポインタです。毎回一人目のデータへのポインタを与えているのですから、lengthOfName()関数は毎回一人目の名前の文字数を返してきます。

やりたい仕事から言えば、lengthOfName()関数には配列の欲しいインデックスの要素へのポインタを与えなければいけません。なので、そのようにするなら

lengthOfName( &pPerson[index] )

または

lengthOfName( pPerson+index )

でしょう。
(直したら動いた、だけじゃなくて意味をよく噛み締めてくださいね。この辺、よくわからないまま*を付けたり&を付けたりして動いたらヨシとする、という初心者をよく見かけます。)

Comments

[do_Shiro_to]

あぁ…そういわれればそうですね！！
難しいことばかり気にしていて、簡単なことを完全に見落としていました！！
灯台下暗しですね…

[do_Shiro_to]

表示部分も直してみたら、ちゃんと名前表示はできるようになったのですが、名前の文字数がまだ正しく表示されないみたいです…
ほかに直すべき部分がありましたら教えてくださると、助かります。

[thkana]

質問では「どうしたいのか」「なにをやったのか」「どういう結果が得られたのか」がきちんと書いてあって素晴らしかったのですが...

出来たかどうかだけが問題なら「出来てません」でいいですけど、トラブルを解決しようという場面では「直してみたら、正しく表示されない」ではなく、「このように変更したら、こういう結果が得られた」という詳細を書いて下さい。
質問は編集出来ますから、その辺のことを追記すればよいでしょう。

想像だけでいうなら、名前の表示の方だけ直して文字数の方は直していないとかいうことじゃないかと疑いますけれど。

[do_Shiro_to]

かさねがさねすいません…
以後気を付けます。
追記を書いたので、よろしければご確認ください。

[do_Shiro_to]

いわれた通り、やったらできました！
そうですね…
わからないまま、コピペをくっつけて作っただけじゃじぶんのためにはなりませんね。
反省してしっかりポインタを勉強しなおします…

[thkana]

> コピペをくっつけて作っただけじゃ
あれ? それ、思ってたけど書かなかったよなぁ。自分で意識してたんですね。
この短いプログラムで構造体アクセスがあっちは.でこっちは->で。きっとあちこちから寄せ集めてきたんだろうなぁ、と思っていはいたのですが。

「メモリ」のイメージを持って、スジを通して考えれば、ポインタは多少ややこしくはあっても本質的に難しいものじゃないと思います。頑張ってください。

Answer 2

    for (int index = 0; index < i; index++) {
        cout << "The name " << pPerson->getName()
            << " has length " << lengthOfName(pPerson) << endl;


    }

一人目をi回表示していますよ

lengthOfName(pPerson)

おそらくこの部分がうまく参照できず、一番最初の名前を引っ張ってきているみたいです。
どうすれば解決するのでしょうか？？

pPersonとpPerson[index]の違いをきちんと理解されていないのでは？

lengthOfName(pPerson + index)

や

lengthOfName(&pPerson[index])

などのようにindex分後ろにずらしてアクセスしないといけません。

Comments

[do_Shiro_to]

なおしたらちゃんと名前表示できました！
けど、今度は名前の長さを正しくできてなかったみたいです…
ほかに直せる部分があったら、ぜひ教えてほしいです！

[SHOMI]

どのように変更し、どうなったのかを質問文に追記して下さい

[do_Shiro_to]

すみません。
いわれた通り、できるだけわかりやすく追記に書きました。