Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > comp.lang.python > #76598 > unrolled thread

Re: More Pythonic implementation

Back to article view | Back to comp.lang.python

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: More Pythonic implementation Chris Kaynor <ckaynor@zindagigames.com> - 2014-08-19 11:57 -0700

#76598 — Re: More Pythonic implementation

[Multipart message — attachments visible in raw view] — view raw

On Tue, Aug 19, 2014 at 10:09 AM, Shubham Tomar <tomarshubham24@gmail.com>
wrote:

> Lets say I have a function poker(hands) that takes a list of hands and
> returns the highest ranking hand, and another function hand_rank(hand) that
> takes hand and return its rank. As in Poker
> http://www.pokerstars.com/poker/games/rules/hand-rankings/.
>
> Which of the following is better and more Pythonic ?
>

Your two code segments will do different things.


>
> def poker(hands):
>     return max(hands, key=hand_rank)
>

In this case, the "hand_rank" function will take a single hand, and return
its rank value. Additionally, the "poker" function will return the highest
ranked hand.


>
> or
>
> def poker(hands):
>     return max(hand_rank(hands))
>

In this case, the "hand_rank" function will take an iterable of hands and
return an iterable of hand ranks. Additionally, the "poker" function will
return the rank of the highest ranked hand.

If that is the desired result, I would recommend writing this as:

def poker(hands):
    return max(map(hand_rank, hands))

which will result in hand_rank taking a single hand and returning its rank
value (similar to the first code segment you provided), rather than having
hand_rank deal with iterables of hands. However, that has more to do with
how you want hand_rank to behave, and where else it might be used.

[toc] | [standalone]

Back to top | Article view | comp.lang.python

csiph-web