#3 JS04(base 2): 8

오픈
gitgod1 년 전을 오픈 · 0개의 코멘트
gitgod 코멘트됨, 1 년 전

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명
로딩중...
취소
저장
아직 콘텐츠가 없습니다.