views:

71

answers:

2

I'm not too familiar with c++ and how instantiating objects work, so this is probably a very simple thing to solve. When I compile with g++ I get the error " undefined reference to 'Foo::Foo(std::string)' ". I want to create an instance of the class Foo that has a string parameter in its constructor. Here is the code:

Foo.h

#include <string>
using namespace std;

class Foo
{
    public:
        Foo(string s);

    private:
        string id;
};

Foo.cpp

#include <string>
#include "Foo.h"
using namespace std;

Foo::Foo(string s)
{
    id = s;
}

main.cpp

#include <string>
#include "Foo.h"
using namespace std;

int main()
{
    Foo foo("bar");

    return 0;
}
+14  A: 

You're probably not including Foo.cpp in your compile line. It should look something like this:

g++ main.cpp Foo.cpp -o testFoo
Fred Larson
Well, it's the lesson I get for compiling command line in a language I'm unfamiliar with; thanks for saving me even more time hunting this down :)
Aaron
+1  A: 

Not related to the problem you were having but consider making a couple minor changes:

  1. Pass the argument in a const reference. const because you do not plan on changing the value of the argument and reference so that you do not create any additional temporary objects.

  2. C++ has an initializer concept that is more efficient then using the assignment operator on member 'id' in the body of the constructor. The current version of the constructor will call member id's default constructor and then its assignment constructor. The initializer concept (i.e. 'id(s)') will just call one method the copy constructor.

    Foo::Foo(const string& s) : id(s)
    {
    }

skimobear
The initializer concept is interesting, thanks for the suggestion.
Aaron