С Новым годом! Форум программистов, компьютерный форум, киберфорум
Python для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.80/5: Рейтинг темы: голосов - 5, средняя оценка - 4.80
0 / 0 / 0
Регистрация: 24.12.2018
Сообщений: 2

Рефакторинг кода

24.12.2018, 13:56. Показов 1083. Ответов 1
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Добрый день.
Нужна помощь опытных “питонистов”
Я создал метод, код ниже, который скорее всего можно хорошенько сократить и сделать его более “питонистым”, но я ещё в этом деле неопытен. Могли бы вы глянуть своим опытным взглядом и дать пару подсказок?
И ещё, если не затруднит, поправить одно условие, оно закомментировано в коде, 2 дня уже ломаю голову над этим условием, не получается.
вот код:
Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
def play(self):
        """Start the WarGame."""
        new_players = self.players[:]   # make list copy
        while len(new_players) != 1:    # start loop until one object remains in list
            self.deck.deal(new_players, per_hand=1)  # hand over to each player one card
            m = new_players[0].total
            n = 0
            for player in new_players:
                print(player)
            """Need this condition but it's not work
            for player in new_players:
            # if the all objects in the list are equal, start the loop over
                if new_players.count(player.total) >= len(new_players):
                    self.play()"""
            # find those who need to kill
            for player in new_players:
                if player.total < m:
                    m = player.total
                    n = new_players.index(player)
            # kill'em all
            for player in new_players:
                if m == player.total:
                    n == new_players.index(player)
                    print("R.I.P.:", player.name)
                    new_players.remove(player)
            # clear copy list
            for player in new_players:
                player.clear()
        print("Win!:", player.name)
        # clear list
        for player in self.players:
            player.clear()
Спасибо.
0
IT_Exp
Эксперт
34794 / 4073 / 2104
Регистрация: 17.06.2006
Сообщений: 32,602
Блог
24.12.2018, 13:56
Ответы с готовыми решениями:

Рефакторинг кода, нужны комментарии для каждой строчки кода
Закомментировать каждую строчку кода !pip install pycountry_convert import warnings import numpy as np import pandas as pd ...

Рефакторинг кода
Имеется следующий код, который не очень хорошо написан. Он содержит две проблемы, которые смешаны - бизнес-логику и логирование. Как...

Рефакторинг кода: вложенный цикл for
Добрый день! Хотелось бы увидеть от коллег, как они делают рефакторинг кода. Вот, накидал такой кусочек: lst = new_lst = ...

1
0 / 0 / 0
Регистрация: 24.12.2018
Сообщений: 2
25.12.2018, 11:42  [ТС]
Добрый день.
Спасибо всем за участие, если кому интересно я нашёл ответ на свой вопрос.
Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
    def play(self):
        """Start the WarGame."""
        new_players = self.players[:]
        while len(new_players) != 1:
            self.deck.deal(new_players, per_hand=1)
            for player in new_players:
                print(player)
 
            min_score = min(player.total for player in new_players)
 
            dead_players = [player for player in new_players if player.total <= min_score]
            living_players = [player for player in new_players if player.total > min_score]
 
            if not living_players:
                # everybody died, play again
                for player in new_players:
                    player.clear()
                    print()
                continue
 
            # kill'em all
            for player in dead_players:
                print("R.I.P.:", player.name)
 
            new_players = living_players
 
            for player in new_players:
                player.clear()
 
        print("Win!:", player.name)
        for player in self.players:
            player.clear()
Тут и рефакторинг и полностью рабочий метод.
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
BasicMan
Эксперт
29316 / 5623 / 2384
Регистрация: 17.02.2009
Сообщений: 30,364
Блог
25.12.2018, 11:42
Помогаю со студенческими работами здесь

Рефакторинг кода
Всем известно, что методы должны содержать 5-7 операторов, лучше 3. Но если я в методе обрабатываю исключения и их много (5 к примеру), то...

Рефакторинг кода
Добрый день. Есть код, он работает, но проблема в том,что его много, как можно сделать рефакторинг кода.В последних 2х функция объявление и...

Рефакторинг кода
Здравствуйте, есть функция private void draw() { if (color2_bt0.IsChecked == true) { drawingBrush = Brushes.Black; ...

Рефакторинг кода
Добрый день, помогите сделать рефакторинг - что можно улучшить? @{ ViewBag.Title = &quot;Каталог товаров&quot;; } &lt;div...

Рефакторинг кода.
Доброго всем дня. помогите упростить кусок кода с if. Первая секция - две строки При нажатии на каждую - раскрываются в список, при...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
Новые блоги и статьи
сукцессия микоризы: основная теория в виде двух уравнений.
anaschu 11.01.2026
https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/
WordPad для Windows 11
Jel 10.01.2026
WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .
Classic Notepad for Windows 11
Jel 10.01.2026
Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .
Почему дизайн решает?
Neotwalker 09.01.2026
В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .
Модель микоризы: классовый агентный подход 3
anaschu 06.01.2026
aa0a7f55b50dd51c5ec569d2d10c54f6/ O1rJuneU_ls https:/ / vkvideo. ru/ video-115721503_456239114
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR
ФедосеевПавел 06.01.2026
Owen Logic: О недопустимости использования связки «аналоговый ПИД» + RegKZR ВВЕДЕНИЕ Введу сокращения: аналоговый ПИД — ПИД регулятор с управляющим выходом в виде числа в диапазоне от 0% до. . .
Модель микоризы: классовый агентный подход 2
anaschu 06.01.2026
репозиторий https:/ / github. com/ shumilovas/ fungi ветка по-частям. коммит Create переделка под биомассу. txt вход sc, но sm считается внутри мицелия. кстати, обьем тоже должен там считаться. . . .
Расчёт токов в цепи постоянного тока
igorrr37 05.01.2026
/ * Дана цепь постоянного тока с сопротивлениями и источниками (напряжения, ЭДС и тока). Найти токи и напряжения во всех элементах. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа и. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru