Skip to content

Conversation

@brooklynharden
Copy link
Contributor

This assignment is focuses on building ontop of the Library app that already has a foundation. My task is to alter the file MediaItemsController, where I need to implement API like GET, POST, & DELETE.

Copy link
Contributor

@anthonydmays anthonydmays left a comment

Choose a reason for hiding this comment

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

🎓 Automated Grading Report

Student: Brooklyn Harden
Date: 10/7/2025

Feedback

The submission demonstrates a good understanding of API development and HTTP protocol usage in Java. The student added methods for retrieving and deleting media items by ID, which aligns with the task requirements. However, there are some areas for improvement. The method getItemID duplicates functionality already present in the sample solution's getItem method, leading to redundancy. Additionally, the deleteItemId method should handle exceptions more robustly, similar to the sample solution's deleteItem method, which accounts for IllegalArgumentException. The use of Optional is appropriate for handling potential null values, but the code could be simplified by directly using isPresent() and get() without the intermediate Optional object. The technical implementation has minor issues with import organization and method naming conventions, which could be improved for clarity and consistency. Overall, the submission meets most functional requirements but falls short in technical execution and code organization.


This is an automated preliminary review. Please review and adjust before finalizing.

@anthonydmays anthonydmays reopened this Oct 8, 2025
}

@PostMapping("/items")
public ResponseEntity<Map<String, MediaItemResponse>> addItem(
Copy link
Contributor

Choose a reason for hiding this comment

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

Could just use MediaItemResponse right?

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.

2 participants