2

介绍：

在我的程序中，一个group对象有一个std::vector完整的多边形（一个环和孔），一个polygon对象有一个std::vector点。实际上，这些类更复杂，我在这里将它们剥离以说明我的问题。

问题：

我希望能够point从其对应的组对象中修改 'sx 和 y 坐标。下面，在 中group.cpp，我有一个名为的虚拟函数void Group::tryChangingAllX()试图完成此任务。但是，之后调用show()该组显示其多边形点的坐标没有变化。

我想我需要使用引用/指针，但我需要朝着正确的方向轻推。

点.cpp：

#include "point.h"
#include <iostream>
Point::~Point(){}
Point::Point(int x, int y){
    _x = x;
    _y = y;
}
void Point::show(){std::cout << "(" << x() << "," << y() << ")";}
void Point::x(int x){_x = x;}
void Point::y(int y){_y = y;}
int Point::x(){return _x;}
int Point::y(){return _y;}

点.h：

#ifndef POINT_GUARD
#define POINT_GUARD
class Point{
    int _x;
    int _y;
    public:
        Point(int x, int y);
        ~Point();
        void show();
        int x();
        int y();
        void x(int x);
        void y(int y);  
};
#endif

多边形.cpp：

#include "polygon.h"
#include "point.h"
#include <iostream>
#include <vector>

Polygon::~Polygon(){}
Polygon::Polygon(){}
std::vector<Point> Polygon::points(){return _points;}
Polygon::Polygon(std::vector<Point> points){_points = points;}
void Polygon::show(){
    std::cout << "Points: ";
    for(std::vector<Point>::size_type i = 0; i != _points.size(); i++) {
        _points[i].show();
    }
}

多边形.h：

#ifndef POLYGON_GUARD
#define POLYGON_GUARD

#include <vector>
#include "point.h"

class Polygon{
    //private:
    std::vector<Point> _points;
    public:
        ~Polygon();
        Polygon ();
        Polygon(std::vector<Point> points);
        std::vector<Point> points();
        void show();
};
#endif

组.cpp：

#include <iostream>
#include <vector>
#include "group.h"
#include "polygon.h"
Group::~Group(){}
Group::Group(std::vector<Polygon> polygons){
    _ring = polygons.front();
    polygons.erase(polygons.begin());
    _holes = polygons;
}
void Group::tryChangingAllX(){
    std::vector<Point> points = _ring.points();
    for(std::vector<Point>::size_type i = 0; i != points.size(); i++) {
        points[i].x(15);
    }
}
void Group::show(){
    _ring.show();
    if(_holes.size()>0){
        for(std::vector<Polygon>::size_type i = 0; i != _holes.size(); i++) {
            _holes[i].show();
        }
    }
}

组.h：

#ifndef GROUP_GUARD
#define GROUP_GUARD

#include <vector>
#include "polygon.h"

class Group{
    Polygon _ring;
    std::vector<Polygon> _holes;
    public:
        ~Group();
        Group(std::vector<Polygon> polygons);
        void show();
        void tryChangingAllX();

};
#endif

谢谢！

4

3 回答 3

5

功能

std::vector<Point> points();

按值返回，所以当你调用它时，你会得到一个成员的副本。您需要将其更改为

std::vector<Point>& points();

完成此操作后

std::vector<Point> points = _ring.points();

还制作返回值的副本。要引用 中的实际成员_ring，请更改为：

std::vector<Point>& points = _ring.points();

那应该这样做。

请注意，您应该通过std::vectorconst 引用传递以防止不必要的复制：

Polygon(const std::vector<Point>& points);

并考虑制作不修改类的方法const：

int x() const;
于 2012-11-26T04:20:42.553 回答
1

这正是你的问题——你得到的是你的观点的副本，而不是参考原始观点本身。

多边形.cpp：

通过引用而不是值返回点：

std::vector<Point>& Polygon::points(){return _points;} // note the '&' in the return

组.cpp：

获得对点的引用，而不是副本

std::vector<Point>& points = _ring.points(); // note the '&' in what you're getting
于 2012-11-26T04:22:42.767 回答
0

Luchian 和 lori 发布的答案在技术上是正确的。但我想指出一些设计注意事项。

返回 areference将允许任何人修改对象private的一部分。Polygon按照设计，您只希望Group类执行此操作。考虑制作Group一个friend。然后将可以访问. 这将确保整体更紧密的封装。PolygonGroupPolygon

在多边形.h

friend class Group;

在group.cpp 中

void Group::tryChangingAllX()
{
    for(std::vector<Point>::size_type i = 0; i != _ring._points.size(); i++)
    {
        _ring._points[i].x(15);
    }
}
于 2012-11-26T04:30:16.703 回答