Правильно ли решена задача по подсчету очков при игре в дартс?

Question

[mikhal_ivanych]

Всем привет.
Решил задачу. Кому не сложно - отпишите плиз, есть ли ошибки / недостатки.
Спасибо.

"""
Написать функцию для подсчета очков при игре в дартс.
Правила начисления очков:
Радиус больше 10 - 0 очков
Радиус между 5 и 10 включительно - 5 очков
Радиус менее 5 - 10 очков
Если все радиусы менее 5, начисляются дополнительные 100 очков.
"""

min_points = 0
mid_points = 5
max_points = 10
bonus_points = 100


def calculate_score(lst):
    total_score = 0

    for i in lst:
        if i < 5:
            total_score += max_points
        else:
            for j in lst:
                if 5 <= j <= 10:
                    total_score += mid_points
                else:
                    total_score += min_points
            return total_score
    return total_score + bonus_points


print(calculate_score([1, 5, 11]))
print(calculate_score([15, 20, 30]))
print(calculate_score([1, 2, 3, 4]))

Comments

[Hcuy]

А зачем Вы два раза используете for? Не проще elif использовать?

Answer 1

[soremix]

Не нужно делать много циклов. Если будет больше 3-х бросков, ваш код не будет их обрабатывать, можете проверить.

Я бы делал так

def calculate_score(lst):
    total_score = 0
    has_bonus = True

    for radius in lst:
        if radius >=5 and radius <=10:
            total_score += mid_points
        
        elif radius < 5:
            total_score += max_points

        if radius > 5:
            has_bonus = False

    if has_bonus:
        total_score += bonus_points

    return total_score

Либо проверку на бонус можно сделать через функцию max(lst), она вернет самое больше число в массиве, соответственно, если максимально число больше 5 - бонуса не будет

def calculate_score(lst):
    total_score = 0

    for radius in lst:
        if radius >=5 and radius <=10:
            total_score += mid_points
        
        elif radius < 5:
            total_score += max_points

    if max(lst) < 5:
        total_score += bonus_points

    return total_score

Comments

[mikhal_ivanych]

SoreMix Спасибо. Но я не очень понял, в каком случае код не будет обрабатывать более 3х бросков? Сейчас же мой код обрабатывает и 3 и 4 и больше бросков.

Вы в своем первом примере не используете else. С точки зрения синтаксиса конструкции это нормально?

[soremix]

mikhal_ivanych, попробуйте [1, 1, 6, 1] передать в функцию. Получите ответ 25, хотя должен быть 35.

В моем коде elif обязателен, так как у нас проверяются три условия (<5; <=5<=10; >10). Если заменить на else, то тогда все радиусы, которые не попали в промежуток от 5 до 10 будут давать максимум очков. Если число меньше 5, то все нормально, но если число больше 10, прибавится максимальное кол-во очков. Хотя так быть не должно конечно. А по синтаксису ну да, нормально

Answer 2

[mikhal_ivanych]

SoreMix да, похоже вариант с max(lst) смотрится получше и цикл один, а не два. Плюс вы убрал вариант с 0 ...
Спасибо за комментарий.