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.

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

Project3311 #195

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

Conversation

simrankanda1911
Copy link

Under the folder Database Connector, there were three files db_bingcovid.py, db_maylasia_states.py and db_malaysia_patient_cases.py. Three of these files were using similar functions of connecting the database on mySQL. Using Factory Method I created one file with three different instantces of Factory_Demo with initial common functions and then their own implementation. Creational Design pattern here was used to showcase polymorphism and to avoid complexity of three different files where my one main class created instances to me implemented by 3 different classes.

For the aggregrate pattern implementation, I applied aggregrate pattern in twitter.py by creating a class called Tweet and made all the entitities as sub entities of the tweet and secondly I merged Bing_covid and scrape covid into one and applied aggregate pattern on that.
I pushed changes in both the files in my previous pull request here:
7fe6ae8. This is just a commented version!
…vid.py, db_maylasia_states.py and db_malaysia_patient_cases.py. Three of these files were using similar functions of connecting the database on mySQL. Using Factory Method I created one file with three different instantces of Factory_Demo with initial common functions and then their own implementation. Creational Design pattern here was used to showcase polymorphism and to avoid complexity of three different files where my one main class created instances to me implemented by 3 different classes.
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.

1 participant