VPF::перегрузка операций - Форум программистов

serendip

Дата 27.1.2006, 02:22 (ссылка)

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

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 32
Регистрация: 20.12.2005

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

Здравствуйте! smile

Кто-нибудь видит что-нибудь нехорошее в этих перегрузках?

Код


DMass::DMass(int m)
{  n=m;
   d=StrToInt(Form2->Edit3->Text);
   a=new float[n];
}
 DMass& DMass::operator=(const DMass &v)
 {
   for (int i=0; i<n; i++)
     a[i]=v.a[i];
      return *this;
 }

 DMass DMass::operator~()const
 { DMass rr(n);
           for (int i=0; i<n; i++)
           rr.a[i]= a[i]-d;
         return rr;
 }
....
DMass x(n);
   ...
     x=~x;

Работает(вторая) совсем непонятно, вадаёт один и тот же неожидаемый рез-т
(1,35244; 1,35244; 0; 0; ...0) И при повторном запуске ошибку(куда-то чего-то не по тому адресу)

JackYF

Дата 27.1.2006, 03:49 (ссылка)

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

Загрузка ...

полуавантюрист

Профиль
Группа: Участник
Сообщений: 5814
Регистрация: 28.8.2004
Где: страна тысячи озё р

Репутация: 18
Всего: 162

что такое DMass?
полный код в студию...

во второй функции не предусмотрено, мне кажется, разных размеров - откуда ты уверен, что они совпадают - опять же таки полный код DMass.

что такое переменная d и чему она равна (или может быть равна?)

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

Пожаловаться на меня как модератора можно здесь.

threef

Дата 27.1.2006, 11:54 (ссылка)

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

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 375
Регистрация: 27.10.2005
Где: Запорожье

Репутация: 9
Всего: 10

Цитата

Код


 DMass& DMass::operator=(const DMass &v)
 {
   for (int i=0; i<n; i++)
     a[i]=v.a[i];
      return *this;
 }

Я думаю здесь:
for (int i=0; i<n; i++)

i<n ?
либо i<v.n
либо
this->resize(v.getsize());

serendip

Дата 27.1.2006, 22:04 (ссылка)

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

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 32
Регистрация: 20.12.2005

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

Итаак...

Код


class DMass
{    float* a;
        int n;
        float d;
       float step(float,float);
public:
        DMass(int = 1);
       ~DMass();
        void vvod1();
        void vvod2();
        void vivod1();
        void vivod2();
        DMass& operator=(const DMass&) ;
        DMass operator^(const DMass&) const; 
        DMass operator~()const;
};
//---------------------------------------------------------------------------
DMass::DMass(int m)
{  n=m;
   d=StrToFloat(Form2->Edit3->Text);
   a=new float[n];
}

Перегрузка та же. Есть ещё одна перегрузка, но она выдает точно такой же результат smile

Код


DMass DMass::operator^(const DMass &v)const  
{ DMass f(n);
   for (int i=0; i<n; i++)
         f.a[i]=step(a[i],v.a[i]);

         return f;
}

Размер массивов всегда совпадает(так задумывалось)
Переменная d - просто обычное число, которое нужно вычесть из каждого эл-та массива.
Причём, без перегрузок"^"и"~" пр-мма работает нормально - всё вводит, выводит и даже присваивает объекты (т.е. срабатывает перегр. "=", я надеюсь)

Цитата

Я думаю здесь:
for (int i=0; i<n; i++)

i<n ?
либо i<v.n
либо
this->resize(v.getsize());

- не помогает.
Такая загадка... smile

[code=cpp]

Это сообщение отредактировал(а) serendip - 27.1.2006, 22:29

BreakPointMAN

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

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

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 725
Регистрация: 15.9.2004
Где: Saratov

Репутация: 14
Всего: 24

Итак, для начала... smile

Твой класс имеет члены-указатели:

Код


class DMass
{    float* a;
...

а в конструкторе ты выделяешь динамически память:

Код


DMass::DMass(int m)
{  n=m;
   d=StrToFloat(Form2->Edit3->Text);
   a=new float[n];
}

Это означает, что должны быть соответствующим образом определены деструктор данного класса, а также конструктор копирования и операция присваивания. В деструкторе ты должен высвобождать выделенную конструктором память (используя delete), а копировать и присваивать объекты данного класса нужно создавая в каждом из них "свои" динамические данные..

Оператор присваивания у тебя уже определен неверно:

Код


 DMass& DMass::operator=(const DMass &v)
 {
   for (int i=0; i<n; i++)
     a[i]=v.a[i];
      return *this;
 }

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

"Разруха не в клозетах, а в головах." © Ф.Ф. Преображенский (М.Булгаков, "Собачье сердце")

serendip

Дата 28.1.2006, 00:35 (ссылка)

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

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 32
Регистрация: 20.12.2005

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

...ну что вам сказать... smile

ничего не понятно, кроме деструктора smile

Могу только ответить:"А в учебнике так!":)
Т.е вы хотите сказать, что в "=" нужно создавать ещё один объект?

BreakPointMAN

Дата 28.1.2006, 00:44 (ссылка)

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

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 725
Регистрация: 15.9.2004
Где: Saratov

Репутация: 14
Всего: 24

Цитата(serendip @ 28.1.2006, 00:35

)

Могу только ответить:"А в учебнике так!":)

Либо в учебнике не так (например, нет выделения памяти с помощью new), либо учебник на свалку... smile

Цитата(serendip @ 28.1.2006, 00:35

)

Т.е вы хотите сказать, что в "=" нужно создавать ещё один объект?

Т.е. хочу сказать, что сначала нужно немного почитать про копирование и присваивание, конструкторы и деструкторы... smile

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

"Разруха не в клозетах, а в головах." © Ф.Ф. Преображенский (М.Булгаков, "Собачье сердце")

serendip

Дата 28.1.2006, 00:59 (ссылка)

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

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 32
Регистрация: 20.12.2005

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

Не могу уже читать, так меня всё это достало smile

Архангельский smile

BreakPointMAN

Дата 28.1.2006, 01:31 (ссылка)

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

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 725
Регистрация: 15.9.2004
Где: Saratov

Репутация: 14
Всего: 24

Цитата(serendip @ 28.1.2006, 00:59

)

Архангельский

Архангельский для изучения среды C++ Builder хорош, но никак не для изучения языка C++... smile

А понятия "конструктор", "деструктор" и пр. - это язык... Так что книгу по C++ в руки ... и читать! smile

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

"Разруха не в клозетах, а в головах." © Ф.Ф. Преображенский (М.Булгаков, "Собачье сердце")

serendip

Дата 28.1.2006, 10:15 (ссылка)

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

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 32
Регистрация: 20.12.2005

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

Я в курсе, что такое конструктор и деструктор,
а вот

Цитата

конструктор копирования

Цитата

присваивать объекты данного класса нужно создавая в каждом из них "свои" динамические данные..

- не совсем понимаю..Думается мне, что в данном случае в этом нет необходимости.
Но мне очень необходмо понять, почему не работают перегрузки! smile

При объявлении объекта я каждый раз создаю в нём свои динамические данные, разве не так?

Fin

Дата 28.1.2006, 10:42 (ссылка)

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

Загрузка ...

Дракон->Спать();

Профиль
Группа: Участник
Сообщений: 687
Регистрация: 4.1.2006

Репутация: 3
Всего: 10

Скрытая логическая ошибка (в класофикации граблей № 2):

Код


DMass DMass::operator~()const
 { DMass rr(n);
           for (int i=0; i<n; i++)
           rr.a[i]= a[i]-d;
         return rr;
 }

В функции создается статический объект, при выходе из функции объект автоматически уничтожается. И ссылка на него некоретна.

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

Пролетал мимо.

Daevaorn

Дата 28.1.2006, 10:44 (ссылка)

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

Загрузка ...

Эксперт

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

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

Цитата(Fin @ 28.1.2006, 10:42

)

Приглядись, там нет ссылки.

Fin

Дата 28.1.2006, 10:51 (ссылка)

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

Загрузка ...

Дракон->Спать();

Профиль
Группа: Участник
Сообщений: 687
Регистрация: 4.1.2006

Репутация: 3
Всего: 10

Это что return rr Да это не ссылка на объект. Но вывод за пределы функции результатов убитого уже экземпляра объекта.

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

Пролетал мимо.

Daevaorn

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

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

Загрузка ...

Эксперт

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

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

Fin
Тогда по твоей логике

Код


int f()
{
   return 123;
}

тоже не коректно? smile

Fin

Дата 28.1.2006, 11:12 (ссылка)

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

Загрузка ...

Дракон->Спать();

Профиль
Группа: Участник
Сообщений: 687
Регистрация: 4.1.2006

Репутация: 3
Всего: 10

Коректно. Компилятор скорее всего сделает такую последовательность команд аcсемблера.

Код


   mov ax,7B
   ret

Это сообщение отредактировал(а) Fin - 28.1.2006, 11:13

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

Пролетал мимо.

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