首先，咱们来揪出你代码里的核心问题：你用四个数的和等于26作为循环终止条件，但这个条件并不能保证四个数是不重复的！比如5+5+8+8=26、6+6+7+7=26这种重复元素的组合，总和也刚好是26，这时候循环会直接终止，但你的输出语句被嵌套在“四个数互不相同”的判断里，自然就不会有任何输出了。

另外，你当前这种“反复随机选数直到凑出不重复组合”的方式效率很低，尤其当vector元素更多时，可能会浪费大量时间在无效循环上。

更好的实现思路 #

其实要实现“随机分配且每个数值仅出现一次”，最直接高效的方法是先对vector进行随机洗牌，再直接取元素赋值——洗牌后vector里的元素顺序被随机打乱，每个元素只会出现一次，直接取前四个元素分配给变量就行，完全不需要循环判断。

修改后的代码 #

这里我给你两种写法，一种是基于你原来的风格，另一种是更现代的C++写法（推荐）：

写法1：基于原有代码的改进（使用shuffle）

#include <iostream>
#include <vector>
#include <algorithm> // 用于shuffle
#include <ctime>
using namespace std;

int main() {
    srand(time(NULL));
    vector<int> number = {5, 6, 7, 8};
    
    // 随机打乱vector中的元素顺序
    random_shuffle(number.begin(), number.end());
    
    // 直接赋值，保证每个元素唯一
    int number1 = number[0];
    int number2 = number[1];
    int number3 = number[2];
    int number4 = number[3];
    
    cout << number1 << number2 << number3 << number4 << endl;
    return 0;
}

写法2：现代C++风格（使用<random>库，替代rand()）

#include <iostream>
#include <vector>
#include <algorithm>
#include <random>
using namespace std;

int main() {
    vector<int> number = {5, 6, 7, 8};
    
    // 使用现代随机数生成器，比rand()更可靠
    random_device rd;
    mt19937 g(rd());
    
    // 洗牌
    shuffle(number.begin(), number.end(), g);
    
    int number1 = number[0], number2 = number[1], number3 = number[2], number4 = number[3];
    cout << number1 << number2 << number3 << number4 << endl;
    return 0;
}

为什么这个方案更好？ #

1. 可靠性：洗牌操作直接保证了每个元素只被分配一次，不会出现重复或遗漏的情况。
2. 效率：不需要反复循环尝试，一次洗牌就能完成任务，时间复杂度是O(n)（n为vector大小）。
3. 可读性：逻辑清晰，一眼就能看出是在做随机分配唯一元素的操作。

补充说明 #

你原来代码里用rand()和srand(time(NULL))其实存在一些问题：比如rand()的随机质量不高，而且time(NULL)的精度是秒，如果程序在同一秒内多次运行，会得到相同的随机序列。现代C++推荐使用<random>库中的生成器（比如mt19937），随机质量更高，也更灵活。

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