Code review

C Smith illusiontechniques at
Wed Nov 5 07:23:39 CET 2014

>Jean-Michel wrote:
> if left_to_act is None:
>    left_to_act = []
>  self.left_to_act = left_to_act
>  self.left_to_act = left_to_act or []

I read that with 2.7 that I had to initialize class variables to
immutable types. I think because I was working with the lists before
they had been altered and were still empty lists. I will mess around
tomorrow with the classes you suggested as I have yet to make use of
decorators. Thanks.

I will work on putting some more comments in as well.

On Tue, Nov 4, 2014 at 4:09 PM,  <sohcahtoa82 at> wrote:
> On Tuesday, November 4, 2014 12:35:32 PM UTC-8, John Gordon wrote:
>> C Smith <illusiontechniques at> writes:
>> > I was wondering if I could get some feedback on the biggest thing I
>> > have done as an amateur Python coder.
>> Comments.
>> You need a *lot* more comments.
>> Like, every line or two of code should have a comment explaining what is
>> being accomplished.  Seriously.  Every line or two.
>> Some blank lines wouldn't hurt either.
>> Also it would be nice to define some of the terms you're using.  What is
>> a blind?  What does "in front" mean?  What is a "muck hand"?
>> --
>> John Gordon         Imagine what it must be like for a real medical doctor to
>> gordon at    watch 'House', or a real serial killer to watch 'Dexter'.
> A "blind" is a required bet that the two people sitting to the left of the button have to make before a hand begins.  The first player pays the small blind and the second player pays the big blind, which is double the small blind.  For tournament games, the blinds increase periodically.
> Mucking a hand means to throw your cards away without revealing them.  In professional Poker, you typically give them to the dealer.
> "In front" just refers to table position which determines play order.
> --

More information about the Python-list mailing list