Array einlesen von Konsole

[pureGewalt]

Schönen Sonntag allen,

ich hab ein problem beim Einlesen eines Arrays als Methode und zwar immer wenn ich das Programm ausführe kommt diese Fehlermeldung:
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 4
at ArraySortOderMax.arrayEinlesen(ArraySortOderMax.java:37)
at ArraySortOderMax.main(ArraySortOderMax.java:18)


Leider verstehe ich in meinem Code nicht wo der Fehler ist .. hoffe ihr könnt mir dabei helfen

import java.util.Scanner;

public class ArraySortOderMax {

    public static void main(String[] args) {

        // Variablen Deklaration
        int n = 0;
       
        // Scanner aufrufen
        Scanner in = new Scanner(System.in);
       
        // Funktion für Array einlesen
        System.out.println("Hallo, bitte geben Sie die größe Ihres Arrays ein: ");
        n = in.nextInt();
       
        System.out.println("Geben Sie nun nach und nach Zahlen zum füllen ein: ");
        arrayEinlesen(n);

        // Ausgabe
        System.out.println(arrayEinlesen(n));

    }// ende von main

    // Platz für Methoden
    public static int[] arrayEinlesen(int n) {
        //Deklaration
        int[] a = new int[n];
        int j = 0;
        for (int i = 0; i <= n; i++) {
            Scanner in = new Scanner(System.in);
            j = in.nextInt();
            i = a[j];

        }
            //Rückgabe
            return a;
    }

}// ende von ArraySortOderMax

Liebe Grüße

 

[mihe7]

Beim Posten von Code bitte Code-Tags verwenden: [code=Java]Dein Code[/code]

Zur Frage:

int[] a = new int[n];
int j = 0;
for (int i = 0; i <= n; i++) {

Die erste Zeile erstellt ein Array mit n Elementen. Die Indizes von Arrays beginnen in Java mit 0, d. h. es gibt die Elemente a[0] bis a[n-1]. In Deiner for-Schleife iterierst Du aber bis einschließlich n, das Element a[n] liegt jedoch außerhalb der Indexgrenzen.

 

[pureGewalt]

Beim Posten von Code bitte Code-Tags verwenden: [code=Java]Dein Code[/code]



Zur Frage:

int[] a = new int[n];
int j = 0;
for (int i = 0; i <= n; i++) {

Die erste Zeile erstellt ein Array mit n Elementen. Die Indizes von Arrays beginnen in Java mit 0, d. h. es gibt die Elemente a[0] bis a[n-1]. In Deiner for-Schleife iterierst Du aber bis einschließlich n, das Element a[n] liegt jedoch außerhalb der Indexgrenzen.

Sorry danke für den Hinweis! Habe es abgeändert.

Genau, und wenn ich dann vom Nutzer beispielsweise 5 eingeben lasse, dann ist mein Array 5 Elemente groß oder? aber in der ersten forschleife wird mein i++ gemacht somit, startet mein Index bereits bei 1 und nicht bei 0. Doch wie lasse ich dann meinen Index bei 0 beginnen? Wenn ich i verringern würde, würde es ja bei jedem schleifen durchlauf passieren.

 

[mihe7]

in der ersten forschleife wird mein i++ gemacht somit, startet mein Index bereits bei 1 und nicht bei 0.

Du solltest Dich ein wenig mehr mit den Grundlagen beschäftigen. Folgendes:

for (int i = 0; i < n; i++) {
    doSomething(i);
}

ist äquivalent zu

int i = 0;
while (i < n) {
    doSomething(i);
    i++;
}

Nachtrag: habe dem doSomething noch den Parameter i spendiert, damit es noch deutlicher wird.

 

[pureGewalt]

Du solltest Dich ein wenig mehr mit den Grundlagen beschäftigen. Folgendes:

for (int i = 0; i < n; i++) {
    doSomething(i);
}

ist äquivalent zu

int i = 0;
while (i < n) {
    doSomething(i);
    i++;
}

Nachtrag: habe dem doSomething noch den Parameter i spendiert, damit es noch deutlicher wird.

Leider verstehe ich es nicht, ich hab meinen Code nochmal abgeändert für die Methode:

public static int[] arrayEinlesen(int n) {
        
        //Deklaration
        int[] a = new int[n];
        int i = 0; 
        int eingabe = 0;
        
        while(i < n) {
            Scanner in = new Scanner(System.in);
            eingabe = in.nextInt();
            eingabe = a[i];
            
            i++;
            
        }

            //Rückgabe
            return a;

Doch jetzt kann ich viele Zahlen eingeben bis es schließlich irgendwas ausgibt. Ich verstehe da leider auch den Fehler nicht einmal mehr. Für mich macht das irgendwie so sinn, aber irgendwie klappt es nicht.
Ich mache meinen Array so groß wie das eingelesene n, initialisiere i und meine eingabe, solange i < n ist, dann nochmal vom Benutzer die Zahlen eingeben lassen womit das Array befüllt werden soll, dann diese eingabe an den Index von 0 - n jeweils immer zuweisen. Danach i erhöhen.

 

[mihe7]

Schau Dir mal die Zuweisungen an.

            eingabe = in.nextInt();
            eingabe = a[i];

Fällt Dir hier etwas auf?

 

[mihe7]

In Deinem Code sind noch mehr Fehler, zum Beispiel:

        System.out.println("Geben Sie nun nach und nach Zahlen zum füllen ein: ");
        arrayEinlesen(n);

        // Ausgabe
        System.out.println(arrayEinlesen(n));

Hier rufst Du zweimal arrayEinlesen(n) auf. Daher wirst Du auch zweimal n Eingaben machen müssen. Außerdem wirst Du eine etwas "kryptische" Ausgabe erhalten.

Die Methode arrayEinlesen(n) gibt ein Array zurück (genauer: sie gibt die Adresse des Arrays zurück). Das kannst Du speichern:

        System.out.println("Geben Sie nun nach und nach Zahlen zum füllen ein: ");
        int[] eingaben = arrayEinlesen(n);

Die Ausgabe eines Arrays funktioniert nicht einfach über System.out. Entweder gibst Du in einer Schleife jedes Element aus oder Du verwendest java.util.Arrays:

        System.out.println("Geben Sie nun nach und nach Zahlen zum füllen ein: ");
        int[] eingaben = arrayEinlesen(n);
        System.out.println(Arrays.toString(eingaben));

Natürlich könnte man die letzten beiden Zeilen auch zusammenlegen, aber wir wollen es nicht gleich übertreiben

 

[pureGewalt]

Schau Dir mal die Zuweisungen an.

            eingabe = in.nextInt();
            eingabe = a[i];

Fällt Dir hier etwas auf?

Was ist hier das genaue Problem? Darf ich es nicht zweimal zuweisen oder muss das einlesen davon außerhalb der Methoden stehen?

In Deinem Code sind noch mehr Fehler, zum Beispiel:

        System.out.println("Geben Sie nun nach und nach Zahlen zum füllen ein: ");
        arrayEinlesen(n);

        // Ausgabe
        System.out.println(arrayEinlesen(n));

Hier rufst Du zweimal arrayEinlesen(n) auf. Daher wirst Du auch zweimal n Eingaben machen müssen. Außerdem wirst Du eine etwas "kryptische" Ausgabe erhalten.

Die Methode arrayEinlesen(n) gibt ein Array zurück (genauer: sie gibt die Adresse des Arrays zurück). Das kannst Du speichern:

        System.out.println("Geben Sie nun nach und nach Zahlen zum füllen ein: ");
        int[] eingaben = arrayEinlesen(n);

Die Ausgabe eines Arrays funktioniert nicht einfach über System.out. Entweder gibst Du in einer Schleife jedes Element aus oder Du verwendest java.util.Arrays:

        System.out.println("Geben Sie nun nach und nach Zahlen zum füllen ein: ");
        int[] eingaben = arrayEinlesen(n);
        System.out.println(Arrays.toString(eingaben));

Natürlich könnte man die letzten beiden Zeilen auch zusammenlegen, aber wir wollen es nicht gleich übertreiben

Danke für deine Hilfe, es funktioniert jetzt. Was ich noch nicht ganz verstanden habe, mit dem int[]eingaben = arrayEinlesen(n); ich weise dann die Methode dem Array "eingaben" zu um es dann mit dem "toString" ausgeben zu lassen, ich dachte die ganze Zeit das wenn ich die Methode einfach aufrufe "arrayEinlesen();" das dann dort einfach der rückgabewert steht und ich diesen mir dann einfach hinschreiben lassen kann und das mein Array dann wäre.

Vielen Dank nochmal!

 

[mihe7]

Was ist hier das genaue Problem? Darf ich es nicht zweimal zuweisen oder muss das einlesen davon außerhalb der Methoden stehen?

Du hast eine Variable eingabe. Die erste Zeile liest einen int-Wert vom Scanner und weist der Variablen eingabe den eingelesenen Wert zu. Anschließend weist die zweite Zeile der Variablen eingabe das i-te Element von a zu. Sprich: Du überschreibst die Varaible eingabe.

Du willst aber doch die Eingaben des Benutzers im Array speichern, dann musst Du schreiben:

            eingabe = in.nextInt();
            a[i] = eingabe;

Jetzt wird in der ersten Zeile der vom Scanner zurückgegebene Wert in der Variablen eingabe gespeichet, dann wird in der zweiten Zeile das i-te Element von a mit dem Wert der Variablen eingabe überschrieben.

ich weise dann die Methode dem Array "eingaben" zu um es dann mit dem "toString" ausgeben zu lassen,

Du weist nicht die Methode zu, sondern den Rückgabewert. D. h. die Methode arrayEinlesen wird aufgerufen. Der Computer arbeitet die Methode ab und liefert als Ergebnis ein int-Array. Und dieser Rückgabewert wird anschließend der Variablen eingaben zugewiesen.