void player::setblock()
{
int flag = 1;
block.push_back({ 0,0 });
srand(static_cast<unsigned int>(time(NULL)));
while (flag)
{
block.end()->x = rand() * (20) + 10;
block.end()->y = rand() * (20) + 10;//随机得到坐标
for (vector<mountain>::iterator iter = block.begin(); iter != block.end(); ++iter)//迭代器(有问题)
{
if (iter->x == block.end()->x && iter->y == block.end()->y)//如果重复
continue;
}
if (block.end()->x < 800 && block.end()->y)//防止超出棋盘
{
cout << "得到正确坐标";
flag--;
drawmountain(block.end()->x, block.end()->y);
block.push_back({ 0,0 });
}
}
}
我想要能检测产生的随机坐标是否重复且不超出最大范围
1.判断x,y是否超出棋盘明显更省时间,所以应该先判断,后判断是否重复,否则循环了半天好不容易找到个不重复的,结果超出范围了,不是白费力气?
而且你应该从随机数那里就直接限制好范围,让它不要超,而不是胡乱随机,再判断是否落在范围内
2.iter 是个迭代器,你确定不应该调用iter->next去获取下一个节点,而是++iter 吗,你定义的是个顺序表吗
if (iter->x == block.end()->x && iter->y == block.end()->y)//如果重复 end 代表一个vector的最后一个元素的下一个,肯定越界
continue;