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

Paketstruktur in Java (im Backend) sauber trennen #287

Open
ErhardSiegl opened this issue Feb 14, 2020 · 5 comments
Open

Paketstruktur in Java (im Backend) sauber trennen #287

ErhardSiegl opened this issue Feb 14, 2020 · 5 comments
Labels
Blocked Enabler exploration, evolving the architecture, improving infrastructure, compliance activities Hogarama goes IoT

Comments

@ErhardSiegl
Copy link
Contributor

ErhardSiegl commented Feb 14, 2020

See comments in #283

ActorServiceImpl implements ActorService and there is a class ActorService in the same directory (class not interface). This is confusing. We have two kind of Actors. Real Actors and data structures that administer Actors. These two concepts are not well separated.

@mhofmann-gepardec
Copy link

@ErhardSiegl @eerofeev @philzelinger @patholzer
Clarification needed

@ErhardSiegl
Copy link
Contributor Author

In my opinion the whole topic of unit-management (creating, configuring of units, actors and sensors) should be in a Java package separated from watering or collecting sensor-data.

@mhofmann-gepardec
Copy link

mhofmann-gepardec commented Mar 13, 2020

@mhofmann-gepardec
Copy link

paketstrukturen

@mhofmann-gepardec mhofmann-gepardec added Enabler exploration, evolving the architecture, improving infrastructure, compliance activities Hogarama goes IoT labels Mar 27, 2020
@mhofmann-gepardec
Copy link

Blocked by #286 - neu evaluieren nach Fertigstellung von #286

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked Enabler exploration, evolving the architecture, improving infrastructure, compliance activities Hogarama goes IoT
Projects
None yet
Development

No branches or pull requests

3 participants