C ++ «Нет подходящего конструктора для инициализации» ошибки компилятора

Question

У меня есть класс, который я пытаюсь инициализировать, но получаю ошибку "Нет подходящего конструктора для инициализации 'TextureCoordinates'";

Класс, который я пытаюсь инициализировать:

class TextureCoordinates
{
public:
    TextureCoordinates(){};
    TextureCoordinates(Point2D& origin, Dimensions2D& dim);
    Point2D getOrigin() const {return origin;};
    Dimensions2D getDim() const {return dim;};
private:
    Point2D origin;
    Dimensions2D dim;
};

Строка с ошибкой компилятора:

TextureCoordinates result(point, Dimensions2D(width, height));

Определение конструктора:

TextureCoordinates::TextureCoordinates(Point2D& origin, Dimensions2D& dim):
origin(origin), dim(dim) {}

Есть идеи, что я делаю неправильно?

Answer 1

Ваш конструктор принимает аргументы по неконстантной ссылке, но вы передаете ему временный объект (Dimensions2D(width, height)). Временные, даже неконстантные, не привязываются к неконстантным ссылкам.

Решение, заставить ваш конструктор принимать константные ссылки (он не должен изменять переданные объекты в любом случае):

TextureCoordinates(Point2D const& origin, Dimensions2D const& dim);

Answer 2

TextureCoordinates result(point, Dimensions2D(width, height))

Второй параметр - это rvalue, которое нельзя привязать к lvalue-ссылке, которую ожидает конструктор:

TextureCoordinates(Point2D& origin, Dimensions2D& dim);

Вы можете исправить это, изменив подпись конструктора на

TextureCoordinates(Point2D& origin, const Dimensions2D& dim);
TextureCoordinates(Point2D& origin, Dimensions2D&& dim); // alternative for c++11

(если можете) или сделать параметр переменной

Dimension2D dim=Dimensions2D(width, height);
TextureCoordinates result(point, dim)

Answer 3

Временные переменные не могут быть переданы как ссылки в C ++, потому что тогда вы можете изменить значение временного объекта в функции, которая скоро исчезнет !! Такой проблемы не существует для ссылки на константы ..... Таким образом, ваше определение функции должно быть как

TextureCoordinates(Point2D const& origin, Dimensions2D const& dim);

Answer 4

Объявить размеры2d снаружи.

Dimension2d d(width, height);
TextureCoordinates result(point, d);

Answer 5

вы написали реализацию:

TextureCoordinates(Point2D& origin, Dimensions2D& dim);