Hi folks
2013/7/10 Doug Hellmann <doug.hellm...@dreamhost.com>:
>
>
>
> On Wed, Jul 10, 2013 at 3:57 PM, David Ripton <drip...@redhat.com> wrote:
>>
>> On 07/10/2013 02:01 PM, Nachi Ueno wrote:
>>>
>>> HI folks
>>>
>>> I would like to ask the review criteria in the community.
>>>
>>> Should we use exception for non-exceptional cases when we can use
>>> parameter checking?
>>>
>>> Example1: Default value for array index
>>>
>>> try:
>>> value = list[5]
>>> except IndexError:
>>> value = 'default_value'
>>>
>>> This can be also written as,
>>>
>>> list_a[3] if len(list_a) > 3 else 'default_value'
>>
>>
>> Both of these are fine. Neither deserves to be banned.
OK if there is no preference in the community, I'l review it as 'This
is my personal opinion but you can choose option'.
>> But LBYL is often naive in the face of concurrency. Just because
>> something was true a microsecond ago doesn't mean it's still true.
>> Exceptions are often more robust.
>
>
> getattr() takes a default and, as it is implemented in C, is thread-safe.
> So:
>
> value = getattr(my_obj, 'might_not_be_there', 'default')
>
> Of course, it's probably better to make sure you've always got the same type
> of object in the first place but sometimes the attributes change across
> versions of libraries.
>
> For accessing elements of a sequence that may be too short,
> itertools.chain() and itertools.islice() are useful.
>
>>>> import itertools
>>>> vals1 = ['a', 'b']
>>>> a, b, c = itertools.islice(itertools.chain(vals1, ['c']), 3)
>>>> a, b, c
> ('a', 'b', 'c')
>>>> vals2 = ['a', 'b', 'd']
>>>> a, b, c = itertools.islice(itertools.chain(vals2, ['c']), 3)
>>>> a, b, c
> ('a', 'b', 'd')
I like this way! :)
Best
Nachi
> Doug
>
>
> _______________________________________________
> OpenStack-dev mailing list
> OpenStack-dev@lists.openstack.org
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
_______________________________________________
OpenStack-dev mailing list
OpenStack-dev@lists.openstack.org
http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
