Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > Ошибка в методе

Автор: Rickert 17.3.2008, 16:58

Ничего понять не могу:

Код

aVector aAux::aaCrossMulty(const aVector& vec, const aVector& vec2)
{
  static const float* v1 = vec.avGetData();
  static const float* v2 = vec2.avGetData();

  return aVector(v1[1]*v2[2] - v1[2]*v2[1], v1[2]*v2[0] - v1[0]*v2[2], v1[0]*v2[1] - v1[1]*v2[0]);
}

Ошибки:

Цитата

error C2662: 'aVector::avGetData' : cannot convert 'this' pointer from 'const aVector' to 'aVector &'
error C2662: 'aVector::avGetData' : cannot convert 'this' pointer from 'const aVector' to 'aVector &'

Метод avGetData() описан в классе aVector:

Код

inline const float* avGetData()
{
  return &pos[0];
}

pos - float[3].
Почему ошибки?

Автор: Daevaorn 17.3.2008, 17:01

Метод avGetData должен быть константным.

Автор: Graf_h 17.3.2008, 18:03

Код


#include "stdafx.h"
#include <iostream>
int i=10;
typedef struct {
inline const int *func2(){

    return &i;
}
}Vec;

Vec z;
int zz(Vec &x1,Vec &x2){
    static const int* v1= x1.func2();
    static const int* v2= x2.func2();
     
    return  (v1[0])+(v2[0]);
}

int main(int argc, char* argv[])
{
    std::cout<<zz(z,z)<<std::endl;
    return 0;
}

Вот попытался смоделировать то что утебя! Все работает! я убрал const от Vec &x1,Vec &x2 так как их функция возвращает const значение! думаю ошибка в этом! я так понял компиятор не может привести к типу & Vec! на мой взгляд это самый правильный выход из даной ситуации

Автор: Rickert 17.3.2008, 18:08

Graf_h, нет, const терять нельзя.
Daevaorn, а в чём смысл этого самого const?
я поставил сразу после объявления функции:

Код

inline const float* avGetData() const
{
  return &pos[0];
}

А заработало.

Автор: bsa 17.3.2008, 18:15

Rickert, этот конст означает, что твой метод не меняет поля класса (другими словами, this указывает на константный объект).

Автор: Graf_h 17.3.2008, 18:26

и у меня заработало smile

Цитата(Rickert @ 17.3.2008, 18:08

)

Graf_h, нет, const терять нельзя.

На мой взгляд тело твоей функции невкоем случае не поменяет значения vec и vec2! Мне кажется здесь можно обойтись и без const! Хотя если я ошибаюсь, тогда скажи вчем! Сам хочу разобратся...

Автор: bsa 17.3.2008, 18:55

Graf_h, const нужен для того, чтобы в будущем не допустить ошибок. А потом, это еще один уровень контроля. Если ты вместо одного оператора в теле функции использовал другой, который меняет данные класса, то компилятор тебя отругает и укажет место ошибки. И ты ее быстро исправишь.
В принципе, можно вообще писать классы, размещая все поля в публичной зоне, не используя const и пр. И, возможно, программа будет работать. Вот только через год, когда в эту программу придется что-то добавить, возникнет куча проблем - вставил одно, сломалось другое, починил, испортилось третье, исправил перестало работать то, ради чего стал переделывать. А почему? А потому, что все забылось, а код позволяет вносить в себя любые изменения, не контроллирая их.

Автор: Graf_h 17.3.2008, 19:00

Спасибо! все понял! smile

Думаю Rickert тему можно пометить решеной! smile

Автор: Rickert 18.3.2008, 04:11

Всем спасибо!