函数内部循环变量出错

代码如图所示：

希望定义一个函数用于摸牌，其中***currcards 与** surpluscards* **分别为手牌vector数组与剩余牌库vector数组**，在程序的试运行中，外循环变量i出错了，i在循环一次过后不知为何产生了变化，部分结果如下：

如图红笔标出i改变为-9，请问这是什么地方有问题？

showcards为打印函数，其代码如下

你这里的i--的意思应该是如果遍历剩余的牌没有随机取得那张牌，当前这次摸牌就不算。
但是你把这个逻辑放到了寻找剩余牌堆里有没有那张随机牌的循环里，它的意义就变了：只要在剩余牌堆了没找到那张随机的牌，就多摸一张牌。
你说怎么可能对。
你可以发现两件很巧的事：

0（初始i=0） -
10（rand=10，在找到10+1前找了10张牌，i--了10次）+
1（i++）=
-10（i=-10）

-9（初始-9） -
42（rand=43，在找到43+1前找了42个数，10已经拿走，i--42次） +
1（i++） =
-50（i=-50）

应该i--放到找牌的for循环外面，循环完了再判断要不要减一。

    if(surpluscards[j] == randindex+1) //如果surpluscards.size()比较大，并且surpluscards[j] 一直不等于 randindex+1，就会一直执行else
    {
        ...
    }
    else 
    {
        i--; //当surpluscards[j] 不等 randindex+1 的次数多了，i会一直减，直到负数出现也可能一直减,直到surpluscards[j] == randindex+1成立。
    }

问题在你第二个循环的else i--里
第二个j循环的时候
你要找剩余牌里的当前随机出来的排，这样遍历没有问题，但是找不到的时候（也就是代码中else语段），不要i--，
反正你的结果是要抽到cardcount张牌，i循环内部就不要改变i的值

建议：这种情况你可以直接指定一个54大小的

vector<vector<int>> pai；

里面是0-53
然后取随机数的时候 int r = rand()%54;
直接用下标访问 pai[r].push_back（）； 省的遍历来浪费时间
当pai[r].size()==0的时候就说明这张牌已经抽到你手里了 牌组中没有了