Как не повторять переменные внутри методов?

переменные self.ledger внутри класса отображают некоторые депозиты и снятия следующим образом:

[{'количество': 50, 'описание': 'Дед Мороз прибыл'}, {'количество': -12,5, 'описание': 'Воры прибыли'}]

Леджер создается с помощью двух методов снятия и депозита, которые просто добавляют информацию в леджер каждый раз, когда они вызываются.

Очевидно, что сумма вывода отрицательна, а сумма депозита положительна, поэтому я написал эту функцию, чтобы получить баланс класса.

def get_balance(self):
   self.Alldeposited = sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] > 0])
   self.Allwithdrawn = abs(sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] < 0]))
   self.balance = self.Alldeposited - self.Allwithdrawn
   return self.balance

Я создал еще одну функцию процент_процентов, где мне нужны точно такие же переменные для get_balance(Alldeposited, Allwithdrawn). Как мне сделать мой код менее повторяющимся? Я попытался инициализировать эти переменные в верхней части класса и внутри конструктора, но это не сработало, я подумал, что могу создать 2 конкретных метода для получения 2 значений, но на этом этапе лучше повторить код. Ниже приведен настолько повторяющийся код, который я написал.

def percentage_spent(self):
    self.Alldeposited = sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] > 0])
    self.Allwithdrawn = abs(sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] < 0]))
    Percentage = round(((self.Allwithdrawn*100)/self.Alldeposited),-1)
    return Percentage

Можете вы помочь мне?

Анализ настроения постов в Twitter с помощью Python, Tweepy и Flair
Анализ настроения постов в Twitter с помощью Python, Tweepy и Flair
Анализ настроения текстовых сообщений может быть настолько сложным или простым, насколько вы его сделаете. Как и в любом ML-проекте, вы можете выбрать...
7 лайфхаков для начинающих Python-программистов
7 лайфхаков для начинающих Python-программистов
В этой статье мы расскажем о хитростях и советах по Python, которые должны быть известны разработчику Python.
Установка Apache Cassandra на Mac OS
Установка Apache Cassandra на Mac OS
Это краткое руководство по установке Apache Cassandra.
Сертификатная программа "Кванты Python": Бэктестер ансамблевых методов на основе ООП
Сертификатная программа "Кванты Python": Бэктестер ансамблевых методов на основе ООП
В одном из недавних постов я рассказал о том, как я использую навыки количественных исследований, которые я совершенствую в рамках программы TPQ...
Создание персонального файлового хранилища
Создание персонального файлового хранилища
Вы когда-нибудь хотели поделиться с кем-то файлом, но он содержал конфиденциальную информацию? Многие думают, что электронная почта безопасна, но это...
Создание приборной панели для анализа данных на GCP - часть I
Создание приборной панели для анализа данных на GCP - часть I
Недавно я столкнулся с интересной бизнес-задачей - визуализацией сбоев в цепочке поставок лекарств, которую могут просматривать врачи и...
2
0
32
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Переместите общую функциональность в отдельную функцию или разделите ее на две отдельные:

def deposited_and_withdrawn():
    return {
        'deposited': sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] > 0])
        'withdrawn': abs(sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] < 0]))
    }

или разделить его дальше:

def deposited():
    return sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] > 0])

def withdrawn():
    return abs(sum([Transaction["amount"] for Transaction in self.ledger if Transaction["amount"] < 0]))

Я не вижу необходимости избегать повторения внутри sum для небольшого блока кода, как это (иначе вы могли бы перенести сравнение как лямбда-функцию, а затем передать это функции «транзакционного лета»):

def transaction_summer(transactions, filter_):
    return sum([transaction["amount"] for transaction in transactions if filter_(transaction)]

def withdrawn():
    return abs(self.transaction_summer(self.ledger, lambda x: x['amount'] < 0))

Если вам это нужно только для работы с леджером, переименуйте метод в ledger_summer и отбросьте аргумент transactions, если это делает код более понятным для вас.

Ответ принят как подходящий

Есть два способа решить эту проблему: либо вы каждый раз, когда совершаете транзакцию, отслеживаете общую сумму, внесенную на данный момент, и общую сумму, снятую на данный момент (что является наиболее эффективным), либо вы просто пересчитываете ее каждый раз. . Я бы порекомендовал использовать первое решение, но, поскольку вы, кажется, каждый раз пересчитываете его, это не должно быть проблемой для вас.

Первое решение: пересчитывать его каждый раз

Используя @property, вы можете сделать что-то похожее на атрибут, но значение которого фактически вычисляется каждый раз, когда вы обращаетесь к нему (и вы не можете его записать). Как раз то, что нам нужно!

class Whatever:
   [...]
   @property
   def all_deposited(self):
      return sum(transaction["amount"] for transaction in self.ledger if transaction["amount"] > 0)
   @property
   def all_withdrawn(self):
      return sum(-transaction["amount"] for transaction in self.ledger if transaction["amount"] < 0)

Второе решение: отслеживать снятые и внесенные деньги до сих пор

class Whatever:
   def __init__(self, ...):
      [...]
      self.all_deposited = 0
      self.all_withdrawn = 0
   [...]
   def deposit(self, amount, ...): # I don't know what the real signature is
      [...]
      self.all_deposited += amount
   def withdraw(self, amount, ...):
      [...]
      self.all_withdrawn += amount

Затем в обоих случаях вы можете получить доступ к этим суммам как к атрибутам:

def percentage_spent(self):
    percentage = round(((self.all_withdrawn*100)/self.all_deposited),-1)
    return percentage

def get_balance(self):
   self.balance = self.all_deposited - self.all_withdrawn
   return self.balance

Другие вопросы по теме