[E.....r]

Da się jakoś lepiej zapisać taką konstrukcję?

if (condtion) {
doSomething()
}

W moim przypapadku mam coś takiego:
#java #naukaprogramowania #programowanie

[joolekk]

@Edelner: dlaczego chcesz usprawniać ifa?

[RYSZARD69]

po if mozesz w tej samej lini pisac bez klamer

[joolekk]

@RYSZARD69: ale to jest niefajne w przypadku gdy jednak doSomething to nie jest jedna funkcja.

No i mieszanie tych konwencji w projekcie mi się nie podoba

[RYSZARD69]

dobra mi sie jezyki pomyliły xD

[fegwegw]

@Edelner: da się usunąć puste linie i dać spację po przecinku. No i wcięcia na +2 a nie +4

[alkb]

@Edelner: wyglada na zly design, jak walidacja nie przejdzie to po cichu sie po prostu nie doda wpis do bazy? xD

[dixtel]

@Edelner: mozna usunac klamry ale po co

[E.....r]

@alkb: A co proponujesz? Rzucić wyjątkiem?

[sezzart]

A co proponujesz? Rzucić wyjątkiem?

@Edelner: najpierw zwalidowac potem dodawac? :D

[E.....r]

@sezzart: No przecież w zależności od walidacji tak się doda lub nie, więc co miałbym zmienić w tym kodzie?

[sezzart]

@Edelner: nie widze dokladnie co tam sie dzieje w tym kodzie, ale po pierwsze czemu metoda nazywa sie add skoro nie wiesz czy wykonane zostanie dodanie do kolejki? po drugie czemu w ogole rozwazasz dodanie commanda ktory nie koniecznie jest poprawny?

[oxfox]

@fegwegw: +2?? czlowieku zycie ci mile?

[E.....r]

@sezzart: Zmieniłem nazwę metody na addIfCommandIsCorrect. Może być? No przecież jest walidacja tego commanda.

[96431e9dfa1a78af1aadcfcc744e7548]

@oxfox: +2 najlepsze, Miras

[ajgoron]

@Edelner: a może pokaż resztę tej klasy? Pytanie czy to odpowiednie miejsce na użycie walidatora? Jeżeli ta klasa powinna służyć do dodawania jakichś elementów (np. tasków) to raczej nie.
Więc w tej metodzie obsługiwałbym tylko dodawanie elementu.
Kolejne pytanie. Czy w trakcie dodawania elementu coś może się nie udać? Jeżeli tak to czy to ta klasa powinna to sprawdzić i obsłużyć wyjątek?

[sezzart]

Zmieniłem nazwę metody na addIfCommandIsCorrect

@Edelner: XD #maybestartservice() XDD

No przecież jest walidacja tego commanda

@Edelner: ale czemu tworzysz command ktory moze nie byc valid?

[ajgoron]

@Edelner jeszcze tak dodatkowo: lubię stosować metodologię "fail fast". W tym przypadku tak zrobiłbym tę metodę:

if (!condition) {
 return;
}

doSomething();

[E.....r]

@ajgoron: To jest taki agregat. Przed dodaniem musi nastąpić walidacja.
@sezzart: Z czego rżysz? Co jest złego w tej nazwie? Ten command przychodzi z controllera od użytkownika, W handlerze jest walidowany i jeśli jest ok to zostaje zmapowany na encje i dodany do repo. Czego nie rozumiesz?

[sezzart]

@Edelner: A to spadaj xdd

[oxfox]

@96431e9dfa1a78af1aadcfcc744e7548: chyba na ekranie 800x600 ( ͡° ͜ʖ ͡°)