Kleiner Rechner - Verbesserungsvorschläge und Kritik

  • Hab's unter Aufsicht von kuhkuh , florian0 und Awesome geschafft, einen kleinen Rechner zu schreiben.
    Gebt mir Feedback , Verbesserungsvorschläge oder Kritik!


    http://paste.www.stagetwo.eu/paste.php?key=nb5ytq4m

    Es ist besser, zu genießen und zu bereuen, als zu bereuen, dass man nicht genossen hat.

    -Giovanni Boccaccio-

    Dieser Beitrag wurde bereits 1 Mal editiert, zuletzt von Mr. Unbekannt ()

  • 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.

    Dieser Beitrag wurde noch nie editiert, zuletzt von »hangman« (Heute , 13:37)



    1355?mode=raw


    Skill ist, wenn Luck zur Gewohnheit wird

  • 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.

    Dieser Beitrag wurde noch nie editiert, zuletzt von »hangman« (Heute , 13:37)



    1355?mode=raw


    Skill ist, wenn Luck zur Gewohnheit wird

  • 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.

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

    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..

    And the reign will kill us all
    We throw ourselves against the wall
    But no one else can see
    The preservation of the martyr in me


    Slipknot - Psychosocial

  • 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.


    :D Ich habe da nur geschaut und nebenbei selber gecodet ^^