Kann jemand mir helfen?

[cannyms]

Hallo zusammen

Ich habe folgendes Programm geschrieben:




am Schluss motzt es immer:



sortieren.java:23: missing return statement
}
^
Note: sortieren.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error



Can someone help me?

 

[byte]

Die Meldung ist doch eindeutig. Es fehlt mindestens ein Return. Wenn Du ne Methode mit Rückgabewert schreibst, dann muss sowohl im try als auch im catch ein return rein. Und gewöhn Dir ne anständige Formatierung (Einrückung) ein, dann blickst Du auch besser durch Deinen Code durch.

 

[tini]

gleich in der ersten Methode arr_sort() fehlt der Rückgabewert

 

[cannyms]

Hallo

Darf ich noch fragen, was ich bei diesen static ArrayList arr_sort returnen kann?

Man kann weder Zahlen noch String. Wie kann man eine ArrayList returnen?

Besten Dank für eure Mühe

 

[byte]

Das musst Du doch wissen, was Du zurückgeben willst. :roll: Ne ArrayList returned man genauso wie Strings, ints oder sonst ein Objekt. Wenn Du nix zurückgeben willst, deklarierst Du die Methode mit void.

 

[Eminent]

Also erstmal vorab, Klassennamen schreibt man Groß (nicht sortieren, sondern Sortieren)

Dann hier mal dein Code wie er eigentlich formatiert sein sollte:

package test;

import java.io.*;
import java.util.*;

public class sortieren {

   private static List zahlenliste = new ArrayList();

   static ArrayList arr_sort() {
      boolean leer = zahlenliste.isEmpty();
      if (leer == true) {
         System.out.println("Die Liste ist leer!");
         System.exit(0);
      } else {
         System.out.println("Hallo");
      }
   }

   static void arr_list() {
      for (int m = 0; m < zahlenliste.size(); m++) {
         System.out.print("" + zahlenliste.get(m) + " ");
      }
   }

   static ArrayList arr_read() {
      BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
      String zahl = " ";
      System.out.print("Geben Sie ein ganze Zahl ein!");

      while (true) {
         System.out.print("Zahl " + " ");
         try {
            zahl = in.readLine();
            zahlenliste.add(zahl);
         } catch (IOException e) {
            System.out.println("Folgende Fehler sind aufgetreten: ");
            System.out.println(e);
            System.exit(0);
         } catch (NumberFormatException e) {
            System.out.println("Es war keine Zahl " + e);
         }
      }

   }

   public static void main(String args[]) throws IOException {

      ArrayList zahlenliste = new ArrayList();

      String input = " ";
      int wahl = 0;
      boolean leer = false;

      BufferedReader in = new BufferedReader(new InputStreamReader(System.in));

      while (true) {
         System.out.println("Zahlenspiel");
         System.out.println("1. Zahlenliste anzeigen!");
         System.out.println("2. Liste sortieren!");
         System.out.println("3. Elemente hinzufügen!");
         System.out.println("4. Spiel verlassen!");
         System.out.print("Ihre Wahl: ");

         try {
            input = in.readLine();
            wahl = Integer.parseInt(input);
         } catch (NumberFormatException e) {
            System.out.println(input + " ist keine Fliesskommazahl!");
         } catch (IOException e) {
            System.out.println("I/O Fehler: +e");
            System.exit(1);
         }

         switch (wahl) {
            case 1:
               leer = zahlenliste.isEmpty();
               if (leer == true) {
                  System.out.println("Die Liste ist leer!");
                  break;
               } else {
                  arr_list();
                  break;
               }
            case 2:
               arr_sort();
               break;
            case 3:
               arr_read();
               break;
            case 4:
               System.out.println("Good bye");
               System.exit(1);
         }
      }
   }
}

Dann kann man ihn nämlich auch lesen. Wie die anderen vor mir schon gesagt haben ist der Fehler ja eindeutig, dass ein Rückgabewert fehlt:

   static ArrayList arr_sort() {
      boolean leer = zahlenliste.isEmpty();
      if (leer == true) {
         System.out.println("Die Liste ist leer!");
         System.exit(0);
      } else {
         System.out.println("Hallo");
      }
   }

In der Methode wird NIE etwas zurück gegeben. Du kannst also entweder

static void arr_sort() {...

schreiben oder musst ein ArrayList-Objekt zurückgeben.

z. B. return zahlenliste;

 

[cannyms]

Hallo
Das geht eben nicht.
ich kann schreiben wie ich will return zahlenliste;
am schluss motzt er immer:

sortieren.java:41: incompatible types
found : java.util.List
required: java.util.ArrayList
return zahlenliste;
^

Ich möchte aber kein void brauchen.
Kann jemand mir bitte helfen?

Besten Dank

 

[Eminent]

Hallo
Das geht eben nicht.
ich kann schreiben wie ich will return zahlenliste;
am schluss motzt er immer:

sortieren.java:41: incompatible types
found : java.util.List
required: java.util.ArrayList
return zahlenliste;
^

Ich möchte aber kein void brauchen.
Kann jemand mir bitte helfen?

Besten Dank

Deine Methode gibt doch (so wie du sie jetzt hast) eh nichts zurück. Was spricht dann dagegen void zu nehmen?

Der Fehler bei der Rückgabe kommt daher, dass die Methode eine ArrayList zurück geben will, du aber zahlenliste oben als List deklariert hast. Du kannst hier jetzt entweder zahlenliste oben als ArrayList deklarieren, oder aber du gibst so zurück

return (ArrayList) zahlenliste;

 

[cannyms]

Hier ist der Fehler

>javac *.java
sortieren.java:42: incompatible types
found   : java.util.List
required: java.util.ArrayList
				return zahlenliste;
                                       ^
Note: sortieren.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error
>Exit code: 1

 

[cannyms]

return (ArrayList) zahlenliste;

mit dem ist es auch nicht gegangen. Der verlangt schon wieder }

 

[cmkatz]

Hallo,

jetzt hast du dein Rückgabe-Problem in der Methode arr_read(), die eine ArrayList zurückgeben soll.

zahlenliste ist aber als List definiert (wenn das überhaupt was ausmacht!), außerdem erfolgt der return nur, wenn kein Fehler aufgetreten ist.
Das mag ja fachlich richtig sein, aber syntaxmäßig nicht wirklich.

Du kannst ja erstmal sowas schreiben:

static ArrayList arr_read() { 
      BufferedReader in = new BufferedReader(new InputStreamReader(System.in)); 
      String zahl = " "; 
       
      while (zahl.length()>0) { 
         System.out.print("Zahl (zurück zum Hauptmenü leer lassen)  "); 
         try { 
            zahl = in.readLine(); 
            zahlenliste.add(zahl); 
         //return zahlenliste; 
         } 
          
         catch (NumberFormatException e ) { 
            System.out.println("Es war keine Zahl "+e); 
         }    
          
         catch (IOException e) { 
            System.out.println("Folgende Fehler sind aufgetreten: "); 
            System.out.println(e); 
            System.exit(0); 
         } 
          
         return zahlenliste;
      }

Falls du jetzt nämlich in das erste catch reinkommst, wird dennoch etwas zurück gegeben. Du mußt halt dann beim Aufruf der Methode arr_read() prüfen, ob der zurückgegebene Wert dem positiven Fall entspricht und dann entsprechend reagieren bzw. anders rum... ;-)

Gruß
cmkatz

 

[cannyms]

Hallo
Vielen Dank für Antwort. Aber es geht einfach nicht. Ich habe es schon mehrmals probiert mit //

am Schluss kommt immer

>javac sortieren.java
sortieren.java:58: missing return statement
}
^
Note: sortieren.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error
>Exit code: 1

 

[cmkatz]

Ja, sorry, mein Fehler,

das letzte return muß natürlich außerhalb der Schleife sein!!,
also so:

static ArrayList arr_read() { 
      BufferedReader in = new BufferedReader(new InputStreamReader(System.in)); 
      String zahl = " "; 
        
      while (zahl.length()>0) { 
         System.out.print("Zahl (zurück zum Hauptmenü leer lassen)  "); 
         try { 
            zahl = in.readLine(); 
            zahlenliste.add(zahl); 
         //return zahlenliste; 
         } 
          
         catch (NumberFormatException e ) { 
            System.out.println("Es war keine Zahl "+e); 
         }    
          
         catch (IOException e) { 
            System.out.println("Folgende Fehler sind aufgetreten: "); 
            System.out.println(e); 
            System.exit(0); 
         } 
      } 
      return zahlenliste; 
}

 

[bygones]

außerdem wenn schon zahlenliste als List deklariert ist dann geb auch List und nciht ArrayList als Returntyp an...

 

[cannyms]

es geht leider auch nicht.

Kann jemand mir noch helfen?

 

[cannyms]

Es geht nun, vielen dank!