Seite 2 von 2

Verfasst: Freitag 10. Juli 2009, 12:03
von Dauerbaustelle
feldmann_markus hat geschrieben:Wie hättest Du das denn sonst formuliert
`point_is_on_line` oder `is_on_line` und `is_in_polygon` oder `in_polygon`.

Verfasst: Freitag 10. Juli 2009, 12:03
von cofi
feldmann_markus hat geschrieben:Also wäre an dieser Stelle ein Modul ohne Klasse mit nur Funktionen ausreichend ?
Ja. Eine Klasse soll ein *Objekt* beschreiben, man sollte es nicht als schlichten Namensraum missbrauchen.
feldmann_markus hat geschrieben:Was ist an den Namen <_checkPointIsOnLine()> und <_checkbounds()> nicht verständlich ?
Es wurde nie gesagt, dass sie unverstaendlich sind, sondern dass sie nicht PEP8 konform sind. Demnach sollten sie ``check_point_is_on_line`` (oder noch besser: ``is_on_line(point)``) und ``check_bounds()`` heissen.

Verfasst: Freitag 10. Juli 2009, 16:12
von Leonidas
feldmann_markus hat geschrieben:Also wäre an dieser Stelle ein Modul ohne Klasse mit nur Funktionen ausreichend ?
Ja, wenn man keine Funktionalität von Klassen nutzt, dann ist das ein gutes Indiz dafür, dass man keine Klassen braucht um das Problem zu lösen.
feldmann_markus hat geschrieben:Wie hättest Du das denn sonst formuliert ?
``point_is_on_line`` etwa...