modelで定義したメソッドを呼び出したい

Question

###開発環境
Python 3.7.1
Django version 2.1.5

困っていること

models.pyに集計ロジックの関数を記述し、views.pyでその関数を呼び出すことで、データの集計作業をしたいと考えています。
しかし、models.pyで集計ロジックを定義したインスタンスメソッド（def total_price(self)）をviews.pyで呼び出すとエラーが出てしまい、困っています。
何か良い方法があれば、ご教授頂けますと幸いです。

###出ているエラー内容
TypeError at /invoice_index
unsupported operand type(s) for +: 'function' and 'int'

Lovcal Vars
...
english_invoice <function BillingPriceCalculator.total_price at 0x1076cdd90>

コード

class Lesson(models.Model):
  name = models.CharField('ジャンル', max_length=255)
  flat_price = models.PositiveIntegerField('基本料金', null=False, default=0)

  def __str__(self):
    return self.name

  def calculate_measured_price(lesson_hour):
    lesson_measured_price = LessonMeasuredPrice.objects.select(record, record.lesson_hour <= lesson_hour ).sort_by(lesson_hours).last
    lesson_measured_price.price

class LessonMeasuredPrice(models.Model):
  lesson_id = models.ForeignKey(Lesson, on_delete=models.PROTECT)
  price = models.PositiveIntegerField('金額', null=False, default=0)
  threshold_hours = models.PositiveIntegerField('受講時間（h）', null=False, default=0)


class CustomerLesson(models.Model):
  customer = models.ForeignKey(Customer, verbose_name='顧客名', on_delete=models.PROTECT)
  lesson = models.ForeignKey(Lesson,verbose_name='ジャンル', on_delete=models.PROTECT)

  lesson_date = models.DateField('受講日')
  lesson_hour = models.PositiveIntegerField(
    '受講時間（h）',
    default = 1,
    validators=[
      MaxValueValidator(12),
      MinValueValidator(1)
    ]
  )

  def __str__(self):
    return self.lesson

class BillingPriceCalculator:

  def __init__(self, lesson, customer_lessons):
    self.lesson = lesson
    self.customer_lessons = customer_lessons

  @staticmethod
  def total_price(self):
    flat_price + measured_price


  def __flat_price(self):
    lesson.flat_price

  def __measured_price(self):
    measured_price_unit * total_hours

  def __total_hours(self):
    customer_lessons.sum(lesson_hour)

  def __measured_price_unit(self):
    lesson.calculate_measured_price(total_hours)

def invoice_index(request):

  today = datetime.today()

  #今月の月初日と月末日の算出
  month_start = today.replace(day=1)
  next_month_start = (today + relativedelta(months=1)).replace(day=1)
  month_end = next_month_start - timedelta(days=1)

  customers = Customer.objects.all()
  sum_template =  {'英語': 0, 'ファイナンス': 0, 'プログラミング': 0}
  invoice_list = []

  for customer in customers:
    columns = {}
    english_invoice = 0
    finance_invoice = 0
    programing_invoice = 0

    genre_sum = sum_template.copy()
    lesson_hour_sum = sum_template.copy()

    lessons = customer.customerlesson_set.all()
    this_month_lessons = lessons.filter(lesson_date__range=(month_start, month_end))
    lesson_sum = this_month_lessons.count()

    columns['id'] = customer.id
    columns['name'] = customer.name
    columns['lesson'] = lesson_sum

    for lesson in this_month_lessons:
      lesson_date = lesson.lesson_date
      columns['lesson_date'] = lesson_date

      genre = lesson.lesson.name
      genre_sum[genre] += 1
      columns['genre'] = genre_sum

      lesson_hour = lesson.lesson_hour
      lesson_hour_sum[genre] += lesson_hour
      columns['lesson_hour'] = lesson_hour_sum

      if genre == '英語':
        english_invoice = BillingPriceCalculator(genre, this_month_lessons).total_price

Answer 1

total_priceをstaticmethodにするならselfを引数に取ってはいけませんし、インスタンスメソッドならstaticmethodデコレータを付けてはいけません。

english_invoice = BillingPriceCalculator(genre, this_month_lessons).total_priceはtotal_priceを呼び出していないので、メソッドオブジェクトがenglish_invoice変数に代入されるだけです。呼び出したければ後ろにかっこをつけます。

total_priceを呼び出したとしてもreturn文がありませんので、Noneが返り無意味というか動きません。

ほかにもdjango以前におかしい気がする部分は多数あるので、pythonの基礎をやるのが先という感じがします。rubyに慣れている人かもしれませんが、文法違うので。

Comments

[shohei0718]

hayataka2049さん
total_priceを呼び出すことができました。
まだPythonの学習を始めたばかりなので、改めて基礎文法から勉強し直します。

Answer 2

関数の戻り値は、明示的にreturnで返す必要があります。

Comments

[shohei0718]

Meganezaruさん有難うございます。
返り値がNoneではありましたが、returnで返ってきました。