Skip to content

Added user confirmation before executing destructive actions #22

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

utkuoztekin58
Copy link

Prompts user before deleting or modifying data.

keremdoleksoz and others added 10 commits March 28, 2025 12:38
"Certain parts of the code lacked exception handling and did not throw exceptions when necessary. The appropriate 'throws Exception' statements have been added where required."
During any operation (such as logging out, depositing money, etc.), a confirmation screen appears asking, "Are you sure?" to verify the user's intent.
Some parts of the code contained errors related to variable visibility. The visibility issues for the necessary variables have been corrected, making the code more executable.
During any operation (such as logging out, depositing money, etc.), a confirmation screen appears asking, "Are you sure?" to verify the user's intent.
"Certain parts of the code lacked exception handling and did not throw exceptions when necessary. The appropriate 'throws Exception' statements have been added where required."
This class provides functionality to remove a bank account from the system based on the account number.
It reads the current data using FileIO.Read(), iterates through the BankAccount[] array, and sets the matching account to null if found.
After removal, it updates the account list and writes the changes back to the data file using FileIO.Write().
A success or failure message is printed to the console depending on whether the account was found and removed.
Some parts of the code contained errors related to variable visibility. The visibility issues for the necessary variables have been corrected, making the code more executable.
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

Successfully merging this pull request may close these issues.

6 participants