Создать шаблон класса, порождающий динамические одномерные массивы

@Ergo_Proxy · Регистрация: 10.05.2014

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

. Дано: число N и последовательность a1, a2, … aN
Создать шаблон класса, порождающий динамические одномерные массивы с элементами различных типов (вещественные, целочисленные, символьные и т.д.). Тип данных и результат являются параметрами по отношению к классу, программа должна иметь методы инициализации, конструктор, деструктор, метод просмотра значений созданного массива, согласно заданному алгоритму.
(a1*a1), (a1*a2), …, (a1*aN);

Правильно ли я сделал?

Код

C++ (Qt)

#include "stdafx.h"
#include "iostream"
using namespace std;
template <typename T>
class Mas {  
private:
    T N;
    T* mas;
public:
     void set(T N);
     void print();
};
 
template <typename T>
Mas<T>::Mas(T N) {  
    mas = new T [N];
    for(int i(0);i<=N;i++) {   
        cin>>mas[i];
    }
}
 
template <typename T>
Mas<T>::~Mas() {
    delete [] mas;
}
 
template <typename T>
 Mas<T>::Mas()
{}
 
template <typename T>
void Mas<T>::set(T N) {
    mas = new T [N];
    for(int i(0);i<=N;i++)  {   
        cin>>mas[i];
    }
};
 
template <typename T>
void Mas<T>::print() {
    for(int i(0);i<=N;i++)  {   
        cout<<mas[i] = mas[1]*mas[i];
    }
}
 
int _tmain(int argc, _TCHAR* argv[]) {
    Mas<int> a;
    a.set(3);
    a.print();
 
    return 0;
}

@schdub · 21.01.2015, 13:40

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

i<=N

1) везде где так, нужно i < N, иначе чтение за границами выделенной памяти;

2) в функции Mas<T>::set(T) утечка памяти;

3) значение Mas<T>::N не запоминается в Mas<T>::set(T);

4) тип Mas<T>::N должен быть числовым, мы можем объявить массив при T=std::string, например;

5) в конструкторе нужно инициализировать члены класса N и mas.

@Ergo_Proxy · 21.01.2015, 13:49 **[ТС]**

а как тогда избежать утечки памяти в функции set?

@ForEveR · 21.01.2015, 13:53

Ergo_Proxy, Очистить память перед тем как выделять новую.

@Ergo_Proxy · 21.01.2015, 15:24 **[ТС]**

А еще вопрос можно,почему задача нечего не выводит?я изменил функцию print

C++ (Qt)

T Mas<T>::print()
{  T j;
    for(int i = 0; i < N; i++)
    {   j = mas[1]*mas[i];
        return j;
    
    }
    
}

@schdub · 21.01.2015, 15:34

Ergo_Proxy, потому что на первой же итерации цикла вы выходите из функции (return j).
Если вы хотите вывести в консоль j, то std::cout << j;

@Ergo_Proxy · 21.01.2015, 17:44 **[ТС]**

все равно не выводит

@schdub · 21.01.2015, 18:55

Ergo_Proxy, мистика? покажите ваш код полностью.

@Ergo_Proxy · 21.01.2015, 19:04 **[ТС]**

C++ (Qt)

#include "stdafx.h"
#include "iostream"
using namespace std;
template <typename T>
class Mas
{  
   private:
       T N;
       T* mas;
   public:
       void set(T N);
       void print();
       Mas(T N); // 
       Mas();
       ~Mas();
};
template <typename T>
Mas<T>::Mas(T N)
{
    mas = new T [N];
    for(int i = 0; i < N; i++) 
    {   
        cin>>mas[i];
    }
}
template <typename T>
Mas<T>::~Mas()
{
    delete [] mas;
}
 
template <typename T>
 Mas<T>::Mas()
{}
 
template <typename T>
void Mas<T>::set(T N)
{   delete [] mas;
    
    mas = new T [N];
    for(int i = 0; i < N; i++)
    {   
        cin>>mas[i];
 
    }
 
};
 
template <typename T>
void Mas<T>::print()
{  T j;
    for(int i = 0; i < N; i++)
    {   j = mas[1]*mas[i];
        cout<< j;
    
    }
    
}
 
int main(int argc, char* argv[])
{
    Mas<int> a;
    a.set(3);
    a.print();
 
    return 0;
}

@schdub · 21.01.2015, 19:17

Ergo_Proxy, как по-вашему, где инициализируется N и сколько раз выполнится цикл в Mas<T> :: print()?

Перечитайте еще раз, все ли вы выполнили?

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

1) везде где так, нужно i < N, иначе чтение за границами выделенной памяти;
2) в функции Mas<T>::set(T) утечка памяти;
3) значение Mas<T>::N не запоминается в Mas<T>::set(T);
4) тип Mas<T>::N должен быть числовым, мы можем объявить массив при T=std::string, например;
5) в конструкторе нужно инициализировать члены класса N и mas.

@schdub 3071 / 1409 / 425 Регистрация: 19.01.2009 Сообщений: 3,880
	21.01.2015, 13:40	2
	Сообщение от Ergo_Proxy i<=N 1) везде где так, нужно i < N, иначе чтение за границами выделенной памяти; 2) в функции Mas<T>::set(T) утечка памяти; 3) значение Mas<T>::N не запоминается в Mas<T>::set(T); 4) тип Mas<T>::N должен быть числовым, мы можем объявить массив при T=std::string, например; 5) в конструкторе нужно инициализировать члены класса N и mas. 0

@Ergo_Proxy 0 / 0 / 0 Регистрация: 10.05.2014 Сообщений: 15
	21.01.2015, 13:49 [ТС]	3
	а как тогда избежать утечки памяти в функции set? 0

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	21.01.2015, 13:53	4
	Ergo_Proxy, Очистить память перед тем как выделять новую. 0

@schdub 3071 / 1409 / 425 Регистрация: 19.01.2009 Сообщений: 3,880
	21.01.2015, 15:34	6
	Ergo_Proxy, потому что на первой же итерации цикла вы выходите из функции (return j). Если вы хотите вывести в консоль j, то std::cout << j; 0

@Ergo_Proxy 0 / 0 / 0 Регистрация: 10.05.2014 Сообщений: 15
	21.01.2015, 17:44 [ТС]	7
	все равно не выводит 0

@schdub 3071 / 1409 / 425 Регистрация: 19.01.2009 Сообщений: 3,880
	21.01.2015, 18:55	8
	Ergo_Proxy, мистика? покажите ваш код полностью. 0

@schdub 3071 / 1409 / 425 Регистрация: 19.01.2009 Сообщений: 3,880
	21.01.2015, 19:17	10
	Ergo_Proxy, как по-вашему, где инициализируется N и сколько раз выполнится цикл в Mas<T> :: print()? Перечитайте еще раз, все ли вы выполнили? Сообщение от schdub 1) везде где так, нужно i < N, иначе чтение за границами выделенной памяти; 2) в функции Mas<T>::set(T) утечка памяти; 3) значение Mas<T>::N не запоминается в Mas<T>::set(T); 4) тип Mas<T>::N должен быть числовым, мы можем объявить массив при T=std::string, например; 5) в конструкторе нужно инициализировать члены класса N и mas. 0