It has been some time since the creation of faculties_tehran.py script (formerly Tehran.py). The script should be reviewed and refactored so that it uses the recent changes in the repo's programming style (like the extensive use of config.json).
Additionally, some helper functions that are useful across multiple scripts may be extracted from faculties_tehran.py to their own helpers.py module.
Suggested solution
Here are some tips:
Use more meaningful names for variables.
Use constants where possible.
Import constants from config.json.
Extract CSV I/O functions to a new sibling module, called helpers.py.
Simplify the script as much as possible (e.g. by removing the part for "education history" of the faculty members).
Feature description
It has been some time since the creation of
faculties_tehran.pyscript (formerlyTehran.py). The script should be reviewed and refactored so that it uses the recent changes in the repo's programming style (like the extensive use ofconfig.json).Additionally, some helper functions that are useful across multiple scripts may be extracted from
faculties_tehran.pyto their ownhelpers.pymodule.Suggested solution
Here are some tips:
config.json.helpers.py.Good Luck!