cyp++ hat geschrieben:[…]
vll weißt du es nicht, aber ich bin anfänger 11. klasse ausbildung zum tai. das hatten wir noch alles gar nicht!
Ich denke nicht das er das Böse gemeint hat. Sondern wollte nur darauf hinweisen, das einige auf dein Post eingegangen sind und du dir dann auch die mühe machen solltest, die posts wenigstens zu lesen. Du hast ja schließlich nach Hilfe gefragt und dann, nach dem alle ein par vorschlage geschrieben haben, nach Verbesserungen zu fragen, ist da schon ein wenig Strange
Es wurden ja viel sehr kurze und gute Sachen gepostet
(ausgenommen von meiner re Variante die echt schlecht ist ^^ Aber das ist ein anderes Thema)
Falls du in den posts was nicht verstehst, kannst du ja gerne nachfragen
Ich verstehe da einige Sachen auch noch nicht wie zum beispiel BlackJacks Funktionales Beispiel ^^ Aber da steige ich auch noch durch.
Aber nun zu deinem Code. Der erinnert sehr stark an C. Und du scheinst auch noch sehr in C/C++ zu denken
(habe auch noch ein wenig das Prob bei einigen Sachen). Z.B. iterierst du über ein range() und benutzt dann den Wert als index-nummer um auf ein Array (list) zuzugreifen. Das ist sehr C Typisch un nicht pythonisch. In diesem Fall iteriert man lieber gleich über das Array und hat dann das Char, anstatt auf den index zurückzugreifen um an den Cahr zu gelangen
Auch Zeile 7 mit alles upper zu machen kann man sich Sparen weil es nicht benötigt wird. Mach gleich alles lower dann kannst du dir Zeile 30 ersparen. Wie gesagt die ganzen
for i in range kannst du dir sparen. Iteriere lieber gleich über den string
Output
Und übrigens das…
XtraNine hat geschrieben:
Dabei fällt mir die regel 1 vom Zen ein: Schön ist besser als hässlich.
und die regel 3: Einfach ist besser als komplex.
…ist nicht böse gemeint, aber dein Code ist zu lang und zu komplex und total unpythonisch.
lg