Author in Vector

[Alina_C]

Bei folgenden Testprogramm habe ich ein Problem.
Die Ausgabe sieht folgendermaßen aus:

null0

ID: 2 Title: title2 Author: [author2.1, author2.2, author2.3, author2.4]

ID: 3 Title: title3 Author: [author3.1, author3.2, author3.3]

Dabei sollte die erste Zeile so aussehen:

ID: 1Title: title1 Author: [author1.1, author1.2, author1.3]

package b1.a2test;

import java.util.Vector;

public class Book {
	//classvar
	private long id;
	private String title;
	private Vector <String> authors;
	private String publisher;
	private Integer year;
	private String isbn;
	
	//constructor
	public Book(long id){
		this.id = id;
		this.authors = new Vector <String> ();
		this.publisher = "";
		this.isbn = "";
		this.year = null;
	}
	
	public long getId(){
		return this.id;
	}
	
	public void setTitle(String title){
		this.title = title;
	}
	
	public String getTitle(){
		return this.title;
	}
	
	public void addAuthors(String author){
		authors.add(author);
	}
	
	public Vector <String> getAuthors(){
		return this.authors;
	}
	
	public void setPublisher(String publisher){
		this.publisher = publisher;
	}
	
	public String getPublisher(){
		return this.publisher;
	}
	
	public void setYear(int year){
		this.year = year;
	}
	
	public int getYear(){
		return this.year;
	}
	
	public void setIsbn(String isbn){
		this.isbn = isbn;
	}
	
	public String getIsbn(){
		return this.isbn;
	}
}

package b1.a2test;
import java.util.Vector;

public class BookAdmin {
	Vector <Book> books = new Vector <Book> ();
	
	public void addBook(Book tmp){
		books.add(tmp);
	}
	
	public String getBook(int id){
		String s = "null"+id;
//		System.out.println(books.get(0).getId());
		for(int i = 0; i < books.size()-1; i++){
			if (id == books.get(i).getId()){
				s = "ID: "+books.get(id).getId()+" Title: "+books.get(id).getTitle()+" Author: "
						+books.get(id).getAuthors();
			}
		}
		return s;
	}
}

package b1.a2test;
import java.util.Vector;

public class Main {
	public static void main (String[] args){
		Book book1 = new Book(1);
		book1.setIsbn("isbn1");
		book1.setPublisher("publisher1");
		book1.setTitle("title1");
		book1.setYear(2001);
		book1.addAuthors("author1.1");
		book1.addAuthors("author1.2");
		book1.addAuthors("author1.3");
		
//		System.out.println("ID: "+book1.getId()+" Title: "+book1.getTitle()+" Author: "
//				+book1.getAuthors());
		
		Book book2 = new Book(2);
		book2.setIsbn("isbn2");
		book2.setPublisher("publisher2");
		book2.setTitle("title2");
		book2.setYear(2002);
		book2.addAuthors("author2.1");
		book2.addAuthors("author2.2");
		book2.addAuthors("author2.3");
		book2.addAuthors("author2.4");
		
//		System.out.println("ID: "+book2.getId()+" Title: "+book2.getTitle()+" Author: "
//				+book2.getAuthors());
		
		Book book3 = new Book(3);
		book3.setIsbn("isbn3");
		book3.setPublisher("publisher3");
		book3.setTitle("title3");
		book3.setYear(2003);
		book3.addAuthors("author3.1");
		book3.addAuthors("author3.2");
		book3.addAuthors("author3.3");
		
//		System.out.println("ID: "+book3.getId()+" Title: "+book3.getTitle()+" Author: "
//				+book3.getAuthors());
		
		BookAdmin ba1 = new BookAdmin();
		ba1.addBook(book1);
		ba1.addBook(book2);
		ba1.addBook(book3);
		System.out.println(ba1.getBook(0));
		System.out.println(ba1.getBook(1));
		System.out.println(ba1.getBook(2));
	}
}

Was mache ich bitte falsch? Ich komm einfach nicht drauf :bahnhof:
(Wenn möglich, bitte nicht direkt die Lösung sagen, sondern nur Hinweise geben)

 

[AquaBall]

Du beginnst mit Book[1], das heißt du lässt Book[0] (= das erste Element im Array) leer, also null.

 

[Alina_C]

Aber ich verwende doch garkein Array?
Die Zahl beim erzeugen von book-Objekten soll als ID verwendet werden.

 

[tfa]

Du hast aber nie ein Book mit der Id 0 erzeugt.

 

[Final_Striker]

Du hast aber kein Buch mir der Id 0, weil du ja mit 1 anfängst. Deshalb bekommst du [c]null+id[/c] ->

String s = "null"+id;

zurück.

 

[Alina_C]

Stimmt, aber selbst wenn ich

System.out.println(ba1.getBook(3));

kommt dann null3 raus

 

[tfa]

Stimmt, aber selbst wenn ich

System.out.println(ba1.getBook(3));

kommt dann null3 raus

Weil deine Schleife ein Element zu wenig besucht. Die läuft nur bis i<(3-1).

 

[Final_Striker]

Das liegt daran, dass deine Schleife nur bis zum vorletzten Element läuft.

 

[Alina_C]

Das Problem war eine "out of range"-Exception.
Wenn ich die for Schleife ändere (in

for(int i = 0; i < books.size(); i++){

), bekomme ich diesen Fehler wieder.

Müssen in dem Fall die Bücher mit der ID 0 beginnen? Aber die ID sollte dem Programm doch eigentlich egal sein.

 

[Final_Striker]

Wenn ich die for Schleife ändere (in for(int i = 0; i < books.size(); i++){ ), bekomme ich diesen Fehler wieder.

Weil du in der Schleife auf verschiedene Indexe zugreifst

[c]books.get(i).getId()[/c] und [c]books.get(id).getId()[/c]

das geht natürlich nicht. ;-)

 

[Pippl]

Müssen in dem Fall die Bücher mit der ID 0 beginnen? Aber die ID sollte dem Programm doch eigentlich egal sein.

Sogesehen egal, aber wenn du keines hinzufügst mit der ID 0, dann kannst du auch nicht erwarten das er zufällig eines findet wenn du danach suchst

 

[Alina_C]

Weil du in der Schleife auf verschiedene Indexe zugreifst

[c]books.get(i).getId()[/c] und [c]books.get(id).getId()[/c]

das geht natürlich nicht. ;-)

Danke, das wars :applaus:

Hab's in

			if (id == books.get(i).getId()){
				s = "ID: "+books.get(id-1).getId()+" Title: "+books.get(id-1).getTitle()+" Author: "
						+books.get(id-1).getAuthors();
			}

geändert.

Hoffe ich hab das jetzt richtig verstanden:
Also der Vector beginnt bei 0 wie ein Array.
Ich habe der Suchmethode aber die ID übergeben (Die, da ich kein Buch mit der ID 0 erstellen wollte, mit 1 beginnt). Um den Index wieder anzupassen, bleibt mir also nichts anderes übrig als die id bei den Abfragen zu ändern.

Im Parameter der Methode sind keine arithmetischen Operationen erlaubt oder?

public String getBook(int id-1){

zb geht nicht.

 

[AquaBall]

Schon klar, dass du kein Array verwendest, aber du wolltest ja auch nur einen Hinweis.

Übrigens ist genau das der Grund von Java's genialen Konstruktionen.
Dein Schleife gehört typisch ohne Zähler konstruiert. (Ist der wesentliche Unterschied zw. Array und List.)

for(Book book:books){
     if (id == book.getId()){
         s = "ID: "+book.getId()+" Title: "+book.getTitle()+" Author: " +book.getAuthors();
     }
}

Dein

getBook

macht eigentlich genau das was die typische Anwendung von

toString

in der Klasse Book sein sollte.

 String toString() {
     return = "ID: "+this.getId()+" Title: "+this.getTitle()+" Author: " +this.getAuthors();
}

Die Funktion

books.getKey(object)

(statt books.get(i).getId()!) wurde in HashList schon erfunden und macht die Schleife überhaupt auch überflüssig.
Somit sollte das so aussehen:

 public String getBook(int id){
        return books.getKey(id).toString();
    }

Jetzt wurde es wohl doch mehr als nur ein Hinweis.

 

[Alina_C]

Trotzdem vielen vielen Dank, denn man will doch immer wissen wie man's besser machen kann (und leichter )

 

[Final_Striker]

Hoffe ich hab das jetzt richtig verstanden:

Nein hast du leider nicht. Der Index muss schon i sein, den brauchst du auch nicht anzupassen.

 

[AquaBall]

Du hast gedanklich INDEX und ID/KEY noch nicht deutlich genug getrennt!

Wenn du die/eine Schleife durchlaufen willst, dann "muss" die von 1 bis n gehen. (=Index)

Versuch aber mal deine IDs z.B 5010,5053,7311 zu verwenden, (= ID, so wie ein ISBNummer) dann erkennst du einen Fehler sofort.
(Das ist ja vermutlich der Grund, warum du ein ArrayList verwendest.)
(Übrigens solltest du Vektor nicht mehr benutzen.)

 

[langhaar!]

Im Parameter der Methode sind keine arithmetischen Operationen erlaubt oder?

public String getBook(int id-1){

zb geht nicht.

Doch. Wenn als Parameter ein int erwartet wird, kann das ein beliebiger Ausdruck sein, der einen int ergibt. Dein Beispiel ist falsch, weil die Deklaration da nichts zu suchen hat. Es muss getBook(id-1) heissen. (Zumindest ist es dann syntaktisch richtig)

 

[Pippl]

Doch. Wenn als Parameter ein int erwartet wird, kann das ein beliebiger Ausdruck sein, der einen int ergibt. Dein Beispiel ist falsch, weil die Deklaration da nichts zu suchen hat. Es muss getBook(id-1) heissen. (Zumindest ist es dann syntaktisch richtig)

sie meint in der signatur der methode. da geht es natürlich nicht,
aber bei einen normalen methodenaufruf schon!

 

[langhaar!]

Oops. Da habe ich nicht genau genug hingesehen.

 

[Alina_C]

Schon klar, dass du kein Array verwendest, aber du wolltest ja auch nur einen Hinweis.

Übrigens ist genau das der Grund von Java's genialen Konstruktionen.
Dein Schleife gehört typisch ohne Zähler konstruiert. (Ist der wesentliche Unterschied zw. Array und List.)

for(Book book:books){
     if (id == book.getId()){
         s = "ID: "+book.getId()+" Title: "+book.getTitle()+" Author: " +book.getAuthors();
     }
}

Dein

getBook

macht eigentlich genau das was die typische Anwendung von

toString

in der Klasse Book sein sollte.

 String toString() {
     return = "ID: "+this.getId()+" Title: "+this.getTitle()+" Author: " +this.getAuthors();
}

Die Funktion

books.getKey(object)

(statt books.get(i).getId()!) wurde in HashList schon erfunden und macht die Schleife überhaupt auch überflüssig.
Somit sollte das so aussehen:

 public String getBook(int id){
        return books.getKey(id).toString();
    }

Jetzt wurde es wohl doch mehr als nur ein Hinweis.

Sorry, da hab ich doch noch Probleme.
Also die toString Methode wird in der Klasse Book <überschrieben> ?
Eine getKey() Methode steht nicht zur Verfügung und eine java.util.HashList kann ich für einen Import nicht finden.

 

[AquaBall]

Also die toString Methode wird in der Klasse Book <überschrieben> ?

Ja, die solltest du in deiner Klasse Book überschreiben.
(@override
String toString() {...})

Eine getKey() Methode steht nicht zur Verfügung und eine java.util.HashList kann ich für einen Import nicht finden.

Naja dann halt:

import java.util.Hash[B]Map[/B];

wer wird das schon so genau nehmen?

 

[Alina_C]

Ja, die solltest du in deiner Klasse Book überschreiben.
(@override
String toString() {...})

Okay danke.

Naja dann halt:

import java.util.Hash[B]Map[/B];

wer wird das schon so genau nehmen?

Kann es sein, dass die Methode einfach nur get heißt? Nur halt eben angewandt an einer Map ?
(laut javadoc eg:

Map m = Collections.synchronizedMap(new HashMap(...));

)
Das machts dann irgendwie doch komplizierter als zunächst von mir gedacht.

 

[AquaBall]

Wenn du aus der HashMap holst, dann machst du das durch:

get(key)

Wenn du loopen willst kommst du mit getKey() an den key:

for( Map.Entry<String,Double> entry : myHashMap.entrySet() ) {
	k=entry.getKey();;
	v =entry.getValue();
	...
}