Flowseal / TravelPractice

0 stars 0 forks source link

lesson-8: Оператор приведения лучше стараться не использовать #11

Closed LehaIvanov closed 1 year ago

LehaIvanov commented 1 year ago

https://github.com/Flowseal/TravelPractice/blob/8e8d193b2c2ee52de62bb03317dd298d916e160b/Lesson8%20(Currency%20Converter)/WebAppUI/src/api-calls/FetchPrice.ts#L27

Оператор as отключает проверку типов Typescript, поэтому его лучше не использовать. В данном случае без него можно обоётись.

Я бы вообще код во втором then переписал бы следующим образом:

...
.then((data: PriceDto[]) => {
  const prices: CurrencyPrice[] = data.map(dto => ({
    dateTime: new Date(dto.dateTime),
    price: dto.price,
  }));

  return {
    [`${payment}/${purchased}`]: prices,
    [`${purchased}/${payment}`]: prices.map(price => ({ price: 1 / price.price, dateTime: price.dateTime })),
  };
});
LehaIvanov commented 1 year ago

Просьба исправить не только в данном месте, но и в других местах тоже.