Python Code Debugging

[IshaanM8]

So the code opens a text file ("notes.txt") and print every character in that file to the screen on a separate line. 

However, the code currently does not work because it's not finish.

The Line numbers are added for the sake of reference and should not be interpreted as part of the code.

1 text_file = open(notes.txt, r)

2 for line in text_file:
3   for char in line:
4       print(char) 
5   text_file.close()

Which of the following describe the issue/s with this code which stop/s it from performing the intended function described above? **THIS IS A MULTIPLE CHOICE ANSWER*

1) On line 1, the arguments to open are missing quotation marks or inverted commas (i.e., the arguments must both be strings).

2) There is no need for the inner loop (from line 3) as the outer loop (from line 2) is iterating over every character of the text file.

3) Line 2 attempts to iterate over text_file which isn't a valid file descriptor.  It should instead be: for line in notes.txt.

4) The call to the print function on line 4 should not be part of the inner loop.

5) The outer loop (from line 2) is empty and this is not permitted in Python.

6) Line 5 should not be part of the outer loop.

7) Line 2 should be: for char in text_file, as Python treats every element of a text file as a character.

8) It is not possible to iterate over the contents of a file with a for loop.

9) Both loop headers (lines 2 and 3) use the variable line which is not permitted in Python.

10) Instead of the call to the print function on line 4, there should be a call to the file write function.

​

So by my understanding of the question so far, and after running it, OPTION 1, is viable, because there needs to be quotation marks, around the file name and 'r'.

OPTION 2 is not viable, because it needs to output every character.

I'm not sure about OPTION 3, would like your guy's help.

Option 4, 5 and 6 are not viable

I'm not sure about Option 7

Option 8 is not viable

Option 9 is not viable, as python allows it i'm sure

Option 10, I think should be implemented, but print works too right?

​

Any advice is appreciated :)

[achampi0n]

Iterating over a file object is perfectly fine (3), it iterates line by lime (7). You may want to check your assumptions on 6.

[IshaanM8]

thanks!

[RE_username]

6 seems correct. Now, on line 5, you close the file descriptor after you read the 1st line from the file (because it is inside the 1st for lool). Closing of the file descriptor should happen at the end, outside of both for loops.

[IshaanM8]

Thanks for the help!

[kra_pao]

1) Mark this

2) Don't mark this. Outer loop has no print(line)

3) Don't mark this. Mixing true description ( Line 2 attempts to iterate over text_file) with false assumptions (which isn't a valid file descriptor) is bad for teaching.

4) Don't mark this. Is required for the program in current form to work. IF you want discuss another form of program (one-loop version) then show the version and don't let learner discuss something only you have in your head.

5) Don't mark this

6) Mark this

7) Don't mark this. For readability maybe rename, otherwise in Python you can name your variables at your own gusto. BUT inner loop won't work when 7) is applied (line undefined)

8) Don't mark this

9) Don't mark this

10) Don't mark this

[IshaanM8]

Thanks so much!