missing return statement

[Gayson]

Hallo!

Ich habe eine Methode geschrieben, die einen String zurückgeben soll.

Leider erhalte ich vom Kompiler nur den oben genannten Fehler, offensichtlich muss das return außerhalb des try-catch-Blockes stehen.
Doch warum und wie behebe ich diesen Fehler?

Vielen Dank!

	public static String getEigenschaft(String MitarbeiterName, String Eigenschaft){
       try {
         SAXBuilder builder = new SAXBuilder();
         Document doc = builder.build("Mitarbeiter.librml");
         Element root = doc.getRootElement();
         List mitarbeiterInfo = root.getChildren("Mitarbeiter");;
         Iterator mitarbeiterIterator = mitarbeiterInfo.iterator();
         while (mitarbeiterIterator.hasNext()) {
            Element mitarbeiterElement = (Element) mitarbeiterIterator.next();
            Element mitarbeiterName = mitarbeiterElement.getChild("Name");
            if(mitarbeiterName.getText().equals(MitarbeiterName)){
               Element mitarbeiterEigenschaft = mitarbeiterElement.getChild(Eigenschaft);
               String wantedEigenschaft = mitarbeiterEigenschaft.getText();
               return wantedEigenschaft;
            }
         }
      } catch (JDOMException e){e.printStackTrace();}
      catch (IOException e) {e.printStackTrace();}
  	}

 

[Anubis]

WEnn die if bedingen nie erfüllt wird, gibt es auch eine ausführung des returns.

 

[Isaac]

WEnn die if bedingen nie erfüllt wird, gibt es auch eine ausführung des returns.

Er meinst "WEnn die if bedingen nie erfüllt wird, gibt es auch keine ausführung des returns"

Wenn deine Methode String zurück geben soll muss sie in allen möglichen Fällen auch String zurück geben.

 

[Gayson]

Wenn deine Methode String zurück geben soll muss sie in allen möglichen Fällen auch String zurück geben.

Irgendwie hab ich es geschafft, mit diesem Satz im Hinterkopf folgenden Code zu basteln, der auch funktioniert. Vielen Dank!!!!

	public static String getEigenschaft(String MitarbeiterName, String Eigenschaft){
		String wantedEigenschaft ="";
       try {
         SAXBuilder builder = new SAXBuilder();
         Document doc = builder.build("Mitarbeiter.librml");
         Element root = doc.getRootElement();
         List mitarbeiterInfo = root.getChildren("Mitarbeiter");;
         Iterator mitarbeiterIterator = mitarbeiterInfo.iterator();
         while (mitarbeiterIterator.hasNext()) {
            Element mitarbeiterElement = (Element) mitarbeiterIterator.next();
            Element mitarbeiterName = mitarbeiterElement.getChild("Name");
            if(mitarbeiterName.getText().equals(MitarbeiterName)){
               Element mitarbeiterEigenschaft = mitarbeiterElement.getChild(Eigenschaft);
               wantedEigenschaft = mitarbeiterEigenschaft.getText();
             }
         }
      } catch (JDOMException e){e.printStackTrace();}
      catch (IOException e) {e.printStackTrace();}
      return wantedEigenschaft;
  	}

 

[meez]

Ich würd den String nicht zuerst mit "" initialiseieren, sondern einfach im catch-Block das schreiben: return "";

 

[L-ectron-X]

Es stimmt schon, dass er nicht erst einen String initialisieren braucht, wenn er dann innerhalb der Methode nicht weiter verwendet wird. Also ist das OK, wenn er einen passenden String direkt per return zurück gibt.
Aber nach Deiner Aussage gibts nur dann eine Rückgabe, wenn eine Ausnahme ausgelöst wurde.