Unclear nesting for compose_ticker_url_function(ticker)

[epogrebnyak]

Не очень понятно зачем оборачивать функцию ticker_url - видимых выигрышей нет:

https://github.com/WLM1ke/PortfolioOptimizer/blob/c87a3b69355fd583cb975a41d18b9594c0bb46df/src/portfolio/loader_history.py#L38-L49

[epogrebnyak]

Также сейчас в конструкторах url много повторяющегося кода, при этом семантически есть похожие составляющие:

• база url
• тикер.json, типа MCFTRR.json
• дата начала
• сдвиг

Можно сделать общий конструктор (функцию) и передавать ему параметры, параметры могут фиксироваться внутри классов.

'http://iss.moex.com/iss/history/engines/stock/markets/index/boards/RTSI/securities/MCFTRR.json'
'https://iss.moex.com/iss/history/engines/stock/markets/shares/securities/{ticker}.json'

[WLM1ke]

Реализовано