Cписки с++

[paravoz1k]

Приветствую. Разработал список, который должен выводить список, элементы которого добавляются в конец. Но он не выводит список элементов, не пойму почему. Прошу проверить!

//---------------------------------------------------------------------------

#include <iostream.h>
#include <conio.h>
#include <vcl.h>
#pragma hdrstop

#pragma argsused

struct link
{
	int data;
	link* next;
};
  /////////////////////////////////////
	/////////////////////////////////////
	  /////////////////////////////////////
class linklist
{
	private:
	link* first;
	link* start;
	public:
		linklist ()
				{first = NULL; start = NULL;}  //first - дополнительный элемент, start - первый элемент списка

	void additem (int d);
	void display ();
	void additemperednomerom ();
   };
   /////////////////////////////////////
	 /////////////////////////////////////
	void linklist::additem( int d )
	{
	  link *newlink = new link;      // отводим память
	  if (start = NULL) { //если стартовый равен нулю
	  start  = newlink ;        //делаем стартовый текущий
	  newlink->data = d;          // и заполняем в него раздел дата
	  first = newlink;            // в дополнительный кидаем  наш предыдущий номер, для того чтобы потом заполнить next.
	  }
	  if (start != NULL) // теперь же если у нас есть стартовый элемент
	  {
	  newlink->data = d;    // в отведенную память кидаем значение
	  newlink->next = NULL;    // Cледующей номер в текущей структуре делаем 0. Он заполниться при добавлении следущего элемента
	  first->next = newlink;  // а вот и заполняем значение прошлой структуры(элемента), отводим ссылку на данный элемент
	   first = newlink ;        // теперь значение предыдущего принимает за текущее.
	  }
	}
	/////////////////////////////////////
	   /////////////////////////////////////
	   /////////////////////////////////////
		void linklist::display( )
	{
   link *current = start; //начинаем со стартого элемента
   while( current )       // пока не дойдем до элемента NULL
   {
	   cout <<current->data << endl;  // выводим его значение
	   current = current->next;       // и ставим следущее следующим.

   }
	}

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

int _tmain(int argc, _TCHAR* argv[])
{
linklist li;
li.additem(27);
li.additem(28);
li.additem(29);
li.additem(30);

 li.display();
getch();
}
//---------------------------------------------------------------------------

[paravoz1k]

Пояснил мой ход, чтобы небыло лишних вопросов.

[Zer0]

для начала видится мне, что нужно перед
if (start != NULL) // теперь же если у нас есть стартовый элемент
поставить else

[paravoz1k]

для начала видится мне, что нужно перед
if (start != NULL) // теперь же если у нас есть стартовый элемент
поставить else

Так помоему смысл не меняет, вначале обрабатываем если стартового нет, а затем если стартовый есть....ведь 3его не предусмотрено

[Zer0]

если else не вписать, то если нету - он его создаст. Т.к. слова "ИНАЧЕ" нету, то проверка на start!=NULL будет истинной и программа запустит уже не то что Вам надо

[paravoz1k]

если else не вписать, то если нету - он его создаст. Т.к. слова "ИНАЧЕ" нету, то проверка на start!=NULL будет истинной и программа запустит уже не то что Вам надо

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

[Zer0]

я сейчас могу ошибиться, но
link *newlink = new link; // отводим память
Вроде бы нужно по другому
link* newlink = new link; // отводим память
аналогично с current в display
А вообще бы поковырять отладчик, думаю сразу станет видно где ошибка

[paravoz1k]

Блин, жесткач господа, последствия программирования на делфи дают о себе знать)
Моя ошибка:

if (start = NULL)

AAAA. должно быть:

if (start == NULL)