'Krzaki' w komunikacji tekstowej ClientSocket - ServerSocket

[mckri]

Witam.
Mam problem z komunikacją pomiędzy ClientSocket oraz ServerSocket. Obie aplikacje widzą się poprawnie.
Problem jest w tym, że przy przesyłaniu wiadomości tekstowych widać po obu stronach (w zalezności która przesłała) tzw. 'krzaki'. Oba programy napisane są w Builder 2009. W starym Builderze 5/6 tego problemu nie było, jednak tam domyslnym typem zmiennych był AnsiString. Domyslam się, że w nowym środowisku chodzi o typ zmiennych (wide?/ansi?/char?), jednak tutaj wypróbowałem chyba juz wszystkie kombinacje a i tak pojawiają się te 'krzaki'.

Po stronie serwera:
-odebranie danych od klienta

Kod: Zaznacz cały

Memo1->Lines->Add(Socket->ReceiveText().Trim());


-wysłanie danych do klienta

Kod: Zaznacz cały

for(int x = 0; x < ServerSocket1->Socket->ActiveConnections; x++)
  {
     ServerSocket1->Socket->Connections[x]->SendText(Memo2->Text.Trim());
  }

Po stronie klienta:
-odebranie danych z serwera:

Kod: Zaznacz cały

Memo1->Lines->Add(Socket->ReceiveText().Trim());

-wysłanie danych do serwera:

Kod: Zaznacz cały

ClientSocket1->Socket->SendText(Memo2->Text.Trim());

Wie ktoś jak poprawnie przesyłac tekst przy wykorzystaniu tych komponentów w środowisku Builder 2009/2010?

PS. komponenty client/server socket można zainstalować na podstawie tej instrukcji http://support.codegear.com/article/36470

Widok krzaków w obu aplikachach:

[Cyfrowy Baron]

Spróbuj tak:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

for(int x = 0; x < ServerSocket1->Socket->ActiveConnections; x++)
  {
   ServerSocket1->Socket->Connections[x]->SendText( AnsiString( Memo2->Text ).Trim() );
  }

lub ze zmienną typu AnsiString:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

for(int x = 0; x < ServerSocket1->Socket->ActiveConnections; x++)
  {
   AnsiString sendText = Memo2->Text;
   ServerSocket1->Socket->Connections[x]->SendText( sendText.Trim() );
  }

Podobnie postąp z klientem. Sockety nie obsługują typu UnicodeString, a klasa TMemo w C++Builder 9 i wyższym operuje właśnie na UnicodeString.

[mckri]

Próbowałem już w ten sposób. 'Krzaki' nadal wystepują, tyle ża są innej postaci - zamiast 'chińszczyzny' są znaki '?'.
Jakby co, przesyłam kod źródłowy do wglądu.

[polymorphism]

Według dokumentacji SendText powinien być wywoływany ze zdarzeń OnWrite, OnClientWrite lub OnSocketEvent. Ty wysyłasz tekst "z buttona", i jeśli sockety działają w trybie asynchronicznym, może być tak, że to wywołanie nie wysyła nic lub tylko fragment tekstu. Sprawdź jaką wartość zwraca SendText.

[Cyfrowy Baron]

Według dokumentacji SendText powinien być wywoływany ze zdarzeń OnWrite, OnClientWrite lub OnSocketEvent.

Swego czasu w środowisku C++Builder 4 używałem socketów i wysyłałem tekst z buttona, i nie było z tym żadnego problemu.

Sprawdź czy ten kod też wywołuje błąd:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

for(int x = 0; x < ServerSocket1->Socket->ActiveConnections; x++)
{
   AnsiString sendText = "Przykładowy tekst";
   ServerSocket1->Socket->Connections[x]->SendText( sendText.Trim() );
}

W tym kodzie nie ma zmiennej UnicodeString, więc teoretycznie nie powinno być żadnego problemu. Jeżeli problem występuje, to oznacz niezgodność Socket'ów ze środowiskiem. Do są już dość przestarzałe biblioteki i w nowszych wersjach środowiska dostępne są nowsze odpowiedniki.

[mckri]

Przyszło mi rozwiązać problem samemu...
Odbiór wiadomości:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

int iBufSize = Socket->ReceiveLength();
   char *Buf = new char [iBufSize + 1];
   Socket->ReceiveBuf(Buf,iBufSize);
   Msg=Buf;
   Memo1->Lines->Add(Msg.Trim());
 

Wysyłanie wiadomości:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

AnsiString Msg = Memo2->Text.Trim();
          ServerSocket1->Socket->Connections[x]->SendBuf(Msg.c_str(), Msg.Length());

[polymorphism]

Pamięć Buf zwalniasz?

I jeszcze taka mała poprawka:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

...
int cb = Socket->ReceiveBuf(Buf, iBufSize);
if(cb > 0)
{
    buf[cb] = 0;
    Msg = Buf;
    ...
}

[mckri]

No właśnie czegoś mi brakowało

[polymorphism]

Tu masz wersję racjonalniejszą, bez alokacji na stercie:

KOD cpp:   �  UKRYJ  �

KOD cpp:   �  POKAŻ  �

int toRead = Socket->ReceiveLength();
char Buf[1024];
int cb;

while(toRead > 0 && (cb = Socket->ReceiveBuf(Buf, min(toRead, sizeof(Buf)))) > 0)
{
        Buf[cb] = 0;
        Msg += Buf;
        toRead -= cb;
}

Memo1->Lines->Add(Msg.Trim());