#3 JS04(base 2): 8

オープン
1 年間 前gitgod によって開かれました · 0 コメント
gitgod1 年間 前 にコメントしました

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) в целом неплохо
会話に参加するには サインイン してください。
ラベルなし
マイルストーンなし
担当者なし
1 参加者
読み込み中…
キャンセル
保存
まだコンテンツがありません