Перегрузка оператора "+" с шаблонами

@hrustash · Регистрация: 05.04.2013

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

нужно написать перегрузку оператора +, который объединяет 2 массива структур в один.
допустим, есть вот такая вот структура

C++

struct Stud1
{
    char name [50];
    char addr [50];
    double price;
    friend istream& operator >> (istream&, Stud1&);
    friend ostream& operator << (ostream&, Stud1&);
    bool operator == (Stud1&);
};

и класс-шаблон, для этого класса и нужна перегрузка +

C++

template <class T>
class STUD
{
    int count;
    T *M;
public:
    STUD();
    STUD(int);
    STUD(const STUD&);
    ~STUD();
    void memory();
    void ReadData(istream&);
    void WriteData(ostream&);
    int GetCount();
    T& operator [] (int index);
    STUD<T> operator + (const STUD<T>&s)
        {
            STUD<T> A(count+s.count);
            int i;
            for (i = 0; i < count; i++)
                A.M[i]=M[i];
            for (int j = 0; j < s.count; i++, j++)
                A.M[i]=s.M[j];
            A.count=count+s.count;
            return A;
        }
};

и вот параметрический конструктор, который используется в реализации перегрузки

C++

template <class T>
STUD<T>::STUD(int a)
{
    M = new T [a];
    count = a;
}

реализацию остальных функций, думаю, нет смысла кидать. но если надо - скину.
в мэйне создаю 3 объекта класса STUD с типом массива Struct1

C++

STUD<Stud1> A, A1, res;
A.ReadData(in1); //информацию, допустим, читаю из одного и того же файла. чтение работает нормально
A1.ReadData(in1);
res=A+A1;// ну и вот на сумме падает с ошибкой

хотелось бы понять, что я сделал не так?

Croessmah · 17.06.2013, 17:58

Где конструктор копий?

@hrustash · 17.06.2013, 18:01 **[ТС]**

Croessmah,
вот

C

STUD<T>::STUD(const STUD<T>&s)
{
    count=s.count;
    M=new T[count];
    for (int i = 0; i < count; i++)
        M[i]=s.M[i];
}

Croessmah · 17.06.2013, 18:07

И деструктор тоже и оператор присваивания. А лучше весь код.
Судя по ошибке где-то с new/delete, наверное, напутали.

@hrustash · 17.06.2013, 18:08 **[ТС]**

Croessmah,

C

#include "structures.h"
template <class T>
class STUD
{
    int count;
    T *M;
public:
    STUD();
    STUD(int);
    STUD(const STUD&);
    ~STUD();
    void memory();
    void ReadData(istream&);
    void WriteData(ostream&);
    int GetCount();
    T& operator [] (int index);
    /*STUD<T> operator + (const STUD<T>&s)
        {
            STUD<T> A(count+s.count);
            int i;
            for (i = 0; i < count; i++)
                A.M[i]=M[i];
            for (int j = 0; j < s.count; i++, j++)
                A.M[i]=s.M[j];
            A.count=count+s.count;
            return A;
        }*/ 
    STUD<T>& operator = (const STUD<T>&);
};
template <class T>
STUD<T>::STUD()
{
    M = new T [1];
    count=1;
}
template <class T>
STUD<T>::STUD(int a)
{
    M = new T [a];
    count = a;
}
template <class T>
STUD<T>::STUD(const STUD<T>&s)
{
    count=s.count;
    M=new T[count];
    for (int i = 0; i < count; i++)
        M[i]=s.M[i];
}
 
 
template <class T>
STUD<T>::~STUD()
{
    delete [] M;
}
 
template <class T>
void STUD<T>::memory()
{
    T *M1=new T [count+1];
    for (int i = 0; i < count; i++)
        M1[i]=M[i];
    delete [] M;
    M=M1;
    count++;
}
template <class T>
void STUD<T>::ReadData(istream& in)
{
    int i=0;
    while (in>>M[i])
    {
        if(in.eof()) break;
        ++i;
        if(i==count)
            memory();
    }
}
template <class T>
void STUD<T>::WriteData(ostream& out)
{
    for (int i = 0; i < count; i++)
        out<<M[i];
}
template <class T>
int STUD<T>::GetCount()
{
    return count;
}
template <class T>
T& STUD<T>::operator[] (int index)
{
    return M[index];
}
template <class T>
STUD<T>& STUD<T>::operator=(const STUD<T>& s)
{
    delete [] M;
    count=s.count;
    M = new T [count];
    for (int i = 0; i < count; i++)
        M[i]=s.M[i];
    return *this;
}

Croessmah · 17.06.2013, 18:40

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

А лучше весь код.

Весь код. Который можно скомпилировать и проверить.

@hrustash · 17.06.2013, 19:03 **[ТС]**

Croessmah,
структуры:

C

#pragma once
#include <fstream>
#include <iostream>
using namespace std;
struct Stud1
{
    char name [50];
    char addr [50];
    double price;
    friend istream& operator >> (istream&, Stud1&);
    friend ostream& operator << (ostream&, Stud1&);
    bool operator == (Stud1&);
};
 
struct Stud2
{
    char name [50];
    friend istream& operator >> (istream&, Stud2&);
    friend ostream& operator << (ostream&, Stud2&);
    bool operator == (Stud2&);
};
 
struct Stud3
{
    char name [50];
    char addr [50];
    char ContrDate [50];
    double price;
};

main:

C

#include "Classes.h"
void main()
{
    STUD<Stud1> A, A1, res;
    STUD<Stud2> B;
    STUD<Stud3> C;
    ifstream in1("input1.txt");
    ifstream in2("input2.txt");
    A.ReadData(in1);
    A.WriteData(cout);
    B.ReadData(in2);
    cout<<endl<<endl;
    B.WriteData(cout);
    A1=A;
    //res=A+A1;
    cout<<endl;
    A1.WriteData(cout);
    A1.sort();
    cout<<endl;
    A1.WriteData(cout);
}

и ниже два инпута

@hrustash · 17.06.2013, 19:21 **[ТС]**

Croessmah,
забыл еще .cpp со структурами, сори

C

#include "structures.h"
 
istream& operator >> (istream& in, Stud1& s)
{
    char temp[10];
    in.getline(s.name, 50);
    in.getline(s.addr, 50);
    in.getline(temp, 30);
    s.price=atoi(temp);
    return in;
}
ostream& operator << (ostream& out, Stud1& s)
{
    out<<s.name<<"  "<<s.addr<<"  "<<s.price<<endl;;
    return out;
}
 
istream& operator >> (istream& in, Stud2& s)
{
    in.getline(s.name, 50);
    return in;
}
 
ostream& operator << (ostream& out, Stud2& s)
{
    out<<s.name<<endl;
    return out;
}
 
bool Stud1::operator == (Stud1& a)
{
    if(strcmp(this->name, a.name)==0) return true;
    else return false;
}
 
bool Stud2::operator == (Stud2& a)
{
    if(strcmp(this->name, a.name)==0) return true;
    else return false;
}

Добавлено через 2 минуты
Croessmah,
БИНГО
прошу прощения за не профессионализм, похоже, что не работало из-за отсутствия перегрузки оператора =

Croessmah · 17.06.2013, 19:25

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

прошу прощения за не профессионализм, похоже, что не работало из-за отсутствия перегрузки оператора =

Ага, потому как с таким текстом работает нормально:

C++

istream& operator >> (istream& lhs, Stud1& rhs){
    lhs.getline(rhs.name,50);
    lhs.getline(rhs.addr,50);
    lhs>>rhs.price;
    lhs.get();
    return lhs;
}
 
ostream& operator << (ostream& lhs, Stud1& rhs){
    lhs<<rhs.name<<std::endl;
    lhs<<rhs.addr<<std::endl;
    lhs<<rhs.price<<std::endl;
    return lhs;
}

Croessmah Неэпический 17869 / 10634 / 2054 Регистрация: 27.09.2012 Сообщений: 26,736 Записей в блоге: 1
	17.06.2013, 17:58	2
	Где конструктор копий? 0

Croessmah Неэпический 17869 / 10634 / 2054 Регистрация: 27.09.2012 Сообщений: 26,736 Записей в блоге: 1
	17.06.2013, 18:07	4
	И деструктор тоже и оператор присваивания. А лучше весь код. Судя по ошибке где-то с new/delete, наверное, напутали. 0

Croessmah Неэпический 17869 / 10634 / 2054 Регистрация: 27.09.2012 Сообщений: 26,736 Записей в блоге: 1
	17.06.2013, 18:40	6
	Сообщение от Croessmah А лучше весь код. Весь код. Который можно скомпилировать и проверить. 0

	17.06.2013, 19:25