Conversation
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Marcelu,
Tutaj jest parę rzeczy do poprawy :) poza nimi jest ok!
| age: 48, | ||
| } | ||
|
|
||
| console.log(user.firstName, user.lastName, user.sex, user.age); No newline at end of file |
| console.log( | ||
| calendarJS[date] === null ? `${date} nie zostało wydane` : `${date} wydano w roku ${calendarJS[date]}` | ||
| ); | ||
| } |
03/app.js
Outdated
| } | ||
|
|
||
|
|
||
| if (lang === 'en') { |
There was a problem hiding this comment.
Zdecydowanie mało elastyczne. Co jeśli pojawią się inne języki? Lepsza opcja to if(title[lang]) - jeśli występuje tytuł w danym języki to... np. zwróć :)
03/app.js
Outdated
| return null; | ||
| } | ||
|
|
||
| if (translator[lang] === null) { |
There was a problem hiding this comment.
nie zawsze jest to null domyślnie (bez ustawienia) jest to undefined
03/app.js
Outdated
| return false; | ||
| } | ||
|
|
||
| if (lang === 'en') { |
04/app.js
Outdated
| return 'Wszystkiego najlepszego z okazji dzisiejszych urodzin!'; | ||
| } | ||
| else { | ||
| return 'Dzis nie masz urodzin' |
There was a problem hiding this comment.
Lepiej zwrócić true/fasle, a potem zwracać komunikat, bo może być on np. różny - co innego napiszesz babci, co innego dziecku, a jeszcze inaczej partnerce ;)
PS. W szczególności, że nazwa funkcji wskazuje sprawdzenie tak/nie - zamiast konkretnego komunikatu. Oczywiście się czepiam, ale w przyszłości może to być pomocne ;P
devmentor-pl
left a comment
devmentor-pl
left a comment
There was a problem hiding this comment.
Marcelu,
Tutaj jeszcze drobna uwaga :)
| return title[lang]; | ||
| } else { | ||
| return title.pl; | ||
| return null; |
| } | ||
|
|
||
| } | ||
| if (checkBirthday(user)){ |
| } | ||
|
|
||
| if (translator[lang] === null) { | ||
| if (translator[lang] === null || translator[lang] === undefined) { |
There was a problem hiding this comment.
Tutaj bym uważał. W zadaniu jest jasno napisane, że null więc tylko bym sprawdzał tą wartość.
undefined to co innego - wtedy ma być zwracane null (ostatni warunek, który masz)
2 participants
No description provided.