Linking error

[Kevin Grigorenko]

Okay, everything seems simple enough but I get the following Link error:

TeXt.obj : error LNK2001: unresolved external symbol "private: static char *
TextDB::TextDB: :myVersion" (?myVersion@Tex tDB@1@0PADA)

My environment is Visual Studio .NET, I think it's C++ 7 from the microsoft
visual studio versioning point of view.

Here is TeXt.h:

// Annoying MSVC++ warning:
#pragma warning( disable : 4290 )

// TeXt - Standalone Pure Text Database Management System

#ifndef TEXT_DB
#define TEXT_DB

#define TEXT_MAJOR_REVI SION 1
#define TEXT_MINOR_REVI SION 0
#define TEXT_BUILD 0

/*
Revision History:
-----------------

1.0.1

---------
- Created. I've run into too many situations
where something like this would have been
useful.
*/

//#include <bool.h>
#include <string>
#include <stdexcept>
/*

// If no bool.h, uncomment:
typedef short int bool;
#ifndef false
#define false 0
#endif
#ifndef true
#define true 1
#endif
*/

namespace TextDB
{
class TextDB
{
public:
// Constructors/Destructor:
TextDB(const std::string filename) throw(std::inva lid_argument);
~TextDB();
TextDB(const TextDB &copy) throw(std::inva lid_argument);

static const char *getVersion();
private:
// Private because it is illogical to instantiate a
// TextDB object without a filename
TextDB();

void initialize(std: :string in_filename)
throw(std::inva lid_argument);

std::string filename;
static char myVersion[20];
}; // class TextDB
} // namespace TextDB

#endif // #ifndef TEXT_DB

AND then here is TeXt.cpp:

//#include <bool>
#include <string>
#include <fstream>
#include "TeXt.h"
namespace TextDB
{
TextDB::TextDB( const std::string filename)
{
initialize(file name);
}

TextDB::~TextDB ()
{
}

TextDB::TextDB( const TextDB &copy)
{
initialize(copy .filename);
}

const char *TextDB::getVer sion()
{
return myVersion;
}

void TextDB::initial ize(std::string in_filename)
{
sprintf(myVersi on, "%d.%d.%d", TEXT_MAJOR_REVI SION,
TEXT_MINOR_REVI SION, TEXT_BUILD);
std::ifstream infile(filename .c_str(), std::ios_base:: in);
if(!infile.is_o pen())
{
throw std::invalid_ar gument("Could not open database file <" +
filename + ">");
}
this->filename = in_filename;
infile.close();
} // initialize()
} // namespace TextDB

I'm sure the answer is simple, but I can't see it obviously.

Thank you very much!

Kevin Grigorenko

[Ron Natalie]

"Kevin Grigorenko" <kz****@psu.edu > wrote in message news:bj******** ***@f04n12.cac. psu.edu...

static char myVersion[20];

This is only a declaration.

You must also define it. Put

char TextDB::myVersi on[20];

inside the namespace TextDB in your cpp file.

[Mike Wahler]

Kevin Grigorenko <kz****@psu.edu > wrote in message
news:bj******** ***@f04n12.cac. psu.edu...

Okay, everything seems simple enough but I get the following Link error:

TeXt.obj : error LNK2001: unresolved external symbol "private: static char * TextDB::TextDB: :myVersion" (?myVersion@Tex tDB@1@0PADA)

My environment is Visual Studio .NET, I think it's C++ 7 from the microsoft visual studio versioning point of view.

Here is TeXt.h: [snip] namespace TextDB
{
class TextDB
{
public:
// Constructors/Destructor:
TextDB(const std::string filename) throw(std::inva lid_argument);
~TextDB();
TextDB(const TextDB &copy) throw(std::inva lid_argument);

static const char *getVersion();
private:
// Private because it is illogical to instantiate a
// TextDB object without a filename
TextDB();

void initialize(std: :string in_filename)
throw(std::inva lid_argument);

std::string filename;
static char myVersion[20];

This is only a declaration, it's not a defintion.
No storage is reserved. Do this in your implementation
file (inside your namespace 'TextDB':

char TextDB::myVersi on[20];

I'd recommend replacing that char array with a std::string.

-Mike

[Kevin Grigorenko]

"Ron Natalie" <ro*@sensor.com > wrote in message
news:3f******** *************** @news.newshosti ng.com...

"Kevin Grigorenko" <kz****@psu.edu > wrote in message news:bj******** ***@f04n12.cac. psu.edu...

static char myVersion[20];

This is only a declaration.

You must also define it. Put

char TextDB::myVersi on[20];

inside the namespace TextDB in your cpp file.

Thanks, that did it. So any variables declared static must explicitly be
defined? Can this be done inline?

Kevin Grigorenko

[Ron Natalie]

"Kevin Grigorenko" <kz****@psu.edu > wrote in message news:bj******** ***@f04n12.cac. psu.edu...

Thanks, that did it. So any variables declared static must explicitly be
defined?
Yes
Can this be done inline?

No, with the exception of static integral constants. The implementations need
the definition to figure out which module actually allocates the storage for the
variable. This all harps back to an age old issue as to whether C (or C++)
could define the same data in multiple object files and have the linker fold
them all into the same object. The early C linkers did support this, but the
powers that were decided to not require this (although any linker that could
do seperate compilation of Fortran (common blocks) had to have the capability).