Можно ли так писать код?

Question

[vlad322111]

Стоит задача принять массив с данными, на основе которых нужно сделать асинхронные запросы, может возникнуть ошибка, по итогу нужно вернуть результаты успешных запросов. Главный вопрос - правильно ли у меня расставлены типы, как лучше сюда внедрить typescript?

import { Idividend, ImainStock } from "~/types/stocks/mainStock";
import { ItinkoffAPITickers } from "~/types/stocks/tinkoffAPITickers";

export default async function(tickers: Array<ItinkoffAPITickers>): Promise<Array<ImainStock>> {
  console.log("[API fetchHistoricalDiv.js]");

  const currentDate: number = Math.floor(Date.now() / 1000);
  const twoPrevYears: number =
    new Date(new Date().getFullYear() - 2, 0).getTime() / 1000;

  let parsedStocks = await Promise.all(
    tickers.map(async (e: ItinkoffAPITickers): Promise<ImainStock | null> => {
      let ticker: string = '';
      
      if (e.currency === "USD") {
        ticker = e.ticker;
      } else if (e.currency === "RUB") {
        ticker = e.ticker + ".ME";
      } else if (e.currency === "EUR") {
        ticker = e.ticker.replace("@", ".");
      }

      try {
        const responsePriceAndDividendsByTicker = await fetch(
          `https://query1.finance.yahoo.com/v8/finance/chart/${ticker}?period1=${twoPrevYears}&period2=${currentDate}&interval=1mo&includePrePost=true&events=div%7Csplit`
        );
        const dataPriceAndDividendsByTicker = await responsePriceAndDividendsByTicker.json();

        const price =  dataPriceAndDividendsByTicker.chart.result[0].meta.regularMarketPrice;
        const dividends: Array<Idividend> = Object.values(dataPriceAndDividendsByTicker.chart.result[0].events.dividends)

        return {
          text: `[${e.ticker}] - ${e.name}`,
          value: {
            ...e,
            price,
            dividends
          }
        };
      } catch (error) {
        console.log("error", error);
        return null
      }
    })
  )

  const resulStocks = parsedStocks.filter((e): boolean => !!e ) as Array<ImainStock>
  
  return resulStocks;
}

Comments

[WbICHA]

Почему в одних местах ты типизируешь даже то, что не надо типизировать:

tickers.map(async (e: ItinkoffAPITickers)

const currentDate: number = Math.floor(Date.now() / 1000);
const twoPrevYears: number =
    new Date(new Date().getFullYear() - 2, 0).getTime() / 1000;

.filter((e): boolean => !!e )

А в других местах НЕ типизируешь то, что надо:

const dataPriceAndDividendsByTicker = await responsePriceAndDividendsByTicker.json();

[WbICHA]

И ещё, почему ты типизируешь массив через Array? Это, конечно, дело вкуса, но, имхо, ImainStock[] смотрится и выделяется лучше.

[WbICHA]

И, кстати, не:

parsedStocks.filter((e): boolean => !!e ) as Array<ImainStock>

а:
parsedStocks.filter((e): e is ImainStock => !!e )

[vlad322111]

WbICHA, почему этот участок не надо типизировать, лучше же знать, что функция получает ?
tickers.map(async (e: ItinkoffAPITickers)

[vlad322111]

WbICHA, спасибо за комментарий, а чем эти два подхода отличаются?

[vlad322111]

WbICHA, я про метод фильтр

[WbICHA]

vlad322111,

лучше же знать, что функция получает ?

Вот именно поэтому форычи, мапы, фильтры и прочее лучше не типизировать, чтобы случайно не изменить тип переменной.

чем эти два подхода отличаются?

В первом случае фильтр возвращает массив того же типа, что и оригинальный массив, после чего ты его принудительно ("грубо") изменяешь.
Во втором случае ты определяешь ("мягко") какой тип вернёт фильтр.

[vlad322111]

WbICHA, спасибо большое за ответы!

Answer 1

[Aetae]

Всякие детали - это дело вкуса, на опасности вам в комментах указали, как и на главную проблему:

const dataPriceAndDividendsByTicker = await responsePriceAndDividendsByTicker.json();

Однако вот тут стоит остановиться по-подробнее.
Даже если вы "типизируете" напрямую возврат от сервера, нет никаких гарантий, что там придёт именно то, что надо. В таком случае вы просто "обанываете" TypeScript. Для всего приходящего с сервера стоит писать хотя-бы минимальный тайпгард.