複数の文字列をランダムに表示する方法

Question

こんばんは。もうすぐくる就職試験に備えて面接ソフトを作りたいのですが、ヘッダファイルにある質問を、毎回ランダムな順番で表示する方法が分かりません。どなたかいい方法をご存じないでしょうか。

###ヘッダーファイルhahahah.h

#include <iostream>
#include <string>

using namespace std;

class 質問
{
public:
  string sibo = "\n当社を志望した理由はなんですか\n";
	string san = "\n三年間頑張ったことはなんですか\n";
	string news = "\n最近のニュースで関心のあるものはありますか？\n";
	string scho = "\n校長先生の名前はなんですか\n";
	string cho = "\nあなたの長所はどんなところですか\n";
	string tan = "\nではあなたの短所はどんなところですか\n";
	string toku = "\nあなたの学校について教えてください\n";
	void chotan();
	void situmon();
}obj;

void 質問::chotan()
{
	cout << cho;
	Sleep(20000);
	cout << tan;
	Sleep(20000);
}


void 質問::situmon()
{
	cout << sibo;
	Sleep(60000);
	cout << san;
	Sleep(60000);
	cout << news;
	Sleep(60000);
	cout << scho;
	Sleep(10000);
	obj.chotan();
};

###ソースファイル

#include <iostream>
#include <string>
#include <windows.h>

#include "hahahah.h"

using namespace std;

int main()
{
	string a;
	cout << "模擬面接を行います。準備ができたら[a]を入力してください\n";
	cin >> a;

	if (a == "a")
	{
		cout << "模擬面接を開始します。\n";
		cout << "出身校とお名前を教えてください\n";

		obj.situmon();


		cout << "\n最後に何か質問はありますか\n";
		cout << "\n以上で面接は終了です。お疲れ様でした。\n";
	}

	else cout << "無効なキーです:-(";
	return 0;
}

Answer 1

質問をstd::arrayで管理して、添え字をランダム指定すればよいと思います。

#include <array>
#include <cstdlib>
#include <iostream>
#include <random>
#include <string>

namespace mol {
    std::array<std::string, 7> questions = {
        "当社を志望した理由はなんですか",
        "三年間頑張ったことはなんですか",
        "最近のニュースで関心のあるものはありますか？",
        "校長先生の名前はなんですか",
        "あなたの長所はどんなところですか",
        "ではあなたの短所はどんなところですか",
        "あなたの学校について教えてください"
    };
}

int main(void) {
    std::random_device rnd_dev;   // 後述の注意書きをご覧ください。
    while(true) {
        std::cout << mol::questions[rnd_dev() % 7] << std::endl;
    }

    system("PAUSE");
    return EXIT_SUCCESS;
}

かなり簡潔に書きました。適宜改良して使ってください。
乱数の質に(多分)課題がありますし、質問者様のコードのようにスリープはしません。

※yumetodo様が乱数の利用について大変重要な情報を寄せてくださっています。
当該のコメントも併せてごらんいただけるようお願いいたします。

---

anndonut様のご指摘を受けて
std::arrayではなくstd::vectorを用いた方が、後々の改良の際に便利です。
どうせなのでクラス化もしました。雑なので改良の余地は大いにありますが、参考になれば幸いです。

class Question {
private:
    const std::vector<std::string> questions = {
        "当社を志望した理由はなんですか",
        "三年間頑張ったことはなんですか",
        "最近のニュースで関心のあるものはありますか？",
        "校長先生の名前はなんですか",
        "あなたの長所はどんなところですか",
        "ではあなたの短所はどんなところですか",
        "あなたの学校について教えてください"
    };
    std::random_device rndDev;   // 前述の注意書きをご覧ください。

public:
    std::string getRandomQuestion(void) {
        return questions[rndDev() % questions.size()];
    }
};

---

using namespace stdは使わない方がいいですよ。
名前空間のメリットをぶち壊すことになります。

Comments

[mol]

回答ありがとうございます！
稚拙な質問で申し訳ないんですが、教えていただいたコードにSleep関数は入れられますか？

[mol]

あとusing namespace stdが、調べてみたところ、いろいろデメリットがあるみたいですね。当たり前のように使っていたので気づきませんでした。次からは名前空間にも配慮してコードを書こうと思います。

[LouiS0616]

Sleepは入れられますよ。
クラスQuestionにgetRandomQuestionメソッドを作り、呼び出し側が出力／スリープする設計が素直かと思います。

[LouiS0616]

色んな入門書に、当たり前のように書いてありますからね... using namespaceが。

[mol]

入れられるんですね、よかった～(´∀｀)

[mol]

そうですね、参考書を改めて読んでみましたが、全部ついてましたよ、using namespace...。

[LouiS0616]

悪しき慣習だと思います。

[mol]

始めたばかりだと何も分かりませんからね...。関数だけでなく、そういうのも教わりたかったです。
あと教えていただいたコードなんですが、無限ループになってしまって、止める方法が分かりません。どうすればいいですか？

[LouiS0616]

上限を決めるのが楽かと思います。

[anndonut]

個人的にはvector::push_backを使う方が好きかなと。７というマジックナンバーを直さないとならないのはあまりいいことだとは思えないので。

[LouiS0616]

おっしゃる通りだと思います。
今回は出来るだけ簡単に見えるコードを書きたかったのでarrayを用いましたが、私も日頃はvectorを使っています。
速度面でarrayの方が勝るのですが、普通にアプリケーションを組む分には無視できますものね。

[LouiS0616]

vectorを用いる場合も回答に追記しておきます。
コメントありがとうございます。

[yumetodo]

random_deviceのもうすこしまともな使い方はこちら・・・擬似乱数とdistribution使いましょ・・・(今回ならstd::uniform_int_distribution)
https://cpprefjp.github.io/reference/random

[yumetodo]

入門書に出てきた関数などは片っ端から
https://cpprefjp.github.io
で調べる癖をつけるといいかもしれませんね。現存する多くのC++本はほぼ例外なくなんかしらの問題を抱えているので鵜呑みにしないことが耐雪です

[LouiS0616]

ご指摘ありがとうございます、非常に勉強になります。
私自身乱数を用いたプログラムの作成経験があまりなく、半ば誤魔化す形で『乱数の質に(多分)課題があります』と表記しました。
もう少し検討して適切な方法を用いるか、あるいは疑似コードで表現すべきだったと反省しております。
コメントを参照する旨、回答に追記しておきます。

[yumetodo]

なおstd::array VS std::vectorの話ですが、個人的にはtemplate化してrandom_access_iteratorに対する操作に抽象化したい思いがありますが今は眠いのでコードは書かないでおきます。