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.

Sign up for GitHub

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

Jump to bottom

Debugging Branch #29

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

Debugging Branch #29

wants to merge 9 commits into from

Conversation

SavixDR
Copy link

@SavixDR SavixDR commented Dec 13, 2022

There were some errors and I have debugged them.
I made the DatabaseHelper.java class a singleton.

SavixDR and others added 9 commits December 12, 2022 18:51
@SavixDR
According to the Lab sheet I have implemented following classes.
1117bde 
>>PersistentAccountDAO.java
>>PersistentTransactionDAO.java
>>PersistentExpenseManager.java

Other than these 3 classes I have created a class to help manage my Database
>>DatabaseHelper.java

Using these classes I have changed SimpleExpenseManager to have a persistent storage.
@SavixDR
Delete BuildConfig.java
16be7a9
@SavixDR
Delete BuildConfig.java
058ceed
@SavixDR
Delete DatabaseHelper.java
12fc5c5
@SavixDR
Delete PersistentExpenseManager.java
87605a8
@SavixDR
Update MainActivity.java
8d9168e
@SavixDR
Delete PersistentAccountDAO.java
1863d7c
@SavixDR
Delete PersistentTransactionDAO.java
91c3f3c
@SavixDR
DatabaseHelper.java made singleton
5830de1 
Some Errors Fixed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@SavixDR