2009-09-04 129 views
0

我会经常代表和工艺折线像这样:每次两个迭代器?

typedef std::vector<Point_t> Polyline_t; 

double PolylineLength(const Polyline_t& line) 
{ 
    double len = 0.0; 
    for(size_t i = 0; i < line.size()-1; ++i) 
     len += (line[i+1]-line[i+0]).length(); 
    return len; 
} 

最直接转化为我想出了双向迭代是:

typedef std::list<Point_t> Polyline_t; 
typedef Polyline_t::const_iterator Polyline_t_cit; 

double PolylineLength(const Polyline_t& line) 
{ 
    double len = 0.0; 
    Polyline_t_cit last = line.end(); 
    last--; 
    for(Polyline_t_cit i = line.begin(); i != last; ++i) 
    { 
     const Point_t& beg = *i; 
     const Point_T& end = *(++i); 
     len += (end - beg).length(); 
     --i; 
    } 
    return len; 
} 

有没有表达这种更简洁的方式用迭代器进行两次处理?

回答

8

我会保留两个迭代器,然后检查第二个迭代器是否已达到end。这将使得它不需要双向迭代器了:

typedef std::list<Point_t> Polyline_t; 
typedef Polyline_t::const_iterator Polyline_t_cit; 

double PolylineLength(const Polyline_t& line) 
{ 
    double len = 0.0; 
    Polyline_t_cit f = line.begin(), s(f), end = line.end(); 
    for(++s; s != end; ++f, ++s) { 
     len += (*s - *f).length(); 
    } 
    return len; 
} 
+1

我认为你需要测试如果行是空的,否则你会有问题。 – 2009-09-05 00:46:03

+0

原始代码有同样的问题。所以我认为多边形对象不能为空。但我同意你的看法,处理空行容器也是一个好主意。 – 2009-09-05 01:06:57

1

立即--i++i - 都是不必要的。

typedef std::list<Point_t> Polyline_t; 
typedef Polyline_t::const_iterator Polyline_t_cit; 

double PolylineLength(const Polyline_t& line) 
{ 
    double len = 0.0; 
    Polyline_t_cit last = line.end(); 
    last--; 
    for(Polyline_t_cit i = line.begin(); i != last;) 
    { 
     const Point_t& beg = *i; 
     const Point_T& end = *(++i); 
     len += (end - beg).length(); 
    } 
    return len; 
}