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

Refactor Layered Architecture pattern #2936

Open
iluwatar opened this issue May 4, 2024 · 1 comment
Open

Refactor Layered Architecture pattern #2936

iluwatar opened this issue May 4, 2024 · 1 comment

Comments

@iluwatar
Copy link
Owner

iluwatar commented May 4, 2024

A couple of specific refactorings are needed for Layered Architecture code example.

  • Runner.java has no main entry point and can't be executed in the IDE. Let's add this.
  • 3 Warnings: Optional used as type for field id. This is related to classes CakeInfo, CakeLayerInfo, and CakeToppingInfo.

Optional is primarily intended for use as a method return type where there is a clear need to represent "no result," and where using null is likely to cause errors.

Acceptance criteria

  • Refactorings implemented
romannimets added a commit to romannimets/java-design-patterns that referenced this issue May 5, 2024
romannimets added a commit to romannimets/java-design-patterns that referenced this issue May 5, 2024
romannimets added a commit to romannimets/java-design-patterns that referenced this issue May 5, 2024
@romannimets
Copy link

Hey! I would like to contribute to this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

2 participants