AY1920S1-CS2113T-F09-1 / main

Duke$$$ is a Command Line based Financial Tracker Assistant
0 stars 5 forks source link

Refactoring of API for currency conversion done as per approved one on forum to solve issue #96 #97

Closed priyan-coder closed 5 years ago

priyan-coder commented 5 years ago

Now we are using European Central Bank Exchange rate as base. So conversion are complicated now :

  1. Either base / to be converted is EUR
  2. Base and to be converted currency is non EUR

So the code had to be refactored quite a bit and hence exchange rates, especially for case 2 is derived by arithmetic calculation. Thus there is some minor loss of accuracy but still very close to expected output.