Python try/except not working
Asked Answered
B

7

14

Trying to get the try/except statement working but having problems. This code will take a txt file and copy the file that is in location row 0 to location of row 1. It works however if i change one of the paths to invalid one it generates an error ftplib.error_perm however the except command is not picking up and everything stops. What am i doing wrong? Python 2.4

import csv
import operator
import sys
import os
import shutil
import logging
import ftplib
import tldftp

def docopy(filename):
        ftp = tldftp.dev()
        inf = csv.reader(open(filename,'r'))
        sortedlist = sorted(inf, key=operator.itemgetter(2), reverse=True)
        for row in sortedlist:
                src = row[0]
                dst = row[1]
                tldftp.textXfer(ftp, "RETR " + src, dst)


def hmm(haha):
    result = docopy(haha);
    try:
        it = iter(result)
    except ftplib.error_perm:
        print "Error Getting File" 


if __name__ == "__main__":
        c = sys.argv[1]
        if (c == ''):
                raise Exception, "missing first parameter - row"
        hmm(c)
Biafra answered 22/1, 2013 at 13:54 Comment(1)
Aside: that's some weird looking indentation, and that's going to lead to problems. Run your code with python -tt your_program_name.py to confirm the inconsistent whitespace use, and then switch to 4-space indentation everywhere.Oviposit
P
11

The except clause will only catch exceptions that are raised inside of their corresponding try block. Try putting the docopy function call inside of the try block as well:

def hmm(haha):
    try:
        result = docopy(haha)
        it = iter(result)
    except ftplib.error_perm:
        print "Error Getting File" 
Pinkiepinkish answered 22/1, 2013 at 13:57 Comment(2)
That made it so it printed the "error getting file", however shouldn't the try command then pick back up and continue with the rest of the result?Biafra
@Biafra Nope, once there is an error in the try block execution of code in the try block is done and will not start again.Subjoin
U
4

The point in the code which raises the error must be inside the try block. In this case, it's likely that the error is raised inside the docopy function, but that isn't enclosed in a try block.

Note that docopy returns None. As such, you will raise an exception when you try to make an iter out of None -- but it won't be a ftplib.error_perm exception, it'll be a TypeError

Ulpian answered 22/1, 2013 at 13:56 Comment(0)
S
1

If you are not sure of what exception will occur, the use the code below, because if especifies for example: except StandardError: and is not that error the exception will not be process.

try:
    # some code
except Exception: # Or only except:
   print "Error" # Python 3: print("Error")
Splash answered 23/8, 2019 at 20:38 Comment(0)
G
0

I know the OP is ancient, but for folks desperate for answers on this question. I had a similar issue, depending on your IDE, if you have a breakpoint on any of the lines with specific exceptions etc, this can conflict and stop try/except executing.

Goddord answered 19/10, 2017 at 16:21 Comment(0)
M
0

I noticed global exception may not works, e.g. , Ctrl+C when epub.py module perform urllib3 connection trigger KeyboardInterrupt but not able to catch in main thread, the workaround is put my clean up code inside finally, e.g.:

try:
    main()
except Exception as e:
    clean_up_stuff()  #this one never called if keyboard interrupt in module urllib3 thread
finally: #but this work
    clean_up_stuff() 
Monostylous answered 5/8, 2018 at 12:14 Comment(0)
E
0

This example is generic for Python3.3+, when decorating a generator function, a decorated generator returns successfully, thus not entering the decorators except, the magic happens with yield from f thus wrapping the yieldable within the decorator:

from types import GeneratorType    

def generic_exception_catcher(some_kwarg: int = 3):
    def catch_errors(func):
        def func_wrapper(*args, **kwargs):
            try:
                f = func(*args, **kwargs)
                if type(f) == GeneratorType:
                    yield from f
                else:
                    return f
            except Exception as e:
                raise e
        return func_wrapper
    return catch_errors

Usage:

@generic_exception_catcher(some_kwarg=4)
def test_gen():
    for x in range(0, 10):
        raise Exception('uhoh')
        yield x

for y in test_gen():
    print('should catch in the decorator')
Ensign answered 13/3, 2019 at 22:9 Comment(0)
B
0

I ran into a weird corner case that caused this problem and that might be worth sharing.

I'd refactored a custom error class into a new package, and there was a place in the code where it was being thrown, but the raise call referenced the old package in a local import statement. For some reason, the exception was constructed and raised without being caught by except Exception: (which I added as a sanity check when I couldn't figure out what was going on.)

The solution of course was to fix the name of the package in the place where the error was imported. I'm guessing my virtual environment or cache is polluted with the old error package and class, which is why it was imported and constructed successfully. However, I'm very surprised I couldn't catch it at all, since even in the old package it inherited from BaseException.

Note: this occurred when I was executing the code in a Python 3.11 runtime.

Basement answered 11/9, 2023 at 21:45 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.