ShayanEdalatjoo/HW-06-exceptions-and-file-handling
1
0
Fork 0
forked from AdvancedProgramming1404/HW-06-exceptions-and-file-handling
Code Pull Requests 1 Activity

Develop #1

Open
ShayanEdalatjoo wants to merge 2 commits from develop into main
pull from: develop
merge into: :main
Conversation 1 Commits 2 Files Changed 2
+92 -70
ShayanEdalatjoo commented 2026-07-16 21:01:57 +00:00
Owner
Copy Link
Copy Source
No description provided.
ShayanEdalatjoo added 2 commits 2026-07-16 21:01:57 +00:00
compliting loadProducts() and processFile() functions. 3763ddd3ab
completing saveReport() function. c07c5caf52
ShayanEdalatjoo requested review from HoseinA05 2026-07-16 21:01:57 +00:00
HoseinA05 requested changes 2026-07-30 10:59:49 +00:00
HoseinA05 left a comment
Collaborator
Copy Link
Copy Source

Grades:

Main: (300*0.7)/500

flaws:

  • Code does not run.
  • Assuming product IDs are sequential, but they're not necessarily, and the task says to look for the product ID but here you're just getting the product with index=ID, it's a fragile logic!
  • Late submission
Grades: Main: (300*0.7)/500 flaws: - Code does not run. - Assuming product IDs are sequential, but they're not necessarily, and the task says to look for the product ID but here you're just getting the product with index=ID, it's a fragile logic! - Late submission
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin develop:develop
git checkout develop
Sign in to join this conversation.
Reviewers
No Reviewers
HoseinA05
Labels
No items
No labels
Milestone
No items
No Milestone
Assignees
Clear assignees
HoseinA05 (Hosein Asfiaee) ShayanEdalatjoo
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ShayanEdalatjoo/HW-06-exceptions-and-file-handling#1