Seite 1 von 1

Verfasst: Dienstag 14. November 2006, 21:40
von tabellar
jens hat geschrieben:Ich wollte das gleich mal ausprobieren...

* Als erstes wuste ich nicht wie man die Dateien benennen muß.
* Die Einrückung stimmt teilweise nicht. Sollte immer 4 Zeichen sein.
* Die DB Typen PgSQL und XML sind doch gar nicht vorhanden, oder wo sind die Dateien pgsqlDAO und xmlDAO ? Ich hab mal ein raise NotImplementedError() eingefügt.

...
Hier ist der Zusammenhang mit dem dazugehörigen Thread

:wink:

Tabellar

Verfasst: Dienstag 14. November 2006, 22:29
von Leonidas
rolgal_reloaded hat geschrieben:Ich finde Leerzeilen angenehm, aber wahrscheinlich habe ich da und dort übertrieben. Gibst mir mal ein paar konkrete Lines, wo du es besonders schlecht findest, dann kann ich leichter eine Vorstellung von einem besseren Gesamtbild bekommen.
Ich sage ja nichts gegen Leerzeilen generell - natürlich sind sie wichtig. Aber nur in Maßen.
Snippet 1:
  • 9-11
  • 14-15
  • 25
  • 30
  • 35
  • 37
  • 59 (die Funktion ist übrigens nicht PEP8 kompatibel)
  • 67
  • 79
  • 81
  • 83
  • 85
  • hier LC hinsetzen
  • ...
  • self.lust = False
Für die anderen Snippets habe ich keine lust, aber du siehst, dass ich zum Beispiel in den Funktionen die erste Leerzeile generell wegstreichen würde, und auch drei oder drei Leerzeilen hintereinander finde ich überflüssig.

Verfasst: Dienstag 14. November 2006, 23:05
von rolgal_reloaded
@Leonidas

Ich wollte dir ja nicht unterstellen, dass du grundsätzlich was gegen Leerzeichen hast.

Natürlich habe ich mir auch nicht erwartet, dass du alle 3 Programme durchforstet.

Auf jeden Fall danke für die Lines, ich werde mir das in Ruhe ansehen.

Warum ist die Funktion nicht PEP kompatibel, wegen dem "\"?

LG

rolgal_reloaded

Verfasst: Mittwoch 15. November 2006, 09:00
von Leonidas
rolgal_reloaded hat geschrieben:Warum ist die Funktion nicht PEP kompatibel, wegen dem ""?
Wegen dem fehlenden Leerzeichen um die Operatoren.