Kleiner Rechner - Verbesserungsvorschläge und Kritik

Freigeschaltet

Sitz grad in der Uni und hab keine Zeit drüber zu schauen ~ 18:50 bin ich daheim

Gruß
florian0

warum in gottes namen hast du alles static gemacht? ganz schlechter stil... hat dir awesome das in den kopf gesetzt?

was macht denn der timer? versteh ich nicht ganz.

naja, wenn du alles static machst, brauchst du keine objekte. also brauchst du auch keine objektorientierte programmiersprache und kannst auch mit C arbeiten.

mach nur static, was unbedingt static sein muss. ansonsten lerne, mit objekten zu arbeiten. außerdem könntest du viel in methoden auslagern. das ist so ein schlauch, wie ich finde.

Ich hatte die Klasse static gemacht, da es keinen Grund gab sie zu instanzieren. Man braucht ja keine Instanz einer Klasse deren Funktionen immer die gleichen sind.

florian0 schrieb:

Ich hatte die Klasse static gemacht, da es keinen Grund gab sie zu instanzieren. Man braucht ja keine Instanz einer Klasse deren Funktionen immer die gleichen sind.

Bei jeder Instanz einer Klasse sind die Funktionen immer die gleichen

@Mr.Unbekannt
Streich bitte das Wort static wieder aus deinem Wortschatz, zumindest für das erste halbe Jahr - Jahr. Sonst wirst du dich sehr schwer tun die ganzen Objekt-orientierten Ansätze zu verstehen. In diesem Fall kann man es noch vertreten die Klasse static zu machen, weils eigentlich nur eine kleine Helper Funktion ist. Aber die Klasse gehört auf jedenfall in eine eigene Datei!
Dann gehört in die Rechne Methode ein switch case rein um die einzelnen Operatoren zu behandeln (oder zumindest - ist aber kein guter stil - if-ELSEIF-else). Und die Fehlerbehandlung fehlt komplett, bei dir gibt es keinen Unterschied zwischen 1-1 und nem falschen Operator (ich weiß das du erst anfängst, aber ich will schon mal gern draufhinweisen ;))

Am besten schreibst du die Rechnung Klasse nochmal neu unter einem objekt-ortientierten Ansatz. Und zwar hast du dafür 2 Methoden:
SetNumbers - um die beiden Zahlen zu setzen
Compute - berechnet das Ergebnis und gibt es zurück

Das wird jetzt vielleicht etwas hoch gegriffen sein für viele und viele werden auch nicht verstehen, was das soll bzw. es als unnötigen Overhead ansehen, aber die Rechenoperationen z. B. bieten sich gerade zu für das Strategy-Pattern an. Das ist zwar nur eine kleine Spielerei, aber dennoch. Und zudem kann man dadurch einen guten Einblick in OOP bekommen. Das löst dann auch das Problem mit dem ganzen static-Geraffel.

Mr. Unbekannt schrieb:

Das hat mir florian0 so gesagt. Warum ist mir nicht ganz klar , aber es wird schon seinen Grund haben.

Das mit dem Zuschauen beim lernen ist ja eine ganz feine Sache.
Allerdings solltest du, wenn du Tipps erhältst, diese auch hinterfragen.

Blind übernehmen kannst du von mir aus bei kleineren Form Fehlern, wie es gestern der Fall war: string blubb[] = new string[index];
Aber, wenn da jemand hingeht und deine Methoden zu statischen Methoden ändert, ist das ein guter Grund zu fragen.

Und ich betone es nochmal, schau in die MSDN Doku (Druck auf F1)
Wenn du den Befehl selektierst springst du automatisch zur Passenden Methode, property..

Mr. Unbekannt schrieb:

Ps: Was für ein Programm könnte man noch als Anfänger programmieren?

Für Anfänger bieten sich Sachen an wie die Türme von Hanoi oder ein Zahlenrate Spiel, das waren zumindest die Sachen mit denen ich immer angefangen habe.