Cameron Simpson wrote:
> On 29Feb2016 10:45, Ian Kelly <[email protected]> wrote:
>>On Mon, Feb 29, 2016 at 10:26 AM, Ganesh Pal <[email protected]> wrote:
>>> On Mon, Feb 29, 2016 at 9:59 PM, Ian Kelly <[email protected]> wrote:
>>>> On Mon, Feb 29, 2016 at 8:18 AM, Ganesh Pal <[email protected]> wrote:
>>>>> 1. usage of try- expect
>>>>
>>>> try-except in every single function is a code smell. You should only
>>>> be using it where you're actually going to handle the exception. If
>>>> you catch an exception just to log it, you generally should also
>>>> reraise it so that something further up the call chain has the
>>>> opportunity to handle it.
>>>
>>> How do we reraise the exception in python , I have used raise not
>>> sure how to reraise the exception
>>
>>raise with no arguments will reraise the exception currently being handled.
>>
>>except Exception:
>> logging.error("something went wrong")
>> raise
>
> Another remark here: if you're going to log, log the exception as well:
>
> logging.error("something went wrong: %s", e)
>
> Ian's example code is nice and simple to illustrate "log and then reraise"
> but
> few things are as annoying as log files reciting "something went wrong" or
> the
> equivalent without any accompanying context information.
>
> Cheers,
> Cameron Simpson <[email protected]>
Or, for that matter:
logging.exception('something went wrong')
Which gives you the whole traceback as well and doesn't require you to
explictly grab the exception.
--
Rob Gaddi, Highland Technology -- www.highlandtechnology.com
Email address domain is currently out of order. See above to fix.
--
https://mail.python.org/mailman/listinfo/python-list
