Vectorクラスの配列へ文字列を代入する方法が知りたいです。

Question

前提・実現したいこと

昨年末からネットや書籍でC++の勉強をしており、現在は動的配列の処理で困っています。
プログラミングビギナーです。
ネットや本で3日粘ってここまで来ましたが、動的配列に対して文字型や文字列の処理を扱っている例がなく、心が折れそうです。

目的としては名簿を作成したいです。
名簿への登録人数をコンストラクタで指定し、クラス関数で名前の入力、出力を実装したいです。

###発生している問題・エラーメッセージ
30行目
error: no type named ‘type’ in ‘struct std::enable_if<false, std::basic_ostream<char>&>’

51行目
error: no match for ‘operator=’ (operand types are ‘__gnu_cxx::__alloc_traits<std::allocator<std::vector<char> > >::value_type {aka std::vector<char>}’ and ‘const char [7]’)

該当のソースコード

1 #include<iostream>
2 #include<stdio.h>
3 #include<stdlib.h>
4 #include<vector>
5
6 using namespace std;
7
8 class StudentList
9 {
10 private:
11 int a;
12 public:
13 int num;
14 int b;
15 vector<vector<char> >student;
16 StudentList();
17 int input_name(int b);
18 int show_name(int c);
19 };
20
21 StudentList::StudentList()
22 {
23 cout << "登録する人数を指定してください" << endl;
24 cin >> num;
25 }
26
27 int StudentList::input_name(int b)
28 {
29 cout << b << "番目に登録する名前を入力して下さい" << endl;
30 cin >> student[b];
31 }
32
33 int StudentList::show_name(int c)
34 {
35 if(c < num)
36 {
37 cout << c << "番目の名前は" << student[c] << endl;
38 }
39 else
40 {
41 cout << "欠番" << endl;
42 }
43 }
44
45 int main(void)
46 {
47 StudentList list;
48 list.student.resize(10);
49 list.student[0].resize(15);
50
51 list.student[0] = "名前";
52
53 for(int i = 1;i<=list.num;i++)
54 {
55 list.student[i].resize(15);
56 list.input_name(i);
57 }
58
59 while(1)
60 {
61 cout << "番号を入力して下さい" << endl;
62 cout << list.b;
63 if(list.b > list.num || list.b < 1)
64 {
65 cout << "番号は1以上かつ" << list.num << "以下" <<endl;
66 break;
67 }
68 else
69 {
70 list.show_name(list.num);
71 }
72 }
73 return 0;
74 }

c++

試したこと

strcpy(student[0],"名前");

補足情報（FW/ツールのバージョンなど）

VirtualBox
Ubunts(64bit)
コンパイラのバージョン,7

Comments

[cateye]

vector<vector<char> >student; ではなくstd::vector<std::string>　student; では、だめですか？・・・あと、Cのヘッダを使っているのはなぜ？・・・<string>をインクルードするように・・

Answer 1

型を std::vectorstd::string にするのが簡潔で扱いやすいかと。

Comments

[Kenny009]

その通りですね。なんでかcharにこだわっていました。
ありがとうございます。

Answer 2

すでに、多くの方から指摘されているとおり、std::string を使用した方がよろしいとは思いますが、あえてこのまま実行できるように変更してみました。
まだ、見直すべき点はありますが、とりあえず最低限の修正です。

#include<iostream>
#include<stdio.h>   
#include<stdlib.h>   
#include<vector>   

using namespace std;

class StudentList
{
private:
	int a;
public:
	int num;
	int b;
	vector<vector<char> >student;
	StudentList();
	int input_name(int b);
	int show_name(int c);
};

StudentList::StudentList()
{
	cout << "登録する人数を指定してください" << endl;
	cin >> num;
}

int StudentList::input_name(int b)
{
	cout << b << "番目に登録する名前を入力して下さい" << endl;
	//cin >> student[b];
	cin >> &student[b][0];				//修正

	return 0;							//???
}

int StudentList::show_name(int c)
{
	if (c < num)
	{
		//cout << c << "番目の名前は" << student[c] << endl;
		cout << c << "番目の名前は" << &student[c][0] << endl;
	} 
	else
	{
		cout << "欠番" << endl;
	}

	return 0;							//???
}

int main(void)
{
	StudentList list;
	list.student.resize(10);
	list.student[0].resize(15);

	//list.student[0] = "名前";

	for (int i = 1; i <= list.num; i++)
	{
		list.student[i].resize(15);
		list.input_name(i);
	}

	while (1)
	{
		cout << "番号を入力して下さい" << endl;
		//cout << list.b;
		cin >> list.b;					//修正
		if (list.b > list.num || list.b < 1)
		{
			cout << "番号は1以上かつ" << list.num << "以下" << endl;
			break;
		}
		else
		{
			//list.show_name(list.num);
			list.show_name(list.b);		//修正
		}
	}
	return 0;
}

Comments

[Kenny009]

皆様のご指摘通りstringに直しました。
しかし、charでも出来るんですね。三日かけても自分は出来ませんでした。
charの配列の場合、直接アドレスを読み書きすれば良かったんですね。
考え方が勉強になりました。
ありがとうございます。

Answer 3

ちょっと ここ(teratail)の使い方が違いますよ。

他の回答者や質問者の書き方を参考にしてください。

---

まず、気になったのは、必ず vector<char>のvectorでないといけませんか?

単に文字列を使うなら std::string が(まだ)いいです。

char* とするとアドレスが確保されるようです。コツがいるので、

std::vector<std::string> strs;

みたいにやったほうがわかりやすいですし、使いやすいです。

また、StudentListクラスにあるnum, b, student はなるべくprivateに。

参考: オブジェクト指向設計　getter, setterを使うなとはどういうことか

また、もしかして日本語を入力していませんか? 特に student を入力させるところ。

日本語は基本的に全角文字です。

半角英数字なら 配列だと 一要素分 で済む場合でも、全角だと基本的に 二要素は必要です。( バイト数によって変わるのかな? ここはわからんが。 )

なので、単純に考えると、

"あいうえお" でも日本語的に考えると5文字。でも半角英数字と比較した場合は、単純計算で 5×2=10 となり、10文字扱いになります。(厳密にはバイト数だけど。)

ケータイサイズのケース( ケータイでもいいし、たばこでもいい。 ) に ノートパソコンを入れようとしているのと同じこと。
「え? 同じ人工物でしょ?」みたいな。

---

[追記1]

ちなみに日本語みたいな全角で一文字とするような文字で構成するなら std::stringよりは std::wstring がいいみたいです。

定義は忘れましたが、std::stringはcharの動的確保系ですが、std::wstring は wchar_t の動的確保系です。

日本向けに発送して、かつ、個人開発なら std::wstringのほうが無難...かな?

ちょっとコツが要りますが、std::wstringだと文字化けしにくいです。( ただしOSの設定とかによるかもしれないが。 )

std::stringだと、"あ" と構成するだけでも、1バイト目と2バイト目をチェックしてやっとわかる。

だから 1バイト目にそれだけで完結するデータが来たらそれで終了してしまう可能性がある。

文字コードはよくわからないので、あくまでイメージですが、

"さすが" と "d*pqが" が同じ...

みたいな感じで文字化けする可能性が高いです。

std::wstring だと wchar_t なので "あ" で一つ。と見なせるらしいので、日本語を扱うならこっちのほうが楽かな?

また、私はあまりわかりませんが、std::stringはバグを持っているっぽいです。
(どっかの記事で見かけた...どこだったっけ?)

Comments

[Kenny009]

>ちょっと ここ(teratail)の使い方が違いますよ。
ご指摘通り他の回答者や質問者の書き方を参考にさせていただきます。

>必ず vector<char>のvectorでないといけませんか?
そんなことないです。
文字列＝char型の配列使わなきゃ。。。という思考になってました。

>また、もしかして日本語を入力していませんか? 特に student を入力させるところ。
はい。日本語にも対応させたいと思っています。なので、ご指摘通りメモリサイズに気を付けます。

解答ありがとうございます。