CODEWITHSUNDEEP
c++stringconstantsstd
Michel Feinstein
Michel Feinstein

Reputation: 14296

C++ const char* to std::string &

I have a code that compiles, and it looks like:

    void test1(const std::string &name)................
    test1("myName");

But, if I remove the const from the declaration as:

    void test2(std::string &name)................
    test2("myName");

The code just doesn't compile.

Why the const std::string makes it compile? As far as I understand the same implicit constructor will be called for the char * to be converted to a std::string.

I am using Visual Studio 2013, in case this might be related to the answer.

Upvotes: 1

Views: 1883

Answers (4)

NathanOliver
NathanOliver

Reputation: 181008

"myName" is a c-style string and it has the type of const char[]. Since it is not a std::string it needs to be converted to one. When that happens a temporary string is created. That temporary string cannot be bound to a reference but it can be bound to a const reference as it will extend its lifetime to the end of the expression.

You could also pass the string by rvalue reference like void foo(std::string && bar). This will move the temporary string into the function and give you the same effect.

A third option is to pass the string by value void foo(std::string bar). This works with temporaries and non temporaries and will wither make a copy or a move depending on the source of the string.

Upvotes: 2

Dietrich Epp
Dietrich Epp

Reputation: 213748

When the implicit conversion happens, you get a temporary std::string (technically, the important thing is that it is an xvalue). You cannot bind a temporary to a lvalue reference std::string & but you can bind to a const lvalue reference const std::string & or an rvalue reference std::string &&.

The same thing happens here:

const int &x = 5;
int &x = 5; // error

Upvotes: 5

Ahmed Akhtar
Ahmed Akhtar

Reputation: 1463

The problem is with the & you are using for reference. When calling the function with a string literal you cannot pass a non-const reference.

Upvotes: 0

P0W
P0W

Reputation: 47844

You get an error in second case because a temporary can not bind to a non-const reference.

test("myName"); tries to bind temporary constructed object to a lvalue in second case, hence an error

Upvotes: 2

Related Questions