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

@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 **[ТС]**

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

Новые блоги и статьи Все статьи Все блоги /
Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.	Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .
Настройки VS Code Loafer 13.04.2026 { "cmake. configureOnOpen": false, "diffEditor. ignoreTrimWhitespace": true, "editor. guides. bracketPairs": "active", "extensions. ignoreRecommendations": true, . . .	Оптимизация кода на разграничение прав доступа к элементам формы Maks 13.04.2026 Алгоритм из решения ниже реализован на нетиповом документе, разработанного в конфигурации КА2. Задачи, как таковой, поставлено не было, проделанное ниже исключительно моя инициатива. Было так:. . .	Контроль заполнения и очистка дат в зависимости от значения перечислений Maks 12.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "ПланированиеПерсонала", разработанного в конфигурации КА2. Задача: реализовать контроль корректности заполнения дат назначения. . .	Архитектура слоя интернета для сервера-слоя. Hrethgir 11.04.2026 В продолжение https:/ / www. cyberforum. ru/ blogs/ 223907/ 10860. html Знаешь что я подумал? Раз мы все источники пишем в голове ветки, то ничего не мешает добавить в голову такой источник, который сам. . .

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

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

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

Опции темы