@toper3902: Ein paar Anmerkungen zum Code:
Man sollte ungebräuchliche Abkürzungen vermeiden. Das `win32com.client`-Modul, das nur ein einziges mal gebraucht wird, in `wincl` umzubenennen, macht den Code eher schlechter als besser verständlich.
Variablen sollten möglichst nah an der Stelle definiert werden, an der man sie benötigt. `durchlauf` wird viel zu weit von der Schleife entfernt definiert. Das macht das Programm schwerer lesbar, weil man die Initialisierung beim Lesen entweder im Kopf behalten muss, während man davon unabhängigen Code liest, oder man gelangt bei der Schleife an, und muss erst wieder zurückgehen um die erste Zuweisung zu finden. Es ist auch schwieriger Codeteile in Funktionen auszulagern, wenn der Code für die Funktion nicht zusammen steht, sondern durch anderen Code ”unterbrochen” wird. Und bei grösseren Programmen entstehen auf diese Weise oft Variablen die irgendwo definiert werden, aber nach Codeänderungen dann nicht mehr verwendet werden. Ganz einfach weil man leichter vergisst die Definition auch zu entfernen wenn man den Code weiter unten so ändert, das die Variable nicht mehr verwendet wird.
`speak` wäre ein guter Name für eine Funktion, weil es eine Tätigkeit beschreibt, aber es ist kein guter Name für ein Objekt das einen *Sprecher* oder eine *Stimme* repräsentiert. Ich würde mich da einfach an der Namensgebung der API orientieren und die Python-Namenskonventionen anwenden. Da käme dann `sp_voice` als Name bei heraus.
Bei `piges` gibt es eine kryptische Abkürzung und die beiden Worte sind nicht durch einen Unterstrich getrennt. Das erschwert das Lesen. `pi_gesamt` ist viel leichter zu lesen (falls ich die Abkürzung richtig geraten habe). Ausserdem würde ich behaupten π ist eine Konstante. Wenn die also literal im Quelltext steht, sollte sie auf Modulebene und komplett in Grossbuchstaben definiert werden. Bei `piok` fehlt auch der Unterstrich zwischen den beiden Worten. Wobei man auch nicht jeden Zwischenwert an einen Namen binden muss.
Ich wäre in 100 Jahren nicht darauf gekommen, das der Name `komma` für die Anzahl der Nachkommastellen stehen soll.
Indexzugriffe in Schleifen sind in Python in der Regel ein „code smell“. Man kann direkt über die Elemente von Sequenztypen wie Listen iterieren, ohne einen Umweg über einen Index. Falls man *zusätzlich* noch eine laufende Zahl benötigt, gibt es die `enumerate()`-Funktion. In jedem Fall ist die ``while``-Schleife hier das falsche Mittel, weil man das einfacher mit einer ``for``-Schleife ausdrücken kann.
Den vorliegenden Code kann man deutlich vereinfachen, wenn man nicht anhand der Durchlaufnummer entscheidet das der Dezimalpunkt gesprochen werden soll, sondern gleich eine Datenstruktur erstellt, wo der Dezimalpunkt an passender Stelle enthalten ist.
Ich komme dann ungefähr bei so etwas heraus (ungetestet):
Code: Alles auswählen
#!/usr/bin/env python3
import win32com.client
PI_GESAMT = '453456 3563'
def main():
#
# TODO Ensure user input is a) an integer value and b) in 0…5000.
#
nachkommastellenanzahl = int(
input(
'Auf wieviele Nachkommastellen soll π ausgegeben werden?'
' (max. 5000) '
)
)
pi_stellen = list(PI_GESAMT.replace(' ', ''))
pi_stellen.insert(1, 'point')
sp_voice = win32com.client.Dispatch('SAPI.SpVoice')
#
# +2 for the integer place and the decimal point.
#
for stelle in pi_stellen[:nachkommastellenanzahl + 2]:
sp_voice.Speak(stelle)
if __name__ == '__main__':
main()
Ich würde noch die Namen in dem Programm auf Englisch übersetzen. Das liest sich besser, weil Programmiersprache, Standardbibliothek, und so ziemlich alle externen Module ebenfalls Englisch verwenden. Ausserdem funktioniert die Mehrzahlbildung von Namen im Englischen deutlich besser als im Deutschen, wo man oft für Einzahl und Mehrzahl ein identisches Wort hat. Das ist wichtig, weil Sequenztypen üblicherweise in der Mehrzahl eines einzelnen Elements benannt werden. Also beispielweise `pi_stellen` für eine Sequenz von einzelnen Stellen von π.
