[kufelmleka]

#java #programowanie
http://pastebin.com/RTNK22Vr
Fragment kodu odpowiedzialny za przyjmowanie wiadomości od klienta przez serwer i wyświetlająca ją na GUI.
Problem w tym, że każda następna odebrana wiadomość nadpisuje się na poprzednią. Dzieje się to 12 linijce.
Chciałbym, aby każda wiadomość była wyświetlana pod poprzednią.

Nie proszę o podanie gotowego rozwiazania, ale o wskazówkę, nakierowanie mnie na sposób rozwiazania.

Może coś z ArrayList?

Dzięki z góry Mircy( ͡° ͜ʖ ͡°)

[Matis666]

@kufelmleka: użyj println

[PanKamil907]

@kufelmleka: Pobierz dane z "displayText". Następnie zrób jakiś append (dodaj do tego nowy)

Coś w stylu: displayText.setText(displayText.getText() + response);

[moriturius]

@kufelmleka: Przemyśl swoje zachowanie! Uruchamiasz obsługę klienta w osobnym wątku i z tego wątku aktualizujesz GUI? Czy to oznacza, że GUI może być aktualizowane przez wiele wątków w tym samym czasie? Potępiam takie rozwiązanie ;)

[PanKamil907]

@PanKamil907: A najlepiej zrób listę z wiadomościami, i dodawaj wiadomości do tej listy, potem pętelka.

[moriturius]

@kufelmleka:
- napisałeś nazwę klasy z małej litery :<
- niektóre zmienne są nazwane po polsku inne po angielsku :<
- nazwałeś klasę sendTheMessage z małej litery
- co tu się w ogóle z tymi wątkami odwala? dlaczego wysyłasz wiadomość w osobnym wątku?
- nie jestem pewien czy robienie System.exit(0) a potem pisanie na stderr to najlepsza praktyka

Każdy minus to jeden zabity kotek. Aktualizacja GUI z wielu threadów to zabicie

[kufelmleka]

@moriturius: generalnie gui zajmuje się jeszcze np. wysyłaniem i wyświetlaniem danych wpisanych z klawiatury.
@PanKamil907: Da się tak z wiadomościami wpisanymi z klawiatury?

[kufelmleka]

@moriturius: Powiem tak. Dopiero zaczynam przygodę z Javą i byłbym bardzo zdziwiony, że mój kod jest dobry. Dzięki za wskazówki.

[PanKamil907]

@kufelmleka: Daj znać jak rozwiążesz to

[moriturius]

@kufelmleka: Pokaż całość, spróbujemy naprawić bo wydaje mi się, że jest co :)

[kufelmleka]

@moriturius: http://pastebin.com/R6kzcisH

[kufelmleka]

@PanKamil907: Bezpośrednie wpisanie zapętla się w nieskończoność. Kombinuję dalej.

[moriturius]

@kufelmleka:
- Swing, którego używasz do GUI ma SwingUtilities.invokeLater(Runnable runnable). To jest JEDYNY dobry sposób na aktualizowanie GUI z wielu wątków. Jak to zawołasz to kod z runnable zostanie wywołany w wątku odpowiedzialnym za GUI.
- JTextArea, którego używasz do displayText ma metodę append(String). Użyj jej zamiast setText i nowa wiadomość nie będzie kasowała poprzednich tylko dopisze się na końcu.
- Robienie niestatycznych klas wewnętrznych jest w 99%

[kufelmleka]

@moriturius: Co do Swinga, to dobrze zrobiłem?

Jeśli tak, to muszę zrezygnować z tego rozwiązania, bo przywraca ono problem, który wczoraj rozwiązałem, a mianowicie, program zawieszał mi się po odebraniu pustego pakietu, a teraz , przy takim rozwiązaniu zawiesza się od razu, przy przyjęciu pierwszego pakietu.

[moriturius]

@kufelmleka: Tylko display.setText powinno być invokeLater. Reszta musi być poza. Przecież nie możemy odbierać danych od klienta w wątkuz GUI ;)

[kufelmleka]

@moriturius: czyli powinienem napisać
invokeLater(response);
?