Das deutsche Python-Forum

Hallo,

da ich im Moment mal wieder Zeit habe, wage ich mich mal langsam in die objektorientierte Welt...

Nun wollte ich ein kleines Schafkopfspiel per Tkinter erstellen und habe so begonnen:

http://paste.pocoo.org/show/39054/

Meine Fragen:
- Was ist hier schlecht?
- Was kann man allgemein verbessern?
- Wieso wird das Frame fr_karten schon von Anfang an riesig dargestellt, obwohl ich noch gar nicht auf den "Karten geben" Button geklickt habe?
- Wieso funktioniert der Karten geben Button nicht? (Ich denke es liegt daran, dass ich aus dem x.pack() kein self.x.pack() gemacht habe (das geht aber leider auch nicht.... ))

Ich bedanke mich für eure Hilfe schon im Voraus!

MfG

Tabs viel zu groß - mehr auf PEP8 achten!

Dir fehlen klare Strukturen! Die Farben und Werte sind ja etwas dauerhaftes, bei Dir werden Sie in einer Methode berechnet und danach weggeworfen.

Trennung von GUI und Logik - MVC-Konzept!

Tja, und mehr Doku!

Das so mal auf die Schnelle!

Und es fehlt irgendwie die Objektorientiertheit. Eine Klasse macht noch keine Objektorientierung aus.