Skip to content

completed OOP homework#2

Open
yared-abera wants to merge 1 commit intomainfrom
oop-homework
Open

completed OOP homework#2
yared-abera wants to merge 1 commit intomainfrom
oop-homework

Conversation

@yared-abera
Copy link
Owner

this code is modified in order to accept the input from the user since our code have to be user interactive

@yared-abera yared-abera self-assigned this Oct 28, 2023
Copy link

@Benawi Benawi left a comment

Choose a reason for hiding this comment

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

Hi @yared-abera 👋

Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

almost

Highlights

  • Invalid code review request
  • You did not commit the required files/classes to the PR

Required Changes ♻️

Check the comments under the review.

Optional suggestions

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.

Cheers, and Happy coding!👏👏👏

Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.
@Benawi

Copy link

@Abdlkerimshemsu Abdlkerimshemsu left a comment

Choose a reason for hiding this comment

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

It looks like you've made a good start on your Java assignment. and used many class and mathod fluntly and created good methods to modularize your code. so i recommed you to use your performance (skill) and you have to practice every day

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

Successfully merging this pull request may close these issues.

3 participants