Suchen nach ArrayObjekten

[Java-Problems]

has beeen deleted!

 

[SebiB90]

Wenn du Arrays vergleichen willst, macht man das über
[highlight=Java]Arrays.equals(arr1, arr2)[/highlight]
Allerdings müssen die Elemente da auch in der gleichen Reihenfolge sein. Du musst dir wohl eine eigene Methode schreiben, da ich denke, dass die Reihenfolgen unterschiedlich sein dürfen oder?


Nachtrag:
So sollte es ungefähr aussehen, wenn beide Element die gleichen Elemente beinhalten sollen, die Reihenfolge aber egal ist. (ungetestet)
[highlight=Java]public boolean equals(Object[] arr1, Object[] arr2) {
if (arr1.length != arr2.length) {
return false;
}
for (Object o1 : arr1) {
boolean found = false;
for (Object o2 : arr2) {
if (o1.equals(o2)) {
found = true;
break;
}
}
if (!found) {
return false;
}
}
return true;
}
[/highlight]

 

[Marco13]

Hast du bei Ingedients auch die Equals-Methode richtig überschrieben?

 

[Java-Problems]

has beeen deleted!

 

[SebiB90]

Vergleicht die equals-methode die Objekte in der entsprechenden Reihenfolge?
Gibt es nicht auch eine andere Methode die schaut ob ein Objekt im anderen Array vorhanden ist, oder nicht?

Bei Arrays#equals(Object,Object) müssen die Objekte in der gleichen Reihenfolge sein, wie ich schon beschrieben hab. Ob es eine fertige Methode gibt ka, aber nimm mein Code von oben, entferne die erste If Abfrage, dann sollte die Methode true zurück geben, wenn alle Element aus arr1 auch in arr2 drin sind.

 

[Java-Problems]

has beeen deleted!

 

[SebiB90]

Ich vermute mal in die, in der die search-Methode steht, oder?

Jop, wäre logisch 😉

 

[Java-Problems]

has beeen deleted!

 

[Java-Problems]

has beeen deleted!

 

[Java-Problems]

has beeen deleted!

 

[SebiB90]

Also erstens:
Mir ist aufgefallen, dass du Ingredient mit RecipeIngredient vergleichst.
Du müsstest dann die equals Methoden der beiden überschreiben, damit ein Vergleich überhaupt möglich ist. Oder erbt RecipeIngredient von Ingredient?

Des weiteren habe ich noch etwas gefunden. Benutz in SearchCriteria auch eine List, dann kannste die Methode containsAll(Collection c) benutzen, dann brauchste keine eigene Methode 😉

 

[Java-Problems]

has beeen deleted!

 

[SebiB90]

Welche Array oder Liste sollte ich durch eine List ersetzen? Anstatt der LinkedList?
Wie könnte ich die equalsMethode überschreiben?

RecipeIngredient enthält wiederum Ingredients. So viel vll. noch als Information.
Die habe ich mit

Ingredient ingredient;

Miteinander verknüpft.

Das Array in SearchCriteria sollst du durch eine List (z.b. ArrayList) ersetzen. Wäre mein Vorschlag.

Dann müsstest du das Ingredient aus RecipeIngredient rausholen und dann damit vergleichen.

Die Methode kannst du einfach überschreiben indem du eine [highlight=Java]public boolean equals(Object o)[/highlight] Methode in deine Ingrendient Klasse einbaust. Du könntest dir die auch einfach über Eclipse, wenn du die IDE nutzt, die equals Methode erstellen lassen.

Evtl wäre besser, wenn du den Quellcode der 3 Klassen(Ingrendient, RecipeIngrendient, und die Klasse mit der Search Methode) mal zeigst, dann kann ich das mal ändern, so wie ich meine.

 

[Java-Problems]

has been deleted

 

[Marco13]

Die Recipe-Klasse könnte noch hilfreich sein...

 

[SebiB90]

Zudem fehlt die Klasse mit der search-Methode

 

[Java-Problems]

has beeen deleted!

 

[Marco13]

Ja, zu entscheiden, was wofür relevant ist, ist manchmal schon eine Kunst. Und ich weiß nicht, ob man erwarten kann, dass jemand den Enthusiasmus aufzubringen bereit ist, der notendig ist, um sich aus zusammenhangslosen Codestücken das zusammenzubasteln, was notwendig ist, um den Fehler nachvollziehen, und sich dann die Arbeit macht, ihn zu beheben, auf die Gefahr hin, dass der Threadersteller dann ganz enttäuscht antwortet mit "Ja, bei DIR geht das jetzt, aber bei MIR ist dies-und-das ja gaaanz anders"...

Wenn ich das richtig sehe, vergleichst du einen Array von Ingredients mit einer List von RecipeIngredients. Ein weiterer Versucht:

Vorher:

if (criteria.getIngredients().equals(recipes.get(i).getRecipeIngredient())) 
{
    recipeList.add(recipes.get(i));
}

Nachher:

Set<Ingredient> a = new HashSet<Ingredient>(Arrays.asList(criteria.getIngredients()));
List<RecipeIngredient> list = recipes.get(i).getRecipeIngredient();
Set<Ingredient> b = new HashSet<Ingredient>();
for (RecipeIngredient ri : list)
{
    b.add(ri.getIngredient());
}
if (a.equals(b)) 
{
    recipeList.add(recipes.get(i));
}