annafragor / playoff

PlayOff

Вводятся названия команд, участвующих в соревновании. Далее будет выявлен победитель, после чего при желании можно будет посмотреть, как выступала та, или иная команда, или же просто завершить программу, напечатав 'exit'.

playoff's Issues

Симуляция плей-офф: замечания

Привет, в целом все верно, вот замечания:

Много лишних скобок, например в условиях вида:

while (client_cmd != "exit"):

можно проще:

while client_cmd != "exit":

Таких мест очень много.

Очень понравилось, что проверяешь кол-во команд логарифмом 👍

Далее

    while (i < n):
        winner = game(shuf_cmds[i], shuf_cmds[i + 1])

        if (winner[0] == shuf_cmds[i + 1]):
            shuf_cmds[i], shuf_cmds[i + 1] = shuf_cmds[i + 1], shuf_cmds[i]

        level[shuf_cmds[i]] = [shuf_cmds[i + 1], "won", winner[1]]
        level[shuf_cmds[i + 1]] = [shuf_cmds[i], "lost", winner[1][::-1]]
        shuf_cmds.remove(shuf_cmds[i + 1])
        n -= 1
        i += 1  # i -= 1; i += 2

выглядит очень нечитаемо, можно по крайней мере создать переменные было:

        team_1 = shuf_cmds[i]
        team_2 = shuf_cmds[i + 1]

И в целом в коде в других местах очень много обращений по индексам - это тоже негативно сказывается на читаемости. Для этой задачи такое кол-во обращений по индексам говорит о том, что структуры данных были выбраны неверно - можно было написать код гораздо проще. Чтобы внести изменения в такой код нужно ооочень осторожно вносить правки. Но умения писать расширяемый код после пары занятий, конечно, мы не ожидаем.

Хранить в статистике текстовый результат это не оч правильно - лучше хранить там только данные достаточные, чтобы впоследствии построить из них текстовый вывод - это более гибко, я имею в виду вот этот момент:

level[shuf_cmds[i + 1]] = [shuf_cmds[i], "lost", winner[1][::-1]]

По логике работы вопросов нет - все как нужно.