C++彩票程序输出顺序异常问题及代码简化咨询

嘿，作为刚入门C++的新手，能写出完整的彩票游戏已经很棒啦！咱们先解决你遇到的输出顺序问题，再一步步简化你的代码，顺便修复一些容易踩的小坑～

一、先搞定"号码先于提示语输出"的问题 #

你看这段代码：

if (ok==1) {
    for (int i=0; i<6; i++) {
        if (i==1) {
            cout<<"\nYour numbers are: "<<endl;
        }
        if (myLotto[i]<=49 && myLotto[i]>=1) {
            cout<<myLotto[i]<<" ";
        }
    }
}

问题出在提示语的输出时机：你是在循环到第2个元素（i=1，数组从0开始计数）的时候才打印提示，但第1个元素（i=0）已经先被输出了，自然就出现了号码跑在提示语前面的情况。

修正方法超简单：把提示语移到循环外面，先打印提示，再循环输出所有号码：

if (ok == 1) {
    cout << "\nYour numbers are: " << endl;
    for (int i = 0; i < 6; i++) {
        cout << myLotto[i] << " ";
    }
    cout << endl;
}

这样就会先显示提示语，再列出你输入的所有号码啦～

二、代码里的其他小坑（必须修复！） #

除了输出顺序，你的代码还有几个容易导致程序崩溃或逻辑错误的问题：

• 数组下标越界：C++数组是从0开始索引的，但你在很多循环里用了for (int i=1; i<=6; i++)，比如检查用户号码重复、匹配中奖号码的循环里，访问myLotto[i]当i=6时，已经超出了数组的范围（数组只有myLotto[0]到myLotto[5]），这会导致程序崩溃、数据乱掉这类未定义行为。所有循环都要改成从0开始，到6结束（不包含6）。
• 重复号码修复逻辑无效：你生成开奖号码时，遇到重复就用(rand()%49+2)/2重新赋值，但这个新值还是可能和其他号码重复，而且计算逻辑没必要这么复杂，直接重新生成1-49的随机数就行。

三、简化后的完整代码 #

我把你的代码重构了一下，用了更简洁高效的方式，同时修复了所有问题：

#include <iostream>
#include <vector>
#include <algorithm>
#include <random>
#include <unordered_set>

using namespace std;

int main() {
    // 初始化现代随机数生成器（比rand()更均匀可靠）
    random_device rd;
    mt19937 gen(rd());
    uniform_int_distribution<> dist(1, 49);

    // 生成不重复的6个开奖号码
    unordered_set<int> winningSet;
    while (winningSet.size() < 6) {
        winningSet.insert(dist(gen));
    }
    vector<int> winningNumbers(winningSet.begin(), winningSet.end());
    sort(winningNumbers.begin(), winningNumbers.end()); // 排序方便查看

    // 获取用户输入的号码
    vector<int> userNumbers(6);
    cout << "Input your Lucky Numbers (6 numbers between 1-49): " << endl;
    for (int i = 0; i < 6; i++) {
        cin >> userNumbers[i];
    }

    // 验证用户号码合法性
    bool isValid = true;
    unordered_set<int> userSet(userNumbers.begin(), userNumbers.end());
    if (userSet.size() != 6) { // 存在重复号码
        isValid = false;
    }
    for (int num : userNumbers) {
        if (num < 1 || num > 49) { // 号码超出范围
            isValid = false;
            break;
        }
    }

    if (!isValid) {
        cout << "\nInappropriate input. You lost this round." << endl;
        return 0;
    }

    // 展示用户号码和开奖号码
    cout << "\nYour numbers are: ";
    for (int num : userNumbers) {
        cout << num << " ";
    }
    cout << "\nWinning numbers this time: ";
    for (int num : winningNumbers) {
        cout << num << " ";
    }
    cout << endl;

    // 计算命中的号码数量
    int hitCount = 0;
    for (int num : userNumbers) {
        if (winningSet.count(num)) {
            hitCount++;
        }
    }

    // 输出结果
    cout << "\nYou hit " << hitCount << " numbers." << endl;
    cout << "You have " << (hitCount * 100) / 6 << "% chances to win." << endl;

    // 奖金计算逻辑（优化为switch更清晰）
    int money = 100;
    switch(hitCount) {
        case 0: money -= 10; break;
        case 1:
        case 2: money += 20; break;
        case 3: money += 50; break;
        case 4: money += 100; break;
        case 5: money += 10000; break;
        case 6: money += 100000; break;
    }
    cout << "Your Money: " << money << endl;

    if (hitCount == 6) {
        cout << "YOU GOT THE JACKPOT!!!" << endl;
    }

    return 0;
}

四、简化的关键点说明 #

1. 用现代随机数库替代rand()：random_device+mt19937生成的随机数更均匀、更可靠，避免了rand()的固有缺陷。
2. 用unordered_set快速去重：不管是生成开奖号码还是检查用户输入的重复，unordered_set可以自动处理重复，不需要嵌套循环手动检查，代码更简洁高效。
3. 用vector替代静态数组：vector是C++的动态数组，比静态数组更灵活，还能直接用范围for循环遍历，代码可读性拉满。
4. 用switch替代多个if：奖金计算部分用switch比一堆if更清晰，逻辑更集中。
5. 提前终止无效输入：一旦发现用户输入不合法，直接输出提示并结束程序，避免后续不必要的计算。

这样修改后，代码不仅解决了你原来的输出问题，还更健壮、更易读，也符合现代C++的写法～

内容的提问来源于stack exchange，提问作者NoobONez