Path: csiph.com!usenet.pasdenom.info!poup.poupinou.org!news.muarf.org!nntpfeed.proxad.net!proxad.net!feeder1-2.proxad.net!weretis.net!feeder4.news.weretis.net!rt.uk.eu.org!newsfeed.xs4all.nl!newsfeed4.news.xs4all.nl!xs4all!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
Date: Fri, 27 Sep 2013 12:27:57 +0100
From: MRAB <python@mrabarnett.plus.com>
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:24.0) Gecko/20100101 Thunderbird/24.0
MIME-Version: 1.0
To: python-list@python.org
Subject: Re: card dealer
References: <0617088e-5aa3-4c5d-b660-22040ee858a1@googlegroups.com>
In-Reply-To: <0617088e-5aa3-4c5d-b660-22040ee858a1@googlegroups.com>
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 8bit
Precedence: list
Reply-To: python-list@python.org
Newsgroups: comp.lang.python
Message-ID: <mailman.377.1380281278.18130.python-list@python.org>
Lines: 65
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:54875

On 27/09/2013 11:24, markotaht@gmail.com wrote:
> from random import *
> from math import floor
>
>
> kaarte_alles = 52
> kaart_tõmmatud = [False for i in range(52)]
>
>
> mast = ["ärtu", "ruutu", "poti", "risti"]
> aste = ["äss", "kaks", "kolm", "neli","viis", "kuus", \
>          "seitse", "kaheksa", "üheksa", "kümme", "soldat",\
>          "emand", "kuningas"]
>
> def tõmba_kaart():
>      global kaarte_alles
>      if kaarte_alles == 0:
>          print("Segan pakki")
>          kaarte_alles = 52
>          for i in range(52):
>              kaart_tõmmatud[i] = False
>      kaarte_alles -= 1
>      n = random(kaarte_alles)
>      kaart = vali_järgmine_vaba(n)
>      m = kaart % 13
>      a = kaart / 13
>      print(mast[int(a)] + " " + aste[int(m)])
>
> def vali_järgmine_vaba(n):
>      i = -1
>
>      while(n  > 0):
>          n -= 1
>          i = i + 1
>          while kaart_tõmmatud[i]:
>              i = i + 1
>      kaart_tõmmatud[i] = True
>      return i
>
> def random(n):
>      return randint(0, n)
>
> while True:
>      n = int(input("Mitu kaarti tõmmata(0 et väljuda): "))
>      if(n==0):
>          break
>
>      for i in range(n):
>          tõmba_kaart()
>
>
> HI im trying to make a card dealer, but this code doesent work correctly. It deasl cards, but the cards to repeat. when i type in 52 ten i might get 3 the same cards. This is a translation from my c++ code. In c++ it works correctly.
>
Suppose random(...) returns 0.

In 'vali_järgmine_vaba', n == 0, so it won't run anything in the
'while' loop, therefore i == -1, therefore:

     kaart_tõmmatud[-1] = True

(which sets the last item in the list).

If random(...) returns 0 again (which is possible),
'vali_järgmine_vaba' will do exactly the same thing.