Any way to simplify this string return in my member function?

**Striph** · 01-21-2012

This is my member function to take the value and suit of a card and return a single string for both variables. (first year of college, go easy on me!) Is there a simpler way to do this? I tried to just return value + suit but I get a conversion error, and its probably more code to convert their type.

Code:

 
string PlayingCard::getCardCode()
{
    string cardCode;

    cardCode = value; // value is a char defined in the Class header
    cardCode += suit; // same for suit, both are defined in the default constructor and set to '0'

    return cardCode;
}

**laserlight** · 01-21-2012

If you just want something concise, then:

Code:

string PlayingCard::getCardCode() const
{
    return string(1, value) + suit;
}

Note that the declared the member function const since it does not change the observable state of the PlayingCard.

**Striph** · 01-21-2012

Interesting, I haven't learned the const after the function method yet. I noticed that it does compile without the const, and also with the const on the end of the member function when I put it in the class declaration.

I assume specifying const like that prevents the getCardCode method from affecting the original values in the class?

Thanks for the reply, learned something new today!

**Elysia** · 01-21-2012

Originally Posted by Striph

I assume specifying const like that prevents the getCardCode method from affecting the original values in the class?

Yep.
Prevents the method from modifying any variables stored in the class.

**grumpy** · 01-21-2012

Originally Posted by Elysia

Yep.
Prevents the method from modifying any variables stored in the class.

You left out "non-mutable"

**Elysia** · 01-21-2012

I did, but I did it on purpose. No need to confuse a newbie.

Thread: Any way to simplify this string return in my member function?

Thread Tools

Search Thread

Display

Any way to simplify this string return in my member function?

Similar Threads

simplify recursive function

How to create a function to return a std::string, what return type is needed?

String member function find error

Getting a function to return a string

member function to return obect