Комаров Ярослав, задания 1 - 5#3
Комаров Ярослав, задания 1 - 5#3YaroslavKomarov wants to merge 3 commits intoRTF-Angular-2021:masterfrom
Conversation
| public isCardTiedToUser(cardId: any): any { | ||
|
|
||
| public isCardTiedToUser(cardId: string): boolean { | ||
| for(const user of this._users) { |
There was a problem hiding this comment.
Почему здесь не используешь this._users.forEach ?
There was a problem hiding this comment.
Исходя из следующего Вашего комментария, не лучше ли будет в методах использовать просто "for"? Или хорошим тоном является все-таки использование "forEach"?
| this._users.forEach(user => { | ||
| if (user.id === userId){ | ||
| if (user.cards.find(card => card.id === cardId && card.pin === cardPin)){ | ||
| return true; |
There was a problem hiding this comment.
Этот метод не отработает, он сделает ретурн для функции .foreach ( callbaack() { return }) , а не для родительской. Попробуй проверить. Авторизация всегда будет false.
| } else { | ||
| return this.registerForUserNewCard(argsForChangeFunction); | ||
| } | ||
| } |
| return this.takeUsersMoney(convertUnits); | ||
| } | ||
| return false; | ||
| } |
src/task_4/index.ts
Outdated
| moneyUnits.forEach(moneyUnit => { | ||
| const tmpObj: IMoneyUnit = { | ||
| moneyInfo: { | ||
| denomination: (+moneyUnit.moneyInfo.denomination * coeff).toString(), |
There was a problem hiding this comment.
Номинал тут считать не надо. Потому что он фиксированный и не может быть дробным.
Тебе тут надо сумму посчитать и вызвать метод MoneyRepository.giveOutMoney передав туда это количество денег и валюту в которую надо превратить, метод MoneyRepository уже сам всё сделает.
3 participants
No description provided.