простенькая задача с классом (как упростить код)

@123orc123 · Регистрация: 27.04.2017

Студворк — интернет-сервис помощи студентам

всем привет.

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

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

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
class MaxSizeList:
 
    def __init__(self,maxsize):
        self.maxsize = maxsize
        self.list = []
 
    def push(self, *args):
        for arg in args:
            if len(self.list) <= self.maxsize:
                self.list.append(arg)
                if len(self.list) > self.maxsize:
                    self.list.pop(0)
                    continue
 
    def get_list(self):
        return self.list
 
 
a = MaxSizeList(2)
 
a.push('sun', 'moon', 'water', 'earth')
print (a.get_list())

@dondublon · 22.05.2017, 14:16

Неплохо бы сначала описать, что нужно сделать.

А так - да, говнокод. Цикл тут совершенно не нужен.

Смотрим сумму длин self.list и пришедшего args. Если она больше self.maxsize, то берём срез:

Python
1
2
3
4
diff = len(self.list)+len(args)-self.maxsize
if diff > 0:
    self.list = self.list[maxsize:]
self.list += args

@Semen-Semenich · 22.05.2017, 15:00

ну я бы так сделал согласно условию только бы имя переменной не задавал бы list а хотя бы l_ist

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class MaxSizeList:
 
    def __init__(self):
        self.maxsize = 2 # в __init__ засовываем максимальную длину списка
        self.list = []
 
    def push(self, *args):
            self.list = list(args) #  одним из методов запихиваем в лист
            while len(self.list) > self.maxsize:
                   self.list.pop(0)
      
    def get_list(self):
        return self.list
 
a = MaxSizeList()
a.push( 'sun', 'moon', 'water', 'earth')
print (a.get_list())

@shsv · 22.05.2017, 15:22

dondublon, а что если len(args) > self.maxsize?

@123orc123 · 22.05.2017, 15:30 **[ТС]**

извиняюсь, что сумбурно описал исходное условие.

В конструктор класса передаем одно значение - максимальную длину списка. С помощью метода класса мы заполняем наш список элементами, при этом надо учитывать макс.длину списка, и если количество переданных в метод элементов больше, то мы оставляем в списке только последние N-элементов (при этом N равен макс.длине списка, переданной в конструктор).

моя вторая попытка (смотрится еще страшнее первой

)

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
class MaxSizeList:
 
    def __init__(self,maxsize):
        self.maxsize = maxsize
        self.list = []
 
    def push(self, *args):
        if len(args) > self.maxsize:
            self.list.extend(args[-(self.maxsize):len(args)+1])
        else:
            self.list.extend(args)
 
    def get_list(self):
        return self.list
 
 
a = MaxSizeList(2)
 
a.push('sun', 'moon', 'water', 'earth')
print (a.get_list())

@dondublon · 22.05.2017, 16:01

shsv, тогда не повезло

Надо отдельно обрабатывать.

@Foxtam · 22.05.2017, 17:58

Можно через дек

Python
1
2
3
4
5
6
7
8
9
10
11
import collections
 
class MaxSizeList:
    def __init__(self, max_size):
        self.dq = collections.deque(maxlen=max_size)
 
    def push(self, *args):
        self.dq.extend(args)
 
    def get_list(self):
        return list(self.dq)

Или если нужно использовать только встроенные типы то

Python
1
2
3
4
5
6
7
8
9
10
11
class MaxSizeList:
    def __init__(self, max_size):
        self.max_size = max_size
        self.dq = []
 
    def push(self, *args):
        self.dq.extend(args)
        self.dq = self.dq[-self.max_size:]
 
    def get_list(self):
        return self.dq

@Ryker · 23.05.2017, 12:47

Смысл в условиях? Результаты верные при входящих аргументов (больше, равно, меньше) длины списка

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class MaxSizeList:
 
    def __init__(self, maxsize):
        self.maxsize = maxsize
        self.list = []
 
    def push(self, *args):
        self.list.extend(args[-(self.maxsize):len(args)+1])
 
    def get_list(self):
        return self.list
 
 
a = MaxSizeList(2)
 
a.push('sun', 'moon', 'water', 'earth')
print(a.get_list())

Добавлено через 14 минут
Прошлая версия выдавала неправильный результат при нескольких вызовах функции push() добавил условие теперь все верно.

Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
class MaxSizeList:
 
    def __init__(self, maxsize):
        self.maxsize = maxsize
        self.spisok = []
 
    def push(self, *args):
        self.spisok.extend(args[-(self.maxsize):len(args)+1])
        if len(self.spisok) > self.maxsize:
            self.spisok = self.spisok[len(self.spisok) - self.maxsize :
                                      self.maxsize + 1]
 
    def get_list(self):
        return self.spisok
 
 
a = MaxSizeList(2)
 
a.push('1')
a.push('2')
a.push('3')
print(a.get_list())

@123orc123 · 24.05.2017, 10:51 **[ТС]**

всем спасибо, что помогли взглянуть на решение задачи с более оптимальной стороны.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11680&d=1772460536 Одним из. . .	Реалии Hrethgir 01.03.2026 Нет, я не закончил до сих пор симулятор. Эта задача сложнее. Не получилось уйти в плавсостав, но оно и к лучшему, возможно. Точнее получалось - но сварщиком в палубную команду, а это значит, в моём. . .	Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .
SDL3 для Web (WebAssembly): Сборка библиотек: SDL3, Box2D, FreeType, SDL3_ttf, SDL3_mixer и SDL3_image из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual Studio. . . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .

@shsv 757 / 306 / 190 Регистрация: 20.05.2016 Сообщений: 593
	22.05.2017, 15:22
	dondublon, а что если len(args) > self.maxsize? 0

@dondublon 4652 / 2072 / 366 Регистрация: 17.03.2012 Сообщений: 10,182 Записей в блоге: 6
	22.05.2017, 16:01
	shsv, тогда не повезло Надо отдельно обрабатывать. 0

@123orc123 24 / 18 / 9 Регистрация: 27.04.2017 Сообщений: 122
	24.05.2017, 10:51 [ТС]
	всем спасибо, что помогли взглянуть на решение задачи с более оптимальной стороны. 0