Problem bei Array

[Guest]

Hallo!
Könnt ihr mir mal grade Helfen? Zeigt mir Fehler folgenden Fehler an: This methode must return a result of double []

Aber ich hab doch ein double-Array... wo liegt denn der Fehler?

danke für die Hilfe!

   public static double [] leseArray() {
        System.out.println("Größe des Arrays eingeben");
        int groeße = package.readInt();
        double array [] = new double [groeße];
        for (int i=0; i<groeße;i++){
            System.out.println("Wert an Stelle "+i+ " angeben");
            array [i] = package.readInt();
            return array;
        }
        
    }

[/code]

 

[Dreezard]

Klar. Für den compiler ist es nicht klar ob jemals die bedingung gegeben ist, dass die Schleife überhaupt einmal ausgeführt wird, schließlich könnte der User ja -1 eingeben. Demzufolge ist der kompiler nicht sicher, ob etwas zurückgegeben wird. Er braucht aber eine eindeutige Rückgabe. Also:

public static double[] leseArray() {
        int groeße = 0;
        do {
              System.out.println("Größe des Arrays eingeben");
              groeße = package.readInt();
        } while(groeße <= 0)                            //überprüfung ob die Zahl überhaupt gültig ist
        double array[] = new double [groeße];
        for (int i=0; i<groeße; i++){
            System.out.println("Wert an Stelle "+ i + " angeben");
            array[i] = package.readInt();
            //Du solltest das Array nicht in der Schleife returnen, die schleife wird dadurch ja nur einmal durchlaufen
        }
        return array;   //hier isses richtig ;-)
       
    }

 

[Guest]

vielen vielen dank für die hilfe!
daran hab ich gar nicht gedacht!

aber eine frage hätte ich da noch, wieso erkennt er denn die überprüfung der zahl nicht per if-abfrage an? sprich wieso muss ich das per schleife machen?

danke nochmal!

 

[mic_checker]

Was meinst du damit , dass er die Überprüfung der Zahl nicht per if "erkennt" ?

 

[Guest]

wenn ich das:

int groeße = 0;
        do {
              System.out.println("Größe des Arrays eingeben");
              groeße = package.readInt();
        } while(groeße <= 0)

als if-abfrage programmiere zeigt er mir immer noch den selben fehler an...

 

[Akimor]

Ich versteh da nicht ganz was du dort mit einer Verzweigung (du nennst es If-Abfrage) anfangen willst...
Eine Verzweigung ueberprueft nur einmal ob die Bedingung zutrifft,
eine Schleife laeuft so lange im Kreis bis die Bedingung erfuellt ist

 

[bygones]

deine forschleife würde so und so nichts bringen -da beim ersten durchlauf schon die return anweisung steht...

public static double [] leseArray() {
        System.out.println("Größe des Arrays eingeben");
        int groeße = package.readInt();
        double array [] = new double [groeße];
        for (int i=0; i<groeße;i++){
            System.out.println("Wert an Stelle "+i+ " angeben");
            array [i] = package.readInt();
        }
            return array;
    }

....

 

[Guest]

meinte so:

public static double [] leseArray() {
        System.out.println("Größe des Arrays eingeben");
        int groeße = package.readInt();
        if (groeße>=0) {
           double array [] = new double [groeße];
           for (int i=0; i<groeße;i++){
               System.out.println("Wert an Stelle "+i+ " angeben");
               array [i] = package.readInt();
           }
        }
            return array;

    }

damit sollte doch auch sichergestellt sein, dass die größe des array nicht negativ ist...

 

[mic_checker]

Nur du deklarierst das Array innerhalb der Bedingung..., dann sollte der Compiler eigentlich meckern.

Davon abgesehen, was gibst du zurück wenn groeße < 0 ?

 

[Dreezard]

Dann wird ein Null-Array zurückgegeben. Aber das ist ja nicht Sinn und Zweck der Sache.
Mit der do-while Schleife wollte ich sicherstellen, dass der eingegebene Wehrt auch gültig ist, indem die Abfrage so lang wiederholt wird bis der Eingegebene Wehrt größer 0 ist. Dann erst wird weitergearbeitet und in (fast) jedem Falle Ein gültiges Array erstellt. Zwei Fallen gibt's aber noch. Der User gibt eine Zahl ein, die Größer als ein Integer ist, oder Buchstaben. Sowas sollte man alles noch abfangen. Am besten immer gleich denken wie ein DAU.
Dein if würde übrigens nicht viel bringen. Denn wenn die eingegebene Zahl wirklich kleiner-gleich 0 ist returnt er halt ein Null-Array und das kommt im code sicher nicht gut.

 

[Bleiglanz]

lass das scharfe ß weg!

verwende nicht "package" als namen!

gib ein leeres array zurück, wenn nichts da ist (das ist oft klarer als der wert null)

 

[mic_checker]

Dann erst wird weitergearbeitet und in (fast) jedem Falle Ein gültiges Array erstellt.

Nur das kann ja nicht Sinn und Zweck des ganzen sein, die Änderungen um dieses Problem zu beheben sind ja nicht sonderlich beanspruchend (zeitlich)....

Dein if würde übrigens nicht viel bringen. Denn wenn die eingegebene Zahl wirklich kleiner-gleich 0 ist returnt er halt ein Null-Array und das kommt im code sicher nicht gut.

Woher willst du das wissen? hab doch gar keinen Code gepostet DU könntest den User ja zwingen ne positive Zahl einzugeben (sprich : wiederholung der eingabe bis > 0).

 

[Dreezard]

Ich zwinge den User immer zu seinem Glück