Spojení string – C / C++ – Fórum – Programujte.com

Ach jo, dost mě štve, že sem pořád píšu takový prkotiny a neumím se s tím vypořádat, ale prostě to nejde :(

Mám následující problém. Zkrátka potřebuju ve třídě přetížit operátor sčítání pro řetězce. Tak jsem to zkusil nějak takhle:

class String

{

      private:

              char * str;

              int len;

      public:

              String(const char * s);

              String();

              String(const String & st);

              ~String();

              int lenght() const { return len; }

      // přetížené operátory

              String & operator=(const String & st);  // operátor přiřazení

              String & operator=(const char * s);     // operátor přiřazení č. 2

              

      // spřátelené funkce

              friend bool operator>(const String &st1, const String &st2);

              friend bool operator<(const String &st, const String &st2);

              friend bool operator==(const String &st, const String &st2);

              friend ostream & operator<<(ostream & os, const String & st);

              friend istream & operator>>(istream & is, String & st);

              friend String & operator+(const String & t1, const String t2);

      };

String & operator+(const String & t1, const String t2)

{

       String temp;

       temp.len = strlen(t1.str) + strlen(t2.str);

       temp.str = new char[temp.len+1];

       temp.str = strcat(t1.str , t2.str);

       

       return temp;

       

}

Zdravím,

z toho, že ti něco nefunguje si nedělej težkou hlavu - C++ je zatraceně složitý jazyk. Počkej až přejdeš na memory managment, ten je, řekl bych, asi nejůležitější částí programu napsaných v C++(sranda teprve začne s používáním vyjímek)
A teď k problému:)
Problém je v tom, že vracíš referenci na LOKÁLNÍ objekt - to už NIKDY nedělej, cokoliv lokálního vracej hodnotou - za to si dej facku. A pokud sis už nafackoval dost, tak tady je řešení:

String  operator+(const String& a, const String& b)

{  

    char *buf = new char[a.length()+b.length()+1]; // alokace dočasného pole

    String temp;

    if(buf)  // pokud se povede alokace(pokud ne asi bych vyhodil vyjimku, ale..)

    {  ::strcpy(buf, a.str); // zkopírování textu, který je v a

        ::strcat(buf, b.str); // připojení dalšího textu

        temp = buf;  // předání operátoru = doufám, že máš tu implementaci dobře(dalsi alokace/dealokace)

        delete [] buf; // smazání dočasného pole

    }

    return temp; // vracení lokalniho objektu hodnotou =  volani kopy konstruktoru(dalsi alokace)

} 

Paráda stačilo to takhle

String operator+(const String & t1, const String t2)

{

       String temp;

       temp.len = strlen(t1.str) + strlen(t2.str);

       temp.str = new char[temp.len+1];

       strcpy(temp.str, t1.str);

       strcat(temp.str, t2.str);

       

       return temp;

}

String::~String()

{

                 delete [] str;

}

To vollf82:

jj, to stačilo a máš tam o 1 volání alokace/dealokace méně:). Já jen, že se obvykle řídím tím, že protředky alokuju v konstrukotru a dealokuju v destructoru, ale tady to bla zjevná blbost -facku jsem si dal sám Btw: kontroluj si jestli se alokace povedla.