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

Remove dependency on reading all projects when it is not needed #17

Open
OliverWich opened this issue Sep 12, 2024 · 0 comments
Open

Remove dependency on reading all projects when it is not needed #17

OliverWich opened this issue Sep 12, 2024 · 0 comments

Comments

@OliverWich
Copy link
Collaborator

Only truly use getAllProjects() when needing all projects. Otherwise create a Project "blindly" just by name and then validate it exists with a single read call that can be the same as the read calls in the constructor.

This might save precious I/O resources if having a lot of projects on the disk.

Hier ist mir aufgefallen, dass deine getProjectForName Funktion ja ein neues Project erstellt, wäre es hier nicht sinnvoller die Logik von dem delete aus der Project Entity rauszunehmen und direkt in den Service einzubauen?

Und jetzt gerade holst du erst alle Projekte und checkst ob das eine existiert, da wäre es auf Dauer vielleicht auch performanter spezifisch nach dem einen Ordner zu suchen ob er existiert, da könnte man ja ein doesProjectExist in den Project Service einbauen.

Originally posted by @gutentag2012 in #15 (comment)

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