C++ 一个字都混不一谈

C++ Can't mix up a word

本文关键字：一个 C++      更新时间：2023-10-16

我是我的源代码，我只是无法混淆我的话，这让我得到了我不想要的任何东西。

我没有错误，没有警告，但是如果我说：爸爸，它给我" p a"，为什么？谢谢您的帮助！

#include <iostream>
#include <string>
#include <random>
#include <chrono>
using namespace std;
string melangeLettre(string mot);
int main()
{
    cout << "Saisissez un mot mystere: n> ";
    string motMystere{};
    cin >> motMystere;
    cout << "Quel est ce mot ? n";
    string const newMot = melangeLettre(motMystere);
    cout << newMot << endl;
    return {0};
}
string melangeLettre(string mot)
{
    size_t random = chrono::system_clock::now().time_since_epoch().count();
    mt19937 gen{random};
    string newMot{};
    for (unsigned int i{}; i < mot.size(); ++i)
    {
        uniform_int_distribution<> getNbr(0, mot.size());
        newMot.push_back(mot[getNbr(gen)]);
        mot.erase(i, 1);
    }
    return newMot;
}

您的代码中有一些问题。您选择的发行版：

uniform_int_distribution<> getNbr(0, mot.size());

可以导致数字，包括 mot.size()，它已经超过字符串的末端。非空字符串中的最后一个字符将具有mot.size()-1的索引。

在此代码中：

newMot.push_back(mot[getNbr(gen)]);
mot.erase(i, 1);

您将字符复制到新单词，然后删除与原始单词不同的字符（它可能是相同的字符，但仅是偶然的）。您可能想删除与新词相同的字符，例如：

auto j = getNbr(gen);
newMot.push_back(mot[j]);
mot.erase(j, 1);

您的循环迭代太少了，因为您正在从每个循环中的单词中删除字符。因此，您确实需要迭代，直到您的原始单词为空为止。

这三件事将您的功能中的循环更改为：

while (mot.size() > 0)
{
    uniform_int_distribution<> getNbr(0, mot.size()-1);
    auto j = getNbr(gen);
    newMot.push_back(mot[j]);
    mot.erase(j, 1);
}
相关文章：