C++ pass list as a parameter to a function

Adrian picture Adrian · Feb 15, 2012 · Viewed 47.4k times · Source

I'm trying to build a very simple address book. I created a Contact class and the address book is a simple list. I'm trying to build a function to allow the user to add contacts to the address book. If I take my code outside of the function, it works OK. However, if I put it in, it doesn't work. I believe it's a passing by reference vs passing by value problem which I'm not treating as I should. This is the code for the function:

void add_contact(list<Contact> address_book)
{
     //the local variables to be used to create a new Contact
     string first_name, last_name, tel;

     cout << "Enter the first name of your contact and press enter: ";
     cin >> first_name;
     cout << "Enter the last name of your contact and press enter: ";
     cin >> last_name;
     cout << "Enter the telephone number of your contact and press enter: ";
     cin >> tel;

     address_book.push_back(Contact(first_name, last_name, tel));
}

I don't get any errors however when I try to display all contacts, I can see only the original ones.

Answer

ildjarn picture ildjarn · Feb 15, 2012

You're passing address_book by value, so a copy of what you pass in is made and when you leave the scope of add_contact your changes are lost.

Pass by reference instead:

void add_contact(list<Contact>& address_book)