Dividing Application Logic into Folder Structure #46
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Move shared mutable datatypes into a config file Provide helpers with minimal interactions and heavy-use an isolated environment to import from
Transported basic Rate function into utils Expanded file structure for further modularity
- Add required keywords for global variables - Reorganize into standard and business custom modules - Keep config as the cross-file mutable variable storage location
Isolates the finisher and update_latest functions with their distinct dynamic website handling purpose Introduces a dest_VAR methodology for QtWebEngine to access otherwise impossible data values TODO: Verify smooth runtime TODO: Alter checker logic to pass more arguments and accept updated output
Place WebPage class within finisher function scope to access dynamic_mangas variable Provide finisher function calls with additional arguments to maintain knowledge about current program state Stop short of re-assigning current or dynamic_indexes because of their end of computation ordering and independent changes to the data text files
JacobK233811
added
the
best practice
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Store global variable definitions in config.py and stateless helper functions in utils.py
Address the full-scope of dynamic website handling in dynamic.py alongside the database update (editing latest.txt) which naturally follows