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 ( ͡ ͜ʖ ͡)

W PHP pisz

To jest właściwie #typescript, w PHP to autor ma tyle lat doświadczenia.
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')"
@DiKey: ehkm...

abstract class Type
{
const web = 0;
const comments = 1;
const forum = 3;
const review = 4;
}

Tzn. ja nie znam PHP, ale jak widzę stringi używane do takich rzeczy... :P
@cevilo: spoko, a po za tym dlaczego

return variable;

jest według Ciebie mniej czytelne od poniższego?

if (variable) {

return true;

} else {

return false;

}
@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