Das deutsche Python-Forum

import random

per=[]
wieoft=[]
x=5 
def perioden():
    for i in range(x):
        per.append(random.randint(1,12))
    wieoft.append(per.count(1))
    wieoft.append(per.count(2))
    wieoft.append(per.count(3))
    wieoft.append(per.count(4))
    wieoft.append(per.count(5))
    wieoft.append(per.count(6))
    wieoft.append(per.count(7))
    wieoft.append(per.count(8))
    wieoft.append(per.count(9))
    wieoft.append(per.count(10))
    wieoft.append(per.count(11))
    wieoft.append(per.count(12))
    print(per)
    print(wieoft)

perioden()

>>>
[2, 11, 10, 3, 10]
[0, 1, 1, 0, 0, 0, 0, 0, 0, 2, 1, 0]
>>>
# bis hierher einwandfrei, aber bei mehrmaliger Ausführung der Funktion:

Code: Alles auswählen

for i in range(2):
    perioden()

>>>
[11, 1, 3, 5, 2]
[1, 1, 1, 0, 1, 0, 0, 0, 0, 0, 1, 0]
[11, 1, 3, 5, 2, 3, 12, 9, 12, 1]
[1, 1, 1, 0, 1, 0, 0, 0, 0, 0, 1, 0, 2, 1, 2, 0, 1, 0, 0, 0, 1, 0, 1, 2]
>>>
die Ergebnisse werden in einen Topf geworfen, warum nur, ich quäle mich seit Tagen damit herum, normal wären doch zwei getrennte
Ergebnisse, etwa so:

>>>
[11, 1, 3, 5, 2]
[1, 1, 1, 0, 1, 0, 0, 0, 0, 0, 1, 0]
[3, 12, 9, 12, 1]
[2, 1, 2, 0, 1, 0, 0, 0, 1, 0, 1, 2]
>>>

oder :K :

>>>
[[11, 1, 3, 5, 2],[3, 12, 9, 12, 1]]
[[1, 1, 1, 0, 1, 0, 0, 0, 0, 0, 1, 0], [2, 1, 2, 0, 1, 0, 0, 0, 1, 0, 1, 2]]
>>>

@haeuslermartin: Warum erwartest Du da etwas anderes? Du erstellst die leeren Listen `per` und `wieoft` und jedes mal wenn `perioden()` aufgerufen wird, werden mehr Werte an diese beiden Listen angehängt. An welcher Stelle erwartest Du die Magie das deren Inhalt bei einem weiteren Aufruf gelöscht wird ohne das das tatsächlich irgendwo explizit im Code stehen würde?

Was ”falsch” ist, ist das `perioden()` keine ”echte” Funktion ist. Da werden keine Werte übergeben, sondern die ”Funktion” greift einfach so auf Werte von ausserhalb zu und verändert sogar Listen ausserhalb der Funktion. So etwas wird ziemlich schnell sehr unübersichtlich und fehleranfällig. Alle Werte (ausser Konstanten) die eine Funktion verwendet sollte die Funktion als Argumente betreten. Ergebnisse sollten sie als Rückgabewerte verlassen. Und ”echte” Funktionen sollten Datenstrukturen nicht verändern, sondern eher neue, veränderte erstellen.

Edit: Für das zählen würde man auch eine Schleife verwenden, statt 12 mal fast die gleiche Codezeile zu schreiben. Oder gleich `collections.Counter` oder in der ersten Schleife schon mitzählen.

vielen Dank für die Antwort, aber so ganz schnall ichs immer noch nicht, wird denn perioden() mit -for i in range- nicht genauso immer wieder neu
gestartet, wie ich es auch "von Hand" machen würde? da kommen die Ergebnisse nämlich getrennt ...

@haeuslermartin: versuch mal genau nachzuvollziehen, wann Du was mit Deinen Listen per und wieoft machst.

Probier's mal hiermit, das könnte beim Verständnis helfen, was in deinem Code passiert: PythonTutor.com/Visualize.html

@haeuslermartin: Wie machst Du das bitte von Hand? Wenn Du da wirklich nur `perioden()` mehr als einmal aufrufst, dann wirst Du dort genau das gleiche Verhalten sehen wie im Programm:

Code: Alles auswählen

In [1]: import random

In [2]: per=[]

In [3]: wieoft=[]

In [4]: x=5

In [5]: def perioden():
   ...:     for i in range(x):
   ...:         per.append(random.randint(1,12))
   ...:     wieoft.append(per.count(1))
   ...:     wieoft.append(per.count(2))
   ...:     wieoft.append(per.count(3))
   ...:     wieoft.append(per.count(4))
   ...:     wieoft.append(per.count(5))
   ...:     wieoft.append(per.count(6))
   ...:     wieoft.append(per.count(7))
   ...:     wieoft.append(per.count(8))
   ...:     wieoft.append(per.count(9))
   ...:     wieoft.append(per.count(10))
   ...:     wieoft.append(per.count(11))
   ...:     wieoft.append(per.count(12))
   ...:     print(per)
   ...:     print(wieoft)
   ...: 

In [6]: perioden()
[6, 4, 4, 2, 9]
[0, 1, 0, 2, 0, 1, 0, 0, 1, 0, 0, 0]

In [7]: perioden()
[6, 4, 4, 2, 9, 12, 3, 3, 8, 12]
[0, 1, 0, 2, 0, 1, 0, 0, 1, 0, 0, 0, 0, 1, 2, 2, 0, 1, 0, 1, 1, 0, 0, 2]

Natürlich wird ``for i in …`` da zweimal ausgeführt, aber warum sollte das auf magische Weise vorhandene Elemente aus den Listen *entfernen* wenn Dein Code doch nur mit `append()` immer mehr Elemente anhängt.

Steve Oualline, "Practical C Programming":

If we don't write code like this, then we don't have to worry about such questions.

ich stand voll auf dem Schlauch, sorry, -klar, wenn ich das Programm neu starte, starte ich auch die Listen immer wieder neu, darum die Einzelergebnisse, ist wahrscheinlich das Alter ... nochmals danke

jetzt so: wenns jemand interessiert, die Ergebnisse könnte man natürlich mittels Binominalfunktion auch math. erhalten, aber ich bin weder Mathematiker, noch Informatiker, sondern nur Rentner, so probier ich halt ein kleines Programm zu schreiben, das mir empirische Ergebnisse ermöglicht:

Code: Alles auswählen

import random

alle=[]
x=10 
def perioden():
    per=[]
    wieoft=[]
    for i in range(x):
        per.append(random.randint(1,12))
    wieoft.append(per.count(1))
    wieoft.append(per.count(2))
    wieoft.append(per.count(3))
    wieoft.append(per.count(4))
    wieoft.append(per.count(5))
    wieoft.append(per.count(6))
    wieoft.append(per.count(7))
    wieoft.append(per.count(8))
    wieoft.append(per.count(9))
    wieoft.append(per.count(10))
    wieoft.append(per.count(11))
    wieoft.append(per.count(12))
    #print(per)
    #print(wieoft)
    alle.append(wieoft.count(2)+wieoft.count(3)+wieoft.count(4))
for i in range(1000):
    perioden()
#print()
#print(alle)
print("von",len(alle), "Perioden a`", x, "coups haben",alle.count(1)+alle.count(2)+alle.count(3),"Perioden Mehrfachtransversalen")

Python 3.4.3 (v3.4.3:9b73f1c3e601, Feb 24 2015, 22:43:06) [MSC v.1600 32 bit (Intel)] on win32
Type "copyright", "credits" or "license()" for more information.
>>> ================================ RESTART ================================
>>>
von 100 Perioden a` 4 coups haben 43 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 1000 Perioden a` 4 coups haben 459 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 100 Perioden a` 5 coups haben 62 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 1000 Perioden a` 5 coups haben 634 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 100 Perioden a` 6 coups haben 76 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 1000 Perioden a` 6 coups haben 769 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 100 Perioden a` 7 coups haben 89 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 1000 Perioden a` 7 coups haben 884 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 100 Perioden a` 8 coups haben 98 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 1000 Perioden a` 8 coups haben 943 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 100 Perioden a` 9 coups haben 98 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 1000 Perioden a` 9 coups haben 959 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 100 Perioden a` 10 coups haben 93 Perioden Mehrfachtransversalen
>>> ================================ RESTART ================================
>>>
von 1000 Perioden a` 10 coups haben 915 Perioden Mehrfachtransversalen
>>>

interessant dabei, dass sich Zufallsergebnisse nicht linear entwickeln ... und hundert Versuche ausreichen

@haeuslermartin: solche Fehler passieren einfach nicht so leicht, wenn man wirklich Funktionen schreiben würde und nicht einfach von irgendwoher Werte zusammensammeln würde:

Code: Alles auswählen

import random
 
def perioden(x):
    per = [random.randint(1,12) for _ in range(x)]
    wieoft = [per.count(i) for i in range(1,13)]
    return sum(wieoft.count(i) for i in range(2,5))

def main():
    x = 10
    alle = [perioden(x) for i in range(1000)]
    count = sum(alle.count(i) for i in range(1,4))
    print("von {} Perioden a `{}` coups haben {} Perioden Mehrfachtransversalen".format(len(alle), x, count))

if __name__ == '__main__':
    main()

Das deutsche Python-Forum

Hilfe erbeten, Funktion und Liste

Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste

Re: Hilfe erbeten, Funktion und Liste