Flask 他のテーブルへのデーター追加（登録）について

Question

FlaskでIPアドレス管理台帳アプリを作成中です。
登録されたIPアドレスを削除する前に、未使用のIPアドレスを保存しているテーブル（freeipテーブル）へ
追加した後に、削除を実行したいのですが、この処理をどの様にすればよいのか分からない状態です。

具体的には、以下の部分です。

ご教示の程、よろしくお願い致します。

# app.py抜粋  

@app.route('/delete/<int:id>')
def delete(id):

    # ipredgeテーブル内の削除対象idを取得
    ipredg = Ipredg.query.get(id)

    # ipredgeテーブルから削除データをfreeipテーブルに登録するため、それぞれのデーターを取得しておく
    ipredg.ip = request.get('ip')
    ipredg.mac = request.get('mac')
    ipredg.hostname = request.get('hostname')
    ipredg.remark = request.get('remark')

    # ipredgeテーブルから削除したデーターをfreeipテーブルへ登録
    # ここから、freeipテーブルに、どうやって追加すれば良いのか？
    db.session.add()

    # ipreddgeテーブルから、削除ボタンを押したデーターを削除
    db.session.delete(ipredg)
    db.session.commit()
    return redirect('/')

# app.py

from datetime import datetime

from flask import Flask, render_template, request, redirect, url_for
from flask_sqlalchemy import SQLAlchemy
import os


app = Flask(__name__)
key = os.urandom(21)
app.secret_key = key

app.config['SECRET_KEY'] = 'mysecretkey'
URI = 'sqlite:///ip.db'
app.config['SQLALCHEMY_DATABASE_URI'] = URI
app.config['SQLALCHEMY_TRACK_MODIFICATIONS'] = False
db = SQLAlchemy(app)


from sqlalchemy.engine import Engine
from sqlalchemy import event

@event.listens_for(Engine, "connect")
def set_sqlite_pragma(dbapi_connection, connection_record):
    cursor = dbapi_connection.cursor()
    cursor.execute("PRAGMA foreign_keys=ON")
    cursor.close()


class Ipredg(db.Model):
     __tablename__='ipreg'
     id = db.Column(db.Integer, primary_key=True)
     ip = db.Column(db.String(20))
     mac = db.Column(db.String(20))
     hostname = db.Column(db.String(50))
     remark = db.Column(db.String(50))
    
     def __init__(self, ip, mac, hostname, remark):
         self.ip = ip
         self.mac = mac
         self.hostname = hostname
         self.remark = remark


class Freeip(db.Model):
     __tablename__='freeip'
     f_id = db.Column(db.Integer, primary_key=True)
     f_ip = db.Column(db.String(20))
     f_mac = db.Column(db.String(20))
     f_hostname = db.Column(db.String(50))
     f_remark = db.Column(db.String(50))
    
     def __init__(self, f_ip, f_mac, f_hostname, f_remark):
         self.f_ip = f_ip
         self.f_mac = f_mac
         self.f_hostname = f_hostname
         self.f_remark = f_remark


@app.cli.command('initialize_DB')
def initialize_DB():
    db.create_all()

@app.route('/')
def index():
    ipredg = Ipredg.query.all()
    return render_template('index.html', ipredg=ipredg)

@app.route('/free')
def free():
    free_ip = Freeip.query.all()
    return render_template('freeip.html', free_ip=free_ip)

@app.route('/delete/<int:id>')
def delete(id):

    # ipredgeテーブル内の削除対象idを取得
    ipredg = Ipredg.query.get(id)

    # ipredgeテーブルから削除データをfreeipテーブルに登録するため、それぞれのデーターを取得しておく
    ipredg.ip = request.get('ip')
    ipredg.mac = request.get('mac')
    ipredg.hostname = request.get('hostname')
    ipredg.remark = request.get('remark')

    # ipredgeテーブルから削除したデーターをfreeipテーブルへ登録
    # ここから、freeipテーブルに、どうやって追加すれば良いのか？
    db.session.add()

    # ipreddgeテーブルから、削除ボタンを押したデーターを削除
    db.session.delete(ipredg)
    db.session.commit()
    return redirect('/')


if __name__ == '__main__':
    app.run(debug=True)

<!-- index.html -->
{% extends 'base.html' %}

{% block content %}
<h1>登録一覧画面</h1>
<table class="table table-striped">
    <tr>
        <th>#</th>
        <th>IP Address</th>
        <th>Mac Address</th>
        <th>Hostname</th>
        <th>備考</th>
    </tr>
    {% for list in ipredg %}
    <tr>
        <td>
            <a href="update/{{list.id}}" role="button">更新</a>
            <a href="delete/{{list.id}}" role="button">削除</a> 
        </td>
        <td>{{list.ip}}</td>
        <td>{{list.mac}}</td>
        <td>{{list.hostname}}</td>
        <td>{{list.remark}}</td>
    </tr>
    {% endfor %}
    
</table>

{% endblock %}

Answer 1

エラーメッセージの掲載もなく、何もかもわからないという質問は、以下にある「コードをください・デバッグしてください等の丸投げの質問」にあたり、回答が付かない可能性が高いです。

https://teratail.com/help/avoid-asking

質問するときは、どのような事をしたくて、どのようなことを実行して、どのような問題があり、どのような解決策を図ったが、結局どうなったのか、といった情報が必要です。
解決策については、「ググりました」だと、回答者が同じことをして時間が無駄になる可能性があるので、具体的にどのような物を見て実行して、どうだったのかというのを記載下さい。

これだけだと微妙なので、一応回答も記載します。

まず、以下がInsert/Update/Deleteのドキュメントです。
https://flask-sqlalchemy.palletsprojects.com/en/3.0.x/queries/

これによると、

db.session.delete(user)
db.session.commit()

とありますので、以下で削除できるはずです。

ipredg = Ipredg.query.get(id)
db.session.delete(ipredg)
db.session.commit()

Ipredg.query.get(id)にヒットしない場合、戻り値はNoneになるはずなので、Noneの場合は処理しないとか、id指定が間違っているとかのエラー処理を入れた方が良いでしょう。

以下の部分は何がしたいのか不明です。
それぞれのデーターを取得しておく、と書いてありますが、既にidで取得済みのデータが入っているところに、Webからの入力値で上書きを実施しています。
updateするなら意味が分かりますが、削除にしか使っていないものを上書きする意味は無いはずです。

    # ipredgeテーブルから削除データをfreeipテーブルに登録するため、それぞれのデーターを取得しておく
    ipredg.ip = request.get('ip')
    ipredg.mac = request.get('mac')
    ipredg.hostname = request.get('hostname')
    ipredg.remark = request.get('remark')

で、肝心の質問っぽいところ以下についてです。

    # ipredgeテーブルから削除したデーターをfreeipテーブルへ登録
    # ここから、freeipテーブルに、どうやって追加すれば良いのか？
    db.session.add()

この質問が書いてある理由が理解できません。
ご自身の1つ前の質問で、add.pyというものを提示されていましたよね？

以下がその抜粋です。

free_ip_list = []
for i in range(100, 111):
    free_ip_list.append(freeip(f"192.168.1.{i}", f"00:{i}:aa:cc:dd", f"host{i}", f"備考{i}"))

db.session.add_all(free_ip_list)
db.session.commit()

これがfreeipの追加コードなのはわかりますか？
自分で書いたのだから分かると信じていますが、これをは複数を追加していますので、1つだけ追加するように変えれば良いだけです。
add_allではなくaddですね。

freeipの引数には、request.getで取得した値を指定すれば良いでしょう。

わからない事があればコメントをお願いします。

2023/01/23 追記

自分が書いたコード内でやっていると思いますが、自分の書いたコードが分からないということですか？

前の質問では、class freeip(db.Model):と書かれていましたが、現在のコードではclass Freeip(db.Model):と、最初の人文字が大文字になっているので、

myip = Freeip("ipaddress", "mac address", "host", "備考")

のようにClassを作って、

db.session.add(myip)
db.session.commit()

として実行すれば追加されると思いますが、何か別の事をやろうとしていますか？

それとも、上の例で書いた("ipaddress", "mac address", "host", "備考")の文字列部分に何を入れれば良いか分からないということでしょうか？

IpredgとFreeipが何を表しているのか不明ですが、Ipredgから削除した物と同じものをFreeipに登録したいというのでしたら、

ipredg = Ipredg.query.get(id)

として取得したipredgの内容を、全部自分でコピーして使えば良いと思います。
例えば、

myip = Freeip(ipredg.ip, ipredg.mac, ipredg.hostname, ipredg.remark)
db.session.add(myip)
db.session.commit()

Comments

[ahosutaman]

def delete(id):の処理内で、選択したidのデーターをfreeipテーブルへ登録し、ipregテーブルから削除を行いたいということです。ipregテーブルから選択したデーターを削除することは、できているのですが、def delete(id):内でどうやって、freeipのインスタンスを呼び出し、freeipテーブルへ登録しする処理を記載すればいいのかが分からない状況です。

[ahosutaman]

FiroProchaainezoさん
色々言葉足らずで申し訳ありませんでした。また、色々と記載いただきありがとうございます。行いたい処理としては、「ipredgから削除した物と同じものをFreeipに登録したいとい」ということです。ご提示いただきましたコードで実現できました。ありがとうございました。