К числу прибавить объект класса

@BenBen · Регистрация: 16.01.2012

Author24 — интернет-сервис помощи студентам

Не получается перегрузить '+' для сложения числа с объектом

Пишу через френд

C++

1	friend Vremya operator + (int a, Vremya );

C++

Vremya operator + (int a, Vremya b)
{
        int sec = getSec(b);         //getSec - перевод в секунды, getHour - перевод в часы
        while (a-- != 0)
        {
        sec++;
        }
        return b = getHour(sec);
}

@retmas · 17.01.2012, 18:22

зачем b = getHour(sec);?
b не должно изменяться (правда в итоге после возврата из ф-ии оно и не изменится))
и передавай параметр в ф-ю по константной ссылке

@kravam · 17.01.2012, 18:49

BenBen, дай весь код

@BenBen · 17.01.2012, 18:57 **[ТС]**

C++

Vremya operator + (int a, const &Vremya b)
{
        Vremya b = c;
        int sec = getSec(c);         
        while (a-- != 0)
        {
        sec++;
        }
        c = getHour(sec);
        return b;
}

Все равно не работает

@kravam · 17.01.2012, 18:59

Код-то дашь весь?

@dimcoder · 17.01.2012, 18:59

Сообщение от BenBen

Vremya b = c;

А 'c' это типа глобальная? Давайте весь код.

@retmas · 17.01.2012, 19:05

Сообщение от BenBen

C++

Vremya operator + (int a, const &Vremya b)
{
        Vremya b = c;
        int sec = getSec(c);         
        while (a-- != 0)
        {
        sec++;
        }
        c = getHour(sec);
        return b;
}

Все равно не работает

чем отличаются 2 b и с которой ты работаешь?

Добавлено через 1 минуту

C++

Vremya operator + (int a, const Vremya& b)
{
        int sec = getSec(b);         
        sec += a;
        return getHour(sec);
}

пробуй

@BenBen · 17.01.2012, 19:07 **[ТС]**

Vremya.cpp

C++

//---------------------------------------------------------------------------
Vremya::Vremya()
{
        sec = 0;
        min = 0;
        hour = 0;
}
//---------------------------------------------------------------------------
Vremya::Vremya(int _hour, int _min, int _sec)
{
        sec = _sec;
        min = _min;
        hour = _hour;
}
//---------------------------------------------------------------------------
Vremya::Vremya(Vremya &t)
{
        hour = t.hour;
        min = t.min;
        sec = t.sec;
}
//---------------------------------------------------------------------------
int getSec(Vremya t)
{
        int sec = t.sec + t.min*60 + t.hour*3600;
        return sec;
}
//---------------------------------------------------------------------------
Vremya getHour(int a)
{
        Vremya t;
        t.hour = a/3600;
        t.min = (a - t.hour*3600)/60;
        t.sec = a - t.hour*3600 - t.min*60;
        return t;
}
//---------------------------------------------------------------------------
 
Vremya &Vremya::operator ++ ()
{
        int sec = getSec(*this);
        sec++;
        *this = getHour(sec);
        return *this;
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator -- ()
{
        int sec = getSec(*this);
        sec--;
        *this = getHour(sec);
        return *this;
}
//---------------------------------------------------------------------------
Vremya Vremya::operator ++ (int)
{
        Vremya a = *this;
        int sec = getSec(*this);
        sec++;
        *this = getHour(sec);
        return a;
}
//---------------------------------------------------------------------------
Vremya Vremya::operator -- (int)
{
        Vremya a = *this;
        int sec = getSec(*this);
        sec--;
        *this = getHour(sec);
        return a;
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator + (int a)
{
        int sec = getSec(*this);
        sec += a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator - (int a)
{
        int sec = getSec(*this);
        sec += a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator += (int a)
{
        int sec = getSec(*this);
        sec += a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator -= (int a)
{
        int sec = getSec(*this);
        sec -= a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
 
Vremya &Vremyae::operator = (Vremya t)
{
        sec = t.sec;
        min = t.min;
        hour = t.hour;
        return *this;
}
//---------------------------------------------------------------------------

Vremya.h

C++

class Vremya
{
        private:
    int sec, min, hour;
        
        public:
    Vremya();
    Vremya(int _sec, int _min, int _hour);
        Vremya(Vremya &t);
 
    void Show();
 
    friend int getSec(Vremya t);
        friend Vremya getHour(int a);
 
    Vremya &operator ++ ();
    Vremya &operator -- ();
 
    Vremya operator ++ (int);
    Vremya operator -- (int);
    
    Vremya &operator += (int a);
    Vremya &operator -= (int a);
 
    Vremya &operator + (int a);
    Vremya &operator - (int a);
 
    friend Vremya operator + (int a, Vremya &);
    friend Vremya operator - (int a, Vremya);
    friend Vremya operator - (Vremya &t1, Vremya &t2);
 
    Vremya &operator = (Vremya t);
};

@retmas · 17.01.2012, 19:17

Сообщение от retmas

C++

Vremya operator + (int a, const Vremya& b)
{
        int sec = getSec(b);         
        sec += a;
        return getHour(sec);
}

пробуй

Добавлено через 1 минуту
если хочешь короче

C++

Vremya operator + (int a, const Vremya& b)
{
        return getHour(getSec(b) + a);
}

Добавлено через 2 минуты
и вместо

C++

        friend int getSec(Vremya t)
        friend Vremya operator + (int a, Vremya &);
        friend Vremya operator - (int a, Vremya);
        friend Vremya operator - (Vremya &t1, Vremya &t2);

лучше переделай чтоб было

C++

        friend int getSec(const Vremya& t)
        friend Vremya operator + (int a, const Vremya &);
        friend Vremya operator - (int a, const Vremya&);
        friend Vremya operator - (const Vremya &t1, const Vremya &t2);

Добавлено через 1 минуту
а еще лучше переписать так, чтоб избавится от френдов (для этого все есть)

Добавлено через 2 минуты
да и не увидел сразу

C++

1 2	Vremya &operator = (Vremya t); Vremya(Vremya &t);

=>

C++

1 2	Vremya &operator = (const Vremya& t); Vremya(const Vremya &t);

Добавлено через 1 минуту
и не забудь: исправляя объявления - исправь определения

@BenBen · 17.01.2012, 19:18 **[ТС]**

чет понять не могу зачем конст нужен?

если без френдов то должен 1 параметр передаваться?

@retmas · 17.01.2012, 19:35

Сообщение от BenBen

чет понять не могу зачем конст нужен?

чтобы например компилятор ругался на вот такое безобразие:

C++

Vremya operator + (int a, Vremya b)
{
        int sec = getSec(b);         //getSec - перевод в секунды, getHour - перевод в часы
        while (a-- != 0)
        {
        sec++;
        }
        return b = getHour(sec);
}

как у вас было.

когда в ф-ю передается параметр в виде const Vremya& b
1) не создается лишний временный обьект типа Vremya (ссылка)
2) компилятор понимает, что b - не должно изменяться (const)

Добавлено через 8 минут

Сообщение от BenBen

если без френдов то должен 1 параметр передаваться?

нет. просто обьявляете их вне класса.
если вы реализуете

C++

1 2	Vremya &operator += (int a); Vremya &operator -= (int a);

как ф-ии - члены, то

C++

1 2	friend Vremya operator + (int a, Vremya &); friend Vremya operator - (int a, Vremya);

реализуются как обычные ф-ии не члены с помощью += / -=
==> отпадает надобность в friend

@kravam · 17.01.2012, 23:38

BenBen, я бы сделал две вещи, поправил определение и объявление operator =, то есть:

C++

Vremya &operator = (const Vremya& t);
 
//тут всякий код
 
Vremya &Vremya::operator = (const Vremya& t)
{
        sec = t.sec;
        min = t.min;
        hour = t.hour;
        return *this;
}

...Вроде компилится...

@BenBen · 19.01.2012, 20:07 **[ТС]**

И все таки не работает число + объект

@kravam · 19.01.2012, 20:15

Дай неработающий код и будь внимательнее на этот раз

@BenBen · 19.01.2012, 20:18 **[ТС]**

C++

Vremya operator + (int a, const Vremya &t)
{
        int sec = getSec(t);
        sec += a;
        return getHour(sec);
}

@kravam · 19.01.2012, 20:24

Короче, что мне надо:
1) Весь код
2) Вводные данные
3) Чё на выходе
4) Чё ожидаешь

Я так работаю, я всегда свои вопросы оформлял так и никто от меня не отмахивался

@retmas · 19.01.2012, 20:25

BenBen, просили дать неработающий код, т.е. покажите где и как вы этим пользуетесь

@BenBen · 19.01.2012, 20:38 **[ТС]**

C++

//---------------------------------------------------------------------------
#include <string.h>
#include <stdio.h>
#include <iostream.h>
#include <vcl.h>
#include <stdlib.h>
#include "Time.h"
#pragma hdrstop
//---------------------------------------------------------------------------
Vremya::Vremya()
{
        sec = 0;
        min = 0;
        hour = 0;
}
//---------------------------------------------------------------------------
Vremya::Vremya(int h, int m, int s)
{
        sec = s;
        min = m;
        hour = h;
}
//---------------------------------------------------------------------------
Vremya::Vremya(const Vremya &t)
{
        hour = t.hour;
        min = t.min;
        sec = t.sec;
}
//---------------------------------------------------------------------------
int getSec(const Vremya &t)
{
        int sec = t.sec + t.min*60 + t.hour*3600;
        return sec;
}
//---------------------------------------------------------------------------
Vremya getHour(int a)
{
        Vremya t;
        t.hour = a/3600;
        t.min = (a - t.hour*3600)/60;
        t.sec = a - t.hour*3600 - t.min*60;
        return t;
}
//---------------------------------------------------------------------------
void Vremya::Show()
{
        cout<<hour<<":"<<min<<":"<<sec;
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator ++ ()
{
        int sec = getSec(*this);
        sec++;
        *this = getHour(sec);
        return *this;
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator -- ()
{
        int sec = getSec(*this);
        sec--;
        *this = getHour(sec);
        return *this;
}
//---------------------------------------------------------------------------
Vremya Vremya::operator ++ (int)
{
        Vremya a = *this;
        int sec = getSec(*this);
        sec++;
        *this = getHour(sec);
        return a;
}
//---------------------------------------------------------------------------
Vremya Vremya::operator -- (int)
{
        Vremya a = *this;
        int sec = getSec(*this);
        sec--;
        *this = getHour(sec);
        return a;
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator + (int a)
{
        int sec = getSec(*this);
        sec += a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator - (int a)
{
        int sec = getSec(*this);
        sec -= a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator += (int a)
{
        int sec = getSec(*this);
        sec += a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator -= (int a)
{
        int sec = getSec(*this);
        sec -= a;
        return *this = getHour(sec);
}
//---------------------------------------------------------------------------
Vremya operator + (int a, const Vremya &t)
{
        int sec = getSec(t);
        sec += a;
        return getHour(sec);
}
//---------------------------------------------------------------------------
Vremya &Vremya::operator = (const Vremya &t)
{
        sec = t.sec;
        min = t.min;
        hour = t.hour;
        return *this;
}
 
 
//---------------------------------------------------------------------------
 
#pragma package(smart_init)

C++

//---------------------------------------------------------------------------
 
#ifndef VremyaH
#define VremyaH
 
 
class Vremya
{
        private:
    int sec, min, hour;
 
        public:
    Vremya();
    Vremya(int s, int m, int h);
        Vremya(const Vremya &t);
 
    void Show();
 
    friend int getSec(const Vremya &t);
        friend Vremya getHour(int a);
 
    Vremya &operator ++ ();
    Vremya &operator -- ();
 
    Vremya operator ++ (int);
    Vremya operator -- (int);
 
    Vremya &operator += (int a);
    Vremya &operator -= (int a);
 
    Vremya &operator + (int a);
    Vremya &operator - (int a);
 
        Vremya &operator = (const Vremya &t);
 
    friend Vremya operator + (int a, const Vremya &t);
    friend Vremya operator - (int a, const Vremya &t);
 
 
};
//---------------------------------------------------------------------------
#endif

C++

В мейне
Vremya t1(12,0,0);
Vremya t2;
t1+= 10; //работает
t1 = t2; //работает
t1 + 10; //работает
10 + t1; //не работает

Добавлено через 2 минуты
Так то он работает (надеюсь правильно) но вот с перегрузкой этой застрял.
И сижу туплю не могу понять как без френдов написать

@retmas · 19.01.2012, 20:39

Сообщение от BenBen

C++

1	10 + t1; //не работает

а что вы от этого хотите? чтобы t1 менялось? так этого быть не должно, ведь когда вы пишите
2 + 3 тройка же не должна магическим образом превращаться в пятерку

@BenBen · 19.01.2012, 20:41 **[ТС]**

То есть должен вернуть копию объекта?

@BenBen 0 / 0 / 0 Регистрация: 16.01.2012 Сообщений: 15
	19.01.2012, 20:07 [ТС]	13
	И все таки не работает число + объект 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	19.01.2012, 20:15	14
	Дай неработающий код и будь внимательнее на этот раз 0

@retmas Жарю без масла 867 / 749 / 225 Регистрация: 13.01.2012 Сообщений: 1,702
	17.01.2012, 18:22	2
	зачем b = getHour(sec);? b не должно изменяться (правда в итоге после возврата из ф-ии оно и не изменится)) и передавай параметр в ф-ю по константной ссылке 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	17.01.2012, 18:49	3
	BenBen, дай весь код 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	17.01.2012, 18:59	5
	Код-то дашь весь? 0

@dimcoder Полярный 476 / 448 / 158 Регистрация: 11.09.2011 Сообщений: 1,156
	17.01.2012, 18:59	6
	Сообщение от BenBen Vremya b = c; А 'c' это типа глобальная? Давайте весь код. 0

@BenBen 0 / 0 / 0 Регистрация: 16.01.2012 Сообщений: 15
	17.01.2012, 19:18 [ТС]	10
	чет понять не могу зачем конст нужен? если без френдов то должен 1 параметр передаваться? 0

@kravam быдлокодер 1724 / 911 / 106 Регистрация: 04.06.2008 Сообщений: 5,679
	19.01.2012, 20:24	16
	Короче, что мне надо: 1) Весь код 2) Вводные данные 3) Чё на выходе 4) Чё ожидаешь Я так работаю, я всегда свои вопросы оформлял так и никто от меня не отмахивался 0

@retmas Жарю без масла 867 / 749 / 225 Регистрация: 13.01.2012 Сообщений: 1,702
	19.01.2012, 20:25	17
	BenBen, просили дать неработающий код, т.е. покажите где и как вы этим пользуетесь 0

@BenBen 0 / 0 / 0 Регистрация: 16.01.2012 Сообщений: 15
	19.01.2012, 20:41 [ТС]	20
	То есть должен вернуть копию объекта? 0