Как вернуть стартовое значение счетчика?

Question

[giorgio85]

Необходимо определить класс, реализующий десятичный счетчик, который может увеличивать или уменьшать свое значение на единицу в заданном диапазоне. Возникает проблема при вызове сразу обоих методов а не по отдельности. Значение self.number обновляется, не возвращается к первоначальному. Возможно ли как-то сбить его изменения?

class Counter:
    def __init__(self, number=5, lower_limit=3, upper_limit=10):
        self.number = number
        self.lower_limit = lower_limit
        self.upper_limit = upper_limit

    def increase_count(self):
        if self.number not in range(self.lower_limit, self.upper_limit):
            return "Error"
        else:
            mylist = list(self.number + 1 for self.number in range(self.number, self.upper_limit))
            return mylist

    def decrease_count(self):
        if self.number not in range(self.lower_limit, self.upper_limit):
            return "Error"
        else:
            mylist = list(self.number - 1 for self.number in range(self.lower_limit + 1, self.number + 1))
            return mylist[::-1]

obj = Counter()
print(obj.increase_count())
print(obj.decrease_count())

Comments

[Vindicar]

Нечитаемо же. Оформи код.

Answer 1

[plustilino]

mylist = list(i + 1 for i in range(self.number, self.upper_limit))

Comments

[giorgio85]

Спасибо огромное!

[Алан Гибизов]

Признаюсь, ничего не понимаю. Можете развернуть свою мысль? Как приведенный код отвечает на вопрос?

[plustilino]

Алан Гибизов, В части for self.number in ... происходит присваивание self.number, а по логике не должно. После for должна стоять просто переменная-счетчик, значения которой меняются от number до upper_limit.
Для наглядности:

>>> i = -1
>>> for i in range(10, 13):
...     print(i)
... 
10
11
12
>>> i
12

[Алан Гибизов]

plustilino, идею понял. Не понял, как вообще всё это соотносится с идеей счетчика.
Как я понимаю, счетчик должен меняться при каждом вызове метода и возвращать текущее значение, а не список возможных значений, иначе это не счетчик, а генератор списков.

Или я не понимаю идею счетчиков…

[plustilino]

Алан Гибизов, это ответ на вопрос: "как сбить изменение поля number". А соотносится ли с идеей счетчика формулировка задачи - другой вопрос.

[o5a]

giorgio85, Судя по постановке задачи, я бы вообще предположил, что требуется всего лишь этими методами класса изменять значение аттрибута self.number +/- 1 (с проверками диапазона), я не вижу, где по постановке задачи требуется возвращать список диапазона значений, как Вы это делаете.
А для вывода значения объекта можно прописать методы __str__/__repr__
Т.е. примерно так:

class Counter:
    def __init__(self, number=5, lower_limit=3, upper_limit=10):
        self.number = number
        self.lower_limit = lower_limit
        self.upper_limit = upper_limit

    # выведет значение счетчика
    def __repr__(self):
        return str(self.number)

    def increase_count(self):
        # делаем проверки диапазона, и если удовлетворяет, увеличиваем self.number

    def decrease_count(self):
        # делаем проверки диапазона, и если удовлетворяет, уменьшаем self.number

obj = Counter()
obj.increase_count()
print(obj) # выведет 6
obj.decrease_count()
print(obj) # выведет 5
obj.decrease_count()
obj.decrease_count()
obj.decrease_count() # ругнется на выход за диапазон

Answer 2

[Til_Ulenspiegel]

Я бы на вашем месте переписал программу так:

class Counter: 
    def __init__(self, number=5, lower_limit=3, upper_limit=10): 
        assert lower_limit <= number <= upper_limit
        self.count = [*range(1, number+1)]
        self.lower_limit = lower_limit
        self.upper_limit = upper_limit  
        
    def increase_count(self):
        if len(self.count) == self.upper_limit: 
            raise ValueError("upper_limit is reached")
        self.count.append(self.count[-1] +1) 
        return self.count
           
    def decrease_count(self): 
        if len(self.count) == self.lower_limit: 
            raise ValueError("lower_limit is reached")
        self.count.pop()
        return self.count
            
            
obj = Counter() 
print(obj.increase_count())
print(obj.increase_count())
print(obj.increase_count())
print(obj.decrease_count())
print(obj.decrease_count())