Python, Conditional Issues

Question

I'm currently writing a database in python, with transitioning files, and they have all been working pretty well. My code after the login is completed is as follows:

PythonDatabase = list("/Users/*****/Desktop/PythonDatabase")



import os
flist = os.listdir(os.getcwd())
for name in PythonDatabase:
    PythonDatabase[PythonDatabase.index(name)]=name[:-3]
out = open('flist.txt','w')
for name in PythonDatabase:
    out.write(name+"\n")
out.close()



#------------------------------------------------------------------------------#
end = False
flist = open('flist.txt','r')

print("SECURITY PASSED")
print("ENTERING DATABASE")

def choosefile():
    fcho = input("ENTER FILE CHOICE, VIEW FILE LIST, OR END: ")

    if (fcho == "view file list" or fcho == "VIEW FILE LIST"):
        print (flist.read())

    elif (fcho == "END"):
        end = True


while (end == False):
    choosefile()

I blanked out my name with the *'s so you can ignore those. Most of this file is working fine, but if you see the while loop, that ignores the elif waiting for an end command. No matter how many times I give it the end command, it ignores, and continues. Help?

instead of end=True in your function, type return True. Also in the while body, instead of simply choosefile(), type end=choosefile(). Basically the function is not returning anything to the caller. — AlessioX
– AlessioX, Commented Feb 15, 2016 at 19:02
Sorry, I accidentally turned off my email notifications so I just saw these. I'll be sure to do so. — Brin Brody
– Brin Brody, Commented Feb 16, 2016 at 15:09

Idos · Accepted Answer · 2016-02-15 19:09:10Z

2

Your function does not return anything. I took the liberty and re-wrote your main part of the program:

end = False
flist = open('flist.txt','r')

print("SECURITY PASSED")
print("ENTERING DATABASE")

def choosefile():
    fcho = input("ENTER FILE CHOICE, VIEW FILE LIST, OR END: ")

    if (fcho == "view file list" or fcho == "VIEW FILE LIST"):
        print (flist.read())

    elif (fcho == "END"):
        return True


while (end == False):
    end = choosefile()

Now it actually returns a value (True) when it finishes, and end will be set accordingly. Keep in mind that the end variable you had inside your choosefile() function is not the same one as you defined outside it.

edited Feb 15, 2016 at 19:09

answered Feb 15, 2016 at 19:03

Idos

15.3k14 gold badges63 silver badges81 bronze badges

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

Comments

Brin Brody · Accepted Answer · 2016-02-16 15:11:55Z

0

Thanks to everyone who helped me out with this. The following is working code, the return statement worked like a charm. Thanks so much.

end = False
flist = open('flist.txt','r')

print("SECURITY PASSED")
print("ENTERING DATABASE")

def choosefile():
    fcho = input("ENTER FILE CHOICE, VIEW FILE LIST, OR END: ")

    if (fcho == "view file list" or fcho == "VIEW FILE LIST"):
        print (flist.read())

    elif (fcho == "END"):
        return True


while (end == False):
    end = choosefile()

answered Feb 16, 2016 at 15:11

Brin Brody

418 bronze badges

Collectives™ on Stack Overflow

Python, Conditional Issues

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related