2013-04-11 71 views
-1

我在存储Point数据类型元素[x,y]的向量上进行迭代,并且我想在每次迭代中实现的是添加第一个4点x元素和前4个点y元素....在下一次迭代期间,我希望与下一个4 x & y ......相同,等等迭代遍历一个向量时的运行时错误

第1次迭代1.x + 2.x + 3.x + 4.x 1.y + 2.y + 3.y + 4.y

第2次迭代5.x + 6.x + 7.x + 8.x 5.y + 6.y + 7.y + 8.y

等.... .................希望我的问题是清楚的

示例代码:

double Pta; 
    double Ptb; 
    Point vect; 
    float R1, R2; 
    float resultant; 
    vector<Point> vect_add; 
    for(vector<Point>::iterator iter_a = Left_Arm_xy.begin()+1; iter_a != Left_Arm_xy.end(); ++iter_a) 
    { 

     if(center.y <= 240) 
     { 
      vect.x = iter_a->x - (iter_a -1)->x; 
      vect.y = iter_a->y - (iter_a -1)->y; 

      vect_add.push_back(Point(vect.x,vect.y)); 


      for(vector<Point>::iterator iter_v = vect_add.begin(); iter_v != vect_add.end(); ++iter_v) 
      { 



        R1 = iter_v->x + (iter_v +1)->x + (iter_v +2)->x + (iter_v +3)->x; 
        R2 = iter_v->y + (iter_v +1)->y + (iter_v +2)->y + (iter_v +3)->y; 

        resultant = sqrt(pow(R1,2) + pow(R2,2)); 



        cout<<"Resultant: "" "<<resultant<<endl; 

      } 

     } 

当我彗星在哪里我想添加的所有部分x的和y的程序工作正常,就像以前一样.....因此,问题必须在那里..我猜语法不对,但我不是最好的程序员,不能这样做什么错,因为没有编译器警告....程序崩溃只是在运行时

谁能可能看到我在做什么毛病此代码 问候

调试: enter image description here

对不起,只是增加调试断点的照片,但我认为图片是值得1000个字在这种情况下:P

+0

你怀疑'vect_add.size()> 4'是什么意思? – 2013-04-11 03:52:23

+0

如果此矢量内有超过4个元素,则在此if语句中继续并执行代码 – Tomazi 2013-04-11 03:54:10

+0

您是对的。所以如果'vect_add.size()'是3,那么这个代码永远不会运行。如果'vect_add.size()'是5,它将运行5次。这与您对该计划的描述不符。 – 2013-04-11 04:00:15

回答

1

向量可以通过控制你的迭代器+ 3是否超过你矢量(vect_add)的限制做到这一点,这样你就不会超过载体界限,也为每个迭代你必须以4比增加迭代器来获得您的设计方案:

for(vector<Point>::iterator iter_v = vect_add.begin(); iter_v - vect_add.begin() + 4 < vect_add.size(); iter_v+=4) 
{ 
     R1 = iter_v->x + (iter_v +1)->x + (iter_v +2)->x + (iter_v +3)->x; 
     R2 = iter_v->y + (iter_v +1)->y + (iter_v +2)->y + (iter_v +3)->y; 

     resultant = sqrt(pow(R1,2) + pow(R2,2)); 



     cout<<"Resultant: "" "<<resultant<<endl; 

} 

这里,iter_v - vect_add.begin()给当前迭代器的索引,并在for循环中,我们检查是否存在的Elemen足够数量的你的加法操作。

+0

嘿你你的男人:)非常感谢你没有任何一点我接近这个解决方案。问候 – Tomazi 2013-04-11 05:14:08

0

其中的一个问题是在这里:

R1 = iter_v->x + (iter_v +1)->x + (iter_v +2)->x + (iter_v +3)->x; 
R2 = iter_v->y + (iter_v +1)->y + (iter_v +2)->y + (iter_v +3)->y; 

iter_v + 1iter_v+ 3可能超出范围,导致尝试访问不属于vector的内存,因此崩溃。

检查条件if(vect_add.size() >4)在这种情况下没有多大帮助。

这一块没有任何意义可言:

if(vect_add.size() > 20) 
{ 
    vect_add.clear(); //does not make sense, you remove all the elements in vector? 
} 
+0

感谢您的回复....你我虽然这将是问题这就是为什么我添加这个条件'如果(vect_add.size()> 4)'所以我怎么能重新排列这个变化,以便我可以实现什么后......? – Tomazi 2013-04-11 03:56:12

+0

@Tomazi你可以改变你的矢量迭代条件为:iter_v +3!= vect_add.end()。这种检查在这种情况下根本没有用。 – taocp 2013-04-11 03:57:42

+0

好吧我删除了检查并厌倦了你的建议bu增加我的iter_v + 3每次迭代和效果是一样的。 – Tomazi 2013-04-11 04:18:50

0

不知道这是相关的,但结算矢量无效迭代器。来自std::vector clear

所有与此容器相关的迭代器,指针和引用都是无效的。

您清除这里

if(vect_add.size() > 20) 
{ 
    vect_add.clear(); 
} 
+0

是的,你是对的,为什么我有这个如果声明的原因是因为我想尝试一些东西:P,并忘记删除这部分,而创建此帖子... – Tomazi 2013-04-11 04:17:33