VPF::Ошибка в методе - Форум программистов

Rickert

Дата 17.3.2008, 16:58 (ссылка)

(нет голосов)

Загрузка ...

Ситхи не пройдут!

Профиль
Группа: Комодератор
Сообщений: 3356
Регистрация: 11.7.2006
Где: Лакрима

Репутация: 0
Всего: 52

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

Код

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].
Почему ошибки?

Это сообщение отредактировал(а) Rickert - 17.3.2008, 17:00

--------------------

Ни что не внушает сна крепче, чем день приисполненный трудов!

Daevaorn

Дата 17.3.2008, 17:01 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 2155
Регистрация: 29.11.2004
Где: Москва

Репутация: 51
Всего: 70

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

Graf_h

Дата 17.3.2008, 18:03 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 482
Регистрация: 18.10.2007
Где: г. Ровно, Украина

Репутация: 2
Всего: 5

Код


#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 (ссылка)

(нет голосов)

Загрузка ...

Ситхи не пройдут!

Профиль
Группа: Комодератор
Сообщений: 3356
Регистрация: 11.7.2006
Где: Лакрима

Репутация: 0
Всего: 52

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

Код

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

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

--------------------

Ни что не внушает сна крепче, чем день приисполненный трудов!

bsa

Дата 17.3.2008, 18:15 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Модератор
Сообщений: 9185
Регистрация: 6.4.2006
Где: Москва, Россия

Репутация: 63
Всего: 196

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

--------------------

Правильно заданный вопрос - половина ответа
Отзывы о работе модератора.

Graf_h

Дата 17.3.2008, 18:26 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 482
Регистрация: 18.10.2007
Где: г. Ровно, Украина

Репутация: 2
Всего: 5

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

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

)

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

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

--------------------

Иногда переменная - это просто переменная

bsa

Дата 17.3.2008, 18:55 (ссылка)

(нет голосов)

Загрузка ...

Эксперт

Профиль
Группа: Модератор
Сообщений: 9185
Регистрация: 6.4.2006
Где: Москва, Россия

Репутация: 63
Всего: 196

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

--------------------

Правильно заданный вопрос - половина ответа
Отзывы о работе модератора.

Graf_h

Дата 17.3.2008, 19:00 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 482
Регистрация: 18.10.2007
Где: г. Ровно, Украина

Репутация: 2
Всего: 5

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

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

--------------------

Иногда переменная - это просто переменная

Rickert

Дата 18.3.2008, 04:11 (ссылка)

(нет голосов)

Загрузка ...

Ситхи не пройдут!

Профиль
Группа: Комодератор
Сообщений: 3356
Регистрация: 11.7.2006
Где: Лакрима

Репутация: 0
Всего: 52

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

--------------------

Ни что не внушает сна крепче, чем день приисполненный трудов!

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| C/C++: Общие вопросы \| Следующая тема »