ポストメソッドで取得してきた値をInt型に直す方法が知りたい

Question

Pythonで家計簿アプリを作っているのですが、ポストメソッドで取得した値（数字）がInt型に直すことができずに困っています。

具体的には（下にpythonと一部のhtmlのコードを載せています）

   used_money = request.form.get('used_money')
        used_money = int(used_money)
        check_category.remaining_budget -=used_money

この部分なんですが、
used_moneyをInt()で囲むと
invalid literal for int() with base 10:
とエラーが出てきました。これはIntに直せないものを直そうとしているので出るエラーだと分かったのですが、値自体はターミナルにprintしてみたところ数字がでてきており、なぜ直らないのか分かりません。

そこでこの値をInt型に直す方法と、今のままでなぜIntに直らないのか、の二点教えていただきたいです。

from flask import Flask
from flask import render_template,redirect,request
from flask_sqlalchemy import SQLAlchemy
from flask_migrate import Migrate
import datetime

app = Flask(__name__)
app.config['SQLALCHEMY_DATABASE_URI'] = 'sqlite:///database.db'
app.config['SQLALCHEMY_TRACK_MODIFICATIONS'] = False
db = SQLAlchemy(app)
migrate = Migrate(app,db)

class Category(db.Model):
    category_name = db.Column(db.String,db.ForeignKey('detail.id'),primary_key=True)
    budget = db.Column(db.Integer,nullable=False)
    remaining_budget = db.Column(db.Integer,nullable=False)
    month = db.Column(db.Integer,nullable=False)

class Detail(db.Model):
    id = db.Column(db.Integer,primary_key=True)
    category_name = db.relationship('Category',backref='detail',lazy=True)
    used_money = db.Column(db.Integer,nullable=False)
    purchased_item = db.Column(db.String,nullable=False)   
    month = db.Column(db.Integer,nullable=False) 
    day = db.Column(db.Integer,nullable=False)

today = datetime.date.today()
missing_entry = 0
duplicate_error = 0

@app.route('/',methods=['GET','POST'])
def index():
    if request.method == 'GET':
        display_categorys = Category.query.filter(Category.month == 6).all()
        display_details = Detail.query.filter(Detail.month == 6).all() 
        check_count = Detail.query.filter(Detail.purchased_item.like('%eh%')).count()
        month = today.month
        if check_count == 0:
            return render_template('index.html',categorys=display_categorys,details=display_details,month=month,count=0)   
        else:
            return render_template('index.html',categorys=display_categorys,details=display_details,month=month)
    else:
        month = request.get('month')
        try:
            search_categorys = Category.query.fileter_by(month = month).all()
            search_details = Detail.query.filter_by(month = month).all()
            return render_template('index.html',categorys=search_categorys,details=search_details,month=month)
        except:
            return render_template('index.html')

@app.route('/create_category' ,methods=['GET','POST'])
def create_category():
    if request.method == 'GET':
        return render_template('create_category.html')
    else:
        category_name = request.form.get('category_name')
        budget = request.form.get('budget')
        category_month = today.month
        add_category = Category(category_name=category_name,budget=budget,remaining_budget=budget,month=category_month)
        db.session.add(add_category)
        try:
            db.session.commit()
            return redirect('/')
        except:
            if category_name == '' or budget == '':
                return render_template('create_category.html' ,missing_entry = 1)
            else: 
                return render_template('create_category.html',duplicate_error = 1)

@app.route('/<string:category_name>/create_detail/' ,methods=['GET','POST'])
def create_detail(category_name):
    check_category = Category.query.filter_by(category_name=category_name).first()
    if request.method == 'GET':
        return render_template('create_detail.html',category=check_category)
    else:
        purchased_item = request.form.get('purchased_item')
        used_money = request.form.get('used_money')
        check_category.remaining_budget -=used_money
        detail_day = today.day
        detail_month = today.month
        try:    
            add_detail = Detail(purchased_item=purchased_item,used_money=used_money,day=detail_day,month=detail_month)
            db.session.add(add_detail)
            db.session.commit()
            return redirect('/')
        except:
            return render_template('create_detail.html',missing_entry = 1)

{% extends 'base.html' %}
{% block body %}
<h1 class="title">{{month}}月の家計簿</h1>
<h1 class="detail_second_title">帳簿</h1>
<h2 class="detail_category_name">カテゴリ：{{category.category_name}}</h2>
{% if error == 1 %}
<h3>記入漏れ、または記入エラーがあります</h3>
{% endif %}
<article class="detail_form">
    <form action="" method="POST">
        <label for="purchased_item" class="detail_label_purchased_money"><h3>買ったもの</h3></label>
        <input type="text" name="purchased_item" class="detail_input_purchased_item">
        <label for="used_money"><h3  class="detail_label_used_money">金額</h3></label>
        <input type="number" name="used_money"  class="detail_input_used_money">
        <input type="submit" value="確定" class="detail_input_submit">
    </form>
</article>
{% endblock %}

Comments

[can110]

printした結果をそのまま文字として提示ください。
そのさいできれば「print(f'[{used_money}]')」のように前後を[]なりで囲み余計な文字など確認しやすくするとよいです。

[quickquip]

invalid literal for int() with base 10:
の右に何も出ていませんか。
の左は ValueError: が出ていませんか。

https://teratail.com/help/question-tips#questionTips34
> 表示されたエラーメッセージをそのままコピー&ペーストしましょう。自分でタイプしなおしたり、自分で解釈・要約しようとしてはいけません。

[asasika_R]

CAN１１０さん、コメントありがとうございます。
quickquipさん、アドバイスありがとうございます。

Answer 1

質問文に記載されているコードを実行してみたところ、実行できませんでした。

質問には、現象が再現する最小のコードを提示ください。
そもそも、下の長いコードの中にused_money = int(used_money)が見当たりません。

また、エラーを記載する場合は、省略すること無くそのままの文章を全文記載してください。
Pythonの場合はTracebackというところからです。

再現に使ったコード

以下で確認してみました。

from flask import Flask, render_template, request

app = Flask(__name__)


@app.route('/', methods=['GET', 'POST'])
def index():
    if request.method == 'GET':
        return render_template('index.html')
    else:
        tmp = request.form.get('used_money', None)
        aaa = int(tmp)
        if tmp is None:
            return render_template('index.html', error=1)
        # if tmp == '':
        #     return render_template('index.html', error=1)
        if tmp.isdigit():
            used_money = int(tmp)
            return render_template('index.html', used_money=used_money)
        else:
            return render_template('index.html')

<!DOCTYPE html>
<html lang="en">
<head>
    <meta charset="UTF-8">
    <title>Title</title>
</head>
<body>
{% if error==1 %}
    <span style="color: #ff0000;">error</span>
{% endif %}
{% if used_money %}
    入力された数値は： {{ used_money }}
{% endif %}
    <article class="detail_form">
        <form action="" method="POST">
            <label for="purchased_item" class="detail_label_purchased_money"><h3>買ったもの</h3></label>
            <input type="text" name="purchased_item" class="detail_input_purchased_item">
            <label for="used_money"><h3  class="detail_label_used_money">金額</h3></label>
            <input type="number" name="used_money"  class="detail_input_used_money">
            <input type="submit" value="確定" class="detail_input_submit">
        </form>
    </article>
</body>
</html>

再現方法

質問文にあるエラーメッセージ(invalid literal for int() with base 10:)と完全一致するものは再現出来ませんでしたが、以下を出力はできました。
ValueError: invalid literal for int() with base 10: ''

フォームに何も入力せず「確定」を押すと再現可能です。

回答

そこでこの値をInt型に直す方法と、
今のままでなぜIntに直らないのか、の二点教えていただきたいです。

変換できる値でないと変換出来ません。
asasika_Rさんのコードでは、来たものを何もかも受け入れて、10進数でない値「''」を変換しようとしているためエラーになっているものと推測できます。

以下のように、意図せぬ値が来た場合、変換を行わないなどの処理を追加してください。(以下で全てカバーできているかはわかりませんので、確認ください。)

@app.route('/', methods=['GET', 'POST'])
def index():
    if request.method == 'GET':
        return render_template('index.html')
    else:
        tmp = request.form.get('used_money', None)
        if tmp is None:
            return render_template('index.html', error=1)
        if tmp == '':
            return render_template('index.html', error=1)
        if tmp.isdigit():
            used_money = int(tmp)
            return render_template('index.html', used_money=used_money)
        else:
            return render_template('index.html')

request.form.getの第2引数は、取得使用としているキーが存在しない場合に取得できる値です。(used_moneyが無い場合、エラーにならずNoneが取得できます)
Noneの場合はエラーなので、エラーで終了させます。
Noneで無い場合は、取得されている値が''で無いかを確認しています。
''の場合、数字で無いのでエラーで終了させます。

この時点でtmpに値が入っていますが、数字では無い可能性を考慮し、数字かどうかを判定しています。

この判定が終わった時点で、数字で構成された文字列であるはずなので、intを使って変換し、templateに渡して表示する、という感じです。

デバッグを全くできていない印象があるのでPyCharmなどのIDE(デバッガ)の使用をお勧めします。

Comments

[asasika_R]

丁寧なご回答ありがとうございます。意図せぬ値が来た時の処理参考になりました。理解してコードに取り込みたいと思います。

Answer 2

質問の修正依頼でエラー文をもう一度出そうしたところ、正常に動きました。動いたコードの下に載せておきます。

@app.route('/<string:category_name>/create_detail/' ,methods=['GET','POST'])
def create_detail(category_name):
    check_category = Category.query.filter_by(category_name=category_name).first()
    if request.method == 'GET':
        return render_template('create_detail.html',category=check_category)
    else:
        purchased_item = request.form.get('purchased_item')
        used_money = request.form.get('used_money')
        check_category.budget -= int(used_money)
        detail_day = today.day
        detail_month = today.month
        try:    
            add_detail = Detail(purchased_item=purchased_item,used_money=used_money,day=detail_day,month=detail_month)
            db.session.add(add_detail)
            db.session.commit()   
            return redirect('/')
        except:
            return render_template('create_detail.html',missing_entry = 1)