Правильно ли я перехватываю исключение?

Question

[Twelfth Doctor]

class Fruit:
    def isexist(self,objid):
        try:
           find_or_fail(objid)
        except ModelNotFound:
            return False
        else: 
            return True

Answer 1

[chupasaurus]

Нет, else нужен для выполнения операций, которые не попали под предыдущие условия в try/except, в вашем случае просто добавьте в блок try return True, исключение будет поймано раньше чем вывод выполнится.

Comments

[JaxxDexx]

Что значит "не попали под предыдущие условия"? Блок else в связке с try, выполняется в том случае, если в try не возникло исключений! Так что ничего криминального ТС в коде не написал, но все же else тут лишний, т.к. не несет особой смысловой нагрузки, но при этом занимает лишнюю строчку

[chupasaurus]

JaxxDexx: except может быть не один.

[JaxxDexx]

chupasaurus: Может, но причем тут это? Если возникло исключение, управление никак не попадет в else. Даже если нет соответствующего обработчика except, питон кинет исключение дальше!

Answer 2

[maxfox]

Else выполняется, если исключение не возникло. При этом, если в блоке else возникло исключение, оно не будет обработано except. Т.е. else в связке с try/excpet нужен, если вы хотите в случае успешного выполнения кода в блоке try выполнить что-то еще и при этом не обрабатывать исключение.
В вашем случае else не нужен, т.к. return True не может вызвать исключения ModelNotFound. В данном конкретном случае вы можете поместить return True как в блок try, так и после блока except, результат будет одинаковый. На мой взгляд, логичнее в try.