шаблонная функция

[zhenya.ya]

У меня такое задание: написать функцию-шаблон, которая переставляет эл-ты в массиве. Правильно ли я это сделал?

template <class T,int n>
void fun()
{
	int m,k,first,second;;
	T *arr = new T [n];
	for(int i=0;i<n;i++){ arr[i] =i;
	cout<<arr[i]<<endl;}cout<<" input first:"<<endl;
	cin>> first;

		for(int i=0;i<n;i++)
			if(arr[i]==first) m=i;

		cout<<" input second:"<<endl;
		cin>>second;

		for(int i=0;i<n;i++)
			if(arr[i]==second) k=i;

		system("cls");
			cout<<" changing..."<<endl;
			arr[m]=second;
			arr[k]=first;
			for(int i=0;i<n;i++)
			{
				cout<<arr[i]<<endl;;
			}
delete []arr;		
system("pause");
}
void main()
{
	fun<float,5>();			
}

[onewho]

работает то правильно. но сделано плохо. функция не должна выполнять работу по вводу выводу.
пусть будет такой прототип
template <typename TYPE> void func (TYPE *arr, int size);

а ввод вывод снаружи

[zhenya.ya]

да,точно, так будет лучше, большое спасибо!

[Vago]

1. Чему равно m, если элемента first не окажется в массиве?
2. Чему равно k, если элемента second не окажется в массиве?

[zhenya.ya]

А мне сказали нужно сделать через swap() ((
Правильно ли я попытался?

void swap( int &a, int &b )
{
    int  temp    = a;
    a = b;
    b = temp;
}
template <class T>
void fun( T *M, int n )
{
	 cout<<"\n";
  swap(M[2],M[3]);
  for(int i=0;i<n;i++) cout<<M[i]<<endl;
}
int main()
{    
	int n=5;
	char *M = new char [n];
	for( int i = 0; i < n; i++ ){ M[i]=i;cout << M[i]<<endl;}
    
    fun<char>(M,n);
	system("pause");
    return 0;
}

[onewho]

абсолютно верно, осталось тока вывод сделать вне функции, и передачу 2 чисел которые меняем как параметров

[onewho]

ах да, вы переписали уже существующую функцию
template <class T> void swap ( T& a, T& b );
подключайте заголовок <algorithm>
и юзайте без переопределения на здоровье

[Vago]

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

абсолютно верно

Ой ли?!...
110225.jpg
(M$ Visual Studio 2008 EE & Visual Studio 6).

Так бесстрашно автор переходит от int к char, а потом - к int&, а потом - опять к char... Аж завидно!..

[zhenya.ya]

подключайте заголовок <algorithm>

А как это может мне помочь? (Я с этим еще не знаком)

Ой ли?!...Так бесстрашно автор переходит от int к char, а потом - к int&, а потом - опять к char... Аж завидно!..

Запутали! И чему тут завидовать??? А как нужно было сделать?

[onewho]

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

Ой ли?!...
Вложение 36372
(M$ Visual Studio 2008 EE & Visual Studio 6).

Так бесстрашно автор переходит от int к char, а потом - к int&, а потом - опять к char... Аж завидно!..

ну да я не обратил внимание что функция void swap( int &a, int &b )
у автора не шаблонная.
а что выдает ромбики всякие так это просто дело в записи массива M[i]=i;