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

Resolved issue #1323 Two-Step View pattern #3106

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sharara6
Copy link

@sharara6 sharara6 commented Dec 3, 2024

created a two-step view design pattern for more details check the read-me file

close issue #1323

@sharara6
Copy link
Author

sharara6 commented Dec 8, 2024

@iluwatar may you review this PR ?

@sharara6
Copy link
Author

Dear @iluwatar may you check my PR?

@sharara6
Copy link
Author

This is a gentle follow up.

@@ -0,0 +1,79 @@
# Book Data Processing and Presentation
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

README.md has a very specific format that needs to be implemented by all the pattern examples. See here: https://github.com/iluwatar/java-design-patterns/wiki/01.-How-to-contribute

Comment on lines +40 to +44
<properties>
<maven.compiler.source>17</maven.compiler.source>
<maven.compiler.target>17</maven.compiler.target>
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
</properties>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comes already from parent pom.xml

<artifactId>junit</artifactId>
<scope>test</scope>
</dependency>
</dependencies>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add maven-assembly-plugin to allow executing the jar from the command line. See the other patterns for examples.


import java.util.logging.Logger;

/** Main class. */
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here, above the App class, explain the pattern and describe how this example implements it.

// Convert raw book data into a structured format
BookStore preparedData = DataPreparation.prepareBook(book);

String htmlOutput = Presentation.presentBook(preparedData);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add comment

@sharara6
Copy link
Author

Thank you for your review. I am in the midst of final exams right now. May I work on it next week?

@iluwatar
Copy link
Owner

Thank you for your review. I am in the midst of final exams right now. May I work on it next week?

Sure, no problem. Good luck with the exams 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants