関数の定義の仕方を教えてください。

Question

modelの中に関数定義を行いましたがins_resultに空欄が入ってしまいます。
以前登録した内容には数字の1が入っています。
間違っていることはわかるのですが、何をどうすればよいかわからないので
ご教授いただきたいです。

内容はnext_calibration_dateが当月になっていれば'校正対象'、
来月以降であれば'合格'、当月より前であれば'不合格'と
ins_result = models.CharField(max_length=10,)に入力されるようにしたいです。

ご教授お願いいたします。

_models.py


class InsLedger(models.Model):
    measuring_ins_group = models.IntegerField(choices=MEASURING_INS_GROUP_CHOICES,default=1)
    measuring_tool = models.ForeignKey(Device, on_delete=models.CASCADE)
    ins_num = models.CharField(max_length=10)
    calibration_type = models.IntegerField(choices=CALIBRATION_CHOICES)#校正期間
    calibration_date = models.DateField(default=now)#datetime.datetime.now())
    next_calibration_date = models.DateField()
    serial_num = models.CharField(max_length=20, null=True, blank=True, default='')
    product_num = models.ForeignKey(Product_num, on_delete=models.CASCADE)#対象品番
    team = models.ForeignKey(TeamModel, on_delete=models.CASCADE, related_name='teams1')
    line = models.ForeignKey(LineModel, on_delete=models.CASCADE, related_name='lines1')
    useapp = models.ForeignKey(Useapp, on_delete=models.CASCADE)#使用用途
    spec1 = models.ForeignKey(Spec1, on_delete=models.CASCADE)#最小目盛
    spec2 = models.ForeignKey(Spec2, on_delete=models.CASCADE)#測定範囲
    spec3 = models.IntegerField(choices=SPEC3_CHOICES)#表示
    check_items = models.ForeignKey(Check_Items, on_delete=models.CASCADE)#日常点検項目
    ins_result = models.CharField(max_length=10,)#点検結果

    def save(self, *args, **kwargs):
        auto_now = kwargs.pop('next_calibration_date_auto_now', True)
        if auto_now:
            if self.calibration_type == CALIBRATION_CHOICES[0][0]:
                calibration_span = 1
            elif self.calibration_type == CALIBRATION_CHOICES[1][0]:
                calibration_span = 2
            elif self.calibration_type == CALIBRATION_CHOICES[2][0]:
                calibration_span = 5
            elif self.calibration_type == CALIBRATION_CHOICES[3][0]:
                calibration_span = 10
            
            self.next_calibration_date = self.calibration_date + relativedelta(years=calibration_span)

            super().save(*args, **kwargs)


    def passorfail(self):
        today = datetime.today()
        if self.next_calibration_date == (today.month):
            pass_result = ('校正対象')
        elif self.next_calibration_date > (today.month):
            pass_result = ('合格')
        else:
            pass_result = ('不合格')

        self.ins_result = pass_result

    def __str__(self):
        return str(self.measuring_tool)

Comments

[maisumakun]

> うまく実行されません。

具体的に、どうなるのですか？

[art35]

空欄になります。
以前に登録した内容は【１】が入っています。

Answer 1

passorfail関数内でself.next_calibration_dateがNoneTypeになっていた様ですが
save関数を1とし、passorfail関数を2とするならば
passorfail関数をsave関数とリネームした事により
1を実行しているつもりが2だけを実行していた為に
self.next_calibration_dateが定義されていなかった事が原因の一つでしょう。

Comments

[art35]

ご指摘の内容で修正しました。修正後は別のerrorが出ましたが一旦改善できました。
まだ1部今回の問題点とは違うコードの気持ち悪さが残っていますが、機能的には問題なく動作します。
super().save(*args, **kwargs)がないと更新時に更新ができていなかったので追加しました。
添付頂いた引数については一度目を通しましたが理解できるまでみてみます。
ありがとうございました。
ベストアンサーにさせていただきます。
---
def save(self, *args, **kwargs):
auto_now = kwargs.pop('next_calibration_date_auto_now', True)
if auto_now:
if self.calibration_type == CALIBRATION_CHOICES[0][0]:
calibration_span = 1
elif self.calibration_type == CALIBRATION_CHOICES[1][0]:
calibration_span = 2
elif self.calibration_type == CALIBRATION_CHOICES[2][0]:
calibration_span = 5
elif self.calibration_type == CALIBRATION_CHOICES[3][0]:
calibration_span = 10

self.next_calibration_date = self.calibration_date + relativedelta(years=calibration_span)

self.passorfail()
super().save(*args, **kwargs)

def passorfail(self):
today = datetime.today()

if self.next_calibration_date.strftime('%Y-%m-%d') == today.strftime('%Y-%m-%d'):
pass_result = ('校正対象')
elif self.next_calibration_date.strftime('%Y-%m-%d') > today.strftime('%Y-%m-%d'):
pass_result = ('合格')
else:
pass_result = ('不合格')

self.ins_result = pass_result

---

[art35]

tabの部分がうまく表示されていません。

[nto]

こういった場合は「tab」ではなくプログラミングでは「インデント」といいます。

[art35]

インデントですね。ありがとうございます。

super()～ですが、ご教授いただいたように抜いてみたのですが、
登録してあるもの対して更新してもcalibration_dateの日付が変わらなかったため
機能的に問題があったので追加しました。
抜いた場合、何を変更すればよろしいでしょうか。

[bsdfan]

> save関数の中で最後に実行しているsuper().save(*args, **kwargs)とありますが
> *args, **kwargsというのは可変長引数といって、関数を実行するときに与えるものではありません。
> 実行する関数で、引数を受け取る時に使用するものです。

については正しくないです。関数を呼び出すときにも使う表現です。
super().save(*args, **kwargs) は、メソッドで受け取った引数をそのままに親クラスの同名メソッドを呼び出すときの普通のやりかたです。
なので、ここは問題とは関係ないと思います。

[nto]

bsdfanさん、オーバーライドですよね
抜けておりました。
djangoに関しても知見が浅はかでした。
部分的に回答とコメントの内容を修正致しました。
そうするとsuperの呼び出しに際してを明示的に行う事で解決に至れないでしょうか。

Answer 2

• passorfail(self)中のif文の結果がただ文字列が書かれているだけですが、個別にreturnするかself.ins_resultとやらに代入すべきなのではないでしょうか。
• self.next_calibration_date < (today.month)これの不等号は逆では？

Comments

[art35]

ご指摘ありがとうございます。
以下のように修正しましたが、これでも反映されませんでした。
def passorfail(self):
today = datetime.today()
if self.next_calibration_date == (today.month):
return self.ins_result('校正対象')
elif self.next_calibration_date > (today.month):
return self.ins_result('合格')
else:
return self.ins_result('不合格')

[jeanbiego]

ins_result = models.CharField(max_length=10,)で定義しているins_resultをなぜcallしているのでしょうか。modelsというのはDjangoの機能なのでしょうが、class内関数の話の前にDjangoの使い方を先にさらったほうが良いのではないでしょうか。
私はDjangoは詳しくないので、すみませんがこれ以上のアドバイスはできかねます。

[art35]

ありがとうございます。
調べてみます。

Answer 3

解決しましたので添付しておきます。
コメント、回答頂きました皆様ありがとうございました。

    def passorfail(self):
            today = datetime.today()
            if self.next_calibration_date.strftime('%Y-%m-%d') == today.strftime('%Y-%m-%d'): 
                pass_result = ('校正対象')
            elif self.next_calibration_date.strftime('%Y-%m-%d') > today.strftime('%Y-%m-%d'):
                pass_result = ('合格')
            else:
                pass_result = ('不合格')

            self.ins_result = pass_result

Comments

[nto]

そもそも同じ名前の関数を定義はできません。
(定義した場合、実行する際には一番最後に定義したものが実行され、それよりも前に定義された同名の関数は無視されます。)
「def passorfailとしていたのをdef saveとして以下を参考にしたところ、動きました。」
との事ですが、定義した関数の名前を変えるのではなく、実行する関数をpassorfail()とすべきでしょう。

先程行っていた別の質問に関してもまずはこちらを訂正する必要がありそうです。

[art35]

はい、その通りでした。
動かしてみたところ前のdef saveが実行されていませんでした。
よくわからなくなってきたので考えてみます。

Answer 4

class InsLedger(models.Model):
    measuring_ins_group = models.IntegerField(choices=MEASURING_INS_GROUP_CHOICES,default=1)
	...
    ins_result = models.CharField(max_length=10,)#点検結果

これは、次の間違いではありませんか？

class InsLedger(models.Model):
	def __init__(self):
		self.measuring_ins_group = models.IntegerField(choices=MEASURING_INS_GROUP_CHOICES,default=1)
		...
    	self.ins_result = models.CharField(max_length=10,)#点検結果

追記
Python は 1つのクラスに、同名のクラス属性とインスタンス属性を持てます。

class A:                 # A はクラス
	result = 3           # result は A のクラス属性
	def __init__(self):
		self.result = 5  # result は A のインスタンス属性

a = A()                  # a は A のインスタンス
print('a.result =', a.result)   # a.result = 5
print('A.result =', A.result)   # A.result = 3

インスタンス属性が無ければ、a.result でクラス属性を参照できます。
インスタンス属性があれば、a.result = 7 でそれを変更できますが、
インスタンス属性が無ければ、a.result = 7 で新たにインスタンス属性を
作ってしまいます。

Java で、同じようなコードを書くと、

class A {
	int result;              // result はインスタンスフィールド
	static int result2 = 3;  // result2 はクラスフィールド
                             // 同名のフィールドは持てない
	A() { result = 5; }
}

class Main {
	public static void main(String[] args) {
		A a = new A();
		System.out.println("a.result = " + a.result);
		System.out.println("A.result2 = " + A.result2);
	}
}

Comments

[art35]

以下の2つ以外は正常に動き、追加したこの２つに悪さがあると認識していますが違いますでしょうか。

ins_result = models.CharField(max_length=10,)#点検結果

def passorfail(self):
today = datetime.today()
if self.next_calibration_date == (today.month):
pass_result = ('校正対象')
elif self.next_calibration_date > (today.month):
pass_result = ('合格')
else:
pass_result = ('不合格')

self.ins_result = pass_result

def __init__(self):
self.～
を付けるとほかの項目を変更する必要が出てくるため、defpassorfailの関数部分のおかしな点があればご教授いただけると幸いです。

[art35]

追記ありがとうございます。
初学者のため、内容が理解できません。
この内容が今回の質問に対してどう作用するのかわからないです。
また、JAVAは全く知らないためわかりません。
具体的に何をどうすればよいかご教授いただけないでしょうか。
よろしくお願いいたします。