トップ Flaskに関する質問 flaskのdelete機能の追加につまづいております。(ドのつく初心者です)

編集履歴

回答編集履歴

追記

2023/10/01 02:54

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -294,10 +294,8 @@
 # 追記
 あまり最初から詰め込みすぎると混乱の原因となるため、ここでは詳細は書きませんが、上記のコードではエラーではないものの、まだ以下の問題点があります。
-+ main.pyの@app.route('/')で、booksリストにインデックスを使ってデータを渡している。
++ main.pyの@app.route('/')で、booksリストにインデックスを使ってデータを渡している→この結果、何か修正の必要が生じた場合に、データベースに格納されているデータのカラムの順番を意識しないといけない
-+ データベースのカラムを追加/削除した場合、カラムの順番を意識して書き換える必要がある。（上記だと、idというカラムを先頭に追加したため、delete.html の表示インデックスを１つずらしている）
+（たとえば上記だと、idというカラムを先頭に追加したため、delete.html の表示インデックスを１つずらしている）
-→この結果、何か修正の必要が生じた場合に、データベースに格納されているデータのカラムの順番を意識しないといけない
-このような問題に対処することを考えると、データベースからデータを取得する際は、row_factory を使用した方がベターです。興味があれば調べてみるのもよいでしょう。
+これについては、データベースからデータを取得する際に row_factory を使用することで少しメンテしやすくなります。興味があれば調べてみるのもよいでしょう。

追記

2023/10/01 02:43

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -290,3 +290,14 @@
 上記修正内容を試す場合は、修正後初回のみ、一旦 flask を終了し、プロジェクト内の database.db ファイルを削除してから run.py を再実行するようにしてください。
 修正前のdatabase.dbが残っている状態で起動した場合、うまく動作しません。
+---
+# 追記
+あまり最初から詰め込みすぎると混乱の原因となるため、ここでは詳細は書きませんが、上記のコードではエラーではないものの、まだ以下の問題点があります。
++ main.pyの@app.route('/')で、booksリストにインデックスを使ってデータを渡している。
++ データベースのカラムを追加/削除した場合、カラムの順番を意識して書き換える必要がある。（上記だと、idというカラムを先頭に追加したため、delete.html の表示インデックスを１つずらしている）
+→この結果、何か修正の必要が生じた場合に、データベースに格納されているデータのカラムの順番を意識しないといけない
+このような問題に対処することを考えると、データベースからデータを取得する際は、row_factory を使用した方がベターです。興味があれば調べてみるのもよいでしょう。

2023/10/01 01:41

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -89,7 +89,7 @@
 ## main.py の修正
 ### root ('/') の修正
-root ('/') では、db_booksに、データベースから取得した id も渡すように修正します。
+root ('/') では、index.html の books に、データベースから取得した id も渡すように修正します。
 ```main.py
         books.append({"title": row[0], "price": row[1], "arrival_day": row[2]})
 　　　　　　　　↓

ｓ

2023/10/01 01:22

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -11,9 +11,9 @@
 ```
 book = con.execute('SELECT * FROM books WHERE id = ?', [id]).fetchone()
 ```
-と言う部分でエラーが発生しているということが分かります。
+と言う部分で「sqlite3.OperationalError: no such column: id 」というエラーが発生しているということが分かります。
-このエラーの原因は、データベースに「id」というカラムが存在していないためです。
+このエラーは、データベースに「id」というカラムが存在していないために発生しています。
 したがって、データベースに「id」というカラムが存在している必要があります。
 以下、データベースに「id」カラムを作成し、id で書籍を管理するように作り変えます。

2023/10/01 00:35

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -1,5 +1,5 @@
 ## エラーの原因
-エラーの最後から７～８行目（空白行除く）を読むと
+エラーの最後から 6～８行目（空白行除く）を読むと
 ```
 File "/Users/kojitakemura/Desktop/pythonProject1 2元本 2/flaskr/main.py", line 58, in delete
 book = con.execute('SELECT * FROM books WHERE id = ?', [id]).fetchone()
@@ -7,11 +7,21 @@
 ```
 と書いてあります。
+このエラーメッセージから、main.py  58 行目の
+```
+book = con.execute('SELECT * FROM books WHERE id = ?', [id]).fetchone()
+```
+と言う部分でエラーが発生しているということが分かります。
 このエラーの原因は、データベースに「id」というカラムが存在していないためです。
-なので、データベースに「id」というカラムが存在している必要があります。
+したがって、データベースに「id」というカラムが存在している必要があります。
-以下データベースに「id」カラムを作成し、id で書籍を管理するように作り変えます。
+以下、データベースに「id」カラムを作成し、id で書籍を管理するように作り変えます。
+---
+# データベースに「id」カラムを作成し、id で書籍を管理するように作り変えるための修正
-# db.pyの修正
+## db.pyの修正
 db.py では、database.db 内に books という名前のテーブルが存在するか判定し、存在しない場合は books テーブルを新規作成しています。
 このとき、books テーブル内に id カラムも作成するように修正します。

2023/10/01 00:29

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -94,7 +94,7 @@
     con.execute('INSERT INTO books (title, price, arrival_day) VALUES(?, ?, ?)',[title, price, arrival_day])
 ```
 ## その他
-form.html の後半に なぜかdelete.html の内容が重複しています。文法的にも誤っていると思われるため、特別な目的がない限り、後半は削除した方が良いでしょう。
+form.html の後半、なぜかdelete.html の内容が重複しています。特別な目的がない限り、後半は削除した方が良いでしょう。
 ---

修正

2023/10/01 00:25

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -80,15 +80,17 @@
 ## main.py の修正
 ### root ('/') の修正
 root ('/') では、db_booksに、データベースから取得した id も渡すように修正します。
-```py
+```main.py
+        books.append({"title": row[0], "price": row[1], "arrival_day": row[2]})
+　　　　　　　　↓
         books.append({"id": row[0], "title": row[1], "price": row[2], "arrival_day": row[3]})
 ```
 ## /register の修正
 id を 自動付番に変えた関係上、/register のSQL文はそのままだとエラーになります。
 ここでは、データベースに追加するレコードのカラム名を明示的に指定するよう修正します。
-```py
+```main.py
     con.execute('INSERT INTO books VALUES(?,?,?)',[title,price,arrival_day])
-　　↓
+　　　　↓
     con.execute('INSERT INTO books (title, price, arrival_day) VALUES(?, ?, ?)',[title, price, arrival_day])
 ```
 ## その他
@@ -98,7 +100,7 @@
 ---
 # 修正後全体
-修正後の各コードは以下です。
+修正した各コード全体は以下のようになりす。
 ```main.py
 from crypt import methods
 from turtle import title

2023/10/01 00:21

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -12,8 +12,8 @@
 以下データベースに「id」カラムを作成し、id で書籍を管理するように作り変えます。
 # db.pyの修正
-db.py では、プロジェクト内にテーブル（database.db）が存在するか判定し、存在しない場合はテーブルを作成しています。
+db.py では、database.db 内に books という名前のテーブルが存在するか判定し、存在しない場合は books テーブルを新規作成しています。
-このとき、テーブル内に id カラムも作成するようにします。
+このとき、books テーブル内に id カラムも作成するように修正します。
 ```db.py
 import sqlite3
@@ -26,7 +26,7 @@
     con.close()
 ```
-CERATE文のところに「**id INTEGER PRIMARY KEY AUTOINCREMENT**」を追加しています。
+CERATE文のところで「**id INTEGER PRIMARY KEY AUTOINCREMENT**」を追加しています。
 これは
 + 「id」と言う名前のカラムを作成
 + id の型（整数）、主キー制約（PRIMARY KEY）、自動付番（AUTOINCREMENT）を設定

ｗ

2023/10/01 00:20

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -66,7 +66,7 @@
 ## delete.html の修正
-delete.html もIDを表示するように修正します。（これも必須ではありませんが、動作をわかりやすくするためです）
+データベースの構造を変更したのに合わせて、delete.html も修正します。
 ```delete.html
 ...略...

ｗ

2023/10/01 00:18

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -65,7 +65,7 @@
 ```
-# delete.html の修正
+## delete.html の修正
 delete.html もIDを表示するように修正します。（これも必須ではありませんが、動作をわかりやすくするためです）
 ```delete.html
 ...略...
@@ -91,6 +91,11 @@
 　　↓
     con.execute('INSERT INTO books (title, price, arrival_day) VALUES(?, ?, ?)',[title, price, arrival_day])
 ```
+## その他
+form.html の後半に なぜかdelete.html の内容が重複しています。文法的にも誤っていると思われるため、特別な目的がない限り、後半は削除した方が良いでしょう。
+---
 # 修正後全体
 修正後の各コードは以下です。

修正

2023/10/01 00:14

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -1,5 +1,5 @@
-# エラーの原因
+## エラーの原因
-エラーの最後から７～８行目（空白行覗く）を読むと
+エラーの最後から７～８行目（空白行除く）を読むと
 ```
 File "/Users/kojitakemura/Desktop/pythonProject1 2元本 2/flaskr/main.py", line 58, in delete
 book = con.execute('SELECT * FROM books WHERE id = ?', [id]).fetchone()
@@ -7,12 +7,14 @@
 ```
 と書いてあります。
-このエラーの原因は、実際のデータベースに「id」というカラムが存在していないためです。
+このエラーの原因は、データベースに「id」というカラムが存在していないためです。
-なので、データベースに「id」というカラムがなければいけません。
+なので、データベースに「id」というカラムが存在している必要があります。
-以下データベースに「id」カラムを作成し、idで書籍を管理するように作り変えます。
+以下データベースに「id」カラムを作成し、id で書籍を管理するように作り変えます。
-# データベース作成時の設定
+# db.pyの修正
+db.py では、プロジェクト内にテーブル（database.db）が存在するか判定し、存在しない場合はテーブルを作成しています。
-一番最初、テーブルを作るときに、idを作ってあげます。
+このとき、テーブル内に id カラムも作成するようにします。
 ```db.py
 import sqlite3
@@ -31,11 +33,12 @@
 の意味です。
 2番目のように設定する理由はidの重複を防ぐためです。仮にidの重複を許してしまうと、削除する時に書籍を区別できず、指定した書籍と別の書籍を削除してしまう可能性があります。
-# index.html の修正
+## index.html の修正
 index.html を下記のように修正します。
-+ idを表示する列を追加
++ データベースの id カラムを表示する列を追加（必須ではありませんが動作をわかりやすくするため）
-+ 削除リンククリック時、idをdelete.htmlに渡すように変更
++ 削除リンクに、コントローラ（main.py）から受け取った id をつける　→　このようにすることで、削除リンククリック時に書籍IDを delete.html に渡すことが可能になります。
 ```index.html
 ...略...
     <table border="1">
@@ -63,7 +66,7 @@
 # delete.html の修正
-delete.html もIDを表示するように修正します。
+delete.html もIDを表示するように修正します。（これも必須ではありませんが、動作をわかりやすくするためです）
 ```delete.html
 ...略...
@@ -74,21 +77,23 @@
 ...略...
 ```
-# main.py の修正
+## main.py の修正
-## root ('/') の修正
+### root ('/') の修正
-root ('/') では下記のように修正します。
+root ('/') では、db_booksに、データベースから取得した id も渡すように修正します。
-+ db_booksに、idも一緒に渡すようにする。
 ```py
         books.append({"id": row[0], "title": row[1], "price": row[2], "arrival_day": row[3]})
 ```
 ## /register の修正
-/register のSQL文で、id を 自動付番に変えた関係上、そのままだとエラーになるので、渡す変数を明示する必要があります。
+id を 自動付番に変えた関係上、/register のSQL文はそのままだとエラーになります。
+ここでは、データベースに追加するレコードのカラム名を明示的に指定するよう修正します。
 ```py
+    con.execute('INSERT INTO books VALUES(?,?,?)',[title,price,arrival_day])
+　　↓
     con.execute('INSERT INTO books (title, price, arrival_day) VALUES(?, ?, ?)',[title, price, arrival_day])
 ```
-# main.py 修正後全体
+# 修正後全体
-main.py全体の修正後のコードは以下です。
+修正後の各コードは以下です。
 ```main.py
 from crypt import methods
 from turtle import title
@@ -133,9 +138,6 @@
     con.commit()
     con.close()
     return redirect(url_for("index"))
 @app.route('/delete/<int:id>', methods=['GET', 'POST'])
 def delete(id):
@@ -154,6 +156,120 @@
             return "Book not found", 404
 ```
+```db.py
+import sqlite3
+DATABASE = "database.db"
+def create_books_table():
+    con = sqlite3.connect(DATABASE)
+    con.execute("CREATE TABLE IF NOT EXISTS books (id INTEGER PRIMARY KEY AUTOINCREMENT, title, price, arrival_day)")
+    con.close()
+```
+```index.html
+<!DOCTYPE html>
+<html lang="ja">
+<head>
+    <meta charset="UTF-8">
+    <title>sapu-app</title>
+    <link rel="stylesheet" href="{{  url_for('static', filename='style.css')  }}">
+</head>
+<body>
+    <h1>書店</h1>
+    <h2>今月の新刊一覧</h2>
+    {% if books == [] %}
+    <p>今月の新刊情報はまだありません</p>
+    {% else %}
+    <table border="1">
+        <tr>
+            <th>ID</th>
+            <th>入荷日</th>
+            <th>タイトル</th>
+            <th>金額</th>
+            <th>削除</th>
+        </tr>
+        {% for book in books %}
+        <tr>
+            <td>{{ book.id }}</td>
+            <td>{{ book.arrival_day }}</td>
+            <td>{{ book.title }}</td>
+            <td>{{ book.price }}円</td>
+            <td>
+                <a href="{{ url_for('delete', id=book.id) }}">Delete</a>
+            </td>
+        </tr>
+        {% endfor %}
+    </table>
+    {% endif %}
+    <a href= "{{ url_for('form') }}">編集</a>
+</body>
+</html>
+```
+```delete.html
+<!DOCTYPE html>
+<html lang="jp">
+<head>
+    <meta charset="UTF-8">
+    <title>Delete Book</title>
+</head>
+<body>
+    <h1>Delete Book</h1>
+    <p>Are you sure you want to delete the following book?</p>
+    <p>ID   : {{ book[0] }}</p>
+    <p>Title: {{ book[1] }}</p>
+    <p>Price: {{ book[2] }}</p>
+    <p>Arrival Day: {{ book[3] }}</p>
+    <form method="POST">
+        <input type="submit" value="Delete">
+    </form>
+</body>
+</html>
+```
+```form.html
+<!DOCTYPE html>
+<html lang="ja">
+<head>
+    <meta charset="UTF-8">
+    <title>sapu-app</title>
+    <link rel="stylesheet" href="{{  url_for('static', filename='style.css')  }}">
+</head>
+<body>
+    <h1>書店</h1>
+    <form method="post" action="{{  url_for('register') }}">
+        <table border="1">
+            <tr>
+                <th>入荷日</th>
+                <th>タイトル</th>
+                <th>金額</th>
+            </tr>
+            <tr>
+                <td><input type="text" name="arrival_day"></td>
+                <td><input type="text" name="title"></td>
+                <td><input type="text" name="price"></td>
+            </tr>
+        </table>
+        <br>
+        <input type="submit" value="登録">
+    </form>
+</body>
+</html>
+```
+# 注意
-上記修正内容を試す場合は、一旦flaskを終了し、プロジェクト内のdatabase.dbファイルを削除してからrun.pyを再実行するようにしてください。
+上記修正内容を試す場合は、修正後初回のみ、一旦 flask を終了し、プロジェクト内の database.db ファイルを削除してから run.py を再実行するようにしてください。
-データベースの構造自体を変える必要があるため、元のdatabase.dbが残っている状態で起動した場合、うまく動作しません。
+修正前のdatabase.dbが残っている状態で起動した場合、うまく動作しません。

初回起動方法について追記

2023/09/30 23:55

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -154,3 +154,6 @@
             return "Book not found", 404
 ```
+上記修正内容を試す場合は、一旦flaskを終了し、プロジェクト内のdatabase.dbファイルを削除してからrun.pyを再実行するようにしてください。
+データベースの構造自体を変える必要があるため、元のdatabase.dbが残っている状態で起動した場合、うまく動作しません。

修正

2023/09/30 23:52

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -13,10 +13,8 @@
 # データベース作成時の設定
 一番最初、テーブルを作るときに、idを作ってあげます。
-db.py
+```db.py
-```py
 import sqlite3
 DATABASE = "database.db"
@@ -38,7 +36,8 @@
 index.html を下記のように修正します。
 + idを表示する列を追加
 + 削除リンククリック時、idをdelete.htmlに渡すように変更
-```html
+```index.html
+...略...
     <table border="1">
         <tr>
             <th>ID</th>
@@ -59,16 +58,20 @@
             </td>
         </tr>
         {% endfor %}
+...略...
 ```
 # delete.html の修正
 delete.html もIDを表示するように修正します。
-```html
+```delete.html
+...略...
     <p>ID   : {{ book[0] }}</p>
     <p>Title: {{ book[1] }}</p>
     <p>Price: {{ book[2] }}</p>
     <p>Arrival Day: {{ book[3] }}</p>
+...略...
 ```
 # main.py の修正
@@ -79,14 +82,14 @@
         books.append({"id": row[0], "title": row[1], "price": row[2], "arrival_day": row[3]})
 ```
 ## /register の修正
-+ /register のSQL文で、id を 自動付番に変えた関係上、そのままだとエラーになるので、渡す変数を明示する必要があります。
+/register のSQL文で、id を 自動付番に変えた関係上、そのままだとエラーになるので、渡す変数を明示する必要があります。
 ```py
     con.execute('INSERT INTO books (title, price, arrival_day) VALUES(?, ?, ?)',[title, price, arrival_day])
 ```
-# main.py 全体
+# main.py 修正後全体
 main.py全体の修正後のコードは以下です。
-```py
+```main.py
 from crypt import methods
 from turtle import title
 #db 追加
@@ -134,7 +137,6 @@
-#以下追記した削除コードです
 @app.route('/delete/<int:id>', methods=['GET', 'POST'])
 def delete(id):
     con = sqlite3.connect(DATABASE)

修正

2023/09/30 23:48

投稿

退会済みユーザー

スコア0

test CHANGED Viewed

@@ -74,5 +74,81 @@
 # main.py の修正
 ## root ('/') の修正
 root ('/') では下記のように修正します。
++ db_booksに、idも一緒に渡すようにする。
+```py
+        books.append({"id": row[0], "title": row[1], "price": row[2], "arrival_day": row[3]})
+```
+## /register の修正
++ /register のSQL文で、id を 自動付番に変えた関係上、そのままだとエラーになるので、渡す変数を明示する必要があります。
+```py
+    con.execute('INSERT INTO books (title, price, arrival_day) VALUES(?, ?, ?)',[title, price, arrival_day])
+```
+# main.py 全体
+main.py全体の修正後のコードは以下です。
+```py
+from crypt import methods
+from turtle import title
+#db 追加
+from flaskr import app, db
+from flask import render_template, request, redirect, url_for
+from flaskr.db import DATABASE
+import sqlite3
+DATABASE = "database.db"
+@app.route('/')
+def index():
+    con = sqlite3.connect(DATABASE)
+    db_books = con.execute("SELECT * FROM books").fetchall()
+    con.close
+    books = []
+    for row in db_books:
+        books.append({"id": row[0], "title": row[1], "price": row[2], "arrival_day": row[3]})
+    return render_template(
+        'index.html',books=books
-+
+    )
+@app.route("/form")
+def form():
+    return render_template("form.html")
+@app.route("/register", methods=["POST"])
+def register():
+    title = request.form["title"]
+    price = request.form["price"]
+    arrival_day =request.form["arrival_day"]
+    con = sqlite3.connect(DATABASE)
+    con.execute('INSERT INTO books (title, price, arrival_day) VALUES(?, ?, ?)',[title, price, arrival_day])
+    con.commit()
+    con.close()
+    return redirect(url_for("index"))
+#以下追記した削除コードです
+@app.route('/delete/<int:id>', methods=['GET', 'POST'])
+def delete(id):
+    con = sqlite3.connect(DATABASE)
+    if request.method == 'POST':
+        con.execute('DELETE FROM books WHERE id = ?', [id])
+        con.commit()
+        con.close()
+        return redirect(url_for('index'))
+    else:
+        book = con.execute('SELECT * FROM books WHERE id = ?', [id]).fetchone()
+        con.close()
+        if book:
+            return render_template('delete.html', book=book)
+        else:
+            return "Book not found", 404
-+
+```