Körper Brechnung - Lokale Variablen in Methoden übergeben

[theqwe]

Hallo zusammen,

wir haben folgende Aufgabe bekommen: Ein Nutzer soll für ein Zylinder den Radius und die Höhe eingeben. Danach werden Umfang, Oberfläche, Mantelfläche, Deckenfläche und Volumen berechnet. Jede Rechnung soll eine eigene Methode besitzen und aus der main-Methode aufgerufen werden.
Soweit habe ich das auch hinbekommen. Aber da ich das ganze effizient gestalten möchte, kann ich für manche Berechnungen die Werte ja übernehmen, z.B. ergibt sich die Mantelfläche aus dem Umfang * Höhe. So möchte ich gerne den Wert von Umfang übernehmen und in der Methode benutzen. Sonst müsste ich ja die komplette Rechnung wieder eingeben.

Wie nehme ich den Wert von der Umfang-Rechnung und nutze ihn für die Mantelfläche-Rechnung?

package VL03;

import java.util.Scanner;
import java.math.*;

public class Dosenberechnung {

    public static void main(String[] args) {
      
        Scanner eingabe = new Scanner(System.in);
        System.out.print("Bitte geben Sie den Radius ein: ");
        System.out.println();
        double radius = eingabe.nextDouble();
        System.out.print("Bitte geben Sie jetzt die Höhe ein: ");
        double höhe = eingabe.nextDouble();
      
        berechneUmfang(radius, höhe);
        berechneDeckenfläche(radius, höhe);
        berechneMantelfläche(radius, höhe);
        berechneOberfläche(radius, höhe);
        berechneVolumen(radius, höhe);

    }
  
    public static void berechneUmfang(double radius, double höhe) {

        double umfang = (2 * Math.PI) * radius;
        System.out.println("Der Umfang beträgt: " + umfang);
    }
  
    public static void berechneDeckenfläche(double radius, double höhe) {

        System.out.println("Die Deckelfläche beträgt: " + Math.PI * radius);
    }
  
    public static void berechneMantelfläche(double radius, double höhe) {

        System.out.println("Die Mantelfläche beträgt: " +  ); // Hier möchte ich die umfang Variable nutzen
    }
  
    public static void berechneOberfläche(double radius, double höhe) {

    }
  
    public static void berechneVolumen(double radius, double höhe) {

    }
}

Ich danke euch im Voraus

 

[mrBrown]

Du kannst in berechneMantelfläche einfach berechneUmfang aufrufen (wenn es dir um das Vermeiden von doppeltem Code ging.)

 

[theqwe]

Du kannst in berechneMantelfläche einfach berechneUmfang[/icode aufrufen] (wenn es dir um das Vermeiden von doppeltem Code ging.) :)

Hi danke für die schnelle Antwort. Wir sollen diese Struktur einhalten. Also jede Berechnung eine Methode und alle Rechnungen aus der main-Methode aufrufen. Ich möchte nur egen der Effizienz die schon vorhandenen Rechnung für die anderen Rechnungen nutzen. Sonst müsste ich bei der Mantelfläche wieder die Formel für den Umfang eingeben.

 

[mrBrown]

Das widerspricht meiner Antwort doch gar nicht

Wenn es dir um "schnellere" Ausführung geht: dafür müsstest du entweder die Struktur zerstören (was du ja nicht darfst), oder statische Variablen nutzen (was du nicht solltest weil's hässlich ist).

 

[theqwe]

Ok, das habe ich dann wohl nicht ganz verstanden. Sind erst bei der 3. Vorlesung und kenne noch nicht so viele Sachen. Wie würde das mit statischen Variablen aussehen? Wir dürfen/sollen natürlich im Moment nur das verwenden, was wir schon gelernt haben

 

[mihe7]

1. Deine Methoden machen zu viel, d. h. Dinge, sie enthalten Dinge, die dort nicht hingehören. Konkret: eine Methode berechneXXX soll etwas berechnen, aber nicht ausgeben
2. Wenn sie nichts ausgeben sollen, dann werden sie etwas zurückgeben müssen.
3. Die Rückgabewerte kannst Du dann für die Ausgabe und/oder weitere Berechnungen verwenden.

Deine Methoden würden sich dann ändern zu (beispielhaft):

public static double berechneUmfang(double radius, double höhe) {
    return (2 * Math.PI) * radius;
}

und in main

double umfang = berechneUmfang(radius, höhe);
System.out.println("Der Umfang beträgt: " + umfang);

double mantelflaeche = berechneMantelfläche(umfang, höhe); // Methode muss dann natürlich den Umfang und nicht den Radius erwarten
System.out.println("Die Mantelfläche beträgt: " + mantelflaeche);

 

[mrBrown]

1. Deine Methoden machen zu viel, d. h. Dinge, sie enthalten Dinge, die dort nicht hingehören. Konkret: eine Methode berechneXXX soll etwas berechnen, aber nicht ausgeben

Oh, noch nicht mal 10 und ich bekomm trotzdem nichts mehr auf die Reihe... bis grad war ich der festen Überzeugung, dass die Methoden bereits die Sachen zurückgeben -.-

 

[theqwe]

Ok, vielen Dank. Dann versuche ich das mal so und hoffe, dass es dem Prof passt
Ich kann mich am Freitag nochmal melden und das Ergebnis mitteilen.

 

[mihe7]

hoffe, dass es dem Prof passt

Das kannst Du besser beurteilen als jeder andere hier. Ich würde sagen: wenn ihr Rückgabewerte bereits behandelt habt, gehe ich stark davon aus, dass es das ist, was er sehen will

 

[theqwe]

Wir hatten return zum Schluss kurz angekratzt. Ich habe es jetzt so übernommen und es funktioniert sehr gut. Eigentlich soll man beim programmieren ja seine Freiheiten haben. Hauptsache man kommt zum Ziel.
Ich werde mich morgen melden und bescheid geben, ob alles nach seinen Wünschen war

 

[M.L.]

Eigentlich soll man beim programmieren ja seine Freiheiten haben.

Kleine Randbemerkung: die Programmiersprache Perl bietet so etwas (Motto: "There is more than one way to do it "). U.a. mit dem Nebeneffekt, dass es mehrere Wege zum Ziel geben kann oder man sich gründlich verlaufen kann.

Hauptsache man kommt zum Ziel.

Die Landkarte inkl. Abkürzungen und offizieller Wege sollte man trotzdem kennen, siehe das Openbook zu return, static