С++ Сортировка массива и вывод количества элементов на чётных позициях с использованием шаблонов

[Desolar]

Здравствуйте! Проблема в том, что не могу разобраться именно с шаблонами, в задание указано : " Шаблон класса должен описывать и обеспечивать набор действий над данными параметризованного массива, а все вычисления должны быть реализованы в виде методов, ввод размера с клавиатуры".
Функции для сортировки и вывода количества элементов на чётных позициях я написал правильно, отдельно проверял.
В остальном ничего не выходит, буду благодарен если поможете мне разобраться и довести программу до ума.

#include <iostream>
using namespace std;

template <class T, size_t size>
class Arr {
private:
	T *arr, count;
	public: 
		Arr(T count_) {
			arr = new T[size];
			count = count_;
		}
		~Arr() {
			delete[] arr;
		}
		
		T Sum();
		T Sort();
		T par();
};
template <class T>
T Arr<T>::Sum() {
	int sum = 0;
	while (N > 0)
	{
		sum += N % 10;
		N /= 10;
	}
	return sum;
}

template <class T>
T Arr<T>::Sort() {
	for (int i = 0; i < N; i++)
	{
		for (int j = N - 1; j > i; j--)
		{
			if (Sum(arr[j]) > Sum(arr[j - 1]))
			{
				swap(arr[j], arr[j - 1]);
			}
		}
	}
	cout << "Sorted arr:" << endl;
	for (int i = 0; i < N; i++)
	{
		cout << arr[i] << " ";
	}
	cout << endl;
}

template <class T>
T Arr<T>::par() {

	for (int i = 0; i<N; i += 2)
		count++;
	cout << "number of Arr elements:" << count << endl;

}

int main()
{
	Arr <int, 0> obj();
	int N, count=0;
	cout << "Enter arr size:" << endl;
	cout << "N = ";
	cin >> N;
	int *arr = new int[N];
	cout << "Enter arr:" << endl;
	for (int i = 0; i < N; i++)
	{
		cin >> arr[i];
	}

	obj.Sort(); //?
	
	obj.par(); //?

	system("pause");
	return 0;
}

[Cuprum5]

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

template <class T, size_t size>

- а зачем size указывать в параметрах шаблона. Там указываются только типы, которые будут использоваться внутри класса - это же Вам не функция.

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

Arr(T count_){
    arr = new T[size];
    count = count_;
}

- Может быть count и size это одно и то же? - Я прав?:D
А, да. Если все-таки я прав, то count это должно быть int, а тип T используется только для создания массива.

А что для каждой функции тоже надо template писать, я думаю что нет - там Вы не создаете шаблоны. У Вас уже есть декларации функций - Вам только нужно их описать. Не нужно писать template перед функциями. Докажите обратное. Или это требует C++?
Намешали все в кашу. Сделайте Ваш код попроще и попонятнее.

[Desolar]

Думал, что нужно указывать size для передачи размера массива, count - это подсчёт количества элементов массива на чётных позициях.

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

template <class T>
T Arr<T>:: par() {

for (int i = 0; i<N; i += 2)
count++;
cout << "number of Arr elements:" << count << endl;

}

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

[Desolar]

Честно говоря, я вообще запутался, что мне делать. Особенно интересно можно ли передать элементы и размер массива здесь, если да, то как, плюс как же вызвать эти методы.

[Desolar]

Немножко подправил, кто может сказать стало ли лучше? Но всеровно не работает.

#include <iostream>
using namespace std;

template <class T>
class Arr {
private:
	T *arr;
	int N, count;
	public: 
		Arr() {
			N = 0;
			count = 0;
			arr = new T[N];
		
			
		}
		~Arr() {
			delete[] arr;
		}
		
		Arr(int N1);

		int Sum();
		void Sort();
		void par();

};

template <class T>
Arr<T>::Arr(int N1) {
	arr = new T[N];
	int N = 0;
	cout << "Enter arr size:" << endl;
	cout << "N = ";
	cin >> N;
	
	cout << "Enter arr:" << endl;
	for (int i = 0; i < N; i++)
	{
		cin >> arr[i];
	}

}




template <class T>
Arr<T>::Sum() {
	int sum = 0;
	while (N > 0)
	{
		sum += N % 10;
		N /= 10;
	}
	return sum;
}


template <class T>
Arr<T>::Sort() {
	for (int i = 0; i < N; i++)
	{
		for (int j = N - 1; j > i; j--)
		{
			if (Sum(arr[j]) > Sum(arr[j - 1]))
			{
				swap(arr[j], arr[j - 1]);
			}
		}
	}
	cout << "Sorted arr:" << endl;
	for (int i = 0; i < N; i++)
	{
		cout << arr[i] << " ";
	}
	cout << endl;



}

template <class T>
Arr<T>::par() {

	for (int i = 0; i<N; i += 2)
		count++;
	cout << "number of Arr elements:" << count << endl;

}





int main()
{
	Arr <int> obj;
	int N;


	
	obj.Sort(); 
	
	obj.par(); 
	
	
	
	system("pause");
	return 0;
}

[Cuprum5]

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

Особенно интересно можно ли передать элементы и размер массива здесь, если да, то как, плюс как же вызвать эти методы.

- Элементы передаются по указателю из другого массива, который заранее заготовлен. Размер передается просто числом.

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

Немножко подправил, кто может сказать стало ли лучше? Но всеровно не работает.

- А что не работает-то? Ну передавайте число элементов в массиве в конструктор Arr или там как-то по-другому надо сделать?

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

template <class T>
T Arr<T>::Sum(){
	int sum = 0;
	while (N > 0)
	{
		sum += N % 10;
		N /= 10;
	}
	return sum;
}

- я не понял вот этот участок кода. Здесь же ведь не находится сумма элементов массива - вообще бред какой-то!!!!!!!!!!!!!!!!!:D Что Вы хотели сказать этим кодом? Также переменная N не имеет инициализации - это плохо там будет мусор и программа будет творить вообще неизвестно что.

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

template <class T>
Arr<T>::Arr(int N1){
	arr = new T[N];
	int N = 0;
	cout << "Enter arr size:" << endl;
	cout << "N = ";
	cin >> N;

- Вот это глупость!!!!!!!!!!!! Сначала создаете динамический массив, а потом уже спрашиваете его размер у Пользователя и заполняете его - в результате расход памяти. - Нужно делать наоборот.
Т.е. спрашиваете сначала размер массива у Пользователя, а потом создаете массив этого размера и заполняете его либо с консоли, либо случайными значениями.

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

Arr<int> obj;

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

[Desolar]

Спасибо, попробую.
Возможно, вас запутало название этой функции :D Она просто необходима, для функции сортировки по убыванию.

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

- Вот это глупость!!!!!!!!!!!! Сначала создаете динамический массив, а потом уже спрашиваете его размер у Пользователя и заполняете его - в результате расход памяти. - Нужно делать наоборот.
Т.е. спрашиваете сначала размер массива у Пользователя, а потом создаете массив этого размера и заполняете его либо с консоли, либо случайными значениями.

О, честно говоря, просто не заметил, что так написал, спасибо.

[Desolar]

Список ошибок, не вижу где их нужно исправить. Потому что, вроде бы всё есть.

[Cuprum5]

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

[Desolar]

Попробуйте приблизить, если не сложно. Раньше он так же не компилировался.