Die Werte zur Berechnung müssen als Parameter übergeben werden?

[AClassyWarius]

Hallo, hier die Aufgabenstellung:

Gegeben: Ein Film mit einer Anzahl Toten, Anzahl Schwerstverletzte und einer Laufzeit in Minuten.

Gesucht: Ausgabe ob der Film angesehen werden soll oder nicht. Im letzten Fall mit Gru ̈nden. Es gibt zwei Gru ̈nde einen Film nicht anzusehen:

1. Der Film ist zu brutal: es gibt mehr als 7 Tote oder mehr als 127 Schwerstverletzte. 2. Der Film ist zu lang: la ̈nger als 118,4 Minuten.

Implementieren Sie drei Funktionen, die jeweils berechnen, ob man den Film anschauen soll, ein Film zu lang oder zu brutal ist. Der Ru ̈ckgabedatentyp dieser Funktionen ist boolean.

Implementieren Sie eine Klassenmethode, die auf dem Bildschirm ausgibt, ob man den Film ansehen sollte oder nicht und im letzten Fall einen oder beide der Gru ̈nde ausgibt.

Drei Ausgabebeispiele:

Film ansehen.
Film nicht ansehen.
Zu brutal.
Zu lang.
Film nicht ansehen.
Zu lang.
Alle zutreffenden Gru ̈nde mu ̈ssen in obiger Reihenfolge ausgegeben werden. Die Werte zur Berechnung müssen als Parameter übergeben werden.

import java.util.Scanner;

public class FilmAuswahl {
    
    public static boolean FilmZuBrutal(int anzahlToten, int anzahlSchwerstverletzte )    {
        return (anzahlToten > 7) | (anzahlSchwerstverletzte > 127);
        
    }
    
    public static boolean FilmZuLang(double laufzeitMinuten)    {
        return (laufzeitMinuten > 118.4);

    }        
    
    public static boolean FilmAnschauen(boolean FilmZuLang, boolean FilmZuBrutal)    {
        return (!FilmZuLang) & (!FilmZuBrutal);
    
    }    
    
    public static void main(String[] args)    {
        
        System.out.println("Ein Film mit einer Anzahl Toten, Anzahl Schwerstverletzte und einer Laufzeit in Minuten");
        
        Scanner s = new Scanner(System.in);
        Scanner d = new Scanner(System.in);
        Scanner f = new Scanner(System.in);
        
        int inputAnzahlToten = s.nextInt();
        int inputAnzahlSchwerstverletzte = d.nextInt();
        double inputLaufzeitMinuten = f.nextDouble();
        
        if ( FilmAnschauen(FilmZuLang(inputLaufzeitMinuten) , FilmZuBrutal(inputAnzahlToten, inputAnzahlSchwerstverletzte)))    {
            System.out.println("Film ansehen.");
        }
        else    
            System.out.println("Film nicht ansehen.");
                if (FilmZuBrutal(inputAnzahlToten, inputAnzahlSchwerstverletzte))                                     {
                    System.out.println("Zu brutal.");
                }
                if (FilmZuLang(inputLaufzeitMinuten))                                                      {
                    System.out.println("Zu lang.");        
                }
    
    }
}

so jetzt zu meiner Frage, mir geht es eigentlich um die Eingabe der Scanner-Werte in die drei Funktionen. Gibt es eine Möglichkeit die Einbindung der Scanner-Parameter besser/einfacher zu gestalten? Das Programm läuft aber ich bin mir nicht sicher ob ich die die letzte Bedingung: ,,Die Werte zur Berechnung müssen als Parameter übergeben werden."" erfülle.

Mit freundlichen Grüßen Warius

 

[JStein52]

ob ich die die letzte Bedingung: ,,Die Werte zur Berechnung müssen als Parameter übergeben werden."" erfülle

Die erfüllst du schon. Aber wozu bitte brauchst du denn drei Scanner-Objekte ?

 

[AClassyWarius]

für meine drei Funktionswerte? Ich wüsste nicht wie ich mit nur einem Scanner-Objekt 3 Variablen in meine Funktion einfügen kann

 

[Cromewell]

Du kannst mit einem Scanner so oft du willst einlesen Der geht nicht kaputt ^.^

 

[AClassyWarius]

ahhhh ok geändert, danke. Des Weiteren sollen wir darauf achten Redundanz zu vermeiden. Fällt euch Redundanz im Programm auf?

 

[JCODA]

ahhhh ok geändert, danke. Des Weiteren sollen wir darauf achten Redundanz zu vermeiden. Fällt euch Redundanz im Programm auf?

Du rufst mehrfach diese Methoden "FilmZuBrutal(inputAnzahlToten, inputAnzahlSchwerstverletzte)" auf. Du könntest dir den Wert in eine Variable speichern und nur noch dieses Variable benutzen.

 

[AClassyWarius]

    import java.util.Scanner;

public class FilmAuswahl {
   
    public static boolean filmZuBrutal(int anzahlToten, int anzahlSchwerstverletzte )    {
        return (anzahlToten > 7) | (anzahlSchwerstverletzte > 127);
       
    }
   
    public static boolean filmZuLang(double laufzeitMinuten)    {
        return (laufzeitMinuten > 118.4);

    }       
   
    public static boolean filmAnschauen(boolean FilmZuLang, boolean FilmZuBrutal)    {
        return (!FilmZuLang) & (!FilmZuBrutal);
   
    }   
   
    public static void main(String[] args)    {
       
        System.out.println("Ein Film mit einer Anzahl Toten, Anzahl Schwerstverletzte und einer Laufzeit in Minuten");
       
        Scanner sc = new Scanner(System.in);
       
        int inputAnzahlToten = sc.nextInt();
        int inputAnzahlSchwerstverletzte = sc.nextInt();
        double inputLaufzeitMinuten = sc.nextDouble();
       
        boolean gewaltBestimmung = filmZuBrutal(inputAnzahlToten, inputAnzahlSchwerstverletzte);
       
        if (filmAnschauen(filmZuLang(inputLaufzeitMinuten) , gewaltBestimmung))    {
            System.out.println("Film ansehen.");
        }
        else   
            System.out.println("Film nicht ansehen.");
                if (gewaltBestimmung)                                     {
                    System.out.println("Zu brutal.");
                }
                if (filmZuLang(inputLaufzeitMinuten))                                                      {
                    System.out.println("Zu lang.");       
                }
   
    }
}

so hab es nochmal verbessert

 

[Cromewell]

Mach es doch dann auch noch für "filmZuLang(...)", denn das rufst du auch zweimal auf.