New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Docs: improvements #5

Open

AnthonyAkentiev opened this issue Oct 7, 2018 · 0 comments

Assignees

Member

AnthonyAkentiev commented Oct 7, 2018

1

в code sample
заменить
WeiRelativeExpenseWithPeriod reserveFundÂ =newWeiRelativeExpenseWithPeriod(250000,0,false);
WeiRelativeExpenseWithPeriod dividendsFundÂ =newWeiRelativeExpenseWithPeriod(750000,0,false);

на фонды

явно сделать именованные переменные для параметров, чтобы было понятно

3

code example в конце
что произойдет, если мы сначала заполним OneTimeFund полностью фонд, потом сделаем из него flush()?
он опять будет собирать деньги?
названия всех элементов переименовал
WeiFund
WeiFundWithPeriod
WeiFundWithPeriodSliding
WeiAbsoluteExpenseWithPeriodSliding
WeiRelativeExpenseWithPeriodSliding

вот таски https://github.com/Thetta/Moneyflow/projects/1
https://github.com/Thetta/Moneyflow/projects/3

4

code examples добавить/пополнить

5

мне кажется, что если в Destination называлось flush/flushTo, то аналогично должно и в Table быть: flushForElement()
переименовать element -> node или item везде в table

MoneyflowTable is distinguished from Destination by the fact that it haveÂ withdrawFundsFromElement()instead of flush()/flushTo().

AnthonyAkentiev assigned enkogu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment