Where to define a const string?

[Riley DeWiley]

I am looking for a graceful way to declare a string const that is to be
visible across many files.

If I do this:

//----hdr.h

const char * sFoo = "foo";

//file.cpp
#include <hdr.h>

strcpy(string, sFoo);
//anotherfile.cpp
#include <hdr.h>

strcpy(string, sFoo);
The linker complains that sFoo is multiply defined.

I don't want to use a #define as it breaks type safety. I don't want to have
multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

RDeW

[Bryan Donlan]

Riley DeWiley wrote:

I am looking for a graceful way to declare a string const that is to be
visible across many files.

If I do this:
[snip example]
The linker complains that sFoo is multiply defined.

I don't want to use a #define as it breaks type safety. I don't want to
have multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

RDeW

Header file:
extern const char sFoo[];

In (one) source file (exactly which doesn't matter):
const char sFoo[] = "foo";

--
λz.λi.i(i((λ n.λm.λz.λi.n z(λq.mqi))((λ n.λz.λi.n(nzi )i)(λz.λi.i(( (λn.λz.λi.n
(nzi)i)(λz.λi .i(iz)))zi)))(( λn.λz.λi.n(n zi)i)(λz.λi.i (iz)))zi))

[Victor Bazarov]

Riley DeWiley wrote:

I am looking for a graceful way to declare a string const that is to be
visible across many files.

If I do this:

//----hdr.h

const char * sFoo = "foo";

//file.cpp
#include <hdr.h>

strcpy(string, sFoo);
//anotherfile.cpp
#include <hdr.h>

strcpy(string, sFoo);
The linker complains that sFoo is multiply defined.

I don't want to use a #define as it breaks type safety. I don't want to have
multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

If you declare it in the header

const char * const sFoo = "foo";

which creates multiple copies. To avoid that you could do

extern const char sFoo[];

in the header and in _one_of_the_C++ _source_files_ do

extern const char sFoo[] = "foo";

The linker will be happy and you will have the only definition of the
string in the program.

V

[Pete Becker]

Riley DeWiley wrote:

I don't want to use a #define as it breaks type safety.
No, it doesn't.

#define sFoo "foo"

Every place you use the identifier sFoo you'll get a string literal. Its
type, however, is array of const char rather than pointer to const char,
which is what your code uses.
I don't want to have
multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

In the header:
const char *sFoo;

In one implementation file:
const char *sFoo = "foo";

The suggestion the other messages make, to use const char sFoo[], also
works, but just like the macro, it makes sFoo a different type from what
you asked for.

--

Pete Becker
Dinkumware, Ltd. (http://www.dinkumware.com)

[Bryan Donlan]

Pete Becker wrote:

Riley DeWiley wrote:

I don't want to use a #define as it breaks type safety.
No, it doesn't.

#define sFoo "foo"

Every place you use the identifier sFoo you'll get a string literal. Its
type, however, is array of const char rather than pointer to const char,
which is what your code uses.

I don't want to have
multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

In the header:
const char *sFoo;

In one implementation file:
const char *sFoo = "foo";

This is wrong. The first will create a pointer, initialized to NULL, in each
file it's included in. The second will create yet another pointer,
initialized to pointing to the constant string needed.

The suggestion the other messages make, to use const char sFoo[], also
works, but just like the macro, it makes sFoo a different type from what
you asked for.

The array will automatically cast into const char * when needed, so it
doesn't matter.

--
λz.λi.i(i((λ n.λm.λz.λi.n z(λq.mqi))((λ n.λz.λi.n(nzi )i)(λz.λi.i(( (λn.λz.λi.n
(nzi)i)(λz.λi .i(iz)))zi)))(( λn.λz.λi.n(n zi)i)(λz.λi.i (iz)))zi))

[Greg]

Victor Bazarov wrote:

Riley DeWiley wrote:

I am looking for a graceful way to declare a string const that is to be
visible across many files.

If I do this:

//----hdr.h

const char * sFoo = "foo";

//file.cpp
#include <hdr.h>

strcpy(string, sFoo);
//anotherfile.cpp
#include <hdr.h>

strcpy(string, sFoo);
The linker complains that sFoo is multiply defined.

I don't want to use a #define as it breaks type safety. I don't want to have
multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

If you declare it in the header

const char * const sFoo = "foo";

which creates multiple copies. To avoid that you could do

extern const char sFoo[];

in the header and in _one_of_the_C++ _source_files_ do

extern const char sFoo[] = "foo";

The linker will be happy and you will have the only definition of the
string in the program.

V

Const declarations by default have internal linkage, so the declaration
can remain in the header file once it's changed from a pointer to an
array. No source files need to be changed.

In other words change this declaration in the header file:

const char * const sFoo = "foo";

to this:

const char const sFoo[] = "foo";

And the multiple definition error will be fixed - no matter how many
source files actually include sFoo's declaration.

Greg

[Pete Becker]

Bryan Donlan wrote:

Pete Becker wrote:

I don't want to have
multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

In the header:
const char *sFoo;

In one implementation file:
const char *sFoo = "foo";

This is wrong. The first will create a pointer, initialized to NULL, in each
file it's included in.

You're right: it needs an extern in front.

The suggestion the other messages make, to use const char sFoo[], also
works, but just like the macro, it makes sFoo a different type from what
you asked for.

The array will automatically cast into const char * when needed, so it
doesn't matter.

First, a cast is something you write in your code to tell the compiler
that you want a conversion. Second, there are two contexts in which the
conversion from array into pointer to its first element is not done. The
two types are not the same.

--

Pete Becker
Dinkumware, Ltd. (http://www.dinkumware.com)

[Pete Becker]

Pete Becker wrote:

Second, there are two contexts in which the
conversion from array into pointer to its first element is not done. The
two types are not the same.

Actually, that's true in C. In C++ there are more. And, in both
languages, this is not a conversion, but a decay: the name of an array
decays into a pointer to its first element in most contexts. And, of
course, it is still true that the two types are not the same. Try this:

a.c
---
char text[] = "abcd";

void f()
{
puts(text);
}

b.c
---
extern char *text;
void f();

int main()
{
*text = 'e';
f();
return 0;
}

--

Pete Becker
Dinkumware, Ltd. (http://www.dinkumware.com)

[Marcus Kwok]

Riley DeWiley <ri***********@ gmail.com> wrote:

I am looking for a graceful way to declare a string const that is to be
visible across many files.

If I do this:

//----hdr.h

const char * sFoo = "foo";

//file.cpp
#include <hdr.h>

strcpy(string, sFoo);
//anotherfile.cpp
#include <hdr.h>

strcpy(string, sFoo);
The linker complains that sFoo is multiply defined.

I don't want to use a #define as it breaks type safety. I don't want to have
multiple copies of '
const char * sFoo = "foo";' littering my code.

What is the most compact and maintainable way to do this?

Would an #include guard work?
//-----hdr.h
#ifndef HDR_H
#define HDR_H

const char* sFoo = "foo";

#endif

--
Marcus Kwok

[Ney André de Mello Zunino]

Riley DeWiley <ri***********@ gmail.com> wrote:

I am looking for a graceful way to declare a string const that is to be
visible across many files.
[...]
What is the most compact and maintainable way to do this?

Probably not the most compact way, but I suppose it would fit into the
maintainable and correct categories.
//--- MagicString.h

#ifndef MAGIC_STRING_H_ _
#define MAGIC_STRING_H_ _

extern const char* const gMagicString;

#endif
//--- MagicString.cpp

#include "MagicStrin g.h"

const char* const gMagicString = "I am a magic string!";
//--- test.cpp

#include "MagicStrin g.h"
#include <iostream>

int main()
{
std::cout << gMagicString << '\n';
}
Regards,

--
Ney André de Mello Zunino