Variabler String

[GRHM93]

Hallo alle zusammen,

ich hab mal wieder eine Anfängerfrage und hoffe, ihr könnt mir helfen.

Also, wir sollen ein Taxi in Java modellieren und den Großteil der Aufgabenstellung hab ich schon fertig, nur beim letzten Punkt haperts...
Folgendes: Ich hab eine Klasse "Taxi" und eine Klasse "Human". Die Klasse Human ist fertig und funktioniert, die beinhaltet nur Vor- und Nachname eines Menschen, die getter und setter dazu und die toString()-Methode wurde überschrieben, sodass sie "Vorname Nachname" eines Menschen ausgibt. Jetzt zur Taxi-Klasse. Jedes Taxi hat einen Fahrer und maximal 4 Passagiere. Ein Taxi kann nur mit Fahrer "erschaffen" werden. Die Passagiere können einzeln einsteigen und nur zusammen aussteigen. Es gibt eine Methode "add" zum Hinzufügen der Passagiere.
Jetzt zu meinem Problem: Die Methode toString() der Taxiklasse soll so überschrieben werden, dass sie je nach Anzahl der Passagiere folgendes ausgibt: "Das ist das Taxi von (Taxifahrername). Es fahren (Name 1. Person bis Name 4. Person) mit."
Also wenn nur 2 Personen mitfahren, dann stehn da halt auch nur die Namen der 2 Personen usw.
Wie bekomme ich es also hin, dass der String, denn die toString-Methode zurückgibt sozusagen variabel ist? Ich dachte mir das iwie mit einer For-Schleife, aber weiß nicht, wie ich das realisieren soll.
Hier erstmal mein bisheriger Code der Taxi-Klasse:

import java.util.ArrayList;

public class Taxi {
	
	private String driverName;
	private String driverForename;
	
	Human driver = new Human(driverName, driverForename);
	
	ArrayList<Human> passengers = new ArrayList<Human>();	
	
	public Taxi(Human driver) {
		this.driver = driver;
	}
	
	public String getDriverName() {
		String output = new String(driver.getForname() + " " + driver.getName());
		return output;
	}
	
	public String add(Human x) {
		String output = new String();
		if (passengers.size()<4) {
			passengers.add(x);
			output = new String(x.toString() + " gets in.");
		} else {
			output = new String("We are sorry, " + x.toString() + ". The Taxi is full.");
		}
		System.out.println(output);
		return output;
	}
	
	public String toString() {
		String output = new String();
		
		if (passengers.isEmpty()) {
			output = new String("This is the taxi of " + driver.toString() + ". He takes nobody along.");
			
		} else {
			for(int i=0; i<passengers.size(); i++) {
				output = new String("This is the Taxi of " + driver.toString()
						+ ". He takes " + passengers.get(i).toString() + " along.");
			}
		}
		
		return output;
		
	}
	
}

Aber so wie es jetzt ist gehts ja nicht, weil der output der Schleife dann nur mit dem einen Passagier an Stelle i ist, aber nicht mit allen Passagieren.
Ich hoffe, ihr könnt mir helfen!

Vielen Dank!

Gruß GRHM93

 

[Androbin]

Meinst du so 8

String s = "Das ist das Taxi von " + Taxifahrername + ". Es fahren "
for ( int i = 0; i < passengers.size(); i++ ) {
	s += passengers.get( i ).toString() + ", ";
}
s += " mit."

 

[turtle]

Ich bin ja Verfechter von Test-driven-development (TDD).

Also habe ich einen Test geschrieben, der das mal nachstellt

	@Test
	public void test() {
		Taxi taxi = new Taxi(new Human("Turtle", "Bob"));
		Human humanMerkel = new Human("Merkel", "Angela");
		Human humanGabriel = new Human("Gabriel", "Sigmar");
		taxi.add(humanMerkel);
		taxi.add(humanGabriel);
		assertEquals(2, taxi.passengers.size());
		assertEquals("Angela Merkel", taxi.passengers.get(0).toString());
		assertEquals("Sigmar Gabriel", taxi.passengers.get(1).toString());
		assertEquals(
				"This is the Taxi of Bob Turtle. He takes Angela Merkel along.This is the Taxi of Bob Turtle. He takes Sigmar Gabriel along.",
				taxi.toString());
	}

Das gibt als Ausgabe (Auszug):

Human@20ad9418 gets in.

DAS ist die Ausgabe der add-Methode, in der du auf Human-toString() aufrufst. Diese hast du aber NICHT überschrieben und daher gibt Java die Adresse des Objektes aus.
Also habe ich danach in Human eingefügt:

	@Override
	public String toString() {
		return vorName + " " + name;
	}

Und schon gibt es einen "schönen" Text aus

Angela Merkel gets in.

Aber am Ende kommt für das Taxi

This is the Taxi of Bob Turtle. He takes Sigmar Gabriel along.

das ist noch falsch und warum? In deiner toString() Methode steht

output = ...

Also überschreibst du den Text jedesmal und daher "siehst" nur den zuletzt hinzugefügten Passagier. Geändert zu

output +=

und es kommt

This is the Taxi of Bob Turtle. He takes Angela Merkel along.This is the Taxi of Bob Turtle. He takes Sigmar Gabriel along.

Ob du ZWEIMAL den Fahrer vom Taxi ausgeben möchtest, weiss ich nicht. Im Test habe ich angenommen das das so beabsichtigt ist

 

[GRHM93]

Hallo,

vielen Dank für eure Hilfe, es funktioniert jetzt eigentlich alles. Ich bin nicht draufgekommen, das ich ja nur einen einzelnen String erzeugen muss, in dem die Namen stehen und den ich dann in die Ausgabe mit reinsetzten kann.
Also die Ausgabe soll dann schon lauten:
Das ist das Taxi von ... . Es transportiert ..., ..., ... und ... .

Bei meiner Ausgabe gibts jetzt nur noch ne Schönheitskorrektur und ich hab kein Plan, wo bzw. wie ich das einbringen könnte: Wenn mehr als ein Passagier im Taxi sitzen, dann soll die Ausgabe vor dem letzten Passagier ein "und" enthalten.
Bei zwei Passagieren also:
Das ist das Taxi von ... . Es transportiert ... und ... .
Bei drei dann:
Das ist das Taxi von ... . Es transportiert ..., ... und ... .
Könntet ihr mir da noch einen Rat geben?
Hier nochmal die vollständige Taxi-Klasse:

import java.util.ArrayList;

public class Taxi {
	
	private String driverName;
	private String driverForename;
	
	Human driver = new Human(driverName, driverForename);
	
	ArrayList<Human> passengers = new ArrayList<Human>();	
	
	public Taxi(Human driver) {
		this.driver = driver;
	}
	
	public String getDriverName() {
		String output = new String(driver.getForename() + " " + driver.getName());
		return output;
	}
	
	public String add(Human x) {
		String output = new String();
		if (passengers.size()<4) {
			passengers.add(x);
			output = new String(x.toString() + " gets in.");
		} else {
			output = new String("We are sorry, " + x.toString() + ". The Taxi is full.");
		}
		System.out.println(output);
		return output;
	}
	
	public Human[] allGetOut() {
		Human[] humanArray = passengers.toArray(new Human[passengers.size()]);
		passengers.clear();
		
		return humanArray;
	}
	
	public String toString() {
		String output = new String();
		String s = new String();
		
		if (passengers.isEmpty()) {
			output = new String("This is the taxi of " + driver.toString() + ". He takes nobody along.");
			
		} else {
			for (int i=0; i<passengers.size(); i++) {
				s = s + passengers.get(i).toString() + " ";
			}
			
			output = "This is the taxi of " + driver.toString() + ". He takes " + s + "along.";
		}
		
		return output;
		
	}
	
}

@turtle
Ich versuche eigentlich auch so oft wie möglich zu testen, denn das ist ja die sicherste Variante, um das erwünschte Ergebnis auch wirklich zu erhalten (;
Ich hatte doch aber nur meine Taxi-Klasse gepostet, desewegen hast du für Human keine überschriebene toString-Methode gefunden, in meiner Human-Klasse hatte ich die geschrieben, genau wie du, und hat auch funktioniert

Vielen Dank euch!

Gruß Manu

 

[turtle]

Das ist "normales" String Handling und du solltest vermeiden Teile über + in einen String zu addieren.

Da String immutable, also unveränderlich, ist, muss die JVM jedesmal Strings anlegen, kopieren etc.

Benutze dafür besser StringBuilder. Ob's genau richtig ist, musst du testen, aber ungefähr so sollte es gehen

	public String toString() {
		StringBuilder sb = new StringBuilder();
		sb.append("This is the taxi of ");
		sb.append(driver);
		sb.append(".");

		if (passengers.isEmpty()) {
			sb.append("He takes nobody along.");

		} else {
			sb.append("He takes");
			for (int i = 0; i < passengers.size(); i++) {
				sb.append(" ");
				if (i > 0) {
					sb.append("und ");
				}
				sb.append(passengers.get(i));
			}

			sb.append(".");
		}
		return sb.toString();

	}//This is the taxi of Bob Turtle.He takes Angela Merkel und Sigmar Gabriel.

 

[GRHM93]

Super, vielen Dank für die schnellen Hilfen, es funktioniert alles so wies soll. Der Tipp mit dem Stringbuilder war Gold wert und wird mir auch in Zukunft viel Arbeit erleichtern!