връщане на низ от функция

lapin, кодът ти е добър C++11 код. В C++98/03 вашият код вероятно ще бъде ефективен поради оптимизации на компилатора, но тези оптимизации не са гарантирани. В C++11 същите тези оптимизации вероятно пак ще направят връщането ви безплатно, но в случай, че не го направят, вашият низ ще бъде преместен вместо копиран.

Така че връщайте по стойност без вина! :-)

Дребна гнида:

Най-добрата практика е да декларирате вашите стойности в точката на първото използване, вместо в горната част на блок:

string sValue = ss.str();
return sValue;

Или може би дори:

return ss.str();

Но това е само малка гнида. Вашият код е добър и ефективен.

Добре, знам, че това не е безопасно за нишки и всичко това и вероятно ще бъда отрицателен до края, но видях следното в библиотека, която използвам (ROOT на CERN):

const char * myfunc(){

  static std::string mystr;

  /*populate mystr */

  return mystr.c_str();
}

Това работи само ако знаете, че никой никога няма да бъде толкова глупав, че да се хване за показалеца.

Това е начин да имате временен, който няма да изтече, независимо какво.

В свят без RVO/NRVO това трябва да избягва конструкцията на копиране в стандартна библиотека преди C++11. В пост C++11 библиотека с конструктор за преместване за низ все още избягва извикването на конструктора за преместване; вероятно това е тривиално малка разлика, но все пак ОП питаше как да се направи по-добре.

(И да, съгласен съм, че наследяването от низ е грозно, но работи.)

#include <ctime>
#include <string>
#include <sstream>
#include <iostream>

using namespace std;

class DateString : public string {

public:
DateString() : string()     {

    stringstream ss;
    time_t t = time(0);
    struct tm * now = localtime(&t);

    ss << now->tm_hour << ":";
    ss << now->tm_min << ":";
    ss << now->tm_sec << " ";
    ss << now->tm_mday + 1 << " ";
    ss << now->tm_mon + 1 << " ";
    ss << now->tm_year + 1900;

    append(ss.str());

}
};

int main()
{
    cout << DateString() << endl;
    return 0;
}