2

ランダムに生成された文字列 rnd を cout で表示しようとすると、エンドラインしか出力されません。これはなぜですか、どうすれば修正できますか? ちなみに、while ステートメントは無限ループを作成します。文字列を正しく比較していませんか? g++ コンパイラを使用しています。

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

int main()
{
    string str;
    string rnd;
    int x;

    cout << "What do you want the string to be?" << endl;
    cin >> str;

    srand(1);

    //assign the initial random string
    for(int i = 0; i < str.size(); i++)
    {
        x = rand() % 26 + 97;
        rnd[i] = static_cast<char>(x);
    }

    cout << rnd << endl;

    //change individual characters of the string rnd until rnd == str
    while(str != rnd)
    {
        for(int i = 0; i < str.size(); i++)
        {
            if (rnd[i] == str[i])
            {
                continue;
            }
            else
            {
                x = rand() % 26 + 97;
                rnd[i] = static_cast<char>(x);
            }
        }

        cout << rnd << endl;
    }

    return 0;
}
4

2 に答える 2

2

のサイズを変更することはないrndため、常に 0 になります。 i > rnd.size() の場合に rnd[i] を設定 (または取得) することは未定義の動作ですが、たとえそれが「機能した」としても (たとえば、実装で短い文字列の最適化とすべての単語が短い)、str == rndそれらのサイズが異なるため、決してそうではありません。

私はお勧めします：

rnd.push_back('a' + rand() % 26);

初期構築中。ループの内側には、それまでに適切なサイズがあるため、while使用できます。rnd[i]rnd

于 2012-10-11T05:00:27.470 に答える
2

rnd.resize(str.size());の後に追加しcin >> str;ます。rndには文字が含まれていないため、文字列を と同じサイズにサイズ変更する必要がありますstr。

于 2012-10-11T04:56:56.173 に答える