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
feature: Add Monolithic Architecture pattern (#2664) #2760
base: master
Are you sure you want to change the base?
feature: Add Monolithic Architecture pattern (#2664) #2760
Conversation
SonarCloud Quality Gate failed. 0 Bugs 69.8% Coverage Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Closed due to inactivity. Thank you for your contributions. |
monolithic-architecture/README.md
Outdated
@@ -0,0 +1,167 @@ | |||
--- | |||
|
|||
title: Exploring the Monolithic Architecture in E-commerce |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest just 'Monolithic Architecture'
@@ -0,0 +1,42 @@ | |||
@startuml | |||
interface com.iluwater.monolithic.architecture.ICustomerService { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In this PR too, please update to com.iluwatar
:)
<properties> | ||
<maven.compiler.source>17</maven.compiler.source> | ||
<maven.compiler.target>17</maven.compiler.target> | ||
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding> | ||
</properties> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can be left out since they come from the parent pom.xml
/** | ||
* The main application class providing a command-line interface to interact | ||
* with customer and order modules. | ||
* <p> | ||
* The application provides menu-driven options to the user for performing various | ||
* operations like fetching customer details by ID, updating credit limits, | ||
* creating new customers, making orders, and fetching order totals for a given customer. | ||
* </p> | ||
* <p> | ||
* The underlying data operations are performed through {@link CustomerModuleInterface} and | ||
* {@link OrderModuleInterface} implementations. | ||
* </p> | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please describe the monolithic architecture pattern briefly and the explain how this example implements it
try { | ||
switch (choice) { | ||
case "1": | ||
System.out.print("Enter Customer ID: "); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to use proper logger. Use the one that Lombok provides.
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. |
Monolithic Architecture pattern (#2664)
Pull request description