>Jean-Michel wrote: >replace > if left_to_act is None: > left_to_act = [] > self.left_to_act = left_to_act >by > 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, <sohcahto...@gmail.com> wrote: > On Tuesday, November 4, 2014 12:35:32 PM UTC-8, John Gordon wrote: >> C Smith <illusiontechniq...@gmail.com> 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 >> gor...@panix.com 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. > -- > https://mail.python.org/mailman/listinfo/python-list -- https://mail.python.org/mailman/listinfo/python-list
