Kritik am Python OpenBook

[BlackJack]

Ich habe meine Kritik am Python OpenBook mal als Webseite verfasst: http://www.marc.rintsch.de/python_openbook.html

[Leonidas]

Gute Idee. Man sollte auch erwähnen, dass die Autoren da eine Unterscheidung zwischen zwei Modultypen einführen, die es in der Realität gar nicht gibt.

[cofi]

Gut, dass du das gemacht hast, denn die Autoren arbeiten weiterhin in unserem Feld, ob sie immernoch wueten ist eine andere Frage, aber anzunehmen waers

Aber deine Alternative zur Durschnittsberechnung ist auch nicht viel besser.
IMHO besser so - mit deren Code brichst du eh schon, also hat es keinen Sinn den Index noch mit zuschleifen:

Code: Alles auswählen

def average(iterable, start_value=0):
    total = start_value
    length = len(iterable)
    if length:
        for value in iterable:
            total += value
    else:
        raise ValueError('iterable must contain at least one argument.')
    return total / length

Eventuell koennte man noch den ``ValueError`` abfangen, wenn das nich alles Zahlen sind, aber das ist wohl besser beim Caller aufgehoben.

[DasIch]

@cofi Deine Variante funktioniert nicht für jedes iterable.

Code: Alles auswählen

>> def mygen():
..     yield 1
..     yield 2
..     yield 3
>> len(mygen())
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
...
TypeError: object of type 'generator' has no len()

[cofi]

Hmm das stimmt natuerlich.

[birkenfeld]

Kleine Zwischenbemerkung: Entweder dein Webserver oder das HTML-Dokument sollten etwas über die Kodierung aussagen. Der IE macht anscheinend keinen Default auf UTF-8...

[birkenfeld]

Achja, und evtl. wäre ``0.0`` als Default für den start_value günstiger, um keine Überraschungen bei ``average([0, 0, 1])`` zu provozieren.

[BlackJack]

Hm, der Seitenquelltext beginnt mit ``<?xml version="1.0" encoding="UTF-8" ?>``. Der IE ist halt doof.

[birkenfeld]

Bah, den XML-Header hab ich übersehen. Bleibt die Frage, warum du XML als "content-type: text/html" auslieferst

[Leonidas]

Bleibt die Frage, warum du XML als "content-type: text/html" auslieferst

Meinst du das ernst? Ich mache das auf meiner Uni-Seite: XHTML 1.1 mit application/xhtml+xml. Was IE damit tut: zum Download anbieten.. wie BlackJack sagte: der IE ist halt doof.

[birkenfeld]

Bleibt die Frage, warum du XML als "content-type: text/html" auslieferst

Meinst du das ernst? Ich mache das auf meiner Uni-Seite: XHTML 1.1 mit application/xhtml+xml. Was IE damit tut: zum Download anbieten.. wie BlackJack sagte: der IE ist halt doof.

Naja, besonders viel lesbarer ist die Seite mit kaputten Umlauten auch nicht. Also warum nicht entweder konformes XHTML oder konformes HTML?
Und außerdem ist XHTML eh ein Auslaufmodell

[Leonidas]

Naja, besonders viel lesbarer ist die Seite mit kaputten Umlauten auch nicht.

Immerhin kann man das in Browser umstellen, ganz im Gegensatz zum "Ach, ich downloade dir mal die Seite auf die Festplatte"-Verhalten.

Also warum nicht entweder konformes XHTML oder konformes HTML?

Ja, von BJ hätte ich durchaus konformes (X)HTML erwartet

Und außerdem ist XHTML eh ein Auslaufmodell

Ich werde wohl bei der XML-Serialisierung bleiben. XHTML5 wird es ja trotz HTML5 zum Glück weiterhin geben.

[BlackJack]

@birkenfeld: Ich liefere ja gar nicht aus, das macht der Webserver.

Hm, muss mal schauen -- wahrscheinlich muss ich '.xhtml' als Dateiendung verwenden.

Edit: So jetzt scheint's unter dem IE zu funktionieren.