Funkcja tekstowa ucinająca znaki

[lukagrom]

Napisałem sobie funkcję, która w zamierzeniu ma ucinać pierwszy i ostatni znak tekstu w zmiennej typu UnicodeString. Z założenia zrobiłem przypisanie do wchar_t, gdzie korzystając z obliczania długości tekstu (wcslen()) stworzyłem nową zmienną typu wchar_t* z długością mniejszą o 2, niż badany tekst (mniejsze o pierwszy i ostatni znak). Potem za pomocą pętli for, proste przypisanie - (nowy_tekst[i]=stary_tekst[i+1] i niby wszystko działa. Jednakże wykonując program za pierwszym razem/pierwszym kliknięcie buttona - wyskakuje mi w wynik w postaci "krzaków", lecz pózniej kazde nastepne uruchomienie/klikniecie jest już zbieżne z okazaniem właściwego wyniku. I teraz sam nie wiem, czy to przypadłość środowiska, czy też moja funkcja ma jakieś braki pamięciowe, że właściwie reaguje po zainicjowaniu po n-tym razie.

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

UnicodeString __fastcall TForm1::first_and_Last(UnicodeString text_Ex)
{
        wchar_t* date_Text=text_Ex.w_str();
        int size_Text=wcslen(date_Text);
        int size_Result=size_Text-2;
        int i;
        wchar_t* new_Text=new wchar_t[size_Result];
        for(i=0;i<size_Result;i++)
         new_Text[i]=date_Text[i+1];
        return (UnicodeString) new_Text;
}
void __fastcall TForm1::Button1Click(TObject *Sender)
{
 UnicodeString days_of_Week[7]={"Monday","Tuesday","Wednesday","Thursday","Friday",
 "Saturday","Sunday"};
 for(int i=0;i<7;i++)
 {
         ListBox1->Items->Add(first_and_Last(days_of_Week[i]));
 }
}
 

[lukagrom]

Zrobiłem praktycznie to samo dla zwykłego stringa w g++ i działa bez zarzutu.

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

string first_and_Last(string text_Ex)
{
        const char* date_Text=text_Ex.c_str();
        int size_Text=strlen(date_Text);
        int size_Result=size_Text-2;
        int i;
        char* new_Text=new char[size_Result];
        for(i=0;i<size_Result;i++)
         new_Text[i]=date_Text[i+1];
        return (string) new_Text;
}


int main()
{
 string days_of_Week[7]={"Monday","Tuesday","Wednesday","Thursday","Friday",
 "Saturday","Sunday"};
 for(int i=0;i<7;i++)
 {
     cout<<first_and_Last(days_of_Week[i])<<endl;

 }
 

[polymorphism]

i działa bez zarzutu.

Pozory mylą Masz wyciek pamięci. Tam, gdzie jest new, tam wcześniej czy później musi pojawić się delete. Już Ci chyba pisałem - nie zarządzaj ręcznie pamięcią! Od tego masz inteligentne wskaźniki i std::vector/std::string.

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

std::string first_and_Last(const std::string &text)
{
    if (text.size() < 3) return "";
    return { text.begin() + 1, text.end() - 1 };
}

Wersja dla UnicodeStringa:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

UnicodeString first_and_Last(const UnicodeString &text)
{
        UnicodeString resultStr;
        for(size_t i = 2; i <= text.Length() - 1; ++i) resultStr += text[i];
        return resultStr;
}

Jak widać, można bez new, tablic, funkcji C i wycieków pamięci

[lukagrom]

Danke. Ja szukałem jescze w tym deseniu (bez new/delete]) coś automatycznego, co używają typy tekstowe i w poradach odnalazłem starego/poczciwego Stringa, którego jeszcze Borland zaadoptowałdo Buildera prosto z Delphi.

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

String __fastcall TForm1::first_and_Last(String text)
{

 text.Delete(1,1);
 text.Delete(text.Length(),1);
 return text;
}
void __fastcall TForm1::Button1Click(TObject *Sender)
{
 AnsiString days_of_Week[7]={"Monday","Tuesday","Wednesday","Thursday","Friday",
 "Saturday","Sunday"};
 for(int i=0;i<7;i++)
 {
  ListBox1->Items->Add(first_and_Last(days_of_Week[i]));
 }
}

 

[polymorphism]

Ten "stary/poczciwy" String to nic innego jak alias typu AnsiString/UnicodeString.

[Cyfrowy Baron]

String w zależności od ustawień projektu reprezentuje typ AnsiString lub UnicodeString. W nowych środowiskach domyślnie jest to UnicodeString.