Skip to content

Assignment 2#2

Open
AnaKanzaTariq wants to merge 4 commits intomainfrom
assignment-2
Open

Assignment 2#2
AnaKanzaTariq wants to merge 4 commits intomainfrom
assignment-2

Conversation

@AnaKanzaTariq
Copy link
Owner

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)

reading writing opening files via python

What did you learn from the changes you have made?

syntax

Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?

I was using the normal approach

Were there any challenges? If so, what issue(s) did you face? How did you overcome it?

It was difficult to create python codes

How were these changes tested?

return function

A reference to a related issue in your repository (if applicable)

Checklist

  • [Y] I can confirm that my changes are working as intended

Copy link

@tianyi21 tianyi21 left a comment

Choose a reason for hiding this comment

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

Hi Ana Kanza,

⚠️ You implemented the function to read and display a file.
✅ You completed the patient_summary() function to summarize a file.
✅ You implemented the detect_problems() function to check the file.

⚠️ In your current version, all contents are being printed as a single list. Based on the instructions given, you're expected to print each row for inspection. Also, to clarify, f is the file handler but not the file content itself. Therefore, your code for item in f didn't produce any output. Please follow the commented lines to revise your code.

Hints:

  • Line 1: You're almost correct. To fix this, please read the file into a variable.
  • Line 2: This is also close. Instead of iterating through the file handler f, please iterate through the variable you declared in the last line.

Thanks,
Tianyi [LS]

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

Comments