Kamaropoulos / COVID19Py

A tiny Python package for easy access to up-to-date Coronavirus (COVID-19, SARS-CoV-2) cases data.
https://pypi.org/project/COVID19Py/
GNU General Public License v3.0
84 stars 57 forks source link

Apply second aggregation #74

Open azhan619 opened 3 years ago

azhan619 commented 3 years ago

Hi,

There is another aggregation and most importantly encapsulation that I think could be applied to this code, it is about the way your code gets the COVID related data for a specific country. The current code uses 3 methods each has same output but different input of country name , id and code. Here, I have made a seperate class which takes one,two or even all three of these inputs within the class, it verifies the location from the dictionary containing all country names and their ISO-code. Then it calls 1 method only to retrieve the data, which reduces the redundancy in the code and makes it cleaner.