Rückgabewert der if-Anweisung in einer for each-Schleife

Habe ich diese Methode, die verwendet eine for-each-Schleife mit einer if-Anweisung, in der es

public static Apartment getApartment(String aNumber)

//for loop to iterate through the list of apartments         
{   
for (Apartment x : listOfApartments) 
  { 
  //Variable to hold the value of returned apartment    
  if (x.getApartmentNo().equalsIgnoreCase(aNumber))
  { Apartment chosen = x;
  }
  else 
  {JOptionPane.showMessageDialog(null, "Apartment not found");
  }

}  
return chosen;
}

Ich erhalte eine Fehlermeldung, dass das Symbol gewählt haben, nicht gefunden. Ich glaube, ich habe deklariert die variable in der falschen Stelle.
Keine Hilfe?
Hinweis: Dies ist eine Methode, die einen String und gibt ein Objekt zurück Wohnung.

InformationsquelleAutor Sasha | 2012-04-25

  1. 0

    Akzeptierte Antwort ist richtig, aber ich würde stark vorschlagen, um zu vermeiden, mehrere return-Anweisungen. Code mit mehreren Rückgabe-Anweisungen ist schwieriger zu verwalten. Mag nicht wie eine große Sache, aber wenn der code wächst, kann dies zu einem echten Problem.

    public static Apartment getApartment(String aNumber)
{   
    Apartment result = null;
    for (Apartment apartment : listOfApartments) 
    {   
        if (apartment.getApartmentNo().equalsIgnoreCase(aNumber))
        {
             result = apartment;
             break;
        }
    }  
    if (result == null)
    {
        JOptionPane.showMessageDialog(null, "Apartment not found");
    }
    return result;
}
    Es gibt keinen Punkt, durch Reifen springen, einfach um zu vermeiden, mehrere return-Anweisungen grundsätzlich ab. Hinzufügen zusätzlicher Logik (break und if(result==null) nach der Schleife) verletzt Lesbarkeit dieses Stück code. Zwar ist dies sicherlich eine legitime Praxis für größere Teile des Codes, kürzere Stücke wie diese können profitieren von mehreren gibt. Es gibt keine one-size-fits-all-Regel auf dieses; es ist am besten Ansatz, das problem von Fall zu Fall.
    Ich bin nicht einverstanden. Es sei denn, Sie und Ihr team sind von mystischen perfekte Entwickler Rasse haben, dann ist dies war in der Regel passiert: dev0 erstellt eine Funktion mit mehreren returns (nicht durch Reifen springen). Dann irgendwann dev1 erweitert und fügt ein paar mehr retuns. dev1 ist nicht überarbeiten Sie den code, weil Alter code Werke. Dann ein paar Jahre später devX hat, um durch zu gehen spaggeti von Retouren und herauszufinden, was man eingeführt, einen Fehler, weil Funktion zurückgegeben zu früh. Bottom line - multi Rücksendungen sind genau so schlimm wie goto und sollte verwendet werden, wie oft, was wesentlich ist - nie

    InformationsquelleAutor Rytis I

  2. 3

    Müssen Sie schreiben Sie Ihre Schleife: Sie sollten nicht machen eine Entscheidung, dass die Wohnung nicht gefunden, bis Sie beenden die Schleife. Sie brauchen nicht, um den Wert zuzuweisen, um die temporäre variable entweder - sobald die Wohnung gefunden ist, kehren Sie sofort.

    Können Sie berichten, dass die Wohnung nicht nur gefunden, wenn Sie beenden die Schleife.

    for (Apartment x : listOfApartments) { 
    //Variable to hold the value of returned apartment    
    if (x.getApartmentNo().equalsIgnoreCase(aNumber)) {
        return x;
    }
}
JOptionPane.showMessageDialog(null, "Apartment not found");
return null;

    InformationsquelleAutor dasblinkenlight

  3. 1

    was ist, wenn deine Bedingung "x".getApartmentNo().equalsIgnoreCase(aNumber)" zu false ausgewertet?

    die Wohnung Datentyp "Auserwählten" nicht erstellt werden!

    wie kann man etwas zurückgeben, das noch nicht instanziiert wurde?

    InformationsquelleAutor Trevor Hickey

  4. 1

    Dies ist ein Bereich Ausgabe Ausgewählt ist, erzeugt in der Seite die if-Anweisung.

    Deklarieren Sie Sie außerhalb und legen Sie es auf null, dann wird die Funktion wieder gewählt, wenn die Bedingung wahr ist, oder null, wenn es nicht so ist. 

    public static Apartment getApartment(String aNumber)

//for loop to iterate through the list of apartments         
{   
Apartment chosen = null;

for (Apartment x : listOfApartments) 
  { 
  //Variable to hold the value of returned apartment    
  if (x.getApartmentNo().equalsIgnoreCase(aNumber))
  { chosen = x;
  }
  else 
  {JOptionPane.showMessageDialog(null, "Apartment not found");
  }

}  
return chosen;
}

    InformationsquelleAutor Andrew T

  5. 0

    Du hast Recht; Sie erklärte entschieden an der falschen Stelle. Deklarieren Sie Sie rechts, bevor Sie Ihren loop, aber weisen Sie ihn (es) zu null. So ist es garantiert Wert, wenn Sie die Rücklaufleitung.

    Alternativ bei nur einer Wohnung entsprechen (oder Sie egal welche), können Sie einfach legen Sie return x; in der if...wahr. Und null zurückgeben, nach der Schleife, vorausgesetzt, die Schleife abgeschlossen ist. Auch, weil Sie Schleifen, jede Wohnung, die nicht match wird zeigen, die "nicht gefunden" - Meldung; Sie haben wahrscheinlich nur wollen, um Sie anzuzeigen, nachdem die Schleife abgeschlossen hat, ohne eine Wohnung zu finden.

    InformationsquelleAutor Carl Manaster

  6. 0

    Die variable chosen im code war gefangen in dem Umfang in der es erstellt wurde, also kann es nicht verwendet werden, außerhalb der {}. Auch der else-Zweig der if-Anweisung ausgelöst wurde, bevor der rest der for-Schleife ran.

    Die Sie nicht wirklich brauchen, um zu speichern Sie die ausgewählte Wohnung, können Sie einfach schicken Sie es sofort, so wie hier:

    public static Apartment getApartment(String aNumber)
{   
    for (Apartment apartment : listOfApartments) 
    {   
        if (apartment.getApartmentNo().equalsIgnoreCase(aNumber))
            return x;
    }  
    JOptionPane.showMessageDialog(null, "Apartment not found");
    return null;
}
    Ich habe getestet wie folgt : - Wohnung b3 = new Wohnung("B3"); Wohnung.listOfApartments.add(b3); Wohnung = Apartment.getApartment("B3"); ein.toString(); (Das sollten sich die Informationen ausdrucken der neu erstellte Wohnung. Es ist nicht zu arbeiten)
    Ohne dass der rest des Codes ist, kann ich nicht sicher sagen, was falsch ist.

    InformationsquelleAutor ricochet1k

  7. 0

    Deklarieren der Variablen 'gewählt' werden, bevor die for-Schleife wie diese:

    Apartment chosen = new Apartment();

    Außerdem können Sie brechen aus der Schleife, wenn Sie sobald Sie finden die Wohnung, die Sie suchen. Wenn die Liste groß ist, es könnte etwas Zeit sparen.

    InformationsquelleAutor Chris911

  8. 0

    Eine einfache (und richtige) Weg zu schreiben, die Methode wäre:

    public static Apartment getApartment(String aNumber) {
    for (Apartment x : listOfApartments)
      if (x.getApartmentNo().equalsIgnoreCase(aNumber))
        return x;
    JOptionPane.showMessageDialog(null, "Apartment not found");
    return null;
}

    Es ist nicht notwendig zu deklarieren Sie eine lokale variable für einen Wert, der zurückgegeben werden soll, sofort. Auch die Meldung sollte dargestellt werden, nur nach wird die Schleife beendet, denn nur dann sind wir sicher, dass keine Wohnung hatte die Nummer gesucht wird.

    Bezug auf den code in der Frage, du hast Recht: die variable chosen sollten deklariert worden vor die for - Schleife, so dass auf Sie verwiesen werden kann nach der Schleife. Der Weg, den Sie erklärt, es ist nur sichtbar innerhalb der if block.

    InformationsquelleAutor Óscar López

  9. 0

    Variable chosen von der Funktion zurückgegeben wird out-of-scope-hier deklarieren Sie am Anfang der Methode, kurz bevor die Schleife beginnt.

    InformationsquelleAutor Ameer Moaaviah

Schreibe einen Kommentar