Skip to content

Fix surface area output & book search syntax in Part 8 examples #469

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

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

Conversation

rohitdaivat
Copy link

Corrected the miscalculation in the sample output for the Rectangle class example. Before, the area of rectangle with width 39 & height 80 was 3920. The correct answer is 3120.

Change Made

  • Update the sample output into correct surface area calculation (3120 instead of 3920).

fixed the miscalculation in the surface area output for the rectangle class example. Before, it was 3920 (for width 39 & 80 height). the correct surface area calculation is 39*80 = 3120.
@rohitdaivat rohitdaivat marked this pull request as ready for review October 31, 2024 11:14
Fix missing closing parenthesis in the if-statement that searches
for "Sense and Sensibility". This caused a compilation error.

Located in Part 8 - Section 2: Hash Map lesson.
@rohitdaivat
Copy link
Author

rohitdaivat commented Apr 6, 2025

🔧Update: I noticed another small issue in the Part 8 section (Part 8, Section 2).
The code example for searching a book is missing a closing parenthesis in the if condition.
I've added the fix to this pull request.

@rohitdaivat rohitdaivat changed the title Fix surface area calculation in Rectangle class example output Fix surface area output & book search syntax in Part 8 examples Apr 6, 2025
A semicolon was missing after book.getName() in the addBook method of the Library class, which caused a compilation error.
@rohitdaivat
Copy link
Author

🔧Update: I noticed another small issue in the same section (Part 8, Section 2).

A semicolon was missing after book.getName() in the addBook method of the Library class, which caused a compilation error. This has been fixed.

@rohitdaivat rohitdaivat reopened this Apr 11, 2025
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.

1 participant