Kopfrechnen-Programm

[Fugover]

Ich habe dieses Programm geschrieben und wollte es euch mal zeigen und eure Meinung dazu hören
[CODE lang="java" title="Kopfrechnen-Training"]package Package1;
import java.util.Scanner;
import java.util.Random;
public class Hallo1 {

public static void main(String[] args) {
System.out.println("Kopfrechnen-Training");
int x2 = 1;
while (x2==1) {

Random zufall = new Random();
int zufallsZahl = zufall.nextInt(10);
System.out.println(zufallsZahl);

Random zufall2 = new Random();
int zufallsZahl2 = zufall2.nextInt(10);
System.out.println(zufallsZahl2);

int x = zufallsZahl * zufallsZahl2;

int inData;
Scanner scan = new Scanner( System.in );
System.out.println("Beide Zahlen multiplizieren");
System.out.println("und Ergebnis eingeben: ");
inData = scan.nextInt();


if (x==inData) {System.out.println("Ergebnis ist richtig");}
else {System.out.println("Ergebnis ist falsch");
System.out.println("richtiges Ergebnis= " + x);}


Scanner scan2 = new Scanner( System.in );
System.out.println("Bitte 1 für weiter eingeben");
System.out.println("und 0 für Stop!");
x2= scan2.nextInt();

}
System.out.println("Programm Ende");
}
}
[/CODE]

 

[Robert Zenz]

Formattierung. Benutze einen Code-Formattierer...immer.

---

Hier sind die Konventionen fuer Paket-Namen beschrieben.

---

  int x2 = 1;
  while (x2==1) {

Hier ist eine einfache Regel fuer das benennen von Variablen: Benenne sie immer nach dem was sie enthalten.

Als Erweiterung dazu: Einbuchstaebige Namen duerfen nur fuer Dimensionen (x, y, z, w) verwendet werden.

---

Random zufall = new Random();

Du musst nicht jedes mal ein neues Random erzeugen, eines reicht vollkommen.

---

int inData;

Definiere Variablen nicht am Anfang von einem Block, sondern dort wo du sie brauchst.

---

  int x2 = 1;
  while (x2==1) {

Ein boolean waere hier schoener.

  boolean runnning = true;
  while (running) {

---

Scanner scan = new Scanner( System.in );

Hier ist es sogar noch wichtiger (weil die darunter liegende Resource, der Stream, geteilt wird). Erzeuge einen Scanner am besten auszerhalb der Schleife und verwende diesen.

 

[httpdigest]

Neben dem, was @Robert Zenz schon erwähnt hat, könntest du noch schönere Ausgaben für den Nutzer anzeigen und Eingaben direkt hinter den Ausgaben erlauben (mit System.out.print() statt System.out.println()) und auch noch etwas sprechendere Bezeichner wählen, z.B. "richtigesErgebnis" statt "x".
Desweiteren könntest du noch eine do..while Schleife nutzen, weil du ja die Schleife immer mindestens einmal betrittst.
Alles zusammen wäre dann z.B.:

import java.util.Scanner;
import java.util.Random;
public class Hallo1 {
    public static void main(String[] args) {
        System.out.println("Kopfrechnen-Training");
        Random zufall = new Random();
        Scanner scan = new Scanner(System.in);
        do {
            int faktor1 = zufall.nextInt(10);
            int faktor2 = zufall.nextInt(10);
            System.out.print(faktor1 + " * " + faktor2 + " = ");
            int userEingabe = scan.nextInt();
            int richtigesErgebnis = faktor1 * faktor2;
            if (userEingabe == richtigesErgebnis) {
                System.out.println("Ergebnis ist richtig");
            } else {
                System.out.println("Ergebnis ist falsch");
                System.out.println("richtiges Ergebnis= " + richtigesErgebnis);
            }
            System.out.print("Weiter? (Y/N): ");
        } while ("y".equalsIgnoreCase(scan.next()));
        System.out.println("Programm Ende");
    }
}

 

[Fugover]

Okay, aber ihr müsst zugeben, dass mein Programm auch leichter zu verstehen ist.

 

[Staarfightaar]

Okay, aber ihr müsst zugeben, dass mein Programm auch leichter zu verstehen ist.

in welcher hinsicht?

 

[temi]

Okay, aber ihr müsst zugeben, dass mein Programm auch leichter zu verstehen ist.

Bezieht man sich auf die Punkte, die @Robert Zenz oben genannt hat, dann ist es schon leichter zu verstehen, wenn

• Variablen aussagekräftige Bezeichner haben
• keine unnötigen Variablen deklariert werden
• Variablen dort deklariert werden, wo sie benötigt werden (allein, weil sie sonst zunächst "für sich alleine stehen", der evtl. engere Gültigkeitsbereich ist ein weiterer Grund dafür)

Was nicht genannt wurde: Code ordentlich einrücken! (EDIT: wurde doch schon genannt )

 

[Robert Zenz]

Okay, aber ihr müsst zugeben, dass mein Programm auch leichter zu verstehen ist.

In Java gibt es haufenweise Konventionen die damals schon am Anfang (in weiser Vorraussicht vermutlich) von Sun vorgegeben wurden. Das hat dazu gefuehrt dass der meiste Java Code einfach gleich aussieht. Dein Code folgt ziemlich genau keinen davon (oder halt nur jenen welche vom Compiler vorgegeben werden). Das Programm von @httpdigest hingegen folgt dem Standard (ein paar mehr Leerzeilen waeren noch nett gewesen).

Dadurch das wir, als Java Programmierer, eine gewisse Erwartung haben wie Programme aussehen ist es relativ schwierig Code zu beurteilen welcher diesen Konventionen nicht folgt, weil er ist von Haus aus schon schwerer zu lesen fuer uns. Klar kann man es lesen, klar sind es nur 41 Zeilen, aber dennoch schwerer.

In Java haben wir die Moeglichkeit Lesbarkeit ueber alles andere zu stellen, wir muessen keinen Speicher verwalten, wir muessen nicht aufpassen wo Pointer hinzeigen, wir muessen uns (so gut wie gar nicht) um irgendwelche Resourcen kuemmern, wir muessen nicht auf Leistung von irgendwelchem Kleinkack achten, wir koennen wirklich Code schreiben welcher in erster Hinsicht auf maximale Lesbarkeit ausgerichtet ist. Und das sollte das Ziel von jedem von uns sein, und dazu gehoert eben auch das man sich an die etablierten Konventionen haelt (mal abgesehen davon existieren die ja aus einem Grund, und der war nicht das zwei Typen die Sprache ueber ein Wochenende zusammen gewuerfelt haben um etwas blinken zu lassen).

Oder um es direkt auszudruecken: Ist dein Code verstehbar? Ja, absolut. Ist er leicht und schnell verstehbar? Nein, gar nicht. Alleine schon durch die fehlende Einrueckung muss man sich den Ablauf erst zusammen suchen.

Was nicht genannt wurde: Code ordentlich einrücken! (EDIT: wurde doch schon genannt )

Da hast du bei mir ohnehin ganz schlechte Karten, das ist bei einem Code Review immer das Erste das ich sage.