Das deutsche Python-Forum

Hyperion hat geschrieben:Naja, Du müßtest einfach die Connection in einem Attribut Deines Objektes hinterlegen :K

Ich würde die Methode zum Verbindungsaufbau auch nicht ``execute`` nennen - so heißt ja beim DB-API2 die Methode zum Absetzen eines SQL-Kommandos. Das ist imho verwirrend.

Klassennamen schreibt man "CamelCase" nach PEP8 - oder ist das bei GtK ggf. eine Ausnahme und bewusst so gehalten?

truehumandesign hat geschrieben: da ''conn'' aber nicht global festgelegt sonder nur im "Verbindung herstellen" object übernimmt er das nicht in das "Verbindung trennen" object.

Hyperion hat geschrieben:

truehumandesign hat geschrieben: da ''conn'' aber nicht global festgelegt sonder nur im "Verbindung herstellen" object übernimmt er das nicht in das "Verbindung trennen" object.

Äh... was sollen das für Objekte sein? Redest Du von den Methoden der Klasse ``portOpt``? Diese haben aber keine "Attribute", sondern kennen nur Namen, die entweder lokal definiert sind - so wie momentan ``conn`` - oder aber das jeweilige "Instanz"-Objekt, welches sie über ``self`` bekommen.

Du musst die Verbindung innerhalb einer Instanz von ``portOpt`` hinterlegen - oder aber direkt als Klassenattribut. Auf jeden Fall muss das ganze "global" innerhalb Deiner Klasse verfügbar sein - ansonsten kannst Du die Klasse auch weglassen und die DB-Verbindung extern verwalten...

import pygtk
pygtk.require("2.0")
import gtk
import psycopg2

class portOptPrg(object):
	
	def con_db(self, *args):
		host = self.builder.get_object('host').get_text()
		db = self.builder.get_object('db').get_text()
		user = self.builder.get_object('user').get_text()
		pw = self.builder.get_object('pw').get_text()
		conn_string = "host=self.host dbname=self.db user=self.user password=self.pw"
		conn = psycopg2.connect(conn_string)

	def con_disc(self, *args):
		conn.close()

class portOptGui(portOptPrg):    
	def __init__(self):
	    self.builder = gtk.Builder()
	    self.builder.add_from_file("portoptv3.glade")
	    self.builder.connect_signals(self)
	    self.window = self.builder.get_object("window1")
	    self.window.show()
				
	def on_bt_quit_activate(self,widget, data=None):
		gtk.main_quit()
		
	def on_window1_destroy(self,widget, data=None):
		gtk.main_quit()
		
	def on_act_con_activate(self, *args):
		self.con_db()
		
	def on_act_discon_activate(self, *args):
		self.con_disc()
				
if __name__ == "__main__":
	app = portOptGui()
	gtk.main()

Traceback (most recent call last):
File "portoopv4.py", line 37, in on_act_discon_activate
self.con_disc()
File "portoopv4.py", line 17, in con_disc
conn.close()
NameError: global name 'conn' is not defined

deets hat geschrieben:Oh mann... das ist wirklich gruselig.

Zuerstmal hakt es immer noch, weil du deine conn nirgendwo speicherst. Du musst

Code: Alles auswählen

      self.conn = psycopg2.connect(conn_string)

machen, und natuerlich auch in disconnect dich auf self.conn beziehen.

Und dann mal ein paar Anmerkungen:

Die Zerteilung in zwei Klassen hat das ganze deutlich verschlimmert. Denn du erstellst eine Basisklasse, die jede Menge Detailwissen vorraussetzt ueber eine Klasse, die von ihr ableitet. Das ist ganz schlechtes Design. Denn wer sagt denn, dass du nicht auch eine GUI programmieren willst, die den connection string als ganzes, inklusive username/passwort usw entgegennimmt - also nur *ein* Texteingabefeld hat? Das wuerde mit deinem jetztigen Ansatz nicht funktionieren.

Also, portOptPrg bekommt den conn_string als Argument an con_db, und das war's.

Und portOptGui erbt nicht von portOptPrg, sondern bekommt eine Instanz davon bei Konstruktion, oder es gibt davon eine globale Variable.

Dann ist deine Namenskonvention ungewoehnlich - Klassen haben CamelCase. Und die ganzen Abkuerzungen sind verwirrend. Warum heissen con_db und con_disc nicht einfach connect_db & disconnect_db?

Last but not least - wenn das, was du hier postest, wirklich dein echter Code ist, dann ist der conn_string voellig sinnfrei zusammengebaut. So funktioniert string-formatting nicht (falls es das sein sollte).

Sorry ich bin ein absoluter python noob

deets hat geschrieben: Und das Argument "ist nur zum ausprobieren" ist immer schwierig... nichts haelt so lang, wie ein Provisorium. Auch und gerade wenn es um's lernen geht, solltes du dich um vernuenftiges programmieren bemuehen. Wann sonst lernst du es - wenn die Zeit im Projekt draengt eher nicht...

def selectall(self, *args):
		cur = self.conn.cursor()
		cur.execute('SELECT * FROM foo')
		text = cur.fetchone()
		buffer = self.builder.get_object('view_log').get_buffer()
		buffer.set_text("%s") % text
		self.conn.commit()
		cur.close()

(1, 'ZL0000001', 'Klemmerle', 'Bushaltestelle 1', '12345', '99999')

<type 'tuple'>
Traceback (most recent call last):
File "portoopv5.py", line 63, in on_act_select_activate
self.selectall()
File "portoopv5.py", line 33, in selectall
buffer.set_text("%s") % text
TypeError: unsupported operand type(s) for %: 'NoneType' and 'tuple'

buffer.set_text(repr(text))

(1, u'ZL0000001', u'Klemmerle', u'Bushaltestelle 1', u'12345', u'99999')

BlackJack hat geschrieben:@truehumandesign: Wenn Du `tuple` nicht kennst, dann solltest Du jetzt Datenbank und GUI mal solange beiseite lassen bis Du die Python-Grundlagen drauf hast. Arbeite am besten mal das Tutorial aus der Python-Dokumentation durch.

Und in der Zeile ist noch ein anderes Problem gewesen: Schau mal was links vom ``%``-Operator steht.

def selectall(self, *args):
cur = self.conn.cursor()
cur.execute('SELECT * FROM foo')
outp = cur.fetchone()
buffer = self.builder.get_object('view_log').get_buffer()
buffer.set_text(outp[2])
self.conn.commit()
cur.close()