search

Kirill-Rush / financier

0 stars 0 forks source link

Refactor issue #1

Open LollyBomb-hub opened 2 years ago

LollyBomb-hub commented 2 years ago

Файловая организация проекта - хромает

Не оптимально. Оптимизировать. Возможно использовать enum с конструктором(см. стандарты C++) https://github.com/Kirill-Rush/financier/blob/926ffacbbf5c6fa0581bc70efbd22c8f43721981/additionalwindow.cpp#L23-L117

Не факт, но эта часть код оптимизируется с помощью цикла + макроса + перечисления https://github.com/Kirill-Rush/financier/blob/926ffacbbf5c6fa0581bc70efbd22c8f43721981/mainwindow.cpp#L17-L36

За регулярку отдельный лайк.))

Можешь ещё почитать про gitlab-ci.yml(Будет прикольно, если здесь реализуешь)

В целом. Прям претензий к написанию кода - нет. Только возможности к оптимизации.

LollyBomb-hub commented 2 years ago

@Kirill-Rush

LollyBomb-hub commented 2 years ago