Kommentare zu: LaTeX Serienbriefe – GUI zur Adressbuch Generierung

Von: Yoshi

Yoshi — Wed, 05 Aug 2009 08:17:19 +0000

@Jazz
Ich danke dir, das werde ich nachher mal ausprobieren

Von: Jazz

Jazz — Wed, 05 Aug 2009 07:59:23 +0000

@Yoshi
Ja das Theme finde ich auch cool. Das ganze heißt Shiki Theme und gibts in verschiedenen Farbvarianten.

http://linuxundich.de/de/2008/12/mein-gnome-desktop-shiki-bites-the-dust/

–> da gibts ne gute Anleitung zum Installieren des Themes

Von: Yoshi

Yoshi — Wed, 05 Aug 2009 07:19:52 +0000

Was mich persönlich interessiert ist, wo man das schöne Theme (braun/grün) von dem Screenshot herbekommt

Von: Christian

Christian — Sat, 25 Jul 2009 09:16:28 +0000

So, ich habs mir mal angesehen und hätte folgendes an Verbesserungsvorschlägen:

- OptionParser kennt eine Methode error (o.ä.). Damit kann man explizit auf fehlende Argumente hinweisen.

- Versuche mal mehr auf PEP8 zu achten. Das hilft nicht nur externen Leuten wie mir, sondern auch Dir den Quelltext später leichter lesen zu können. Im Speziellen betrifft das auf den ersten Blick die Klassennamen, die Länge der Zeilen (speziell beim Optparse Teil), Zeilenabstände usw.

- Es gibt in der Python-Doku zum CSV-Modul ein Beispiel, wie man CSV beim Einlesen bereits in Unicode umwandelt. Das kannst Du so fast immer 1:1 in Deinen Quellcode übernehmen. Dann bekommst Du eben die Wandlung in und von Unicode hin.

- Wieso packst Du alles in eine Datei? Wenn Du fremden Quellcode benutzt, lass ihn doch in einer eigenen Datei und importe dann die benötigten Teile?

- ich würde den if __name__ == “__main__”-Hook kurz halten und innerhalb nur eine main()-Funktion aufrufen, in der dann der Rest steht.

- Doc-Strings benutzen statt simpler Kommentare über den Methoden / Funtkionen! Dazu dann ggf. die übergebenden Parameter genauer beschreiben – bei DuckTyping-Sprachen imho unerlässlich.

Ansonsten aber schön zu sehen, wie sich Leute mit Python befassen

Von: Christian

Christian — Sat, 25 Jul 2009 08:58:50 +0000

Hallo,

wie “müme” schon sagte, sollte man innerhalb des Scriptes nur mit unicode-Objekten arbeiten. Dabei gilt die Regel: So früh wie möglich in Unicode wandeln (decodieren), so spät wie möglich in Byte-Strings zurückwandeln (encodieren).

Das Beispiel oben zeigt schon sehr gut, wie man das z.B. beim Laden einer Datei hinbekommt.

Hier mal ein nützlicher Link, um sich die Basics dazu anzulesen:
http://wiki.python.de/Von%20Umlauten%2C%20Unicode%20und%20Encodings

Bei Fragen kannste Dich auch gerne an mich wenden

Von: Zaidira

Zaidira — Sat, 25 Jul 2009 02:05:51 +0000

An sich find ich das Programm von der Idee her echt gut. Aber über die Gestaltung der GUI würde kich mir nochmal Gedanken machen: Zum Einen würde ich die Liste auf der Rechten Seite nach links “verschieben” und zum Anderen würde ich die Buttons “OK” und “Cancel” nach unten verschieben, fände ich gewohnter. Ansonsten gefällt mir das schonmal, auch wenn ich es noch nicht testen konnte. Das werd ich aber schnellstmöglichst nachholen.

MfG Zaidira

Von: Jazz

Jazz — Fri, 24 Jul 2009 16:28:06 +0000

Ja is mit Python 2.6 programmiert.

Da die Spaltenüberschriften der CSV als Keys in einem Dictionary und als Keys für das Templatesystem von Python verwendet werden. Der eigentlich Inhalt kann also auch Sonderzeichen enthalten, nur die Spaltenbezeichnung nicht.

Von: µme

µme — Fri, 24 Jul 2009 16:22:19 +0000

Ich erhalte mit Python 2.5.4 unter Windows folgende Fehlermeldung, offenbar ist Python 2.6 nötig?
C:\Users\Mein Name\Desktop\LatexAddressbook.py:135: Warning: ‘as’ will become a reserved keyword in Python 2.6
File “C:\Users\Mein Name\Desktop\LatexAddressbook.py”, line 135
except IOError as ioerror:
^
SyntaxError: invalid syntax

Zu dem Encoding-Problem:
Warum müssen die Schlüssel UTF-8 sein?
Versuche die Dateien mit codecs.open(filename, ‘r’, ‘utf-8′) zu importieren, dann ist alles ausgelesene ein Unicode-Objekt und kann einfach verwendet werden.