Schleifen-Problem mit Arrays

[Java-Problems]

Guten Morgen zusammen,
ich habe ein Problem mit den Schleifen...

und zwar sollte beim Aufrufen der MEthode "add" (siehe 1. Code) ja ein Objekt Recipe erstellt werden und die dazugehörigen Ingredients in eine ArrayList aufgenommen werden.
Das Eingeben auf der Konsole etc. klappt auch alles, nur wird irgendwie nur das letzte IngredientsObjekt dann nur in den Array übernommen, sodass das erste irgendwie nicht gespeichert wurde.

	public void add(Recipe recipe) throws CocktailException {

		String inputNa = IOTools.readLine("Name of the recipe: ");
		int inputNmb = IOTools.readInt("Number of needed Ingredients: ");
		Recipe.recipeList.add(new Recipe(inputNa, inputNmb));

	
		for (int i = 0; i < inputNmb; i++) {
			System.out.println(Recipe.ingredient.ingredientList.get(i));
		}
	}

	public Recipe(String nameRec, int inputNmb) {
		this.nameRec=nameRec;
		for (int i = 0; i < inputNmb; i++) {
			Recipe.ingredient.ingredientList.add(new Ingredient());
		}
	}

public Ingredient() {
		this.nameIng = IOTools.readLine("Name of the ingredient: ");
		this.quantityNec = IOTools.readInt("Needed quantity: ");
	}

main-Methode:
Die For-Schleife sollte eig. beide Array-Objekte ausgeben, gibt aber nur das letzte aus, obwohl ja eig. der gesamte Array aufgerufen wird... D.h. wohl dass das erste Objekt nicht in die ArrayList übernommen wurde.
Was mache ich falsch?

		String inputNa = IOTools.readLine("Name of the recipe: ");
		int inputNmb = IOTools.readInt("Number of needed Ingredients: ");

activity.add(null); 
System.out.println("");
	for (int i=0; i<Recipe.ingredient.ingredientList.size();i++)}
		 System.out.println(Recipe.ingredient.ingredientList.get(i));
		 }

 

[hdi]

Soweit ich das überflogen hab, müsste es stimmen.

Poste bitte mal deinen ganzen Code, also alle Klassen die damit zusammenhängen sowie
die komplette(!) main Methode.

Sag bitte ausserdem, was du genau wo eingibst, und was als Ergebnis rauskommt, und was
du eigentlich erwartet hättest.

Danke

 

[Java-Problems]

has been deleted!

 

[hdi]

Die Übeltäter:

static String inputIng;
static String nameIng;

Das sind statische, also nicht an ein Objekt gebundene Variablen. D.h. beim Einlesen
des 1. Ingredients setzt du den Namen auf "test1", beim Einlesen des zweiten auf "test2".
In der Schleife gibtst du immer wieder diesen statischen String aus, der jetzt auf test2 ist.

Lösung:

Die Variablen musst du zu Member-Variablen machen, also NICHT static.
So hat jedes Ingredient-Objekt seinen eigenen Namen + Quantity.

EDIT: Ich seh grad du hast ja auch noch nicht-statische. Alles sehr strange..
Ich frag mich wieso denn das list.toString() dir überhaupt solche Namen ausspuckt...

Lösch einfach mal die 2 statischen Einträge, dann müsste dir erstmal eine Liste mit komischen
Zeichen rauskommen, was aber auch gut so ist.

Dann füge in die Klasse Ingredient noch folgende Methode ein:

@Override
public String toString(){
     return this.name; // <- oder wie auch immer dein Attribut da heisst hab den Code grad nicht
}

 

[Java-Problems]

herzlichen Dank!
Es Funktioniert soweit!

Noch eine Allgemeine Frage zu meinem Projekt:
Bin ich hierbei überhaupt auf dem richtigen Weg, wenn ich Ingredients und Recipes anlegen will, und dann dementsprechend einem Recipe mehrere Ingredients zuordnen will?
Benötige ich hierzu keine Datenbankanbindung oder sowas? Frage mich nämlich, wie das mit dem Speichern der Recipes laufen soll. Es wird ja nicht nur eines angelegt... Wird dann nicht jedesmal das andere überschrieben?
Eine Datenbankanbindung ist eigentlich nicht erwünscht, aber ich frage mich halt, ob das überhaupt ohne geht...

 

[hdi]

Ja dein Problem ist generell, dass du zuviel mit statischen Dingen arbeitest.
Generell gibt es in einem guten kleinen Programm kein einziges mal das Wort "static" bis
in der main-Methode.

Also davon musst du dich trennen, die Klassen sollten nur individuelle Attribute haben,
nicht static.

Und dann haste auch kein Problem mit mehrere Rezepten, weil du über Konstruktoren einfach
mehrere Objekte vom Typ Recipe anlegen kannst etc..

Weisst du was ich mein?

 

[Java-Problems]

benötige ich "static" nicht, damit ich meine Methoden etc. in anderen Klassen die keine main-Methode haben aufrufen kann etc.?
Beispielsweise: Wenn ich eine Methode "getRecipe()" erstelle, und in der BarActivity noch eine habe, die heißt:
"search(recipe recipe)", dann müsste ja diese die getRecipe-Methode implementieren und dazu würde ich doch "static" benötigen, oder?

 

[hdi]

static heisst nur, diese Methode ist nicht an ein Objekt gebunden, sondern einfach eine Funktion die
eine Klasse bereitstellt. Aber sowas ist in gutem OO-Design nicht nötig, weil alles über Objekte geht.

Hier mal Bsp, wie ich das angehen würde, vllt kannst du dir dann besser vorstellen worauf ich hinauswill:

public class Recipe {

	private String name;
	private List<Ingredient> ingredients;

	public Recipe(String name) {

		this.name = name;
		this.ingredients = new ArrayList<Ingredient>();
	}

	public void addIngredient(Ingredient i) {
		this.ingredients.add(i);
	}

	// Wird aufgerufen wenn man ein Recipe-Objekt zB per System.out.println()
	// ausgibt. Zeigt eine Liste aller Zutaten.
	@Override
	public String toString() {
		String asString = "";
		asString += "---" + this.name + "---" +"\n";
		for (Ingredient i : ingredients) {
			asString += i + "\n";
		}
		return asString;
	}
}

public class Ingredient {

	private String name;
	private int quantity;

	public Ingredient(String name, int quantity) {
		this.name = name;
		this.quantity = quantity;
	}

	public String getName() {
		return name;
	}

	public int getQuantity() {
		return quantity;
	}
	
	// Wird aufgerufen wenn man ein Ingredient-Objekt zB per System.out.println()
	// ausgibt. Zeigt den Namen und in Klammern die Menge der Zutat.
	@Override
	public String toString() {
		return this.name + " [" + this.quantity + "]";
	}
}

public class Bar {

	private List<Recipe> recipes;

	public Bar() {
		recipes = new ArrayList<Recipe>();
	}

	public void addRecipe(Recipe r) {
		this.recipes.add(r);
	}
	

	public static void main(String[] args) {

		Bar bar = new Bar();

		Recipe recipe = new Recipe("Java-Profi-Trunk");
		recipe.addIngredient(new Ingredient("Geduld", 100));
		recipe.addIngredient(new Ingredient("Übung", 500));
		recipe.addIngredient(new Ingredient("Spass", 1000));
		bar.addRecipe(recipe);

		System.out.println(recipe);
	}
}

Merkst du wo der Unterschied ist? Ich arbeite nur auf Objekten, nicht auf irgendwelche statischen Dingen.
Alles bei mir ist ein individuelles Objekt, und so sollte man es auch immer machen

 

[Java-Problems]

Okay!
Ich werde es berherzigen!
Danke für deine Hilfe!