Author in Vector

A

Alina_C

Mitglied
Bei folgenden Testprogramm habe ich ein Problem.
Die Ausgabe sieht folgendermaßen aus:
Code: 
null0
Code: 
ID: 2 Title: title2 Author: [author2.1, author2.2, author2.3, author2.4]
Code: 
ID: 3 Title: title3 Author: [author3.1, author3.2, author3.3]

Dabei sollte die erste Zeile so aussehen:
Code: 
ID: 1Title: title1 Author: [author1.1, author1.2, author1.3]


Java: 
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;
	}
}


Java: 
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;
	}
}


Java: 
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)
 
Zuletzt bearbeitet von einem Moderator:
Du hast aber kein Buch mir der Id 0, weil du ja mit 1 anfängst. Deshalb bekommst du [c]null+id[/c] ->
Code: 
String s = "null"+id;
zurück.
 
Das Problem war eine "out of range"-Exception.
Wenn ich die for Schleife ändere (in
Code: 
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 hat gesagt.:
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. ;-)
Zum Vergrößern anklicken....

Danke, das wars :applaus:

Hab's in
Java: 
			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?
Code: 
public String getBook(int id-1){
zb geht nicht.
 
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.)
Java: 
for(Book book:books){
     if (id == book.getId()){
         s = "ID: "+book.getId()+" Title: "+book.getTitle()+" Author: " +book.getAuthors();
     }
}

Dein
Code: 
getBook
macht eigentlich genau das was die typische Anwendung von
Code: 
toString
in der Klasse Book sein sollte.
Java: 
 String toString() {
     return = "ID: "+this.getId()+" Title: "+this.getTitle()+" Author: " +this.getAuthors();
}

Die Funktion
Code: 
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:

Java: 
 public String getBook(int id){
        return books.getKey(id).toString();
    }
Jetzt wurde es wohl doch mehr als nur ein Hinweis. 😉
 
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.)
 
Alina_C hat gesagt.:
Im Parameter der Methode sind keine arithmetischen Operationen erlaubt oder?
Code: 
public String getBook(int id-1){
zb geht nicht.
Zum Vergrößern anklicken....

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)
 
Zuletzt bearbeitet:
langhaar! hat gesagt.:
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)
Zum Vergrößern anklicken....

sie meint in der signatur der methode. da geht es natürlich nicht,
aber bei einen normalen methodenaufruf schon!
 
AquaBall hat gesagt.:
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.)
Java: 
for(Book book:books){
     if (id == book.getId()){
         s = "ID: "+book.getId()+" Title: "+book.getTitle()+" Author: " +book.getAuthors();
     }
}

Dein
Code: 
getBook
macht eigentlich genau das was die typische Anwendung von
Code: 
toString
in der Klasse Book sein sollte.
Java: 
 String toString() {
     return = "ID: "+this.getId()+" Title: "+this.getTitle()+" Author: " +this.getAuthors();
}

Die Funktion
Code: 
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:

Java: 
 public String getBook(int id){
        return books.getKey(id).toString();
    }
Jetzt wurde es wohl doch mehr als nur ein Hinweis. 😉
Zum Vergrößern anklicken....



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.
 
Alina_C hat gesagt.:
Also die toString Methode wird in der Klasse Book <überschrieben> ?
Zum Vergrößern anklicken....
Ja, die solltest du in deiner Klasse Book überschreiben.
(@override
String toString() {...})

Alina_C hat gesagt.:
Eine getKey() Methode steht nicht zur Verfügung und eine java.util.HashList kann ich für einen Import nicht finden.
Zum Vergrößern anklicken....
Naja dann halt:
Code: 
import java.util.Hash[B]Map[/B];
wer wird das schon so genau nehmen? 😉
 
AquaBall hat gesagt.:
Ja, die solltest du in deiner Klasse Book überschreiben.
(@override
String toString() {...})
Zum Vergrößern anklicken....
Okay danke.

AquaBall hat gesagt.:
Naja dann halt:
Code: 
import java.util.Hash[B]Map[/B];
wer wird das schon so genau nehmen? 😉
Zum Vergrößern anklicken....

Kann es sein, dass die Methode einfach nur get heißt? Nur halt eben angewandt an einer Map ?
(laut javadoc eg:
Code: 
Map m = Collections.synchronizedMap(new HashMap(...));
)
Das machts dann irgendwie doch komplizierter als zunächst von mir gedacht.
 
Wenn du aus der HashMap holst, dann machst du das durch:
Code: 
get(key)

Wenn du loopen willst kommst du mit getKey() an den key:
Java: 
for( Map.Entry<String,Double> entry : myHashMap.entrySet() ) {
	k=entry.getKey();;
	v =entry.getValue();
	...
}
 

Aktuelle Jobs

Neue Themen

Ähnliche Themen

Eclipse: @author Standard ändern?
Antworten: 1
Eclipse - @author anpassen
Antworten: 5
Author Name ändern
Antworten: 9
@author vor Methode (eclipse)
Antworten: 1
@Author-Tags entfernen
Antworten: 2

Zurück
Oben