Wpis z mikrobloga

@bonus: ja bym to zrobił jakoś tak:

isInSearchedDataSource(type: string): boolean {

return (['web', 'comments', 'forum', 'review'].indexOf(type) !== -1);

}

Ale ja nie mam tyle komercyjnego doświadczenia (
  • Odpowiedz
Nie rozumiem co Ty krytykujesz.

Twoje rozwiązanie jest jeszcze gorsze niż jego, bo mniej czytelne, a chodzi o wygodę osób czytających, poprawiających kod po Tobie.

Literówka w nazwie metody, też nie taka straszna.

Dziwne jest tutaj podanie tak tych stringów na sztywno, ale żeby to ocenić trzeba znać całą otoczkę.

Niczego karygodnego tutaj nie ma.

@cevilo: co? xD Wciśnij F12 i wklej "('web' || 'comments' || 'forum' || 'review')"
  • Odpowiedz
@DiKey: W zasadzie, też bym zrobił jak Ty i dopiero teraz zauważyłem fail programisty 10k ( ͡° ͜ʖ ͡°)
Cóż każdemu się zdarza, ale beka, że skomitował błędny kod
  • Odpowiedz