Kompilator nie widzi zaprzyjaźnionych funkcji w klasie (C++)

Question

Witam!

Problem polega na tym, że mimo pzypisania do funkcji zaprzyjaźnionych obiektów klas kompilator zwraca błąd wywołania (lub po prostu nie widzi ich w klasie). Dlaczego ?

Oto błędy programu

main4.cpp: In function ‘int main()’:
main4.cpp:20:32: error: ‘class Time’ has no member named ‘operator+’
         total = coding.operator+(fixing); //zapis operacyjny
                                ^
main4.cpp:24:32: error: ‘class Time’ has no member named ‘operator-’
         minus = fixing.operator-(coding); //zapis operacyjny
                                ^
main4.cpp:36:37: error: ‘class Time’ has no member named ‘operator+’
         total = more_fixing.operator+(total); //dodanie wartości do total !

class Time
{
        private:
                int hours, minutes;
        public:
                Time();
                Time(int hour, int minut = 0);
                friend Time operator+(const Time & all_time);
                friend Time operator-(const Time & all_time);
                friend Time operator*(double value, const Time & all_time);
                void show() const;
};

 

Comments

Po co friend przed operatorami z linii 8 i 9, przecież to są metody klasy, więc siłą rzeczy mają dostęp do niepublicznych pól?

---

Takie jest polecenie w zadaniu. Zamienić wszystkie przeciążenia operatorów (metod) na funkcję zaprzyjaźnione.

---

W takim razie operatory + i - powinny być dwuargumentowe (tak jak teraz masz *).

---

Dalej jest błąd 'candidate': ...

---

Pokaż więcej kodu.

---

plik nagłówkowy:

#ifndef SUMTIME_H_
#define SUMTIME_H_

class Time
{
        private:
                int hours, minutes;
        public:
                Time();
                Time(int hour, int minut = 0);
                friend Time operator+(double value, const Time & all_time);
                friend Time operator-(double value, const Time & all_time);
                friend Time operator*(double value, const Time & all_time);
                void show() const;
};
#endif

deklaracja klasy:

#include <iostream>
#include "sumTime.h"

Time::Time() {hours = minutes = 0; }

Time::Time(int hour, int minut) { hours=hour; minutes=minut; }

Time operator+(double value, const Time & all_time)
{
        Time total;
        total.minutes = all_time.minutes + all_time.minutes;
        total.hours = all_time.hours + all_time.hours + total.minutes/60;
        total.minutes %= 60;
        return total;
}

Time operator-(double value, const Time & all_time)
{
        Time minus;
        int total1, total2;
        total1 = all_time.minutes + 60 * all_time.hours * value;
        total2 = all_time.minutes + 60 * all_time.hours * value;
        minus.minutes = (total2 - total1) % 60;
        minus.hours = (total2 - total1) / 60;
        return minus;
}

Time operator*(double value, const Time & all_time)//nie dodajemy kwalifikatora Time::
{
        Time result;
        int total_result = all_time.hours * value * 60 + all_time.minutes * value;
        result.hours = total_result / 60;
        result.minutes = total_result % 60;
        return result;
}

void Time::show() const { std::cout << hours << " godzin, " << minutes << " minut.\n"; }

główny:

#include "sumTime.h"
#include <iostream>

int main()
{
        Time planning; //ustawienie wartości na 0
        Time coding(2, 40);
        Time fixing(5, 55);
        Time total, minus, result;

        std::cout << "Czas planowania: ";
        planning.show();

        std::cout << "Czas kodowania: ";
        coding.show();

        std::cout << "Czas poprawiania: ";
        fixing.show();

        total = coding + fixing; //zapis operacyjny
        std::cout << "Łącznie \"(coding + fixing)\": ";
        total.show();

        minus = fixing - coding; //zapis operacyjny
        std::cout << "Róznica pomiedzy poprawkami a tworzeniem \"(fixing - coding)\": ";
        minus.show();

        result = operator*(1.5, total); // wywolanie zaprzyjaznionej funkcji z przeciazonym operatorem
        std::cout << "Doliczenie przerw do czasu pracy \"(operator*(1.5, total))\": ";
        result.show();

        Time more_fixing(3, 15);
        std::cout << "Dodatkowe poprawki: ";
        more_fixing.show();

        total = more_fixing + total; //dodanie wartości do total !
        std::cout << "Łącznie \"(more_fixing.operator+(total))\": ";
        total.show();
        return 0;
}

 

---

Przeciążyłeś operatory z argumentem typu double, a próbujesz dodawać do/odejmować od siebie dwa obiekty typu Time. Pisząc wcześniej o operatorze * miałem na myśli jego dwuargumentowość (binary operator). Zrób takie operatory:

friend Time operator+(Time ltime, const Time & rtime);
friend Time operator-(Time ltime, const Time & rtime);

 

---

Dziękuję Ci bardzo mi pomogłeś !.

Wszystko już działa sprawnie.

Answer 1

total = coding.operator+(fixing);

Tak się operatorów nie wywołuje. Robi się to po prostu tak:

total = coding + fixing;

To samo dotyczy pozostałych przypadków. No i oczywiście jak masz inne operatory (-, *, /, <<, >> to też używasz ich bezpośrednio, a że dla obiektów twojej klasy zostały przeciążone, to będą działały tak, jak sobie tego zażyczysz).

Comments

main4.cpp: In function ‘int main()’:
main4.cpp:20:24: error: no match for ‘operator+’ (operand types are ‘Time’ and ‘Time’)
         total = coding + fixing; //zapis operacyjny
                        ^
In file included from main4.cpp:1:0:
sumTime.h:12:29: note: candidate: Time operator+(const Time&)
                 friend Time operator+(const Time & all_time);
                             ^
sumTime.h:12:29: note:   candidate expects 1 argument, 2 provided
main4.cpp:24:24: error: no match for ‘operator-’ (operand types are ‘Time’ and ‘Time’)
         minus = fixing - coding; //zapis operacyjny
                        ^
In file included from main4.cpp:1:0:
sumTime.h:13:29: note: candidate: Time operator-(const Time&)
                 friend Time operator-(const Time & all_time);
                             ^
sumTime.h:13:29: note:   candidate expects 1 argument, 2 provided
main4.cpp:36:29: error: no match for ‘operator+’ (operand types are ‘Time’ and ‘Time’)
         total = more_fixing + total; //dodanie wartości do total !
                             ^
In file included from main4.cpp:1:0:
sumTime.h:12:29: note: candidate: Time operator+(const Time&)
                 friend Time operator+(const Time & all_time);
                             ^
sumTime.h:12:29: note:   candidate expects 1 argument, 2 provided

Niestety to również wywołuje błąd.