Arrays min- maxWerte bestimmen

[Fkglr]

Hallo Liebe Programmier Community,

ich habe von der Uni die Aufgabe bekommen, eine Methode zu schreiben, die den kleinsten und größten Wert, sowie den Durchschnittswert eines Arrays, der gefüllt ist mit Zufallszahlen, auszugeben. Jetzt hab ich es mit folgendem Code probiert:

public static int minWert (int[] zahlen) {

        int minArray = 0;
      
        for (int i = 0; i < zahlen.length - 1; i++) {
            if ( zahlen[I][I] < zahlen[i+1] ) {
                minArray = zahlen[I];
            }
            else {
                minArray = zahlen[i+1];
            }
        }
        return minArray;
    }

Allerdings bekomme ich als return-Wert immer nur eine 0 raus.
Für den maxWert hab ich den gleichen Code, nur mit anderen Operatoren natürlich, benutzt. Wo liegt der Fehler?
Danke im vorraus

MfG[/I][/I][/I]

 

[Yaslaw]

Was ist I in zahlen [I][I]?

Kannst du nicht einfach das folgende machen?

        int minArray = zahlen[0];
        for (int i = 0; i < zahlen.length - 1; i++) {
            if (zahlen[i] < minArray) minArray = zahlen[i];
        }

 

[Fkglr]

Was ist I in zahlen [I][I]?

das zweite war ein kopierfehler, bitte einfach ignorieren, habs im code auch nicht

funktioniert leider trz nicht, ich bekomme immer noch 0 als return wert raus

 

[Technipion]

funktioniert leider trz nicht, ich bekomme immer noch 0 als return wert raus

Ist vielleicht 0 einfach der kleinste Wert im Array?

Am besten wäre es, wenn wir den ganzen Code hätten...

 

[Zvoni]

Hab zwar keine Ahnung von Java, aber.....
Wieso nicht einfach das Array an eine Sortier-Routine schicken (Quicksort/BubbleSort?)?
Als Abfall-Produkt bekommst du automatisch den kleinsten und grössten Wert (erstes bzw. letztes Element des Arrays), danach einmal durch das Array rennen, die Werte aufaddieren, zum Schluss durch Anzahl Elemente teilen.

EDIT: Ohne Sortier-Routine würde ich wie folgt vorgehen:
1) Reservier dir drei Variablen (min, max und summe), und initialisiere alle mit dem ersten Element des Arrays!!
(ist übrigens im Beispiel von Yaslaw)
2) laufe durch das Array, beginnend ab dem zweiten Element, und mache folgendes:
a) vergleiche aktuelles Element mit min. Falls kleiner setze min auf aktuelles Element (ist übrigens das Beispiel von Yaslaw)
b) vergleiche aktuelles Element mit max. Falls grösser setze max auf aktuelles Element
c) bilde die summe mit dem aktuellen Element (summe=summe+Array[aktuell])
3) Nach Ende der Schleife mache Summe geteilt durch Anzahl Elemente --> ergibt Durchschnitt, in min steht der kleinste wert, in max der grösste

 

[Zvoni]

Ist vielleicht 0 einfach der kleinste Wert im Array?

Am besten wäre es, wenn wir den ganzen Code hätten...

er initialisiert minArray mit 0. Wenn die Zufallszahlen >=0 sind, was wird wohl ein Vergleich "kleiner als" ergeben?

 

[Fkglr]

Danke für eure hilfreichen Antworten! Hab's dann mit viel ausprobieren und unter Berücksichtigung eurer Tipps zum laufen gebracht!! )

 

[ComFreek]

Wieso nicht einfach das Array an eine Sortier-Routine schicken (Quicksort/BubbleSort?)?
Als Abfall-Produkt bekommst du automatisch den kleinsten und grössten Wert

Weil vergleichsbasiertes Sortieren immer Ω(n log n) ist und Minimum/Maximumbestimmung immer in θ(n) geht.

 

[Zvoni]

Weil vergleichsbasiertes Sortieren immer Ω(n log n) ist und Minimum/Maximumbestimmung immer in θ(n) geht.

Weiss ich, aber für nen Anfänger wäre das, zumindest meiner Meinung nach, der erste Anlauf was Algorythmen-Entwicklung betrifft.
Die Optimierung etc. sind dann der zweite Schritt (siehe auch mein EDIT in welchem ich dann auf ein Sortieren verzichte)

 

[HonniCilest]

Falls ihr Streams verwenden dürft, dann würde auch das gehen:

IntSummaryStatistics stats = IntStream.of(array).summaryStatistics();

IntSummaryStatistics hält die Werte, welche du brauchst, siehe auch IntSummaryStatistics (Java SE 11 & JDK 11 )