#3 JS04(base 2): 8

Ouvert
Créé il y a 1 an par gitgod · 0 commentaires
gitgod a commenté il y a 1 an

http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L148 - преждевременно спрашивается пароль;

http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L207-L212 - вместо кучи кода внутри тернарки лучше в тернарку засунуть только курс; результат тернарного оператора сохранить в переменную курса (rate), а prompt и умножение написать один раз ниже;

http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L276-L280 - лишнее сравнение - есть ничья (http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L268), есть выигрыш, все остальное - проигрыш;

http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L294-L318 - лучше использовать объект вида

let ratios = {
    usd: {sale: xxx, buy: yyy},
    eur: {sale: xxx, buy: yyy},
}

использование квадратных скобок подразумевает ненужность switch-case, так как в квадратные скобки можно вписать любое выражение, в том числе переменную с валютой (как это было сделано здесь: http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L330)

в целом неплохо

http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L148 - преждевременно спрашивается пароль; http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L207-L212 - вместо кучи кода _внутри_ тернарки лучше в тернарку засунуть только курс; результат тернарного оператора сохранить в переменную курса (`rate`), а `prompt` и умножение написать один раз ниже; http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L276-L280 - лишнее сравнение - есть ничья (http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L268), есть выигрыш, все остальное - проигрыш; http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L294-L318 - лучше использовать объект вида ```javascript let ratios = { usd: {sale: xxx, buy: yyy}, eur: {sale: xxx, buy: yyy}, } ``` использование квадратных скобок подразумевает ненужность `switch-case`, так как в квадратные скобки можно вписать любое выражение, в том числе переменную с валютой (как это было сделано здесь: http://gitlab.a-level.com.ua/suslov-dmytro/homework/src/master/03/src/App.js#L330) в целом неплохо
Connectez-vous pour rejoindre cette conversation.
Pas d'étiquette
Aucun jalon
Pas d'assignataire
1 Participants
Chargement…
Annuler
Enregistrer
Il n'existe pas encore de contenu.