Improve `/balance` command performance

[rodrigo-brito]

When we request the balance summary, ninjabot will request the position for each trading pair and report a summary. But in this case, we request the same function multiple times: binance.Account()

https://github.com/rodrigo-brito/ninjabot/blob/1a3585bedbaf66ba35e9377635877aa688e5a1e0/exchange/binance.go#L444-L455

A good approach to improve the performance are: Request binance.Account() once and calculate the summary for all pairs in the same time.

[panapol-p]

I would like to do this feature ;)

[rodrigo-brito]

Hi @panapol-p, thank you for the help!

[panapol-p]

I find better solution for this issue, I will work on it first, the pull request was cloesd that need implement 2 function with the same functional for new exchange it is not good I think

[rodrigo-brito]

Ok @panapol-p, thank you for the help!

[panapol-p]

Hi @rodrigo-brito after I try to work with my new idea, it's not worked now I re-open the pull request. could you please review this one thank you.

[rodrigo-brito]

Hi @panapol-p, thank you. I will check soon.

[panapol-p]

Hi, @rodrigo-brito I think this issue was fixed. may be we can close this issue. or you have any suggestion for this one?

[rodrigo-brito]

Sure! Closing due #184