C++ łączenie dwóch C-stringów - błędny wynik

Question

Witam! Napisałem sobie ostatnio szablon funkcji, który dodaje/łączy do siebie dwie wartości, zwracając sumę tych wartości. Wszystko było dobrze, ale gdy chciałem w ten sposób połączyć dwie wartości typu const char* , to kompilator pokazał błąd, że nie można łączyć operatorem + dwóch wartości tego typu. Dlatego napisałem, więc specjalizację dla tego typu i cały kod prezentuje się tak:

template<typename T> T dodaj(T a, T b)
{
    return a + b;
}

// specjalizacja dla typu const char*
template<> const char* dodaj<const char*>(const char* a, const char* b)
{
    string aa = a;
    string bb = b;

    string ret = aa + bb;

    return ret.c_str();

}

Problem polega na tym, że gdy program wyświetli wynik zwrócony przez tę funkcję, to wyświetlają się dziwne znaczki, np: dla takiego wywołania funkcji:

cout << dodaj("tekst1 ","tekst2");

Wyświetlają się takie znaki:

└■( Ű♠K P■( 2

W takim razie, chce was zapytać co zrobiłem źle w tym kodzie?

Z góry dzięki za pomoc!

Answer 1

"c-string" to po prostu wskaźnik do pamięci ze znakami. Konkatenacja c-stringów polega na stworzeniu obszaru pamięci, który pomieści oba napisy i przekopiowaniu ich do tej pamięci. Pamiętaj o kończącym zerze.

edit: sorry, nie zauważyłem tej specjalizacji. Problem polega na tym, że ret jest obiektem lokalnym i jest niszczony po wyjściu z funkcji.

Comments

Dzięki wielkie

Teraz zmodyfikowałem kod aby mogło wszystko ładnie działać

template<typename RET = string,typename T> RET dodaj(T a, T b)
{
    return a + b;
}


template<> string dodaj<string,const char*>(const char* a, const char* b)
{
    string aa = a;
    string bb = b;

    string ret = aa + bb;

    return ret;

}

Jeśli mógłbyś mi powiedzieć, czy powyższe rozwiązanie jest najlepsze, czy jest lepszy sposób?

EDIT:

Znalazłem sam jeszcze lepszy sposób, wystarczy tylko zmienną ret uczynić zmienną statyczną:

static string ret = aa + bb;

 

---

@dawid2002, lepszy to on nie jest, bo nie możesz swojej funkcji bezpiecznie używać w programach wielowątkowych. Tym bardziej, że używasz klasy std::string, którą możesz po prostu zwrócić przez wartość.

---

@j23 Ok dzięki.

Answer 2

return ret.c_str();

Zwracasz wskaźnik na nieistniejącą (po wyjściu z funkcji) pamięć.

Comments

Dzięki