int値をsaveするために、モデルフィールドではなくモデルインスタンスで保存を実行する方法が知りたいです。

Question

実現したいこと

商品＝カニがカートに入れられたら、カニ商品データ（Crabdb.model）から在庫数を取得して、カニの在庫数を一つ減らして新たに在庫数を保存したいです。

発生している問題・エラーメッセージ

'int' object has no attribute 'save'

該当のソースコード

from django.shortcuts import render, redirect, get_object_or_404
from django.http import HttpResponse, Http404, JsonResponse
from django.contrib import messages
from django.contrib.auth.decorators import login_required
from django.contrib.auth.models import User
from django.core.paginator import Paginator
from ECrab.models import Profile, Crabdb, Cart

import random

def list_view(request):
    Crabdb_list = Crabdb.objects.all().order_by('-id')
    paginator = Paginator(Crabdb_list, 20) 

    try:
        page = int(request.GET.get('page'))
    except:
        page = 1

    posts = paginator.get_page(page)
    return render(request, 'ECrab/post_list.html', {'posts': posts, 'page': page, 'last_page': paginator.num_pages})

def detail_view(request, post_id):
    post = get_object_or_404(Crabdb, id=post_id)

    try:
        page = int(request.GET.get('from_page'))
    except:
        page = 1
    return render(request, 'ECrab/post_detail.html', {'post': post, 'page': page })

def edit_cart(request, post_id):
    
    post = get_object_or_404(Crabdb, id=post_id)
    
    try:
        page = int(request.GET.get('from_page'))
    except:
        page = 1
    
    user_id = request.user.id
   
    previous_cart = Cart.objects.filter(user_id=user_id, product_id=post_id)
    
    if previous_cart.exists():
        
        previous_cart.delete()
    else:
        product = Cart(user_id=user_id, product_id=post_id)
        product.save()

    #在庫数を取得
    x = Crabdb.objects.get(id=post_id)
    stock_quality = x.stock
    if  stock_quality > 0: #在庫がある場合
        stock_quality -= 1 #在庫を一つ減らす
        stock_quality.save()
    else:
        print("在庫がありません")

    return render(request, 'ECrab/post_detail.html', {'post': post, 'page': page })

試したこと

stock_quality=stockはCrabdbモデルのデータの一部の整数だから、
モデルフィールドではなく、モデルインスタンスで保存を実行する必要がある。という文章を見つけました。

そのため、12行目のようにしてみたのですが、そしたらsave() missing 1 required positional argument: 'self'
とエラーが出てきました。

・・・
1 else:
2        #無ければユーザID・商品IDの組をカートに追加
3        #productはCartクラスのインスタンス
4        product = Cart(user_id=user_id, product_id=post_id)
5        product.save()
6
7    #在庫数を取得
8    x = Crabdb.objects.get(id=post_id)
9    stock_quality = x.stock
10    if  stock_quality > 0: #在庫がある場合
11        stock_quality -= 1 #在庫を一つ減らす
12        Crabdb.save() #モデルインスタンスで保存してみた
13    else:
14        print("在庫がありません")

djangoを始めたばかりで、引数などを理解していないから起こっているエラーだとは思います。
解決方法を教えてもらえると助かります。よろしくお願いします。

補足情報（FW/ツールのバージョンなど）

Mac M1
shell zsh
python,django

Answer 1

生成したインスタンスの方で　save()を使用するのが正しいのかなと思いました。

・・・
1 else:
2        #無ければユーザID・商品IDの組をカートに追加
3        #productはCartクラスのインスタンス
4        product = Cart(user_id=user_id, product_id=post_id)
5        product.save()
6
7    #在庫数を取得
8    x = Crabdb.objects.get(id=post_id)
9    stock_quality = x.stock
10    if  stock_quality > 0: #在庫がある場合
11        stock_quality -= 1 #在庫を一つ減らす
12　　　　　　　　　　　　　　　　x.stock = stock_quality
13        x.save() # Crabdbではなくxでsave()を使用する
14    else:
15        print("在庫がありません")

Comments

[momonga8316]

教えていただいた方法を実行したところ、エラー文は出なくなりました！ありがとうございます！

しかし、、、、在庫に反映されないという現象が起こってしまいました・・・。管理ページに移っても在庫は減っていません。
htmlでの在庫数を「在庫数:{{ post.stock }}」と表示しているのが原因か、はたまた何かコードが違うのでしょうか。。。すみません、質問ばかりしてしまい。

[momonga8316]

shellコマンドでレコードの取得、変更、保存を試してみました。
x.save()　で試したところ、stock_quality（在庫数）は変化していませんでした。

[__horito]

DBが更新されているかも確認してみてください
Crabdbクラスのコードも見てみたいです

[__horito]

更新する前のデータを渡してるので画面には更新してないように見えるのかもしれません。
#在庫数を取得の処理をpost取得する前行うようにしてみてください

[momonga8316]

CrabdbおよびCartクラスのmodels.pyです。

```
class Crabdb(models.Model):
class Meta:
verbose_name = 'カニデータ'
verbose_name_plural = 'カニデータ'

id = models.CharField('カニID',max_length=6,primary_key=True) # 'A00001'
type = models.CharField('カニの種類',max_length=30) # 'ケガニ'
area = models.CharField('生産地',max_length=30) # '北海道'
weight = models.IntegerField('重さ')
stock = models.IntegerField('在庫数',default=0)
price = models.FloatField('カニの値段')
delivery_date = models.IntegerField('お届け日')

owner = models.ForeignKey(Profile,
verbose_name='所有者',
null=True, blank=True,
on_delete=models.CASCADE
)

def __str__(self):
return self.id

class Cart(models.Model):
class Meta:
verbose_name = 'カートデータ'
verbose_name_plural = 'カートデータ'

user_id = models.CharField('ユーザーID',max_length=6) # 'B00001'
product_id = models.CharField('商品ID',max_length=6) # 'A00001'
timestamp = models.DateTimeField('追加日時', auto_now_add=True) #カートに追加時の現在時刻'13:15'

def __str__(self):
return self.user_id + ' ' + self.product_id
```

[momonga8316]

＞#在庫数を取得の処理をpost取得する前行うようにしてみてください
実行してみましたが、管理画面および表ページでも変化ありませんでした（T.T）

[__horito]

申し訳ないですが原因はわからなかったです。正直ほかにおかしいところ見当たらないです、、

[momonga8316]

holy_さん

```
else:
#無ければユーザID・商品IDの組をカートに追加
#productはCartクラスのインスタンス
product = Cart(user_id=user_id, product_id=post_id)
product.save()

x = Crabdb.objects.get(id=post_id)
if x.stock > 0: #在庫がある場合
x.stock -= 1 #在庫を一つ減らす
x.save() #モデルインスタンスで保存
print("成功しました")
else:
print("在庫がありません")

return render(request, 'ECrab/post_detail.html', {'post': post, 'page': page })
```

こちらに変更したところ、在庫を減らすことができました。
たくさん回答していただきありがとうございました！mm

Answer 2

・・・
（略）

else:
#無ければユーザID・商品IDの組をカートに追加
#productはCartクラスのインスタンス
product = Cart(user_id=user_id, product_id=post_id)
product.save()

x = Crabdb.objects.get(id=post_id)

stock_quality = x.stock　　←置き換えた部分を消す

if x.stock > 0: #在庫がある場合
x.stock -= 1 #在庫を一つ減らす
x.save() #モデルインスタンスで保存
print("成功しました")
else:
print("在庫がありません")

return render(request, 'ECrab/post_detail.html', {'post': post, 'page': page })