Adressdatenbank

[CrackPod]

Hallo,

da ich mir derzeit eine Adressdatenbank schreib - obwohl Gerold derzeit eine schreibt, um PySQLite zu erklären. Treten ja immer ein paar Probleme auf
Sicherlich hab ich ja auch schon einen, sonst gäbs den Thread hier nich
Also Fehler:

Code: Alles auswählen

$ python Adressenverwaltung.py
INSERT INTO adresses (
                vorname,
                zuname,
                nachname,
                anrede,
                adresse,
                plz,
                ort,
                land,
                privatnummer,
                privatfaxnummer,
                mobilnummer,
                bueronummer,
                buerofaxnummer
            ) VALUES (
                Max,
                Maximilian,
                Musterman,
                Herr,
                Musterstr 12,
                1234865,
                Musterstadt,
                Deutschland,
                004912326548,
                /,
                01701234586,
                /,
                /)
Traceback (most recent call last):
  File "Adressenverwaltung.py", line 154, in ?
    main()
  File "Adressenverwaltung.py", line 149, in main
    adr.add('Max','Maximilian','Musterman','Herr','Musterstr 12','1234865','Musterstadt',
'Deutschland','004912326548','/','01701234586','/','/')
  File "Adressenverwaltung.py", line 107, in add
    self.cur.execute(sql)
pysqlite2.dbapi2.OperationalError: near "12": syntax error

Der Dazugehörige Codeabschnitt

Code: Alles auswählen

    def add (self,
             vorname = None, zuname = None, nachname = None, anrede = None, adresse = None, plz = None, ort = None, land = None,
             privatnummer = None, privatfaxnummer = None, mobilnummer = None, bueronummer = None, buerofaxnummer = None
        ):
        '''Fügt einen neuen Kontakt in das Adressbuch ein

            :param vorname: Der Vorname des Kontakts
            :param zuname: Der Zuname des Kontakts
            :param nachname: Der Nachname des Kontakts
            :param anrede: Die Anrede des Kontakts
            :param adresse: Die Adresse der Kontakts (Incl. Hausnummer)
            :param plz: Die Postleitzahl des Kontakts
            :param ort: Der Ort/Wohnort des Kontakts
            :param land: Das Land, in welchem der Kontakt wohnt
            :param privatnummer: Die Privatnummer des Kontakts
            :param privatfaxnummer: Die Privatfaxnummer des Kontakts
            :param mobilnummer: Die Mobiltelefonnummer des Kontakts
            :param bueronummer: Die Büronummer des Kontakts
            :param buerofaxnummer: Die Bürofaxnummer des Kontakts
        '''

        sql = '''INSERT INTO adresses (
                vorname,
                zuname,
                nachname,
                anrede,
                adresse,
                plz,
                ort,
                land,
                privatnummer,
                privatfaxnummer,
                mobilnummer,
                bueronummer,
                buerofaxnummer
            ) VALUES (
                %s,
                %s,
                %s,
                %s,
                %s,
                %s,
                %s,
                %s,
                %s,
                %s,
                %s,
                %s,
                %s)''' % (vorname, zuname, nachname, anrede, adresse, plz, ort, land,
privatnummer, privatfaxnummer, mobilnummer, bueronummer, buerofaxnummer)
        print sql #daher kommt die lange SQL ausgabe oben :wink:

        self.cur.execute(sql)
        self.con.commit()

Kann mir da einer helfen?
Ich versteh nich, wo da der Syntaxerror sien soll

[BlackJack]

Dein SQL ist fehlerhaft. Das ist im Grunde der gleiche Fehler, den Du auch unter Python bekommen würdest:

Code: Alles auswählen

In [3]: a = Musterstrasse 12
------------------------------------------------------------
   File "<ipython console>", line 1
     a = Musterstrasse 12
                        ^
SyntaxError: invalid syntax

Bevor Du jetzt anfängst ``'%s', '%s', ...`` in Deinen Quelltext zu schreiben -- lass es. Gib den SQL Befehl so wie er ist als erstes Argument an `execute()` und die Werte als zweiten. Dann sorgt das DB Modul dafür, dass die vernünftig in den SQL Befehl eingefügt werden.

Insbesondere passiert dann auch kein Murks wenn jemand 'Müller's Backerei' eintragen will oder gar jemand absichtlich "bösen" SQL-Code eingibt.

[gerold]

Ich versteh nich, wo da der Syntaxerror sien soll

Hi CrackPod!

Die korrekte Syntax zum Einfügen von Textfeldern ist folgende:

Code: Alles auswählen

INSERT INTO adresses (
  vorname,
  zuname
) VALUES (
  'Max',
  'Maximilian'
)

Achte auf die einfachen Anführungszeichen.

Folgendes Codebeispiel kann so nicht funktionieren, da es sich nicht um die Anfürhungszeichen kümmert:

Code: Alles auswählen

sql = """
INSERT INTO adresses (
  vorname,
  zuname
) VALUES (
  %s,
  %s
)
""" % (vorname, zuname)

Die %s in Anführungszeichen zu setzen ist auch keine Lösung, da du auch None übergeben können musst. Aber None darf nicht in Anführungszeichen gesetzt werden.

Also hast du mehrere Möglichkeiten. Entweder du kümmerst dich selbst darum,

Code: Alles auswählen

if vorname:
    vorname = "'%s'" % vorname
else:
    vorname = "None"

oder du lässt es dir von pysqlite erledigen.

Code: Alles auswählen

sql = """
INSERT INTO adresses (
  vorname,
  zuname
) VALUES (
  :vorname,
  :zuname
)
"""
cur.execute(sql, {"vorname": vorname, "zuname": zuname})

Achte auf die Doppelpunkte vor den Variablenamen im SQL-String.

So kümmert sich pysqlite um die Anfürhrungszeichen, wo sie benötigt werden. None und Zahlen werden nicht in Anfürhungszeichen eingeschlossen.

Da sich pysqlite um das alles kümmert, ist es für andere viel schwerer, Code in die SQL-Anweisung einzuschmuggeln. (Stichwort: SQL-Injection)

lg
Gerold

[CrackPod]

auf das mit den Anführungszeichen bin ch kurz nachdem ich gepostet hab selber gekommen, hab haber vergessen, den Post zu löschen(is aber vorteilhaft hab nen neuen Trick gelernt)
Danke

[CrackPod]

Naja, hab trotzdem noch Probleme:
Fehler

Code: Alles auswählen

$ python Adressenverwaltung.py
INSERT INTO adresses (
                vorname,
                zuname,
                nachname,
                anrede,
                adresse,
                plz,
                ort,
                land,
                privatnummer,
                privatfaxnummer,
                mobilnummer,
                bueronummer,
                buerofaxnummer
            ) VALUES (
                :vorname,
                :zuname,
                :nachname,
                :anrede,
                :adresse,
                :plz,
                :ort,
                :land,
                :privatnummer,
                :privatfaxnummer,
                :mobilnummer,
                :bueronummer,
                :buerofaxnummer)
Es ist ein Fehler beim Erstellen des Kontakts aufgetreten adresses.id may not be NULL

Methode:

Code: Alles auswählen

        sql = '''INSERT INTO adresses (
                vorname,
                zuname,
                nachname,
                anrede,
                adresse,
                plz,
                ort,
                land,
                privatnummer,
                privatfaxnummer,
                mobilnummer,
                bueronummer,
                buerofaxnummer
            ) VALUES (
                :vorname,
                :zuname,
                :nachname,
                :anrede,
                :adresse,
                :plz,
                :ort,
                :land,
                :privatnummer,
                :privatfaxnummer,
                :mobilnummer,
                :bueronummer,
                :buerofaxnummer)'''
        print sql

        self.cur.execute(sql,{'vorname':vorname, 'zuname':zuname, 'nachname':nachname, 'anrede':anrede, 'adresse':adresse, 'plz':plz,
                              'ort':ort, 'land':land, 'privatnummer':privatnummer, 'privatfaxnummer':privatfaxnummer,
                              'mobilnummer':mobilnummer, 'bueronummer':bueronummer, 'buerofaxnummer':buerofaxnummer}
                         )
        self.con.commit()

Aufruf

Code: Alles auswählen

adr = Adress('./daten/adress.sqldb')
adr.add('Max','Maximilian','Musterman','Herr','Musterstr 12',
'1234865','Musterstadt','Deutschland','004912326548',
'','01701234586','','')

Muss ich das Feld id mit angeben bzw einfügen? Bei MySQL muss man das nicht (Falls man der id AUTO_INCREMENT gegeben hat - was ich gemacht hab)
Datenbankstruktur:

Code: Alles auswählen

CREATE TABLE adresses(
      id INTEGER PRYMARY KEY AUTO_INCREMENT NOT NULL,
      vorname VARCHAR(30),
      zuname VARCHAR(40),
      nachname VARCHAR(30),
      anrede VARCHAR(20),
      adresse VARCHAR(50),
      plz VARCHAR(15),
      ort VARCHAR(25),
      land VARCHAR(30),
      privatnummer VARCHAR(35),
      privatfaxnummer VARCHAR(35),
      mobilnummer VARCHAR(35),
      bueronummer VARCHAR(35),
      buerofaxnummer VARCHAR(35)
)

(Und warum geht das nich, wenn ich noch IF NOT EXISTS mit dazu mach, dann bekomm ich nen 'SyntaxError near "NOT"'....)


Noch was anderes: Wie bekomm ich es hin, keine Werte übergeben zu müssen? Wenn ich satt ,'','') (im letzten Teil) ,,) schreib, bekomm ich nen Syntaxerror-.- Geht das nich auch noch anders, als einen leeren String zu übergeben?

Danke greetz

[gerold]

Es ist ein Fehler beim Erstellen des Kontakts aufgetreten adresses.id may not be NULL

Code: Alles auswählen

CREATE TABLE adresses(
      id INTEGER PRYMARY KEY AUTO_INCREMENT NOT NULL,
      ...

Hi CrackPod!

Bei SQLite gibt es kein AUTO_INCREMENT. Es wird einfach ignoriert. Wichtig ist einzig "INTEGER PRIMARY KEY". Das ist das Zeichen für SQLite, dass automatisch hoch gezählt wird. (du hast dich nur verschrieben...)

Siehe dieses Beispiel: http://gelb.bcom.at/trac/misc/browser/s ... rev=11#L99

lg
Gerold

[gerold]

Noch was anderes: Wie bekomm ich es hin, keine Werte übergeben zu müssen? Wenn ich satt ,'','') (im letzten Teil) ,,) schreib, bekomm ich nen Syntaxerror-.- Geht das nich auch noch anders, als einen leeren String zu übergeben?

Hi CrackPod!

In der Deklaration der Funktion hast du für diese Parameter einen Standardwert übergeben. Das bedeutet, dass du diese restlichen Argumente einfach weg lassen kannst. Keinen Beistrich und keine Anführungszeichen.

lg
Gerold

[CrackPod]

Achso... So is das
Danke!

Zu deinem Link: Den kenn ich, aber ich schaus mir nich allzu oft an, weil ich ja nich alles vorgekaut bekommen will. Mit der Datenbankerstellung hab ich schon reingeschaut und mich gefragt, wie das bei dir denn ohne AUTO_INCREMENT funzt
Wieso geht das IF NOT EXISTS bei mir nicht, wenn ich ne Datenbank erstell?

Code: Alles auswählen

    def install(self):
        sql = '''CREATE TABLE adresses IF NOT EXISTS(
                id INTEGER PRIMARY KEY NOT NULL,
                vorname VARCHAR(30),
                zuname VARCHAR(40),
                nachname VARCHAR(30),
                anrede VARCHAR(20),
                adresse VARCHAR(50),
                plz VARCHAR(15),
                ort VARCHAR(25),
                land VARCHAR(30),
                privatnummer VARCHAR(35),
                privatfaxnummer VARCHAR(35),
                mobilnummer VARCHAR(35),
                bueronummer VARCHAR(35),
                buerofaxnummer VARCHAR(35)
            )'''
        self.cur.execute(sql)
        self.con.commit()

$ python Adressenverwaltung.py
Traceback (most recent call last):
File "Adressenverwaltung.py", line 154, in ?
main()
File "Adressenverwaltung.py", line 145, in main
adr.install()
File "Adressenverwaltung.py", line 52, in install
self.cur.execute(sql)
pysqlite2.dbapi2.OperationalError: table adresses already exists

Code: Alles auswählen

    def install(self):
        sql = '''CREATE TABLE IF NOT EXISTS adresses(
                id INTEGER PRIMARY KEY NOT NULL,
                vorname VARCHAR(30),
                zuname VARCHAR(40),
                nachname VARCHAR(30),
                anrede VARCHAR(20),
                adresse VARCHAR(50),
                plz VARCHAR(15),
                ort VARCHAR(25),
                land VARCHAR(30),
                privatnummer VARCHAR(35),
                privatfaxnummer VARCHAR(35),
                mobilnummer VARCHAR(35),
                bueronummer VARCHAR(35),
                buerofaxnummer VARCHAR(35)
            )'''
        self.cur.execute(sql)
        self.con.commit()

$ python Adressenverwaltung.py
Traceback (most recent call last):
File "Adressenverwaltung.py", line 154, in ?
main()
File "Adressenverwaltung.py", line 145, in main
adr.install()
File "Adressenverwaltung.py", line 52, in install
self.cur.execute(sql)
pysqlite2.dbapi2.OperationalError: near "NOT": syntax error

[gerold]

Code: Alles auswählen

    def install(self):
        sql = '''CREATE TABLE IF NOT EXISTS adresses(
                id INTEGER PRIMARY KEY NOT NULL,
                vorname VARCHAR(30),

Hi CrackPod!

Den einzigen Unterschied, den ich erkenne: Das Leerzeichen zwischen addresses und der Klammer.

Code: Alles auswählen

CREATE TABLE IF NOT EXISTS addresses (
    id INTEGER PRIMARY KEY NOT NULL,
    salutation VARCHAR(50),

Ob das der ausschlaggebende Grund für den Fehler ist?

Nein, das habe ich soeben ausprobiert. Das Leerzeichen ist nicht der Grund.

lg
Gerold


PS: http://www.woerterbuch.info/?query=Adressen&s=dict

[CrackPod]

Ja aber an was liegst denn dann?
An dem Leerzeichen liegst nich und auch an dem fehlendem 2. d liegst nich

Ach man Will das nich immer überprüfen müssen, obs die Tabelle schon gibt, wenn ein User install macht, obwohls gar nich notwendig ist>.<

[gerold]

Ja aber an was liegst denn dann?

Hi CrackPod!

Ich habs.

IF NOT EXISTS funktionert erst ab SQLite Version 3.3.0.

http://www.sqlite.org/changes.html

Hätte ich das vorher gewusst, dann hätte ich es nicht in mein Beispiel rein getan. Ich werde mir also für mein Beispiel etwas anderes ausdenken.

lg
Gerold

[gerold]

... ach was solls

Ich werde es nicht austauschen. "IF NOT EXISTS" bleibt im Beispiel. Aber einen Hinweis auf diese Version werde ich rein schreiben.

lg
Gerold

[CrackPod]

und wie kann man das mit älteren versionen überprüfen?
Mann ka ja geschachtelete try...catch Blöcke nehmen.
Der erste versucht ...IF NOT EXISTS...
Wenn das fehlschlägt kommt im Expect ein Try, das schaut, ob man einen Datensatz aus der Tabelle holen kann, wenn ein Tabellenfehler entseht(pysqlite2.dbapi2.OperationalError: no such table: addresses) Dann erstellt man die Tabelle.
War das verständlich:?:
Würde das so funzen?

Greetz

EDIT:
Hab das jetz mal so gemacht:

Code: Alles auswählen

    def install(self):
        '''Erstellt die Datenbankstruktur'''

        
        sql = '''CREATE TABLE IF NOT EXISTS addresses (
                id INTEGER PRIMARY KEY NOT NULL,
                vorname VARCHAR(30),
                zuname VARCHAR(40),
                nachname VARCHAR(30),
                anrede VARCHAR(20),
                adresse VARCHAR(50),
                plz VARCHAR(15),
                ort VARCHAR(25),
                land VARCHAR(30),
                privatnummer VARCHAR(35),
                privatfaxnummer VARCHAR(35),
                mobilnummer VARCHAR(35),
                bueronummer VARCHAR(35),
                buerofaxnummer VARCHAR(35)
            )'''

        try:
            self.cur.execute(sql)
            self.con.commit()
        except sqlite3.OperationalError, err:
            sql = '''SELECT vorname FROM addresses'''
            try:
                self.cur.execute(sql)
                self.con.commit()
            except sqlite3.OperationalError, err:
                error = str(err)
                if error.startswith('no such table:'):
                    sql = '''CREATE TABLE addresses (
                            id INTEGER PRIMARY KEY NOT NULL,
                            vorname VARCHAR(30),
                            zuname VARCHAR(40),
                            nachname VARCHAR(30),
                            anrede VARCHAR(20),
                            adresse VARCHAR(50),
                            plz VARCHAR(15),
                            ort VARCHAR(25),
                            land VARCHAR(30),
                            privatnummer VARCHAR(35),
                            privatfaxnummer VARCHAR(35),
                            mobilnummer VARCHAR(35),
                            bueronummer VARCHAR(35),
                            buerofaxnummer VARCHAR(35)
                        )'''

                    self.cur.execute(sql)
                    self.con.commit()

Scheint auch wunderbar zu funktionieren.

Greetz

[CrackPod]

So... Neue Methode neues Problem:
getall()

Code: Alles auswählen

    def getall (self):
        '''Rückgabe von allen Kontakten des Telefonbuchs'''

        sql = '''SELECT
                    id,
                    vorname,
                    zuname,
                    nachname,
                    anrede,
                    adresse,
                    plz,
                    ort,
                    land,
                    privatnummer,
                    privatfaxnummer,
                    mobilnummer,
                    bueronummer,
                    buerofaxnummer
                FROM
                    addresses'''

        self.cur.execute(sql)

        adresses    = {}
        i           = 1
        while True:
            row = self.cur.fetchone()

            if row:
                adresses[i] = {'id': row['id'],
                               'vorname': row['vorname'],
                               'zuname': row['zuname'],
                               'nachname': row['nachname'],
                               'anrede': row['anrede'],
                               'adresse': row['adresse'],
                               'plz': row['plz'],
                               'ort': row['ort'],
                               'land': row['land'],
                               'privatnummer': row['privatnummer'],
                               'privatfaxnummer': row['privatfaxnummer'],
                               'mobilnummer': row['mobilnummer'],
                               'bueronummer': row['bueronummer'],
                               'buerofaxnummer': row['buerofaxnummer'],
                            }
                i = i+1
            else:
                return adresses

adr = Adress('./daten/address.sqldb')

adr.install()

adr.add('Max','Maximilian','Musterman','Herr','Musterstr 12',
'1234865','Musterstadt','Deutschland','004912326548','','01701234586','','')

adresses = adr.getall()
print adresses

Traceback (most recent call last):
File "Adressenverwaltung.py", line 225, in ?
main()
File "Adressenverwaltung.py", line 221, in main
adresses = adr.getall()
File "Adressenverwaltung.py", line 189, in getall
adresses = {'id': row['id'],
TypeError: tuple indices must be integers

[gerold]

und wie kann man das mit älteren versionen überprüfen?

Hi CrackPod!

Indem man in der "sqlite_master"-Tabelle nachsieht.

http://www.sqlite.org/faq.html#q9

lg
Gerold

[gerold]

TypeError: tuple indices must be integers

Hi CrackPod!

Damit du die Rückgabe wie ein Dictionary verwenden kannst, musst du, vor dem Erstellen des Cursors, eine spezielle RowFactory zuweisen.

Code: Alles auswählen

self.conn.row_factory = sqlite3.Row

Code: Alles auswählen

[...]
    #----------------------------------------------------------------------
    def __init__(self, dbfilename = DBFILENAME):
        """
        :param dbfilename: Name und Pfad zur Datenbankdatei. Existiert
            sie nicht, wird sie automatisch erstellt.
        """

        # Verbindung herstellen
        self.conn = sqlite3.connect(dbfilename)

        # Abfrageergebnisse wie Dictionarys verwenden lassen.
        # Diese Zeile muss **vor** dem Erstellen des Cursors stehen.
        self.conn.row_factory = sqlite3.Row

        # Cursor erstellen
        self.cur = self.conn.cursor()

[...]

    def getall (self):
        '''Rückgabe von allen Kontakten des Telefonbuchs'''

        sql = '''SELECT
                    id,
                    vorname,
                    zuname,
                    [...]
                FROM
                    addresses'''

        self.cur.execute(sql)

        return self.cur.fetchall()


adr = Adress('./daten/address.sqldb')
adr.install()
adr.add('Max','Maximilian','Musterman','Herr','Musterstr 12',
'1234865','Musterstadt','Deutschland','004912326548', None,'01701234586')

addresses = adr.getall()
for address in addresses:
    print "vorname:", address["vorname"]
    print "zuname:", address["zuname"]
    print

lg
Gerold

[CrackPod]

Wenn ich jetz mit sqlite_master arbeite, dann bekomm ich folgendes:

Code: Alles auswählen

        sql = """SELECT name FROM sqlite_master WHERE type='table'"""

        self.cur.execute(sql)
        while True:
            row = self.cur.fetchone()
            print '-'*15
            print row
            print '-'*15
            if row == 'addresses':
                created = True
                break
            if row == None:
                break

---------------
(u'addresses',)
---------------

Wie kann ich jetz auslesen addresses auslesen? Mit dem if Teil oben geht schonmal nicht.

[gerold]

(u'addresses',)

Hi CrackPod!

Ich habe die Methode create_dbstructure() komplett umgebaut. Bitte sieh sie dir an. Da drinnen zeige ich auf, wie man prüfen kann ob eine Tabelle oder ein Index bereits existieren.

Code: Alles auswählen

(u'addresses',)

Das ist ein Tupel. Dieser verhält sich ähnlich wie eine Liste. Der große Unterschied ist der, dass ein Tupel sich nich verändern lässt. Dadurch ist er aber auch schneller als eine Liste.

Code: Alles auswählen

>>> t = (u'addresses',)
>>> print t[0]
addresses
>>>

In unserem speziellen Fall allerdings, wurde dieser Tupel durch den Code ``self.conn.row_factory = sqlite3.Row`` in ein Objekt vom Type **pysqlite2.dbapi2.Row** "umgewandelt". Deshalb kann man darauf nicht nur mit einem Index, sondern auch mit einem Schlüssel, wie bei einem Dictionary zugreifen.

Code: Alles auswählen

print row[0]
# oder
print row["name"]

lg
Gerold

[CrackPod]

So bin jetz größtenteils fertig.
Jetz muss halt noch "Interaktivität" rein und fertig is der Spaß danach wird das ausgebaut
http://nopaste.info/7176b9c5ed.html
Könnt ihr mir sagen, was ich noch verbessern kann/soll?

Danke Greetz

[CrackPod]

naja irgendwie wollte mir wohl keiner so richtig auf die Frage antworten. Naja nich so tragisch. Hab jetz mal ein wenig weitergearbeitet um die Interaktivität herzustellen.
Hier is das Skript: http://nopaste.info/d22f2a9a91.html
Problem: Irgendwie bekomme ich Endlosschleifen, wenn ich bei dem Interaktiven Teil als Spalte vorname und als Wert Max eingeben. Auch, wenn ich mit der "Statischen" Klasse suche - mit denselben Daten - , bekomm ich eine Endlosschleife...
Ich weiß nur nich warum.
Bevor ich mit der Klasse `Menu` angefangen habe, hat alles wunderbar funktioniert und ich habe seitdem die Klasse `Adresses` nichmehr angerührt - außer um alles einmal zu kommentieren, aber das macht ja den Code nicht kaputt. Hab ja deswegen trotzdem nix an dem Code, der für den Interpreter wichtig is, geändert.
Kann mir einer Helfen?
Danke schonma im voraus

Greetz