Methode- Fehlermeldung

[TomTom]

Hallo zusammen!

Wir haben gerade mit Methoden begonnen und ich wollte ein wenig üben.

Nun wollte ich eine Methode schreiben, die die kleinste Zahl ausliest und diese dann übergibt. Leider bekomme ich dabei eine Fehlermeldung, die mir nicht wirklich viel sagt.

static double Wertemin {double min (int z) {
	int min = 0;
	for (int i = 1; i < n; i++) {
		if (z[i] < z[min])
			min = i;
		return min;
		}
	}
}

Die Fehlermeldung lautet:

Syntax error on token "double", interface expected Wertemethoden.java line 1

Vielleicht könnte mir jemand hier erklären, wo der Fehler liegt bzw. was genau das Problem ist.

Habt Dank!

Tom

 

[PaiMei]

Hi,

es muss heißen

static double Wertemin (int z) {

 

[TomTom]

Danke für die fixe Antwort.

Leider behebt das mein Problem nicht, sondern erschafft nur viele neue Fehlermeldungen.

Gruß

 

[PaiMei]

Hi,

so genau habe ich mir deinen Code gar net angeschaut...da sind einige Fehler

1.) in der for schleife....n ist ja niergends deklariert und initialisiert;

2.) du übergibst der Methode Wertemin ja einen int Parameter ...z; dann kanst du z nacher nicht wie ein Array behandeln

z[i]

 

[TomTom]

Ah, ok- da steckt der Fehler also.

Hab den Code nun so umformuliert

static double Wertemin {double min (int n, double z[]) {
	int min = 0;
	for (int i = 1; i < n; i++) {
		if (z[i] < z[min])
			min = i;
		return min;
		}
	}
}
	
static double Wertemax {double max(int n, double z[] ) {
	int max = 0;
	for (int i = 1; i < n; i++) {
		if (z[i] > z[max])
			max = i;
		return max;
		}
	}
}

Leider meckert er immer noch, jetzt heißt es:

Syntax error on token "double", class expected Wertemethoden.java line 2/ 12

 

[PaiMei]

du erstellst den Methodenkopf falsch.

er muss folgendemassen aufgebaut sein

static "Typ des Rückgabewertes" "Methodenname" ("Paramater") {

dh.

static double Wertemin (int n, double z[]) {

double ist dein Rückgabetyp;
Wertemin dein Methodenname;
int n und double []z deine Parameter;

wieso du das schreibst

{double min

.... versteh ich nicht und ist auch falsch

 

[Guest]

1. Du willst n double zurück geben, hast aber nur int-Werte
2. Die Übergabe von n ist vollkommen unnötig, die Anzahl der Schleifendürchläufte wird durch die Länge von z[] definiert
3. Dein return ist viel zu früh. Selbst wenn dein int[] 1000 Werte hat wird immer der zweite oder 0 zurück gegeben.
4. min zu Beginn auf 0 setzen ist sehr ungünstig da das automatisch schon die kleinstmögliche nicht negative Zahl ist
5. Beim Durchgehen eines Arrays beginnt der Zähler bei 0 nicht bei 1, außer du machst es wie ich in meinem Beispiel unten und setzt den ersten Wert schon vor der Schleife
6. Dein Vergleich z < z[min] kann zu einer Exception führen da min kein index für das Array ist, sondern ein Wert aus dem Array, und damit kann er die grenze des Arrays ggf. schnell erreichen

Hier mal n kleines Beispiel

static int werteMin (int[] z) {
    int iMin;
    if (z != null && z.length > 0) {
        iMin = z[0];
    } else {
        // Fehlerausgabe
    }
    for (int i = 1; i < z.length; i++) {
        if (z[i] < iMin) {
            iMin = z[i];
        }
    }
    return iMin;
}

 

[Guest]

Also erstmal sind die Anmerkungen von PaiMei antürlich richtig!

Dann noch welche von mir:

a) n als Übergabeparameter brauchst du nicht, die Anzahl der Schleifendurchgänge definiert sich durch die Länge von z[]
b) Dein return kommt zu früh und du würdest schon eine Zahl zurück geben, die u. U. noch nicht das Minimum oder Maximum ist
c) Beim return darfst du dann nicht den Index des Wertes zurück geben, sondern musst schon den Wert selbst zurück geben (außer du arbeitest dann nur mit dem index weiter, dann würde allerdings auch int als Rückgabewert reichen)

static double Wertemin (double z[]) {     
    int min = 0; 
    for (int i = 1; i < z.length; i++) { 
        if (z[i] < z[min])  {
             min = i; 
        }
    }
    return z[min];
}

 

[TomTom]

Wow, tausend Dank für die lieben und vor allem sehr hilfreichen Ratschläge.

Habe nun nochmals das Ganze alleine programmiert, nachdem ich meine Fehler gelesen und korrigiert hatte und - o Wunder- ich verstehe es

Nun habe ich also meine Methode bzw. einige andere Methoden- fertig und will sie in der class Werte aufrufen bzw. sie ausgeben.

Kann ich dies einfach über

"Die kleinste Zahl ist: "  
				+ Wertemethoden.min(imin))

machen?

 

[Guest]

Wow, tausend Dank für die lieben und vor allem sehr hilfreichen Ratschläge.

Habe nun nochmals das Ganze alleine programmiert, nachdem ich meine Fehler gelesen und korrigiert hatte und - o Wunder- ich verstehe es

Nun habe ich also meine Methode bzw. einige andere Methoden- fertig und will sie in der class Werte aufrufen bzw. sie ausgeben.

Kann ich dies einfach über

"Die kleinste Zahl ist: "  
				+ Wertemethoden.min(imin))

machen?

So ungefähr sollte das gehen. Ich nehme jetzt mal an, dass Wertemethoden die Klasse ist, inder deine ganzen Static-Methode (wie eben die wertemin) drin sind und in deinem Beispiel imin dann ein doblue[] oder int[] ist. Wenn das alles so ist sollte auch deine Ausgabe so gehen! :bae: