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

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

Форум программистов > Центр помощи > Проблемы с перегрузкой обератора << в С++

Автор: Pakshin A. S. 20.5.2006, 11:12

Учебная прога:

Код


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

using namespace std;

template <class X>
    class CComplex
    {
    private:
        X Re;
        X Im;

        X _abs(X temp)
        {
            if (temp < 0)
                return -temp;
            else
                return temp;
        }
    public:
        CComplex operator+= (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im + obj.Im;
            temp.Re = Re + obj.Re;

            return temp;
        }
        CComplex operator+ (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im;
            temp.Re = Re;

            temp += obj;

            return temp;
        }
        CComplex operator-= (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im - obj.Im;
            temp.Re = Re - obj.Re;

            return temp;
        }
        CComplex operator- (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im;
            temp.Re = Re;

            temp -= obj;

            return temp;
        }
        CComplex operator= (const CComplex &obj)
        {
            Re = obj.Re;
            Im = obj.Im;
        }

        friend ostream& operator<< (ostream& stream, CComplex &obj);
        friend CComplex SetData (X _Re, X _Im);
    };

template <class X>
    CComplex <X> SetData (X _Re, X _Im)
    {
        CComplex <X> temp;
    
        temp.Re = _Re;
        temp.Im = _Im;

        return temp;
    }

template <class X>
    ostream& operator<< (ostream& stream, CComplex /*<X>*/ &obj)
    {
        int flag = 0;
        
        if (obj.Re != 0)
        {
            stream << obj.Re;
            flag = 1;
        }
    
        if (obj.Im == 0)
        {
            if (flag == 0)
                stream << "0";
        }
        else
        {
            if (obj.Im < 0)
                stream << "-";
            else
                if (flag == 1)
                    stream << "+";
            stream << obj._abs(obj.Im) << "i";
        }
    }

int main(int argc, char* argv[])
{
    CComplex <int> C1;
    CComplex <int> C2;
    
    C1 = SetData(1, 2);
    C2 = SetData(2, 3);

    cout << C1 << endl << C2;

    cout << C1 + C2;

    cout << C1 - C2;
    
    return 0;
}

Ошибки:

Цитата(Microsoft Visual C++ 6.0)

error C2955: 'CComplex' : use of class template requires template argument list - на строке с перегрузкой оператора вне класса

Автор: DeadSoul 20.5.2006, 12:20

Код


ostream& operator<< (ostream& stream, const CComplex<X> &obj)

Автор: MAKCim 20.5.2006, 12:59

Код


template<class X> class CComplex;  // Объявление чтобы работал код ниже

template<class X> ostream& 
    operator<<(ostream&, CComplex<X>&);

template<class C> class CComplex
{
...
    friend ostream& operator<< <>(ostream&,CComplex<X>&);
...
};

template<class X> ostream& 
    operator<<(ostream& stream, CComplex<X>& obj)
{
...
}

Автор: Pakshin A. S. 20.5.2006, 16:00

Не компилицца...

Код


// Project5.cpp : Defines the entry point for the console application.
//

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

using namespace std;

template <class X> class CComplex;

template <class X> ostream& operator<<(ostream&, CComplex<X>&);

template <class C>
    class CComplex
    {
    private:
        C Re;
        C Im;

        C _abs(C temp)
        {
            if (temp < 0)
                return -temp;
            else
                return temp;
        }
    public:
        CComplex operator+= (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im + obj.Im;
            temp.Re = Re + obj.Re;

            return temp;
        }
        CComplex operator+ (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im;
            temp.Re = Re;

            temp += obj;

            return temp;
        }
        CComplex operator-= (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im - obj.Im;
            temp.Re = Re - obj.Re;

            return temp;
        }
        CComplex operator- (const CComplex& obj)
        {
            CComplex temp;

            temp.Im = Im;
            temp.Re = Re;

            temp -= obj;

            return temp;
        }
        CComplex operator= (const CComplex &obj)
        {
            Re = obj.Re;
            Im = obj.Im;
        }

        friend ostream& operator<< (ostream& stream, CComplex <C> &obj);
        friend CComplex SetData (C _Re, C _Im);
    };

template <class X>
    CComplex <X> SetData (X _Re, X _Im)
    {
        CComplex <X> temp;
    
        temp.Re = _Re;
        temp.Im = _Im;

        return temp;
    }

template <class X>
    ostream& operator<< (ostream& stream, CComplex <X> &obj)
    {
        int flag = 0;
        
        if (obj.Re != 0)
        {
            stream << obj.Re;
            flag = 1;
        }
    
        if (obj.Im == 0)
        {
            if (flag == 0)
                stream << "0";
        }
        else
        {
            if (obj.Im < 0)
                stream << "-";
            else
                if (flag == 1)
                    stream << "+";
            stream << obj._abs(obj.Im) << "i";
        }
    }

int main(int argc, char* argv[])
{
    CComplex <int> C1;
    CComplex <int> C2;
    
    C1 = SetData(1, 2);
    C2 = SetData(2, 3);

    cout << C1 << endl << C2;

    cout << C1 + C2;

    cout << C1 - C2;
    
    return 0;
}

Автор: DeadSoul 20.5.2006, 16:21

Pakshin A. S., в какой строчке, какое сообщение об ошибке?

Автор: Pakshin A. S. 20.5.2006, 17:09

error C2248: 'Re' : cannot access private member declared in class 'CComplex<int>'

на 94'ой

Автор: Damarus 20.5.2006, 17:14

Код


#include <iostream>    

using namespace std;    

template <class X> class CComplex;
template <class X> CComplex <X> SetData (X _Re, X _Im);
template <class X> ostream& operator<< (ostream& stream, CComplex <X> &obj);

template <class X> class CComplex    
{
private:    
   X Re;    
   X Im;    
   X _abs(X temp)    
   {    
      if (temp < 0)    
         return -temp;    
      else    
         return temp;    
   }

public:    
   CComplex& operator+= (const CComplex& obj)    
   {    
      Im += obj.Im;    
      Re += obj.Re;    
      return *this;    
   }    
   CComplex operator+ (const CComplex& obj)    
   {    
      CComplex temp = *this;
      temp += obj;    
      return temp;    
   }
   CComplex& operator-= (const CComplex& obj)    
   {        
      Im -= obj.Im;    
      Re -= obj.Re;    
      return *this;    
   }    
   CComplex operator- (const CComplex& obj)    
   {    
      CComplex temp = *this;
      temp -= obj;    
      return temp;    
   }    
   CComplex& operator= (const CComplex &obj)    
   {    
      Re = obj.Re;    
      Im = obj.Im;
      return *this;
   }

private:
   template<class X> friend ostream& operator << (ostream&, CComplex<X>&);
   template <class X> friend CComplex<X> SetData (X _Re, X _Im);
};

template <class X>    
CComplex <X> SetData (X _Re, X _Im)    
{    
   CComplex <X> temp;    

   temp.Re = _Re;    
   temp.Im = _Im;    
   return temp;    
}

template <class X>
ostream& operator<< (ostream& stream, CComplex <X> &obj)    
{    
   int flag = 0;    

   if (obj.Re != 0)    
   {    
      stream << obj.Re;    
      flag = 1;    
   }    

   if (obj.Im == 0)    
   {    
      if (flag == 0)    
         stream << "0";    
   }    
   else    
   {    
      if (obj.Im < 0)    
         stream << "-";    
      else    
         if (flag == 1)    
            stream << "+";
      stream << obj._abs(obj.Im) << "i";    
   }

   return stream;
}

int main(int argc, char* argv[])    
{    
   CComplex <int> C1;    
   CComplex <int> C2;    

   C1 = SetData(1, 2);    
   C2 = SetData(2, 3);    
   cout << C1 << endl << C2 << endl;    
   cout << C1 + C2 << endl;    
   cout << C1 - C2 << endl;

   return 0;    
}

Автор: Pakshin A. S. 20.5.2006, 17:19

Damarus,

error C2563: mismatch in formal parameter list
на
cout << C1 << endl << C2;

Автор: MAKCim 20.5.2006, 17:24

Pakshin A. S.

Цитата

...
friend ostream& operator<< <>(ostream& stream, CComplex <C> &obj);
friend CComplex SetData (C _Re, C _Im);
...

смотрите мой пост выше
Добавлено @ 17:25

Цитата

error C2248: 'Re' : cannot access private member declared in class 'CComplex<int>'

ошибка потому что без <> operator<< просто сторонняя функция, не имеющая доступа к реализации CComplex

Автор: Damarus 20.5.2006, 17:51

Pakshin A. S., не знаю, у меня всё работает:

Автор: Pakshin A. S. 20.5.2006, 18:01

MAKCim, поставлены <>... Ругается на эту строчку: error C2143: syntax error : missing ';' before '<'

Код


        friend ostream& operator<< <>(ostream& stream, CComplex <C> &obj);
        friend CComplex SetData (C _Re, C _Im);
    };

Damarus, я не спорю, но у меня Microsoft Visual C++ 6.0, а на картинке вроде не тот компилятор, что у меня... smile

Автор: MAKCim 20.5.2006, 18:17

Цитата

Ругается на эту строчку: error C2143: syntax error : missing ';' before '<'

не знаю, может проблема в VS6.0
у меня все OK, проверял (g++ 4.0.2)

сделайте public методы SetRe, SetIm, GetRe, GetIm
и шаблон оператора operator<< (не используя friend), вместо obj.Re пишите obj.GetRe() и т д

Автор: Damarus 20.5.2006, 18:34

Pakshin A. S., как я делал:
1. Скопировал твой код (9-ю постами выше). Попытался скомпилировать. Получил:

Цитата

error C4716: 'CComplex<int>::operator=' : must return a value

2. Добавил

Код

return *this;

в CComplex<int>::operator=. Попытался скомпилировать. Получил:

Цитата

error LNK2019: unresolved external symbol "class std::basic_ostream<char,struct std::char_traits<char> > & __cdecl operator<<(class std::basic_ostream<char,struct std::char_traits<char> > &,class CComplex<int> &)" (??6@YAAAV?$basic_ostream@DU?$char_traits@D@std@@@std@@AAV01@AAV?$CComplex@H@@@Z) referenced in function _main Test.obj

Цитата

error LNK2019: unresolved external symbol "class CComplex<int> __cdecl SetData(int,int)" (?SetData@@YA?AV?$CComplex@H@@HH@Z) referenced in function _main Test.obj

3. Посмотрел MSDN по вопросу LNK2019. Там есть подходящий пример:

Цитата

The following sample generates LNK2019 on a user-defined operator.

Код


// LNK2019e.cpp
// compile with: /EHsc
// LNK2019 expected
#include <iostream>
using namespace std;

template<class T> class 
Test {
   friend ostream& operator<<(ostream&, Test&);
   // Uncomment the following line to resolve.
   // template<typename T> friend ostream& operator << (ostream&, Test<T>&);
};

template<typename T>
ostream& operator<<(ostream& os, Test<T>& tt) {
   return os;
}

int main() {
   Test<int> t;
   cout << "Test: " << t << endl;   // unresolved external
}

4. На основе этого примера добавил:

Код


template<class C> friend ostream& operator<< (ostream& stream, CComplex <C> &obj);    
template<class C> friend CComplex<C> SetData (C _Re, C _Im);

Попытался скомпилировать. Получил:

Цитата

error C4716: 'operator<<<int>' : must return a value

5. Добавил

Код

return stream;

.

После этого программа нормально скомпилировалась, но выдавала неверные результаты (вроде). Изменил операции в классе CComplex (см. выше).

Автор: Pakshin A. S. 20.5.2006, 19:51

Цитата(MAKCim @ 20.5.2006, 19:17

)

сделайте public методы SetRe, SetIm, GetRe, GetIm

Задача - использование именно дружественности.

Damarus, Спасибо за подробный ответ. Но вот только пытаюсь пихать в разные места.. что-то пока не нашел комбинации верной... smile

МОжет проще будет выложить код? Я бы его сравнил с тем, что у меня и бытрее нашел ошибки у себя...

Добавлено @ 19:52
Говорил именно про проблемы с перегрузкой, а не с забытием возврата значения функциями...

Добавлено @ 19:57

Цитата(MAKCim @ 20.5.2006, 19:17

)

сделайте public методы SetRe, SetIm, GetRe, GetIm

Так-то работат.. smile

Сделал public все... smile

а вот на private все так же ругается, что не может получить доступ.

Автор: Damarus 20.5.2006, 20:31

Pakshin A. S., вот немного изменённый твой код (все отличия прокоментированы):

Код


#include <iostream>

using namespace std;

template <class C> class CComplex
{    
private:
   C Re;
   C Im;
   C _abs(C temp)
   {    
      if (temp < 0)
         return -temp;
      else
         return temp;
   }

public:
   CComplex operator+= (const CComplex& obj)
   {
      CComplex temp;
      temp.Im = Im + obj.Im;
      temp.Re = Re + obj.Re;
      return temp;
   }

   CComplex operator+ (const CComplex& obj)
   {
      CComplex temp;
      temp.Im = Im;
      temp.Re = Re;
      temp += obj;
      return temp;
   }

   CComplex operator-= (const CComplex& obj)
   {
      CComplex temp;
      temp.Im = Im - obj.Im;
      temp.Re = Re - obj.Re;
      return temp;
   }

   CComplex operator- (const CComplex& obj)
   {
      CComplex temp;
      temp.Im = Im;
      temp.Re = Re;
      temp -= obj;
      return temp;
   }

   CComplex operator= (const CComplex &obj)
   {
      Re = obj.Re;
      Im = obj.Im;

      return *this; // Добавленно для нормальной компиляции.
   }

   // Перед friend ostream& operator<< (ostream& stream, CComplex<C>& obj);
   // добавленно template<class C>, параметор obj изменён с CComplex&
   // на CComplex<C>&
   template<class C> friend ostream& operator<< (ostream& stream, CComplex<C>& obj);

   // Перед friend CComplex SetData (C _Re, C _Im);
   // добавленно template<class C>, после friend CComplex
   // добавленно <C>
   template<class C> friend CComplex<C> SetData (C _Re, C _Im);
};

template <class X> CComplex <X> SetData(X _Re, X _Im)
{
   CComplex <X> temp;

   temp.Re = _Re;
   temp.Im = _Im;
   return temp;
}    

template <class X> ostream& operator<< (ostream& stream, CComplex<X>& obj)
{
   int flag = 0;

   if (obj.Re != 0)
   {
      stream << obj.Re;
      flag = 1;
   }

   if (obj.Im == 0)
   {
      if (flag == 0)
         stream << "0";
   }
   else
   {
      if (obj.Im < 0)
         stream << "-";
      else
         if (flag == 1)
            stream << "+";
      stream << obj._abs(obj.Im) << "i";
   }

   return stream; // Добавленно для нормальной компиляции.
}

int main(int argc, char* argv[])
{
   CComplex <int> C1;
   CComplex <int> C2;

   C1 = SetData(1, 2);
   C2 = SetData(2, 3);
   cout << C1 << endl << C2;
   cout << C1 + C2;
   cout << C1 - C2;

   return 0;    
}

Автор: Pakshin A. S. 20.5.2006, 21:30

А у меня не компилицца! smile

Похоже, нужно брать нормальную систему...

Автор: MAKCim 21.5.2006, 11:33

Цитата

Задача - использование именно дружественности.

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

Код


template<class Type> class complex
{
public:
    typedef Type value_t;
private:
    typedef complex<value_t> self_t;
private:
    value_t re_, im_;
public:
    explicit complex(
        const value_t& p_re=value_t(),
        const value_t& p_im=value_t()
    ): re_(p_re), im_(p_im) {}
    
    void set_re(const value_t& p_re) {re_=p_re;}
    void set_im(const value_t& p_im) {im_=p_im;}

    value_t get_re() const {return re_;}
    value_t get_im() const {return im_;]

    self_t& operator+=(const self_t& obj)
    {
        re_+=obj.re_;
        im_+=obj.im_;
        return *this;    
    }
};

template<class Type> complex<Type> operator+
    (const complex<Type>& obj_a, const complex<Type>& obj_b)
{
    complex<Type> obj(obj_a);
    obj+=obj_b;
    return obj;
}

template<class Stream, class Type> Stream& operator<<
    (Stream& stream, const complex<Type>& obj)
{
    stream<<obj.get_re()<<obj.get_z();
    return stream;
}

template<class Stream, class Type> Stream& operator>>
    (Stream& stream, complex<Type>& obj)
{
    Type re, im;
    stream>>re>>im;
    obj.set_re(re);
    obj.set_im(im);
    return stream;
}