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

Under-the-hood improvement: separate data model from output model #123

Open
croesnick opened this issue Nov 26, 2021 · 0 comments
Open

Under-the-hood improvement: separate data model from output model #123

croesnick opened this issue Nov 26, 2021 · 0 comments

Comments

@croesnick
Copy link
Contributor

Situation

Currently, there is just one model used throughout the whole extension: the CsvEntry.
This model, however, is tightly coupled with the storage format.

Problem

One model does not fit all. For instance, the location information is stored as a string in this model. But this is a choice only reasonable for the csv export, but not at other places in the program, where having explicit access to, say, the stating line would be needed.

Solution

Introduce a program internal data model.
Also: Consider applying the Repository Pattern to reduce the coupling to the explicit Csv output format. This would also make switching to future output formats (json? yaml?) easier.

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

No branches or pull requests

1 participant