//kill, username, email, kill count
soldier Bob(1,"Twinsen","Twhj@jher.com",2345);

soldier( int rank, std::string NickName, std::string EmailAddress, int totalKills );

List Players;

Players.insert(Players.begin(),Bob);

error C2664: 'List::insert' : cannot convert parameter 2 from 'soldier' to 'std::string'

Quote:Original post by DrEvil
Looks like your not using std::list, so you'd have to post more information about your list representation to get more help.

/////////This is the declaration inside the list class//////////////////////////////////////// in the header file//void insert(Iterator pos, std::string s);///in list.cpp ///Here is the insert funcctionvoid List::insert(Iterator iter, std::string s){	if(iter.location == NULL)	{		send_back(s);		return;													// everything went ok.	}	Node *after	=	iter.location;	Node *before =	after->backward;	Node *newNode = new Node(s);	newNode->backward	=	before;								// The new Node gets told where to go in the list	newNode->forward	= after;	after->backward = newNode;									// Realign Nodes	if(before ==NULL)	{		first = newNode;	}else	{		before->forward = newNode;	}}

Quote:Original post by skittleo
Look ---

void List::insert(Iterator iter, std::string s)

The second paramater is a string. So why are you trying to send it a soldier?

You have a few options. You could redesign your list object using templates. You could find a way to use strings to represent your soldier. OR, and my favorite, you could use a tried and true list/vector object such as std::vector. You can pick it up in less than a day.

Quote:Original post by DrEvil
I'd vote for std::list/std::vector also. I'd question your ability to write a custom linked list when a very clear compiler error stumps you. No offense intended.

Your insert/push_back/push_front methods should all take whatever type you are trying to store in the list. This why templates keep these kinds of classes generic, by allowing you to store anything you want.

std::list<soldier> mylist;

soldier Bob(1,"Twinsen","Twhj@jher.com",2345);
mylist.push_back(Bob);