[adi1tysdz]

Piszę prosty edytor tekstu w #java i mam pewien problem z zapisywaniem. Otóż, gdy napiszę jakiś tekst i chcę go zapisać z poziomu aplikacji do np. "plik.txt", w pliku znajduję takie coś: javax.swing.JTextArea[,0,26,496x448,lay ... =184,g=207,b=229],colums=0,columWidth=0,rows=0,rowHeight=0,word=false,wrap=false] (Część wyciąłem).

Tutaj link do funkcji zapisz: Link
Co jest nie tak? ( ͡° ʖ̯ ͡°)

[Patres]

@adi1tysdz: Zapisujesz obiekt klasy w postaci stringa, a nie to co przechowuje. Nie znam się na Swingu, ale podejrzewam, że powinieneś wstawić coś w tym stylu: edytorek.getText().
Protip: olej Swinga i zacznij uczyć się JavaFX

[ppawel]

@adi1tysdz: zdecydowanie wygląda na wyjście toStinga() co pozwala przypuszczać, że (sic!) edytorek jest typu JTextArea.

Code review gratis:
- nie powinno się używać polskich nazw
- psst, klasy mają metody nie funkcje
- dwukrotne wyrzucanie na wyjście informacji o błędąch jest trochę nadmiarowe,
- zapisz() oraz edytorek są statyczne podczas gdy wszystko wskazuje na to, że jednak mamy do czynienia z jakimś stanem

( ͡° ͜ʖ ͡°)

[sorhu]

@adi1tysdz: Code review dalsza część.
Czemu ta metoda jest statyczna?
W pewnym przypadku może wystąpić NullPointerException. Zabezpiecz się przed tym.
Skoro masz GUI, to ewentualne komunikaty błędu pokazuj w GUI, a nie w konsoli.

[adi1tysdz]

@Patres: @ppawel: @sorhu: Dzięki panowie, jakoś wyszło. Generalnie moje pierwsze podrygi w tym języku, ledwo 2 zajęcia przerobiłem. Jeszcze raz dzięki za pomoc i rady.

[sorhu]

@adi1tysdz: To jak ledwo zacząłeś ledwo Javy się uczyć, to nie leć od razu w GUI. Naucz się najpierw samego języka.

[adi1tysdz]

@sorhu: Takie dostałem zadanie na laborki. Muszę skończyć ten edytor, na szczęście nie musi być rozbudowany.

[ppawel]

jakoś wyszło

@adi1tysdz: i dopiero tutaj zaczyna się przyjemność tworzenia dobrego kodu :)