Класс-шаблон создание конструктора по умолчанию

@BabaSveta · Регистрация: 17.10.2014

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

Хочу создать класс-шаблон. Начал с создания конструктора по умолчанию, с этого же момента и началась проблема, причём такая, что смысл её даже не понятен)))

Почему-то не создаёт Elements <int> A;
В чём может быть проблема?
С меня "спасибо" и "лучший ответ" за правильно данный ответ.

Вот код программы:

C++

//файл main.cpp
#include <iostream>
#include <locale>
#include "Elements.h"
 
int main()
{
    Elements <int> A;
    //A.Add(5.);
 
    return 0;
}

C++

//файл Elements.h
#pragma once
template <class T> 
class Elements
{
    //T Array;      //массив
    int razmer;     //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *, int);          //Конструктор
    void ShowArray();   //показать весь массив
    void Add(T);        //Добавить элемент в массив
    void ShowElement(); //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    //T operator=(&T);
    //T operator*(&T, &T);
    ~Elements();
};

C++

//файл Elements.cpp
#include "Elements.h"
 
const int n = 10;
 
template <class T>
Elements<T>::Elements()
{
    name = new char[n]
    name = "NoneName";
    int razmer = 10;
    T Array[n];
    count = 0;
}

Вот ошибка, которая возникает при компиляции, при нажатии на неё ни на что ни ссылается...

@Gr1f0nn · 11.04.2015, 18:08

BabaSveta, в данном случае, определение конструктора (и других public функций) должно быть в .h ,а не в .cpp

@DrOffset · 11.04.2015, 18:10

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

@BabaSveta · 11.04.2015, 18:43 **[ТС]**

перенёс, всё равно ошибка..

C++

//файл main.cpp
#include <iostream>
#include <locale>
#include "Elements.h"
 
int main()
{
    Elements <int> A;
    //A.Add(5.);
 
    return 0;
}

C++

//файл Elements.h
#pragma once
template <class T> 
class Elements
{
    //T Array;      //массив
    int razmer;     //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *, int);          //Конструктор
    void ShowArray();   //показать весь массив
    void Add(T);        //Добавить элемент в массив
    void ShowElement(); //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    //T operator=(&T);
    //T operator*(&T, &T);
    ~Elements();
};
 
 
 
const int n = 10;
 
template <class T>
Elements<T>::Elements()
{
    name = new char[n];
    name = "NoneName";
    int razmer = 10;
    T Array[n];
    count = 0;
}

@DrOffset · 11.04.2015, 18:48

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

перенёс, всё равно ошибка..

Нужно перенести все методы, а не только конструктор.

@BabaSveta · 11.04.2015, 18:56 **[ТС]**

перенёс абсолютно все, и даже удалил файл Elements.cpp, но всё равно возникает ошибка..

C++

//файл main.cpp
#include <iostream>
#include <locale>
#include "Elements.h"
 
int main()
{
    Elements <int> A;
    //A.Add(5.);
 
    return 0;
}

C++

//файл Elements.h
#pragma once
template <class T> 
class Elements
{
    //T Array;      //массив
    int razmer;     //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *, int);          //Конструктор
    void ShowArray();   //показать весь массив
    void Add(T);        //Добавить элемент в массив
    void ShowElement(); //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    //T operator=(&T);
    //T operator*(&T, &T);
    ~Elements();
};
 
 
 
const int n = 10;
 
template <class T>
Elements<T>::Elements()
{
    name = new char[n];
    name = "NoneName";
    int razmer = 10;
    T Array[n];
    count = 0;
}
 
//template <class T>
//Elements::Elements(char *Name, int Razmer)
//{
//  name = new char[n];
//  strcpy(Name,name);
//
//  razmer = Razmer;
//  count = 0;
//
//  T Array[razmer];
//}
 
//template <class T>
//Elements<T>::Add(T  a)
//{
//  if(count < razmer)
//  {
//      Array[count] = a;
//      count++;
//  }
//  else
//      cout << "Массив переполнен" << endl;
//}

ошибка ни на что не ссылается..

@DrOffset · 11.04.2015, 19:04

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

перенёс абсолютно все, и даже удалил файл Elements.cpp, но всё равно возникает ошибка..

Что-то ты мудришь. В приведенном фрагменте не видно ни реализации конструктора, ни остальных методов. Как ты мог их перенести, если их нет? Естественно он их не находит при компоновке.

C++

//файл Elements.h
#pragma once
template <class T>
class Elements
{
    //T Array;      //массив
    int razmer;     //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *, int);          //Конструктор
    void ShowArray();   //показать весь массив
    void Add(T);        //Добавить элемент в массив
    void ShowElement(); //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    //T operator=(&T);
    //T operator*(&T, &T);
    ~Elements();
};
 
const int n = 10;
 
template <class T>
Elements<T>::Elements()
{
    name = new char[n];
    name = "NoneName";
    int razmer = 10;
    T Array[n];
    count = 0;
}
 
template <class T>
Elements<T>::Elements(char *Name, int Razmer)
{
    //добавить реализацию
}
 
template <class T>
void Elements<T>::Add(T a)
{
    //добавить реализацию
}
 
template <class T>
Elements<T>::~Elements()
{
    //добавить реализацию
}

Остальные методы тоже нужно сделать, само собой.

@BabaSveta · 11.04.2015, 19:06 **[ТС]**

всё, решилось! Спасибо!))) Сейчас проставлю вам "Спасибо"!

@BabaSveta · 11.04.2015, 20:57 **[ТС]**

При добавлении элемента с помощью метода Add выводит ошибку, смысл её понять не могу, уже менял тип принимаемых значений и тип класса, всё равно одно и то же...

C++

//файл main.cpp
#include <iostream>
#include <locale>
#include "Elements.h"
 
int main()
{
    Elements <int> A;
    Elements <float> B("название");
    //int a= 5;
    A.Add(5);
 
    return 0;
}

C++

//файл Elements.h
#pragma once
#include <iostream>
 
using namespace std;
const int n = 10;
 
template <class T> 
class Elements
{
    T Array[n];     //массив
    //int razmer;       //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *);           //Конструктор
    //void ShowArray(); //показать весь массив
    void Add(T &);      //Добавить элемент в массив
    //void ShowElement();   //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    //T operator=(&T);
    //T operator*(&T, &T);
    //~Elements();
};
 
 
 
template <class T>
Elements<T>::Elements()
{
    name = new char[n];
    name = "NoneName";
    //int razmer = 10;
    T Array[n];
    count = 0;
}
 
template <class T>
Elements<T>::Elements(char *Name)
{
    name = new char[n];
    strcpy(name,Name);
 
    count = 0;
 
    T Array[n];
}
 
template <class T>
Elements<T>::Add(T &a)
{
    if(count < razmer)
    {
        Array[count] = a;
        count++;
    }
    else
        cout << "Массив переполнен" << endl;
}

Нажимая на ошибку, программа ссылается на 59-ю строчку файла Elements.h

@DrOffset · 11.04.2015, 21:11

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

Нажимая на ошибку, программа ссылается на 59-ю строчку файла Elements.h

Тип возвращаемого значения добавь. В моем примере выше это сделано, кстати.

@BabaSveta · 12.04.2015, 16:23 **[ТС]**

кажется не так прописываю оператор присваивания. Где именно ошибка?

C++

//файл Elements.h
#pragma once
#include <iostream>
 
using namespace std;
const int n = 10;
 
template <class T> 
class Elements
{
    T Array[n];     //массив
    //int razmer;       //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *);   //Конструктор явный
    void ShowArray();   //показать весь массив
    void Add(T);        //Добавить элемент в массив
    void ShowElement(const T);  //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    Elements operator=(Elements <T> &obj);  //оператор присваивания
    //T operator*(&T, &T);
    //~Elements();
};
 
 
template <class T>
Elements Elements<T>::operator=(const Elements<T> &obj)
{
    if(this==&obj)
        return *this;
    else
        count = obj;
}

сам оператор ещё не дописал, т.к. даже не могу обратиться к obj...
Почему не могу обратиться, не понимаю..

@DrOffset · 12.04.2015, 16:30

BabaSveta, Банальная невнимательность.
В классе (забыл const у параметра, возвращать надо ссылку):

C++

1	Elements & operator=(const Elements &obj); //оператор присваивания

В определении (параметр шаблона у типа возвращаемого значения нужен, а вот у аргумента как раз необязателен)

C++

template <class T>
Elements<T> & Elements<T>::operator=(const Elements &obj)
{
    if(this != &obj)
        count = obj;
 
    return *this;
}

@BabaSveta · 12.04.2015, 16:52 **[ТС]**

Спасибо! Фиг поймёшь где его ставить

@BabaSveta · 13.04.2015, 14:31 **[ТС]**

При копировании атрибута name в операторе присваивания возникает такая ошибка

C++

//файл Elements.h
#pragma once
#include <iostream>
 
using namespace std;
const int n = 10;
 
template <class T> 
class Elements
{
    T Array[n];     //массив
    //int razmer;       //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *);   //Конструктор явный
    void ShowArray();   //показать весь массив
    void Add(T);        //Добавить элемент в массив
    void ShowElement(const T);  //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    Elements & operator=(const Elements &obj);  //оператор присваивания
    //T operator*(&T, &T);
    //~Elements();
};
 
template <class T>
Elements<T> & Elements<T>::operator=(const Elements &obj)
{
    if(this==&obj)
        return *this;
    else{
        count = obj.count;
        
        for(int i=obj.count-1; i>=0; i--){
            Array[i] = obj.Array[i];
        }
        strcpy(name, obj.name);                //ВОТ ЗДЕСЬ ВЫХОДИТ ЭТА ОШИБКА
    }
}

Ставлю "спасибо" и "лучший ответ", если кто-нибудь укажет на мою ошибку..

@aLarman · 13.04.2015, 14:36

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

strcpy(name, obj.name);

под name не выделена память

C++

1	name = new char[strlen(obj.name) + 1];

@BabaSveta · 13.04.2015, 14:45 **[ТС]**

но в конструкторе по умолчанию и в явном конструкторе я её выделял... Ещё раз выделять? Что-то тогда ничего не могу понять....

C++

//файл Elements.h
#pragma once
#include <iostream>
 
using namespace std;
const int n = 10;
 
template <class T> 
class Elements
{
    T Array[n];     //массив
    //int razmer;       //какой должен быть размер массива
    int count;      //кол-во заполненных элементов в массив Array
    char *name;     //название множества
public:
    Elements();         //Конструктор по умолчанию
    Elements(char *);   //Конструктор явный
    void ShowArray();   //показать весь массив
    void Add(T);        //Добавить элемент в массив
    void ShowElement(const T);  //Посмотреть, есть ли в массиве Array тот элемент, который мы запрашиваем
    Elements & operator=(const Elements &obj);  //оператор присваивания
    Elements(const Elements &obj);//конструктор копирования
    //T operator*(&T, &T);
    //~Elements();
};
 
 
 
template <class T>
Elements<T>::Elements()
{
    name = new char[n];
    name = "NoneName";
    //int razmer = 10;
    T Array[n];
    count = 0;
}
 
template <class T>
Elements<T>::Elements(char *Name)
{
    name = new char[n];
    strcpy(name,Name);
 
    count = 0;
 
    T Array[n];
}
 
template <class T>
void Elements<T>::Add(T a)
{
    if(count < n)
    {
        Array[count] = a;
        count++;
    }
    else
        cout << "Массив переполнен" << endl;
}
 
 
template <class T>
void Elements<T>::ShowArray()
{
    cout << this->name << endl;
    for(int i = count-1; i>=0; i--){
        cout << Array[i] << " ";
    }
    cout << endl << endl;
}
 
template <class T>
void Elements<T>::ShowElement(T seach)
{
    for(int i = count-1; i>=0 ;i--)
    {
        if(seach==Array[i]){
            cout << "Элемент " << seach <<" в массиве " << "'" << this->name << "'" <<  " найден" << endl;
            return;
        }
    }
    cout << "Элемент в массиве не найден.." << endl;
}
 
template <class T>
Elements<T> & Elements<T>::operator=(const Elements &obj)
{
    if(this==&obj)
        return *this;
    else{
        count = obj.count;
        
        for(int i=obj.count-1; i>=0; i--){
            Array[i] = obj.Array[i];
        }
        strcpy(name, obj.name);
    }
}
 
//Elements(const Elements &obj);//конструктор копирования
 
template <class T>
Elements<T>::Elements(const Elements &obj)
{
    name = new char[n];
    strcpy(name,obj.name);
 
    count = obj.count;
    T Array[n];
 
    for(int i=obj.count-1; i>=0; i--){
            Array[i] = obj.Array[i];
    }
}

C++

//файл main.cpp
#include <iostream>
#include <locale>
#include "Elements.h"
 
int main()
{
    setlocale(LC_ALL, "rus");
 
    Elements <int> A("asd");
    Elements <double> B("название");
    Elements <int> C;
    A.Add(5); A.Add(6); A.Add(7); A.Add(8);
    B.Add(1.); B.Add(3.7); B.Add(3.8); B.Add(3.2);
    C=A;
    A.ShowArray();
    B.ShowArray();
 
    B.ShowElement(3.8);
    C.ShowArray();
    system("pause");
    return 0;
}

@aLarman · 13.04.2015, 14:59

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

name = new char[n];
name = "NoneName";

угу Выделили, а потому указатель поменяли на указатель в памяти под константный строковый литерал
хотя бы так сделайте

C++

1 2	name = new char[n]; strcpy(name, "NoneName");;

@DrOffset · 13.04.2015, 15:24

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

T Array[n];

Кстати, вот эта запись в каждом конструкторе бессмысленна в данном контексте. Ты определяешь локальный массив, который уничтожится по выходу из конструктора. И что это дает? Ничего.

@aLarman · 13.04.2015, 15:34

DrOffset, ну так то по идее при присваивании надо name удалять(потому что даже в деф.констркуторе выделяется память), косяком много...

@DrOffset · 13.04.2015, 15:42

Не по теме:

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

ну так то по идее при присваивании надо name удалять(потому что даже в деф.констркуторе выделяется память), косяком много...

Ты это не мне говори :) А автору. Я не лезу в это, раз уж ты взялся помогать, вот и объясняй :)

Класс-шаблон создание конструктора по умолчанию

Решение

Решение

Решение

@Gr1f0nn 244 / 164 / 133 Регистрация: 30.09.2012 Сообщений: 690
	11.04.2015, 18:08	2
	BabaSveta, в данном случае, определение конструктора (и других public функций) должно быть в .h ,а не в .cpp 1

@DrOffset 18842 / 9841 / 2408 Регистрация: 30.01.2014 Сообщений: 17,284
	11.04.2015, 18:10	3
	BabaSveta, реализацию конструктора, как и остальных методов шаблона, надо перенести в h-файл. Соль в том, что шаблон должен быть доступен в точке использования полностью, чтобы компилятор смог сгенерировать по нему конкретную реализацию. 0

@DrOffset 18842 / 9841 / 2408 Регистрация: 30.01.2014 Сообщений: 17,284
	11.04.2015, 18:48	5
	Сообщение было отмечено BabaSveta как решение Решение Сообщение от BabaSveta перенёс, всё равно ошибка.. Нужно перенести все методы, а не только конструктор. 1

@BabaSveta 1 / 1 / 1 Регистрация: 17.10.2014 Сообщений: 153
	11.04.2015, 19:06 [ТС]	8
	всё, решилось! Спасибо!))) Сейчас проставлю вам "Спасибо"! 0

@DrOffset 18842 / 9841 / 2408 Регистрация: 30.01.2014 Сообщений: 17,284
	11.04.2015, 21:11	10
	Сообщение от BabaSveta Нажимая на ошибку, программа ссылается на 59-ю строчку файла Elements.h Тип возвращаемого значения добавь. В моем примере выше это сделано, кстати. 0

@BabaSveta 1 / 1 / 1 Регистрация: 17.10.2014 Сообщений: 153
	12.04.2015, 16:52 [ТС]	13
	Спасибо! Фиг поймёшь где его ставить 0

@DrOffset 18842 / 9841 / 2408 Регистрация: 30.01.2014 Сообщений: 17,284
	13.04.2015, 15:24	18
	Сообщение от BabaSveta T Array[n]; Кстати, вот эта запись в каждом конструкторе бессмысленна в данном контексте. Ты определяешь локальный массив, который уничтожится по выходу из конструктора. И что это дает? Ничего. 1

@aLarman 654 / 575 / 164 Регистрация: 13.12.2012 Сообщений: 2,124
	13.04.2015, 15:34	19
	DrOffset, ну так то по идее при присваивании надо name удалять(потому что даже в деф.констркуторе выделяется память), косяком много... 0

@DrOffset
	13.04.2015, 15:42 Класс-шаблон создание конструктора по умолчанию #20
	Не по теме: Сообщение от aLarman ну так то по идее при присваивании надо name удалять(потому что даже в деф.констркуторе выделяется память), косяком много... Ты это не мне говори :) А автору. Я не лезу в это, раз уж ты взялся помогать, вот и объясняй :) 0