Erste Schritte Parameterübergabe - Prozentrechner

[excuseme]

Hallo,

ich bin gerade überfragt warum mein Code nicht funktioniert. Vorne weg ich bin ein Anfänger, daher bitte Rücksicht. Ich soll von dem 1 Parameter den jeweiligen Prozentteil (2 Parameter) ausrechnen und ausgeben lassen. (Bsp: berechneProzent(180/10 = 18.0). Wenn einer der Parameter negativ ist, soll es -1.0 ausgeben, da habe ich auch ein bisschen getrickst, aber das Ergebnis ist ja dasselbe. Weiß aber auch nicht genau wie ich p1 und p2 mit der Methode verbinde. Beim ausführen terminated es aber. Danke für eure Hilfe!

import java.util.Scanner;
public class berechneProzent {

    public berechneProzent(int p1, int p2) {
        Scanner sc = new Scanner(System.in);
        System.out.println("Erster Parameter:");
        p1 = sc.nextInt(); //100 Prozent
        System.out.println("Zweiter Parameter:");
        p2 = sc.nextInt(); //entsprechnender Prozentteil
        double berechnung = p1 * (p2/100); 
       
        if (p1 > 0 && p2 >0) //Parameter sollen größer 0 sein
        System.out.println(berechnung);
       
        else { //andernfalls soll es -1.0 ausgeben
        double fehler = berechnung/berechnung;
        System.out.println("-" + fehler); }
            sc.close();
            }
        }

 

[Robat]

Kleiner Tipp für später: "Code funktioniert nicht" ist keine Fehlerbeschreibung

1. Die Methode berechneProzent(int,int) ist nicht dafür zuständig die Zahlen einzulesen. Die Werte bekommst du als Parameter übergeben. Das einlesen sollte woanders geschehen.
2. double berechnung = p1 * (p2/100); p1, p2 und 100 sind Integer d.h. hier wird auch mit Integer-Division gerechnet. p2 / 100 wird immer 0 sein wenn p2 < 100 ist. Daher ist dann p1 * 0 auch 0. Caste p2 oder die 100 zu einem double.

 

[excuseme]

Vielen Dank schonmal für den Denkanstoss. Okay, also muss ich die Werte in einer main Methode einlesen lassen, korrekt? Die Rechnung findet dann in meiner berechneProzent Methode statt. Und wie übergebe ich die eingelesenen Werte dann an meine Methode?
Bei dem zweiten Problem, kann ich doch das "nextInt" in einen "nextDouble" umändern

 

[Javinner]

Und wie übergebe ich die eingelesenen Werte dann an meine Methode?

In dem du deine Methode zu einer Rückgabe aufforderst. Oder ganz einfach ausgedrückt:
Wenn du willst, dass deine Methode ein Wert liefert, sollte die Methode von Wert sein
http://openbook.rheinwerk-verlag.de/javainsel9/javainsel_02_008.htm

 

[Robat]

Bei dem zweiten Problem, kann ich doch das "nextInt" in einen "nextDouble"

Das kommt auf die Aufgabenstellung an. Wenn p1 und p2 explizit ints sein sollen dann nicht. Dann muss gecastet werden

 

[temi]

Und wie übergebe ich die eingelesenen Werte dann an meine Methode?

Deine Methode hat doch zwei Parameter, dort musst du die eingelesenen Werte übergeben.