Ошибка в методе шаблона класса. С++

119
16 ноября 2019, 21:10
#ifndef __GEOMETRY
#define __GEOMETRY
#include <iostream>
#include <cmath>
#define PI 3.14159265

template <typename T>
struct Vec2
{
    union {
        struct { T x, y; };
        T raw[2];
    };
    Vec2() : x(0), y(0) {}
    Vec2(T _x, T _y) : x(_x), y(_y) {}
    Vec2(const Vec2<T> &vec): x(vec.x), y(vec.y){}
    operator Vec2<int>() { return Vec2<int>(x + 0.5, y + 0.5); }
    operator Vec2<float>() { return Vec2<float>(x, y); }
    Vec2<T> operator +  (const Vec2<T> &vec) { return Vec2<T>(x + vec.x, y + vec.y); }
    Vec2<T> operator -  (const Vec2<T> &vec) { return Vec2<T>(x - vec.x, y - vec.y); }
    Vec2<T> operator *  (const float value) { return Vec2<T>(x*value, y*value); }
    Vec2<T> operator /  (const T value) { return Vec2<T>(x / value, y / value); }

    double       operator *  (const Vec2<T> &vec) { return x * vec.x + y * vec.y; }
    T       operator [] (int index) { if (index == 0) return x; else return y; }

    friend std::ostream& operator << (std::ostream& ostr, const Vec2<T> &vec) {
        ostr << "Vector: " << vec.x << " " << vec.y;
        return ostr;
    }
    double length() { return sqrt(pow(x, 2) + pow(y, 2)); }
    double angle() { return acos(x / length()) * 180 / PI; }
    double angle_to(const Vec2<T> &vec) { return vec.length(); } 
  /* Выдает ошибку  из-за того что я пишу "vec.length()"
Ошибка  C2662   double Vec2<float>::length(void): невозможно преобразовать указатель "this" из "const Vec2<float>" в "Vec2<float> &"    */
};
typedef Vec2<float> Vec2f
#endif 

//main.cpp
#include <iostream>
#include "geo.h"
int main()
{
  Vec2f test(2,4);
  std::cout << test.angle_to(Vec2f(1,0));
  return 0;
}
Answer 1

Константные функции лучше делать константными:

double length() const { return sqrt(pow(x, 2) + pow(y, 2)); }
double angle() const { return acos(x / length()) * 180 / PI; }
double angle_to(const Vec2<T> &vec) const { return vec.length(); }

Обратите внимание на const. Кстати, последняя вообще не использует данные объекта - она точно не должна быть статической?..

Я указал только некоторые функции, но вы проверьте все - если они не меняют состояние объекта - объявляйте их как константные...

И все же решите, вы используете Vec2 или Vec2f...

READ ALSO
Перегрузка оператора == и шаблоны

Перегрузка оператора == и шаблоны

Почему присутствие структуры Hard Logic приводит к ошибке компиляции, а структуры SoftLogic — нет?

111
Перевести строку в число не использую стандартные функции [закрыт]

Перевести строку в число не использую стандартные функции [закрыт]

Мне нужно написать программу которая переводит строку в числоНапример: вводится строка "1234" на выход получаем число 1234, все было бы проще...

125
Область применения std::weak_ptr

Область применения std::weak_ptr

std::weak_ptr может быть создан из std::shared_ptr, но у меня такой вопросweak_ptr может вернуть информацию про умный указатель, точнее висячий он или нет

124