Fehler bei Vergleich auf den grössten Wert

[-jojo0714-]

Hallo,
ich sitze schon seit Tagen vor einer Klasse und komme nicht weiter.
Die Klasse soll mir die beste und schlechteste Note sowie den Mittelwert aller Noten ermittelt. Die Ermittlung des Mittelwertes funktioniert soweit . Mein Problem ist allerdings das beim vergleich auf die schlechteste Note bei der Methode void WorstVergleichDrei() keine mehr ausgegeben wird. Woran könnte es liegen? Wie kann ich es lösen? Hab ich einfach nur vergessen was einzutippen???
Hier die Kasse und schonmal Danke im Vorraus

package uebung08;

import javax.swing.JOptionPane;

public class Mittelwert {
    int anzahlWerte;    // Variablendeklaration
    double wertEins;
    double wertZwei;
    double wertDrei;
    double wertVier;
    double wertFünf;
    double mittelwert;
    double besteNote;
    double schlechtesteNote;
    double verVaEins;    // For comparing two  double variables with the same value
    double verVaZwei;
    double verVaDrei;
    double verVaVier;
    double verVaFünf;
   
    void GleichheitZwei(){
        if(anzahlWerte == 2){
       
            if (wertEins == wertZwei){
            verVaEins = wertEins;
            verVaZwei = 0;
            System.out.println("[@Mittelwert GleichheitZwei]: Gleichheit gefunden!");
            }
            System.out.println("[@Mittelwert GleichheitZwei]: Gleichheitsprüfung abgeschlossen!");
        }
    }

    double BerechnenFuenf(){
     mittelwert = (wertEins + wertZwei + wertDrei + wertVier + wertFünf)/anzahlWerte; // method for returning the intersection
        return mittelwert;
    }
   
    double BerechnenVier (){
         mittelwert = (wertEins + wertZwei + wertDrei + wertVier)/anzahlWerte;
        return mittelwert;
    }
   
    double BerechnenDrei(){
         mittelwert = (wertEins + wertZwei + wertDrei)/anzahlWerte;
        return mittelwert;
    }
   
    double BerechnenZwei(){
         mittelwert = (wertEins + wertZwei)/anzahlWerte;
        return mittelwert;
    }
   
    void BestVergleichZwei(){                //Methoden um die beste Note zu ermitteln
        if(wertEins > wertZwei){
            besteNote = wertZwei;
            System.out.println("[@Mittelwert VergleichZwei]: Vergleich erfolgreich." );
        }else{
            besteNote = wertEins;
            System.out.println("[@Mittelwert VergleichZwei]: Vergleich erfolgreich." );
        }
    }
   
    void BestVergleichDrei(){
        if(wertDrei > wertEins){
            besteNote = wertEins;
            System.out.println("[@Mittelwert VergleichDrei]: Vergleich erfolgreich." );
        }else{
            if(wertDrei > wertZwei){
                besteNote = wertZwei;
                System.out.println("[@Mittelwert VergleichDrei]: Vergleich erfolgreich." );
            }else{
                besteNote = wertDrei;
                System.out.println("[@Mittelwert VergleichDrei]: {Drei} Vergleich erfolgreich." );
            }
        }
    }
   
    void BestVergleichVier(){
        if (wertVier > wertEins){
            besteNote = wertEins;
            System.out.println("[@Mittelwert VergleichVier]: Vergleich erfolgreich." );
        }else{
            if (wertVier > wertZwei){
                besteNote = wertZwei;
                System.out.println("[@Mittelwert VergleichVier]: Vergleich erfolgreich." );
            }else{
                if(wertVier > wertDrei){
                    besteNote = wertDrei;
                    System.out.println("[@Mittelwert VergleichVier]: Vergleich erfolgreich." );
                }else{
                    besteNote = wertVier;
                    System.out.println("[@Mittelwert VergleichVier]: Vergleich erfolgreich." );
                }
            }
        }
    }
   
    void BestVergleichFünf(){
        if (wertVier > wertEins){
            besteNote = wertEins;
            System.out.println("[@Mittelwert VergleichFünf]: Vergleich erfolgreich." );
        }else{
            if (wertVier > wertZwei){
                besteNote = wertZwei;
                System.out.println("[@Mittelwert VergleichFünf]: Vergleich erfolgreich." );
            }else{
                if(wertVier > wertDrei){
                    besteNote = wertDrei;
                    System.out.println("[@Mittelwert VergleichFünf]: Vergleich erfolgreich." );
                }else{
                    if(wertFünf > wertVier){
                        besteNote =wertVier;
                        System.out.println("[@Mittelwert VergleichFünf]: Vergleich erfolgreich." );
                    }else{
                        besteNote = wertFünf;
                        System.out.println("[@Mittelwert VergleichFünf]: Vergleich erfolgreich." );
                    }
                     
                }
            }
        }
    }
   
    void WorstVergleichZwei(){        //Methoden um die schlechteste Note zu ermitteln
        if(wertEins > wertZwei ){
            schlechtesteNote = wertEins;
            System.out.println("[@Mittelwert WorstVergleichZwei]: Vergleich erfolgreich.");
        }else{
            schlechtesteNote = wertZwei;
            System.out.println("[@Mittelwert WorstVergleichZwei]: Vergleich erfolgreich.");
        }
    }
   
    void WorstVergleichDrei(){
       
                    if(wertEins > wertZwei && wertEins > wertDrei){
                        schlechtesteNote = wertEins;
                        System.out.println("[@Mittelwert WorstVergleichDrei]: {@wertE} Vergleich erfolgreich!");
                    }else{
                        if(wertZwei > wertEins  && wertZwei > wertDrei){
                            schlechtesteNote = wertZwei;
                            System.out.println("[@Mittelwert WorstVergleichDrei]: {@wertZ} Vergleich erfolgreich!");
                        }else{
                            if(wertDrei > wertEins && wertDrei > wertZwei){
                                schlechtesteNote = wertDrei;
                                System.out.println("[@Mittelwert WorstVergleichDrei]: {@wertE} Vergleich erfolgreich!");
                            }
                        }
                    }
                }
   
           
       

   
    void WorstVergleichVier(){
        if(wertVier>wertDrei && wertVier > wertZwei && wertVier > wertEins){
            schlechtesteNote = wertVier;
            System.out.println("[@Mittelwert WorstVergleichVier]: Vergleich erfolgreich.");
        }else{
            if(wertDrei > wertVier && wertDrei > wertZwei && wertDrei > wertEins){
                schlechtesteNote  = wertDrei;
                System.out.println("[@Mittelwert WorstVergleichVier]: Vergleich erfolgreich.");
            }else{
                if(wertZwei > wertVier && wertZwei > wertDrei && wertZwei> wertEins){
                    schlechtesteNote = wertZwei;
                    System.out.println("[@Mittelwert WorstVergleichVier]: Vergleich erfolgreich.");
                }else{
                    if(wertEins > wertVier && wertEins > wertDrei && wertEins >wertZwei){
                        schlechtesteNote = wertEins;
                        System.out.println("[@Mittelwert WorstVergleichVier]: Vergleich erfolgreich.");
                    }else{
                        System.out.println("[@Mittelwert WorstVergleichVier]): FEHLER! Kein Vergleich möglich");
                    }
                }
            }
        }
       
    }
   
    void WorstVergleichFünf(){
       
    }

    public void CalculateAll (){
        if(anzahlWerte == 1){
            System.out.println("[Mittelwert]: Zu wenig Werte");
            JOptionPane.showMessageDialog (null, "Zu wenig Werte" + " "
                        + anzahlWerte);   
        }else{
            if(anzahlWerte == 2){
                BerechnenZwei();
                BestVergleichZwei();
                WorstVergleichZwei();
                System.out.println("[@Mittelwert CalculateAll]: Berechnung erfolgreich.");
            }else{
                if(anzahlWerte == 3){
                    BerechnenDrei();
                     BestVergleichDrei();
                     WorstVergleichDrei();
                     System.out.println("[@Mittelwert CalculateAll]: Berechnung erfolgreich.");
                }else{
                    if(anzahlWerte == 4){
                    BerechnenVier();
                    BestVergleichVier();
                    WorstVergleichVier();
                    System.out.println("[@Mittelwert CalculateAll]: Berechnung erfolgreich.");
                }else{
                    if(anzahlWerte == 5){
                        BerechnenFuenf();
                        BestVergleichFünf();
                        WorstVergleichFünf();
                        System.out.println("[@Mittelwert CalculateAll]: Berechnung erfolgreich.");
                    }
                }
                   
                }
            }
               
        }
        System.out.println("[@Mittelwert CalculateAll]: Alle Prüfungen wurden durchlaufen!");
    }

    void MittelwertAusgeben(){
        System.out.println(mittelwert);
    }
}

 

[Kababär]

Heiliges Quadrat.. wieso machst du das so kompliziert und unübersichtlich? Der Code besteht ja nur aus ineinandergeschachtelten if-else.
Ist das Absicht?

Wie sind denn deine Noten abgespeicherg? In einer Liste, Array?
Wäre auf jeden Fall angebrachter, denn dann brauchst du nicht so viele unübersichtliche Methoden und vor allem brauchst du zum Finden der schlechtesten Note (Max-Wert) oder besten Note (Min-Wert). Also eigentlich nur drei Methoden getBestGrade(), getWorsrGrade() und getGradeAverage().
Das hat zudem den Vorteil, dass du nichts ändern musst am Code, falls du mal 20 Noten hast oder sogar 70, 127 oder 2012.

Zu deiner Frage:
Ich kenne die Noten nicht aber theoretisch ist es möglich, dass tatsächlich nichts ausgegeben wird. Mach mal ein System.out.println ans Ende der Methode, um herauszufinden, ob sie überhaupt aufgerufen/durchlaufen wird.

 

[InfectedBytes]

Ich kann @Kababär nur zu stimmen, komplizierter geht es kaum. Du solltest dich mal dringend mit Arrays beschäftigen, denn das was du dort versuchst, geht eigentlich locker in wenigen Zeilen. Hier mal beispielhaft für das Maximum:

double max = noten[0]; // wobei noten ein double array ist
for(int i=1;i<noten.length;i++) {
  if(noten[i] > max) max = noten[i];
}