Ich weiß, wo der Fehler ist, kann ihn aber nicht beheben

[Xeopos]

Hi,
ich hab ein Problem mit der unten aufgeführten Methode. Zum Hintergrund: Kunden haben mehrere Konten, von diesen wird das, mit dem höchsten Betrag, mit Hlfe von getKontoMitHoechstemBetrag(), ermittelt, das funktioniert so weit auch. Nun soll der Kunde bestimmt werden, welcher insgesamt den höchsten Betrag auf einem seiner Konten hat.

	public Kunde gibKundenMitHoechstemBetragAus(){
		Kunde bigktmp = null;
		Kunde ktmp1 = null;
		Kunde ktmp2 = null;
		for(int i=0; i<kundenListe.size()-1;i++){
			ktmp1 = kundenListe.get(i);
			TermingeldKonto big1 = ktmp1.getKontoMitHoechstemBetrag();
				for(int j=0;j<kundenListe.size();j++){
					ktmp2 = kundenListe.get(j);
					TermingeldKonto big2 = ktmp2.getKontoMitHoechstemBetrag();
					if(big1!=null&&big2!=null){
					int test = big2.compareTo(big1);
					if(test == 1){
						bigktmp = ktmp2;
					}else{
						bigktmp = ktmp1;
					}
				}
				}
		}
		System.out.println("Kundennummer: " + bigktmp.getKundennummer() + " Name: " + bigktmp.getName() 
				+ " Kontonummer: " + bigktmp.getKontoMitHoechstemBetrag().getKnummer() + " Saldo: " 
				+ df.format(bigktmp.getKontoMitHoechstemBetrag().getBetrag()) + "\n");
		return bigktmp

So wie es oben steht, bekomme auf die Konsole nur die letzten beiden Einträge verglichen, soweit klar, da sie zum Ende der For-schleife verglichen werden, und einer der beiden höher ist, aber wie bekomme ich es hin das die zuvor verglichenen Kunden und deren Konten, in der Variablen erhalten bleibt, wenn sie denn einen höheren Betrag aufweisen

 

[eRaaaa]

Wieso vergleichst du nicht einfach immer aktuelles Konto mit größtem Konto?

Kunde bigktmp = kundenListe.get(0);
Schleife von 1 bis n
Vergleiche Konto von Kunde i mit Konto von bigktmp (wenn größer, setze bigktmp einfach auf aktuellen Kunde, wenn nicht, weiter in Schleife ...)

/edit:

	Kunde bigktmp = kundenListe.get(0);
	for (int i = 1; i < kundenListe.size(); i++) {
	    Kunde aktuell = kundenListe.get(i);
	    if (aktuell.getKontoMitHoechstemBetrag().compareTo(bigktmp.getKontoMitHoechstemBetrag()) > 0) { //vllt auch immer noch das Konto merken, damit nicht jedes mal getKontoMitHoechstemBetrag aufgerufen werdne muss
		bigktmp = aktuell;
	    }
	}
	return bigktmp;

p.s.: netter Thementitel übrigens

 

[Xeopos]

Danke, jetzt funzt es, wie es soll.