Programm erkennt nicht an das Array zurückgegeben wird trotz Initialisierung *einfach*

[infonoob1234]

Hey,
kann mir jemand sagen warum das Programm nicht funktioniert? Eigentlich wird in jedem der drei Fälle ein int array zurückgegeben, trotzdem sagt mir die Fehlermeldung, dies wäre nicht der Fall.

public static int[] take(int[] a ,int[] b ,int start ,int end){
        if(end <= start) {
            int []c = {};
            return c;
        }
        else if(end > start && Arrays.compare(a, b) != 0) {
            int []c = a.clone();
            return c;
        }
        else if(end > start && Arrays.compare(a, b) == 0) {
            Arrays.sort(a);
            int []c = Arrays.copyOfRange(a, start, end);
            return c;
        }
    }

Wäre Klasse wenn mir jemand den offensichtlich einfachen Fehler einmal erklären könnte!
Danke

 

[kneitzel]

Das Problem dürfte sein, dass Du in allen Zweigen if Anweisungen hast. Aus Sicht des Compilers können die zutreffen aber auch nicht. Dass diese drei if alle Fälle abprüfen, sieht der Compiler nicht.

Einfachste Lösung: Lösche das letzte if heraus. Ein reines else reicht doch aus.

Also vom Aufbau her ein:

if (...) {
    // ...
} else if (...) {
    // ...
} else { // Hier kein if mehr!
    // ...
}

 

[mrBrown]

Aus Sicht des Compilers können die zutreffen aber auch nicht. Dass diese drei if alle Fälle abprüfen, sieht der Compiler nicht.

Durch mehrere Threads könnten sich a und b zwischen zwei Bedingungen ändern, wodurch keine der Bedingungen zutrifft.

Einfachste Lösung: Lösche das letzte if heraus. Ein reines else reicht doch aus.

Alternativ einen AssertionError am Ende werfen.

 

[mrBrown]

Den Code könnte man übrigens kürzen zu:

public static int[] take(int[] a ,int[] b ,int start ,int end){
        if(end <= start) {
            int []c = {};
            return c;
        }

        if(Arrays.compare(a, b) != 0) {
            int []c = a.clone();
            return c;
        }
        else if(Arrays.compare(a, b) == 0) {
            Arrays.sort(a);
            int []c = Arrays.copyOfRange(a, start, end);
            return c;
        }
    }

(Der Fehler ist extra noch drin, sonst wär's ja langweilig )

 

[kneitzel]

Durch mehrere Threads könnten sich a und b zwischen zwei Bedingungen ändern, wodurch keine der Bedingungen zutrifft.

Ja, das ist richtig, aber das ist hier etwas seltsam und würde ich so erst einmal nicht beachten.

Die Problematik bleibt ja auch mit dem zweiten if bestehen. Du springst in einen Bereich hinein, obwohl er nicht mehr gültig ist. Also das Arrays.compare(a,b) läuft, ein Element, das schon verglichen wurde, ändert sich. Das Ergebnis vom Aufruf ist dann schlicht falsch. Die Arrays, die vorher gleich waren, sind jetzt nicht mehr gleich (oder umgekehrt).

Das gehört also generell zu den Racing Conditions und wenn die hier problematisch sind, dann gehören die entsprechend behandelt (z.B. durch ein Locking verhindert o.ä.)

Aber wie schon gesagt: Der Hinweis ist natürlich richtig.

 

[infonoob1234]

okay, danke.
ich hab meinen Code jetzt verbessert, jedoch bekomme ich beim Testen mehrere Fehlermeldungen.

public static void main(String[] args) {
        System.out.println(A2_main.take({2,7,1,9}, {5,6,7}, 2, 4);
    }

Fehler:
Syntax error on token ".", @ expected after this token
Syntax error on token(s), misplaced construct(s)
Syntax error on token "}", delete this token
Syntax error, insert "SimpleName" to complete ArgumentList

Eigentlich müsste das Programm doch funktionieren oder nicht

 

[Gelöschtes Mitglied 65838]

 System.out.println(A2_main.take({2,7,1,9}, {5,6,7}, 2, 4);

hier fehlt schonmal die richtige klammersetzung

 

[kneitzel]

Öhm, das Konstrukt nur mit den { ... } funktioniert bei der Initialisierung aber nicht bei der Übergabe von Parametern. Das musst Du noch ein new int[] davor schreiben, also ein System.out.println(A2_main.take(new int[] {2,7,1,9}, new int[] {5,6,7}, 2, 4));
Und es fehlt eine schließende Klammer am Ende!

 

[kneitzel]

Dazu einfach einmal noch die Details aus der Java Language Specification.

Das, was Du hier verwenden wolltest, ist ein Array Initializer und diese werden in der JLS in 10.6 behandelt:

Chapter 10. Arrays

docs.oracle.com

Und der erste Satz ist wichtig:

An array initializer may be specified in a field declaration (§8.3, §9.3) or local variable declaration (§14.4), or as part of an array creation expression (§15.10.1), to create an array and provide some initial values.

Also dieser Array Initializer kann benutzt werden:
a) in einer field declaration (Instanzvariable)
b) local variable (lokale Variable)
c) einer array creation expression (ein new Ausdruck, der ein Array erzeugen soll)

Damit ist es als Parameter so nicht zulässig. Wir erweitern den Ausdruck daher um eine array creation expression, so dass wir dies verwenden können.

Edit: Typo behoben.

 

[infonoob1234]

DANKE!