[DiKey]

Właśnie tak Mireczki wygląda kod programisty 10k z 8-letnim doświadczeniem (w phpie xD) ( ͡€ ͜ʖ ͡€)
Powyższe metody zostały utworzone w commicie z jakże opisowym tytułem "branch update".
A zaryzykuję #programowanie #webdev #javascript #kod #programista15k #rakcontent #gorzkiezale

[B.....c]

@DiKey: przecież działa

[DiKey]

@BazaltowyTubylec: i to jak! ( ͡° ͜ʖ ͡°)

[ozzie]

@DiKey: adekwatne do poziomu 10k ( ͡° ͜ʖ ͡°)

[DiKey]

@interface: taka prawda, niestety w Polsce przy rekrutacji liczy się tylko komercyjne (podkreślam: komercyjne) doświadczenie.

[Ragnarokk]

@DiKey:
Czy ja wiem. Często widzę, że Open Sauce mile widziane i sporo firm ma swoje otwarte repka.
Chyba że mówisz o korpo.

[bonus]

@DiKey: A jak ty byś to zrobił?

[Benzen]

@bonus: Tak żeby działało ( ͡° ͜ʖ ͡°)

[DiKey]

@Ragnarokk: mówię z autopsji

[bonus]

@DiKey: W PHP pisz

[DiKey]

@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 (⌐

[DiKey]

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')"

[Dijuna]

@DiKey: ehkm...

abstract class Type
{
 const web = 0;
 const comments

[DiKey]

@cevilo: spoko, a po za tym dlaczego

return variable;

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

if

[DiKey]

@Dijuna: do tego to typy wyliczeniowe (enum) jak już ;) To #typescript, a nie #php.

[bonus]

@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