Python prints error message io.UnsupportedOperation: not readable

Question

I have searched this site for similar issues, but have not found any solutions that worked, hence, this question. I am writing a Python 3.4 program in which I have a function, export, that essentially appends data to a text file. The function checks to make sure that there is an appropriate file, if not, it creates one, and then it gets the contents of the file, adds the addendum, and overwrites the file. Python is throwing the error at for line in file: Also, upon running this program again, once the text file is already created, this error doesn't occur. Here is the function:

def export(addendum, user):
    filename = user + '.txt'
    try:
        file = open(filename, 'r')
    except OSError:
        file = open(filename, 'w')
        export(addendum, user)
    file_contents = ''
    print('What day is it? (1-5)')
    day = input()
    day = int(day)
    if day >= 1 and day <= 5:
        for line in file:
            file_contents += line
        file = open(filename, 'w')
        new_file = file_contents + '\n' + addendum
        file.write(new_file)
        file.close()
    else:
        print('Invalid weekday number...')
        sys.exit()

jwilner · Accepted Answer · 2015-04-09 03:10:36Z

4

It's happening when the file doesn't exist yet because that's when the file was opened in write mode. Write mode is not readable.

My understanding of what's happening here is when the file doesn't exist on the first call, your except block opens a file and puts one there; you then recurse for some reason, it hits the first block on this call, and completes at that level of the stack; when it returns back up to the next level, your first invocation continues, but the file reference is still in write mode, whatever the other levels of the stack have done. When it reaches the for line in file it blows up.

I would suggest you greatly simplify what you have going on here.

def export(addendum, user):
    filename = user + '.txt'
    try:
        with open(filename, 'r') as file:
            contents = file.read()
    except OSError:
        contents = ""
    day = input("What day is it (1-5)?")
    day = int(day)
    if not (1 <= day <= 5):
        print("Invalid weekday number...")
        sys.exit()
    contents += '\n' + addendum
    with open(filename, 'w') as file:
        file.write(contents)

answered Apr 9, 2015 at 3:10

jwilner

6,6067 gold badges39 silver badges48 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

Brandon Over a year ago

I see. Thank you greatly for your assistance. I just was using the loop to avoid running into memory limitations, though from what I understand, in python 3 on a modern system that isn't really an issue. Also, I need to look into the with keyword. Looking at it now, I'm not really sure why I used recursion. Again, thank you, I've been trying to debug this for the last hour.

Collectives™ on Stack Overflow

Python prints error message io.UnsupportedOperation: not readable

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related