Project Status
- Project has made progress with many basic features working
- Team has been getting things in order and establishing a better workflow
- Some members expressed uncertainty about whether everything is truly fixed
Bug Discoveries
- Withdrawals take money from the wrong account - when selecting account 1002, it withdraws from account 1000 instead
- System allows negative withdrawals which adds money to the account instead of subtracting
- Some unnecessary empty curly braces were found in the code
- Inconsistent error messages when entering invalid account numbers
Currency Exchange Implementation
- Current implementation is overly complex with two separate variables (amount1 and amount2)
- Suggestion to use a Dictionary<string, decimal> to store currency codes and their values
- Using SEK as base currency (value 1.0) simplifies conversion between currencies
- This approach would eliminate the need for multiple if-statements and make adding new currencies easier
- Current hard-coded values make updating exchange rates difficult
UI and Code Structure
- UI class should handle all console output for consistency
- Methods like PrintMessage() and PrintErrorMessage() should be implemented in the UI class
- This centralizes formatting (like colors) and allows easier design changes
- Using While(true) loops was identified as a bad practice that had been flagged before