Каковы семантические ошибки с этим кодом - PullRequest
Новая
       17

Каковы семантические ошибки с этим кодом

0 голосов
/ 25 марта 2019

У меня есть этот код, в котором есть некоторые семантические ошибки, которые я не могу найти. Первая строка не говорит нам, что это была за монета. 1000 сальто - математически невероятные 1000 хвостов. Кубик бросает 0, когда значения должны быть между 1 и 6. Результат двух кубиков просто говорит нам о функции, и результат 10 кубиков также содержит нули. 

import random
import numpy as np

def flipacoin():
    #This function flips a virtual coin and returns "Heads" or "Tails"
    coin = random.randint(1, 2) #returns a 1 or a 2
    if coin == 1:
        coin = "Heads"
    elif coin == 2:
        coin = "Tails"

def flipNcoins(n):
    #This function flips a coin a number of times (n flips total) and returns a string stating results.
    total_heads = 0
    total_tails = 0
    for i in range(n):
        if flipacoin() == "Heads":
            total_heads += 1
        else:
            total_tails += 1
    return "Heads " + str(total_heads) + ", Tails " + str(total_tails)


def rolladie():
    #This function rolls a virtual die and returns a number between 1 and 6.
    die = random.randint(0,6)
    return die

def rolltwodice():
    #This function rollstwo virtual dice and returns a tuple of the two values.
    die1 = rolladie()
    die2 = rolladie()
    return die1, die2

def rollNdice(n):
    #This function rolls n virtual dice and returns an array of roll values between 1 and 6.
    rolls = np.zeros((n,))
    for i in range(n):
        die = rolladie()
        rolls[i] = die
    return rolls

1 Ответ

1 голос
/ 25 марта 2019

coin = "Heads" устанавливает локальную переменную coin в "Heads". не возвращает его.return "Heads" будет (или, что эквивалентно, return coin в конце функции).Предполагается, что любая функция, которая не выполняет return до ее окончания, возвращает None.None явно не "Heads", поэтому вы накапливаете 1000 total_tails.

Не должно быть большим сюрпризом, что random.randint(0,6) бросает числа от 0 до 6, а не от 1 до 6.последний, вместо этого используйте random.randint(1,6).

Я не понимаю, что вы подразумеваете под "Результатом двух кубиков просто говорит нам о функции".

...