Comments (1)
Team's Response
Hi this is intended. We did not try to solve this issue because we did not felt a need to do it. As long as the values entered by the user is correct (no rounding off issues or miscalculation of expenses), we do not try to rectify it. Having one decimal places although not the typical convention to represent money, the value that it is trying to represent is the same. If the user were to enter a value with 2 d.p, the total expense value will also show 2dp, hence there is no loss in accuracy of the amount.
as shown in the image, when I enter an expense of 4.5, the total expense shown is 4.5. There is no loss in precision of the value. When I added another expense with 4.55, the total expenses is now 9.05. The total expense now shows 2dp, which also does not result in any loss precision of the value. Hence we did not rectify the decimal problem.
Items for the Tester to Verify
❓ Issue response
Team chose [response.Rejected
]
- I disagree
Reason for disagreement: Yes while it might not affect functionality, that is not the be all end all of the program. The consistency especially with monetary values is important for readability for the users. Furthermore, your description of we didn't solve it because we didn't feel a need to is precisely the definition of response.notInScope, which is what it should be classified under.
from pe.
Related Issues (20)
- Complicated addExpense Command HOT 1
- I can edit expense to 0 HOT 1
- Income is not loaded on restart HOT 1
- Error in Command Component Sequence Diagram HOT 1
- Typo in DG EntryList component UML diagram HOT 1
- EntryList UML diagram has incorrect UML notation HOT 1
- Uneccessary and innacurate information in DG EntryList UML diagram HOT 1
- Incorrect Dependancy arrow in EntryList UML diagram HOT 1
- Wrong arrow notation used in UML diagram for UI HOT 1
- Missing arrow heads in UI UML diagram HOT 1
- UI class diagram errors HOT 1
- Adding large total expense causes hard to read format of total expense HOT 1
- Parser class UML class diagram errors HOT 1
- Grammatical error in intro message HOT 1
- Time format differences between commands HOT 1
- Adding spaces to time input to addExpense command results in wrong time format HOT 1
- Categories for addIncome does not fit its use case HOT 1
- Type SALARY of addExpense does not fit its use case HOT 1
- Program crashes do not save data HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from pe.