try/except with multiple files

[Robert Hicks]

Is it good practice to do something like:

try:
f1 = file('file1')
f2 = file('file2')
except:
# catch the exception

Or do you do a try/except for each open?

Robert

[Matimus]

It depends, what are you going to do if there is an exception? If you
are just going to exit the program, then that works fine. If you are
going to just skip that file, then the above wont work. If you are
going to return to some other state in your program, but abort the
file opening, you might want to close any files that were opened. The
closing can be taken care if in the except block, but you will have to
know which ones opened successfully.

In general I would do something like this for multiple files:

filenames = ["fname1","fname2","fname3"]
for fn in filenames:
try:
f = open(fn)
except IOError:
# handle exception
#do something with f
 

But, that might not work for you if the files aren't homogeneous (each
have similar contents). If the files have distinctly different
purposes, I would just wrap them each in their own try/except block.

I rambled a bit there, but I hope it helps.

Matt

[Robert Hicks]

On Jun 21, 3:11 pm, Matimus <[e-mail]:

It depends, what are you going to do if there is an exception? If you
are just going to exit the program, then that works fine. If you are
going to just skip that file, then the above wont work. If you are
going to return to some other state in your program, but abort the
file opening, you might want to close any files that were opened. The
closing can be taken care if in the except block, but you will have to
know which ones opened successfully.

In general I would do something like this for multiple files:

filenames = ["fname1","fname2","fname3"]
for fn in filenames:
    try:
        f = open(fn)
    except IOError:
        # handle exception
    #do something with f
 

But, that might not work for you if the files aren't homogeneous (each
have similar contents). If the files have distinctly different
purposes, I would just wrap them each in their own try/except block.

I rambled a bit there, but I hope it helps.

Matt

Helps a bunch...thank you very much.

Robert

[bruno.desthuilliers]

On Jun 21, 9:00 pm, Robert Hicks <[e-mail]:

Is it good practice to do something like:

try:
f1 = file('file1')
f2 = file('file2')
except:
# catch the exception

It's bad practice. Because you use a bare except clause, and don't do
anything useful with the exceptions you catch.

Or do you do a try/except for each open?

If what you want is to make sure that resources will be released, you
can use a try/finally block or (Python 2.5) a with block.

[Stargaming]

Matimus wrote:

It depends, what are you going to do if there is an exception? If you
are just going to exit the program, then that works fine. If you are
going to just skip that file, then the above wont work. If you are
going to return to some other state in your program, but abort the
file opening, you might want to close any files that were opened. The
closing can be taken care if in the except block, but you will have to
know which ones opened successfully.

In general I would do something like this for multiple files:

filenames = ["fname1","fname2","fname3"]
for fn in filenames:
    try:
        f = open(fn)
    except IOError:
        # handle exception
    #do something with f
 

But, that might not work for you if the files aren't homogeneous (each
have similar contents). If the files have distinctly different
purposes, I would just wrap them each in their own try/except block.

I rambled a bit there, but I hope it helps.

Matt

Heh, reminded me of http://worsethanfailure.com/Articles/Code-Reuse.aspx
at the first glance. SCNR.

To add something on-topic, I'd just stick with the "one try for all
files" option if your need is really "Try to open all files, if *at
least one* of them fails, recover/abort/foobar/...".

[Scott David Daniels]

br*****************@gmail.com wrote:

On Jun 21, 9:00 pm, Robert Hicks <[e-mail]:

>Is it good practice to do something like:

try:
f1 = file('file1')
f2 = file('file2')
except:
# catch the exception

If what you want is to make sure that resources will be released, you
can use a try/finally block or (Python 2.5) a with block.

You could do something like this:

files = []
try:
for name in ['abc.txt', 'def.txt', 'ghi.txt']:
files.append(open(name))
a, b, c = files
<code using the three files>
finally:
while files:
files.pop().close()

--Scott David Daniels
sc***********@acm.org