Überarbeitung

app/app.py

@@ -1,4 +1,4 @@
-from flask import Flask, request, session, redirect
+from flask import Flask, request, session, redirect, url_for, abort
 import sqlite3
 
 app = Flask(__name__)
@@ -11,9 +11,11 @@ def get_db():
 
 @app.route("/", methods=["GET", "POST"])
 def start():
+    html = '<h1>Verlies</h1>' 
 
     if request.method == "POST":                 # Formular abgeschickt?
-        name = request.form["name"].strip()
+        # .get() verhindert Abstürze, falls "name" mal fehlt
+        name = request.form.get("name", "").strip()
         if name != "":
             session["spieler"] = name            # App merkt sich: das bist du
             db = get_db()
@@ -23,9 +25,10 @@ def start():
             )
             db.commit()
             db.close()
-            return redirect("/app/raum/1")
+            # url_for generiert den Pfad dynamisch (sehr viel sicherer!)
+            return redirect(url_for("raum", id=1)) 
 
-            html = '<h1>Verlies</h1>'
+    # Wenn kein POST gesendet wurde, läuft der Code hier einfach weiter:
     if "spieler" in session:
         db = get_db()
         s = db.execute(
@@ -34,23 +37,15 @@ def start():
         ).fetchone()
         db.close()
         if s:
+            # Dynamische URL-Generierung auch hier für den Weiterspielen-Link
+            weiterspielen_url = url_for("raum", id=s["raum_id"])
             html = html + '<p>Willkommen zurück, ' + session["spieler"] + '!</p>' \
-                 + '<a href="/app/raum/' + str(s["raum_id"]) + '">Weiterspielen</a><hr>'
+                        + '<a href="' + weiterspielen_url + '">Weiterspielen</a><hr>'
+                        
+    # Jetzt existiert 'html' garantiert immer
     return html + '<form method="post">Dein Name: ' \
                 + '<input name="name"> <button>Abenteuer starten</button></form>'  
              
-    if "spieler" in session:
-        db = get_db()
-        s = db.execute(
-            "SELECT raum_id FROM spielstand WHERE spieler=?",
-            (session["spieler"],)
-        ).fetchone()
-        db.close()
-        if s:
-            html = html + '<p>Willkommen zurück, ' + session["spieler"] + '!</p>' \
-                 + '<a href="/app/raum/' + str(s["raum_id"]) + '">Weiterspielen</a><hr>'
-    return html + '<form method="post">Dein Name: ' \
-                + '<input name="name"> <button>Abenteuer starten</button></form>'
 
 @app.route("/raum/<int:id>", methods=["GET", "POST"])
 def raum(id):
@@ -63,27 +58,40 @@ def raum(id):
         )
         db.commit()
     
-    r = db.execute("SELECT * FROM raeume WHERE id=?", (id,)).fetchone()
+    # Raum aus der DB holen
+    r = db.execute("SELECT * FROM r_aeume WHERE id=?", (id,)).fetchone()
     
-    if r["raetsel_frage"]:                    # Rätselraum?
+    # Fehlerbehandlung: Was, wenn die ID nicht existiert?
+    if r is None:
+        db.close()
+        abort(404, description="Dieser Raum existiert nicht!")
+
+    # Rätselraum?
+    if r["raetsel_frage"]:                    
         if request.method == "POST":
-            if request.form["antwort"].strip() == r["raetsel_antwort"]:
+            user_antwort = request.form.get("antwort", "").strip()
+            if user_antwort == r["raetsel_antwort"]:
+                db.close() # DB schließen vor dem Return!
                 return "<h1>Befreit!</h1>"
-            return 'Leider falsch. <a href="/app/raum/' + str(id) + '">nochmal</a>'
+            
+            db.close() # DB schließen vor dem Return!
+            nochmal_url = url_for("raum", id=id)
+            return 'Leider falsch. <a href="' + nochmal_url + '">nochmal</a>'
+        
+        db.close() # DB schließen vor dem Return!
         return "<h1>" + r["name"] + "</h1><p>" + r["beschreibung"] + "</p>" \
              + '<form method="post">' + r["raetsel_frage"] \
              + ' <input name="antwort"><button>OK</button></form>'
 
-    # sonst: normaler Raum mit Ausgängen (wie bisher)
+    # Sonst: normaler Raum mit Ausgängen
     ausgaenge = db.execute("SELECT richtung, nach_raum FROM ausgaenge WHERE von_raum=?", (id,)).fetchall()
-    db.close()
+    db.close() # DB schließen für den normalen Pfad
 
     html = "<h1>" + r["name"] + "</h1>"
     html = html + "<p>" + r["beschreibung"] + "</p>"
     for a in ausgaenge:
-        html = html + '<a href="/app/raum/' + str(a["nach_raum"]) + '">' \
+        nach_raum_url = url_for("raum", id=a["nach_raum"])
+        html = html + '<a href="' + nach_raum_url + '">' \
                     + a["richtung"] + '</a> &nbsp; '
-    return html
     
-if __name__ == "__main__":
-    app.run(host="0.0.0.0", port=9011)
+    return html # Das hat gefehlt!