grundlegende Fragen

[Hasenherz]

Hi zusammen,

ich habe vor einigen Tagen angefangen Java mithilfe eines Tutorials (Video2Brain) zu lernen. Früher (vor 30 Jahren!) habe ich Basic und Assembler (Schneider CPC) programmiert, jetzt im gediegeneren Alter (hust) reizt mich das Neue, auch wenn ich noch nicht allzu weit gekommen bin.

Ich habe mich an einem kleinen Testprogramm versucht, welches (meine Kinder mögen Minecraft) berechnet, wieviele Blöcke man für eine Pyramide mit variabler Kantenlänge benötigt. (Pyramide = Vollkörper)

Wenn ich in meinem Code Variablen definiere und Ihnen der reinen Übersicht halber direkt während der Definition Werte zuweise, welche aber im Programm nicht genutzt werden, makelt NetBeans daran herum und meint, dieser zugewiesene Wert werde nie genutzt.
Ist das schlimm (mit Blick auf 'größere' Programme) oder kann ich es getrost drinlassen (Zeilen 8-11)?

Ich habe hier im Forum in einem alten Thread eine Methode zur Eingabe einer Zahl gefunden (und die von Arthur177 geforderte Lösung bei Falscheingabe hoffentlich elegant gelöst). Ist diese Methode (Zeilen 12-15) noch aktuell oder liegt schon Staub drauf?

Das mag in den Augen Vieler trivial aussehen, aber da noch alles ziemlich neu ist beschäftigt mich das schon noch ein wenig.

import java.util.Scanner;

public class Pyramide {

    public static void main(String[] args) {

        int kanteLang = 1;
        int kanteBreit = 1;
        int kanteHoch = kanteBreit / 2;
        int blöckeGesamt = 0;
        Scanner kante;

        System.out.println("Bitte die Seitenlänge deiner Pyramide angeben: ");
        kante = new Scanner(System.in);
        if (kante.hasNextInt()) {
            kanteLang = kanteBreit = kante.nextInt();
        } else {
            System.out.println("fehlerhafte Eingabe!");
            System.exit(0);
        }

        kanteHoch = (kanteBreit + 1) / 2;

        System.out.println("Die Pyramide ist " + kanteLang + " Blöcke breit und " 
                + kanteHoch + " Ebenen hoch.");
        System.out.println("");

        for (int i = 0; kanteHoch > i; i++) {
            System.out.println("Die " + (i + 1) + ". Ebene besteht aus " +
                    ((kanteBreit - i - i) * (kanteLang - i - i)) + " Blöcken");

            blöckeGesamt = blöckeGesamt + ((kanteBreit - i - i) * (kanteLang - i - i));
        }

        System.out.println("");
        System.out.println("Die Pyramide besteht aus insgesamt " + blöckeGesamt + " Blöcken.");
    }
}

 

[fLooojava]

Ich bin zwar auch noch nicht vertieft in Java, jedoch hätte ich es genau gleich gemacht wie du.

Zu den Variablen - denen du direkt einen Wert zuweist: Ich nehme mal an du machst es für dich - damit du dich im Code "besser auskennst". Wieso dann nicht einfach kleine Kommentare, wenn die Werte keine Rolle spielen?

Das mit dem Scanner passt auch, solange du keine Oberfläche haben möchtest. Dann sieht das Auslesen natürlich anderst aus.

Die Abfrage ob es ein Integer ist oder nicht passt auch, schließlich macht sie das was sie machen soll .

Was du noch machen könntest ist bei einer falschen Eingabe einfach die Abfrage solange wiederholen bis eine richtige Eingabe dem Scanner geliefert wird.

 

[kaoZ]

Mal davon Abgesehen das du nie ein Pyramiden Objekt erzeugst und die Klasse einfach nur als Einstiegspunkt dient (main Methode)

Normalerweise hält genau dieses Objekt alle Informationen über sich selbst. ( also Datenfelder , auch Attribute genannt, wie u.B Höhe, Kantenlänge usw. usw.)

und du würdest z.B über einen Scanner ( in der main Methode ) nutzen, um die Daten für genau dieses Objekt einzulesen und das Objekt damit zu initialisieren.

 

[Hasenherz]

Ok. Ich habe jetzt eine Klasse Pyramide, in der Pyramiden-Objekte gebaut werden können.
Dafür werden aus der Main-Methode beim Erstellen Breite und Höhe mitgegeben.

Ich habe die Abfrage nach der richtigen/falschen Eingabe geändert, so dass solange gefragt wird, bis eine Zahl eingegeben wurde.

Ganz langsam wächst das Verständnis ^^

edit: Ich habe noch eine Kleinigkeit geändert doch sehe jetzt, dass ich mich mit den ganzen Variablen irgendwie verhaspelt habe :/


Main-Methode

import java.util.Scanner;

public class PyramideBauen {

    public static void main(String[] args) {

        int kanteLang = 1, kanteBreit = 1, kanteHoch = 1, flaecheInsg = 0;
        Scanner kante;
        boolean eingabe = false;

        while (!eingabe) {
            System.out.print("Bitte die Seitenlänge deiner Minecraft-Pyramide angeben: ");
            kante = new Scanner(System.in);
            if (kante.hasNextInt()) {
                kanteLang = kanteBreit = kante.nextInt();
                kanteHoch = (kanteBreit + 1) / 2;
                eingabe = true;
            } else {
                System.out.println("Fehlerhafte Eingabe! Bitte eine Zahl eingeben.");
                eingabe = false;
                //System.exit(0);
            }
        }
        Pyramide neuePyramide = new Pyramide(kanteLang, kanteHoch);
        System.out.println(neuePyramide.getStatus());

        neuePyramide.volumenRechnen(kanteLang);

    }

}

Pyramiden-Klasse

public class Pyramide {

    private int seiteL, hoeheH, gesamtB;

    public Pyramide(int seiteL, int hoeheH) {
        this.seiteL = seiteL;
        this.hoeheH = hoeheH;
    }

    public void flaecheRechnen(int seiteL) {
        gesamtB = (seiteL * seiteL);
        System.out.println("Die Fläche der Ebene ist: " + gesamtB);
    }

    public int getFlaeche() {
        return gesamtB;
    }

    public void volumenRechnen(int seiteL) {
        int flaecheInsg = 0, kanteLang = seiteL, kanteHoch = (seiteL + 1) / 2;

        for (int i = 0; i < kanteHoch; i++) {
            flaecheRechnen(kanteLang - i);
            flaecheInsg = flaecheInsg + getFlaeche();
            kanteLang = kanteLang - 1;
        }
        System.out.println("Die Gesamtzahl an Blöcken beträgt: " + flaecheInsg + " Blöcke.");

    }

    public String getStatus() {
        return "Die Pyramide ist " + seiteL + " Blöcke breit und " + hoeheH + " Ebenen hoch.";
    }

}