Compiler-Fehler NullPointerException

[Call_He_Tempi]

Hallo zusammen,

ich möchte einen Bubblesort Algorithmus schreiben.
Die jeweiligen Methoden befinden sich in unterschiedlichen Klassen.
Das erstellen und ausgeben des Array´s "zusortieren" funktioniert, ich habe die Klasse zu Probe einzeln getestet.
Wenn ich jetzt aber die Bubblesort Klasse dranhänge gibt mir die Konsole folgendes aus.

[CODE lang="java" title="Konsole"]Wie viele Werte sollen sortiert werden?
3

Geben Sie nun die 1 zu speichernde Zahl ein
1

Geben Sie nun die 2 zu speichernde Zahl ein
2

Geben Sie nun die 3 zu speichernde Zahl ein
3

Exception in thread "main" java.lang.NullPointerException: Cannot read the array length because the return value of "Array.getZusortieren()" is null
at Bubblesort.sortieren(Bubblesort.java:8)
at Main.main(Main.java:10)[/CODE]

Ich habe schon unendlich lange rumprobiert und gegoogelt aber ich finde einfach keine Lösung für mein Problem.
Ich hoffe, ihr könnt mir da weiterhelfen.
Vielen Dank im Voraus.


[CODE lang="java" title="Array Class"]import java.util.Scanner;

public class Array {
private double[] zusortieren;

public double[] getZusortieren() {
return zusortieren;
}

public static void arrayVoid(String[] args) {

System.out.println("Wie viele Werte sollen sortiert werden?");
Scanner eingabe = new Scanner(System.in);
Array arrayObjekt = new Array();
int arrayGröße = eingabe.nextInt();
arrayObjekt.zusortieren = new double[arrayGröße];

for (int i = 0; i < arrayGröße; i++) {
System.out.println("Geben Sie nun die " + (i + 1) + " zu speichernde Zahl ein");
double eingaben = eingabe.nextDouble();
arrayObjekt.zusortieren = eingaben;
}

}
}[/CODE]

[CODE lang="java" title="Bubblesort Class"]public class Bubblesort {
public void sortieren() {
Array arrayObjekt = new Array();

double tausch;

for (int i = 1; i < arrayObjekt.getZusortieren().length; i++) {
for (int j = 0; j < (arrayObjekt.getZusortieren().length - i); j++) {
if (arrayObjekt.getZusortieren()[j] > arrayObjekt.getZusortieren()[j + 1]) {
tausch = arrayObjekt.getZusortieren()[j];
arrayObjekt.getZusortieren()[j] = arrayObjekt.getZusortieren()[j + 1];
arrayObjekt.getZusortieren()[j + 1] = tausch;

}
}
}
}

public static void bubblevoid(String[] args) {

Array arrayObjekt = new Array();

System.out.println("Vor dem sortieren");
for (int i = 0; i < arrayObjekt.getZusortieren().length; i++) {
System.out.println(i + 1 + ":" + arrayObjekt.getZusortieren());
}
double[] arrayA = arrayObjekt.getZusortieren();
System.out.println("Nach dem sortieren");
for (int j = 0; j < arrayA.length; j++) {
System.out.println(+1 + ":" + arrayA[j]);
}
}
}
[/CODE]

 

[Jw456]

du musst natürlich der Methode bubblevoid auch das zu sortierende Array übbergeben.
nicht String[] args was bestimmt von deiner MainMethode war.

wie schaut denn deine Main Methode aus

 

[Call_He_Tempi]

du musst natürlich der Methode bubblevoid auch das zu sortierende Array übbergeben.
nicht String[] args was bestimmt von deiner MainMethode war.

wie schaut denn deine Main Methode aus

Die Main Klasse
[CODE lang="java" title="Main Class"]public class Main {

public static void main(String[] args) {

Array.arrayVoid(args);

Bubblesort sortierer = new Bubblesort();
sortierer.sortieren();

Bubblesort.bubblevoid(args);

}
}
[/CODE]


Hier müsste das zu sortierende Array eigentlich übergeben werden oder ?

[CODE lang="java" title="Bubblesort" highlight="7,8,9,10,11,12"]public class Bubblesort {
public void sortieren() {
Array arrayObjekt = new Array();

double tausch;

for (int i = 1; i < arrayObjekt.getZusortieren().length; i++) {
for (int j = 0; j < (arrayObjekt.getZusortieren().length - i); j++) {
if (arrayObjekt.getZusortieren()[j] > arrayObjekt.getZusortieren()[j + 1]) {
tausch = arrayObjekt.getZusortieren()[j];
arrayObjekt.getZusortieren()[j] = arrayObjekt.getZusortieren()[j + 1];
arrayObjekt.getZusortieren()[j + 1] = tausch;

}
}
}
}[/CODE]

 

[Jw456]

import java.util.Scanner;

public class Array {
private double[] zusortieren;

public double[] getZusortieren() {
return zusortieren;
}

public static void arrayVoid(String[] args) {

System.out.println("Wie viele Werte sollen sortiert werden?");
Scanner eingabe = new Scanner(System.in);
Array arrayObjekt = new Array();
int arrayGröße = eingabe.nextInt();
arrayObjekt.zusortieren = new double[arrayGröße];

for (int i = 0; i < arrayGröße; i++) {
System.out.println("Geben Sie nun die " + (i + 1) + " zu speichernde Zahl ein");
double eingaben = eingabe.nextDouble();
arrayObjekt.zusortieren = eingaben;
}

}
}

[/CODE]

hier machst du deine Eingaben in ein anderes array als das was du in getZusortieren() zurück gibst.

lasse das mit dem neuen Objekt und benutze das Array aus der Klasse.

[/I]
import java.util.Scanner;

public class Array {
    private double[] zusortieren;

    public double[] getZusortieren() {
        return zusortieren;
    }

    public void arrayVoid() {

        System.out.println("Wie viele Werte sollen sortiert werden?");
        Scanner eingabe = new Scanner(System.in);
        
        int arrayGröße = eingabe.nextInt();
        zusortieren = new double[arrayGröße];

        for (int i = 0; i < arrayGröße; i++) {
            System.out.println("Geben Sie nun die " + (i + 1) + " zu speichernde Zahl ein");
            double eingaben = eingabe.nextDouble();
            zusortieren[i] = eingaben;
        }

    }
}
[I]

 

[Jw456]

Hier müsste das zu sortierende Array eigentlich übergeben werden oder ?

die Methode zu erstellen des Arrays arrayVoid() ruft du gar nicht auf
also es gibt kein Array

 

[Jw456]

public class Main {

    public static void main(String[] args) {
        
        //Array.arrayVoid(args); sinnlos
        
        Bubblesort sortierer = new Bubblesort();
        sortierer.sortieren();
    
        //Bubblesort.bubblevoid(args); sinnlos

    }
}

public class Bubblesort {
    public void sortieren() // hier solltest du das Array übergeben
        //nicht ein neues erstellen
    {
        Array arrayObjekt = new Array();

        double tausch;

        for (int i = 1; i < arrayObjekt.getZusortieren().length; i++) {
            for (int j = 0; j < (arrayObjekt.getZusortieren().length - i); j++) {
                if (arrayObjekt.getZusortieren()[j] > arrayObjekt.getZusortieren()[j + 1]) {
                    tausch = arrayObjekt.getZusortieren()[j];
                    arrayObjekt.getZusortieren()[j] = arrayObjekt.getZusortieren()[j + 1];
                    arrayObjekt.getZusortieren()[j + 1] = tausch;

                }
            }
        }
    }

    public static void bubblevoid(String[] args) // hier solltest du das Array übergeben
        //nicht ein neues erstellen
        // das mit dem string ist sinnlos

        
    {
    
    {

        Array arrayObjekt = new Array();

        System.out.println("Vor dem sortieren");
        for (int i = 0; i < arrayObjekt.getZusortieren().length; i++) {
            System.out.println(i + 1 + ":" + arrayObjekt.getZusortieren()[i]);
        }
        double[] arrayA = arrayObjekt.getZusortieren();
        System.out.println("Nach dem sortieren");
        for (int j = 0; j < arrayA.length; j++) {
            System.out.println(+1 + ":" + arrayA[j]);
        }
    }
}

 
Like Antwort
Report

 

[Call_He_Tempi]

hier machst du deine Eingaben in ein anderes array als das was du in getZusortieren() zurück gibst.

lasse das mit dem neuen Objekt und benutze das Array aus der Klasse.

[/I]
import java.util.Scanner;

public class Array {
    private double[] zusortieren;

    public double[] getZusortieren() {
        return zusortieren;
    }

    public void arrayVoid() {

        System.out.println("Wie viele Werte sollen sortiert werden?");
        Scanner eingabe = new Scanner(System.in);
        
        int arrayGröße = eingabe.nextInt();
        zusortieren = new double[arrayGröße];

        for (int i = 0; i < arrayGröße; i++) {
            System.out.println("Geben Sie nun die " + (i + 1) + " zu speichernde Zahl ein");
            double eingaben = eingabe.nextDouble();
            zusortieren[i] = eingaben;
        }

    }
}
[I]

[CODE lang="java" title="Array Class" highlight="25"]import java.util.Scanner;

public class Array {
private static double[] zusortieren; // Erstellen eines private double "zusortieren"

public double[] getZusortieren() { // getMethode für private double "zusortieren"
return zusortieren;
}

public static void arrayVoid(String[] args) {

System.out.println("Wie viele Werte sollen sortiert werden?");
Scanner eingabe = new Scanner(System.in);
Array arrayObjekt = new Array();
int arrayGröße = eingabe.nextInt();
arrayObjekt.zusortieren = new double[arrayGröße]; // Da sich public double[]

// "arrayObjekt.zusortieren" ist dazu da, um das private array
// "zusortieren" abzurufen um die Größe des array´s festzulegen.

// "arrayObjekt.zusortieren" ersetzt ">double[] arrayname< = new double[]"
for (int i = 0; i < arrayGröße; i++) {
System.out.println("Geben Sie nun die " + (i + 1) + " zu speichernde Zahl ein");
double eingaben = eingabe.nextDouble();
zusortieren = eingaben;
}

}
}[/CODE]

Japp! Vielen dank! Genau das ist es gewesen. Jetzt klappt es!

 

[Jw456]

Versuche deinen Code mal ohne static zu schreiben.
Static sollten besonders Anfänger sehr sparsam benutzen.
In der main erstellt du ja Objekte von den Klassen also benutze sie auch.

Was du hier machst hat nicht viel mit OOP zu tun.

 

[temi]

Versuche deinen Code mal ohne static zu schreiben.
Static sollten besonders Anfänger sehr sparsam benutzen.

Konkret bedeutet das: Genau ein "static" vor der Methode main().

 

[Jw456]

Konkret bedeutet das: Genau ein "static" vor der Methode main().

Das die main Methode static ist versteht sich doch von selbst. 😏

 

[mihe7]

Das die main Methode static ist versteht sich doch von selbst. 😏

Es dürfte @temi auch mehr um das "genau ein static" gegangen sein.