SQLite3 Datenbank überschreibt db file

[nfehren]

Hallo,
Ich bin wieder an einer kleinen Übungsaufgabe. Ich will eine Abfrage über die Shell machen, die Informationen zu einem "ToDo" sammelt.
Das soll dann in eine Datenbank übernommen werden. Soweit so gut, es klappt auch alles, das ToDo wird übernommen und gespeichert. Blos wenn ein neues ToDo dazu kommt wird das alte überschrieben.

Hier erstmal der Code:

Code: Alles auswählen

import time
import datetime
import sqlite3

def tableCreate():
    c.execute("CREATE TABLE ToDo("
              "Nr INT,"
              "Datum TEXT,"
              "Notiz TEXT,"
              "Menge REAL)")

def dataEntry():
    Datum = str(datetime.datetime.fromtimestamp(int(time.time())).strftime('%d.%m.%Y %H:%M'))
    c.execute('INSERT INTO ToDo(Nr,Datum,Notiz,Menge) VALUES (?,?,?,?)',
              (Nr, Datum, Notiz, Menge))
    conn.commit()

def readData():
    sql = "SELECT * FROM ToDo"
    for row in c.execute(sql):
        print(row)

conn = sqlite3.connect('tut.db')
c = conn.cursor()
c.execute("drop table if exists ToDo")
conn.commit()

tableCreate()

Nr = int(input("Welche Nummer soll der Eintrag haben? "))
Notiz = input("Welche Notiz soll der Eintrag haben? ")
Menge = float(input("Geben sie die Menge an "))

dataEntry()
readData()

Wo habe ich etwas vergessen oder falsch gemacht, dass jedes mal überschrieben wird?

Danke für die Antworten, sonstige Verbesserungsvorschläge sind auch sehr erwünscht.

[Sirius3]

@nfehren: was meinst Du macht Zeile 25?

[nfehren]

Es überschreibt, aber wenn ich das nicht drin habe lässt sich das Skript nicht ausführen, weil die Tabelle schon existiert.

[BlackJack]

@nfehren: Dann darfst Du die Tabelle halt nur erstellen wenn sie noch nicht existiert. Oder Du behandelst die dadurch entstehende Ausnahme, und *nur die*, durch ignorieren.

Die Zeile überschreibt auch nichts. Die löscht die Tabelle.

[/me]

Es überschreibt, aber wenn ich das nicht drin habe lässt sich das Skript nicht ausführen, weil die Tabelle schon existiert.

Man erstellt eine Tabelle nur einmal und zwar nur einmal während der gesamten Existenz der Datenbank, nicht einmal pro Programmaufruf.

[nfehren]

Es überschreibt, aber wenn ich das nicht drin habe lässt sich das Skript nicht ausführen, weil die Tabelle schon existiert.

Man erstellt eine Tabelle nur einmal und zwar nur einmal während der gesamten Existenz der Datenbank, nicht einmal pro Programmaufruf.

Code: Alles auswählen

def tableCreate():
    c.execute("CREATE TABLE IF NOT EXISTS ToDo("
              "Nr INT,"
              "Datum TEXT,"
              "Notiz TEXT,"
              "Menge REAL)")

So, ist das richtig? Funktionieren tuts.

[BlackJack]

@nfehren: Jup, das geht, wobei das ``IF (NOT) EXISTS`` soweit ich weiss kein Standard-SQL ist.

Ich würde aus der `Nr` einen Primärschlüssel machen und aus dem `Datum` den Typ `TIMESTAMP`. Oder `DATE`, dann müsste man dem SQLite-Modul leider extra sagen dass das in ein entsprechendes Python-Objekt umgewandelt wird und nicht als Zeichenkette geliefert werden. Einer der Gründe warum ich auch bei SQLite gerne SQLAlchemy verwende, da werden solche sinnvollen Konvertierungen automatisch erledigt.

[nfehren]

@nfehren: Jup, das geht, wobei das ``IF (NOT) EXISTS`` soweit ich weiss kein Standard-SQL ist.

Ich würde aus der `Nr` einen Primärschlüssel machen und aus dem `Datum` den Typ `TIMESTAMP`. Oder `DATE`, dann müsste man dem SQLite-Modul leider extra sagen dass das in ein entsprechendes Python-Objekt umgewandelt wird und nicht als Zeichenkette geliefert werden. Einer der Gründe warum ich auch bei SQLite gerne SQLAlchemy verwende, da werden solche sinnvollen Konvertierungen automatisch erledigt.

Habe ich so übernommen, danke

Ich möchte jetzt auch noch eine Funktion erstellen, mit der ich einen Eintrag löschen kann.
Dafür soll erst abgefragt werden, welcher Eintrag (Nr) gelöscht werden soll.
Wie ich das aber umsetzen soll mit der Variable beim löschen, da bin ich mir nicht ganz sicher.

Soweit bin ich:

Code: Alles auswählen

def delData():
    auswahl = int(input("Welchen Eintrag wollen sie löschen? "))
    c.execute("DELETE FROM ToDo WHERE Nr = (auswahl) VALUES (?)",
              (auswahl))

Es kommt aber eine Error meldung. Also stimmt irgendwas mit der Syntax nicht..

[Hyperion]

Ich möchte jetzt auch noch eine Funktion erstellen, mit der ich einen Eintrag löschen kann.
Dafür soll erst abgefragt werden, welcher Eintrag (Nr) gelöscht werden soll.
Wie ich das aber umsetzen soll mit der Variable beim löschen, da bin ich mir nicht ganz sicher.

Ich würde ja Nachfragen (also Benutzerinteraktion) und Funktionalität trennen! Erstelle einfach zwei Funktionen, eine für die Nachfrage und eine, die das Löschen durchführt und als Parameter die Nummer erhält.

Es kommt aber eine Error meldung. Also stimmt irgendwas mit der Syntax nicht..

Probiere das doch zunächst in einer SQL-Shell aus! Wenn Du dann die richtige Syntax hast, brauchst Du nur noch den Platzhalter einbauen.

(Ja, ich verrate Dir hier absichtlich nicht die Lösung, denn die ist wirklich trivial )

[nfehren]

Probiere das doch zunächst in einer SQL-Shell aus! Wenn Du dann die richtige Syntax hast, brauchst Du nur noch den Platzhalter einbauen.

(Ja, ich verrate Dir hier absichtlich nicht die Lösung, denn die ist wirklich trivial )

delete from Todo where nr = 1 , klappt. Aber das ist halt ohne Variable

[BlackJack]

@nfehren: Das würde sich aber auch mit Variable von dem Unterscheiden was Du vorher mit dem Programm probiert hast.

[nfehren]

Code: Alles auswählen

def delData():
    auswahl = int(input("Welchen Eintrag wollen sie löschen? "))
    c.execute("DELETE FROM ToDo WHERE Nr IS '{0}'".format(auswahl))
    conn.commit()

[Sirius3]

@nfehren: Nein. Du formatierst wieder was in SQL-Asudrücke rein. Nr ist eine Zahl, du vergleichst mit einem String, was Ausnahmsweise bei SQLite keine Rolle spielt, an sich aber falsch ist.

[/me]

@nfehren: Nein. Du formatierst wieder was in SQL-Asudrücke rein. Nr ist eine Zahl, du vergleichst mit einem String, was Ausnahmsweise bei SQLite keine Rolle spielt, an sich aber falsch ist.

Ergänzend dazu: Wenn man einen exakten Stringvergleich haben will nimmt man natürlich das Gleichheitszeichen und nicht LIKE.

[BlackJack]

@nfehren: Geht das tatsächlich mit ``IS``? Und Du formatierst da einen vom Benutzer eingegebenen Wert direkt in eine SQL-Anweisung. Das ist ein absolutes No-Go, weil eine riesige Sicherheitslücke und Fehlerquelle.

[Hyperion]

Und Du formatierst da einen vom Benutzer eingegebenen Wert direkt in eine SQL-Anweisung. Das ist ein absolutes No-Go, weil eine riesige Sicherheitslücke und Fehlerquelle.

Was besonders erstaunlich ist, da er den richtigen Mechanismus ja schon mal benutzt und gezeigt hat!

[pillmuncher]

Aha. Ist wohl mal wieder Zeit hierfür: http://xkcd.com/327/

[Hyperion]

Den kannte ich noch nicht - nice!