Skip to content
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.

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

ICurrencyConverter might need a datatime also passed in #9

Open
rahulpnath opened this issue Mar 9, 2016 · 1 comment
Open

ICurrencyConverter might need a datatime also passed in #9

rahulpnath opened this issue Mar 9, 2016 · 1 comment

Comments

@rahulpnath
Copy link
Collaborator

Just thinking out loud -
either the entire conversion could happen on a single data in which case it might be fine for Evaluate to take in the datetime, but there could be also Transactions of different dates that we are dealing with in which case it be along with the Transaction

@zpbappi
Copy link
Owner

zpbappi commented Mar 9, 2016

@rahulpnath I think you are expecting the operations happening over a long period of time (says, few days or week). One possible, but less preferred way of doing this would be to run the money calculator for that period of time. In that case, the solution would be to Evaluate the wallet as soon as you operate. However, that, as I mentioned, would be less preferred. Because everything will be lost if the system crashes. So, the more accurate of doing it would be to store the wallet into some permanent storage so that it can be brought into memory when needed. In that case, the DateTime stamp with each transaction will make sense. Along with that, the wallet also needs to be serializable. I am just trying to think a strong use-case for this feature, as this will require quite a large amount of change in the way I currently handle any mathematical operation, or transaction if you like.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants