C++ Списки

[sir.andrey]

Помогите пожалуйста исправить ошибки:

Две функции: создание массива и вывод на экран
Но они работают не правильно, подскажите пожалуйста где ошибки.

//Создание динамического массива размером "n" и заполнение его случайными числами
void CStack::adding( int n ) 
{
	//Создание первого элемента//
	link* newlink=new link;

	newlink->next=NULL;
	newlink->data=rand();		//Заполняем случайным числом
	newlink->prev=NULL;
	first=newlink;				// Указывает на первый элемент				
	buffer=newlink;				// Вспомогательная переменная
	last=newlink;				// Указывает на последний элемент

	//Создание последущих элементов
	for( int i=1; i<n; i++ ){
		link* newlink=new link;
		
		buffer->next=newlink;
		newlink->prev=buffer;
		newlink->data=rand();
		newlink->next=NULL;
		buffer=newlink;
		last=newlink; 
	}

}

//////////////////////////////////////////////////////////////////////

//Вывод динамического массива на экран
void CStack::showing()
{
	link* current=first;

	cout << "Massiv: ";
	current=first;
	while( current != NULL ){
		cout << current->data << " ";
		current=current->next;
	}
}

[Stilet]

А почему ты решил что работа не верна?

[sir.andrey]

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

А почему ты решил что работа не верна?

Потому что, это я баран в main забыл добавить ф-цию вывода

И заодно еще один вопросик: rand всегда выводит числа с возрастанием?
И такие здоровые (четырехзначные).

[sir.andrey]

И еще вопрос, можно ли то, что я сделал, сделать менее громоздко в плане кода?
Если можно, то как?

[sir.andrey]

У нас завелся знаток-интеллектуал!

[Stilet]

сделать менее громоздко в плане кода?

А по -моему это не такой плохой код... Я видал и подлинее

[sir.andrey]

Посмотрите пожалуйста, почему не работает разрушитель динамического массива?
first & last - первый и последний эл-т.
Начинаем удалять с последнего.

void CStack::destroy()
{
	for( int i=0; i<n; i++ ){
		if( last==first ){  //Удаление первого эл-та
			delete last;
			last=NULL;
			first=NULL;
			buffer=NULL;
		}
		else{
			last->prev=buffer;
			buffer->next=NULL;
			last->prev=NULL;
			delete last;
			last=buffer;
		}
	}
}

[Раввин]

Таки это у тебя массив, связанный список или стек? Вижу в нём признаки всех троих.

[sir.andrey]

Сообщение от Раввин

Таки это у тебя массив, связанный список или стек? Вижу в нём признаки всех троих.

Скорее это двусвязный список

[sir.andrey]

Помогите пожалуйста подправить ф-цию для деструктора!!!
По моему все сделано верно, но блин не работает!