Test2: Mustertestangabe

[Neveragain]

Die Angabe zum 2. Mustertest ist online!

Fragen zum 2. Mustertest bitte nur hier stellen.

Link zum Thread über den 1. Mustertest:
http://www.informatik-forum.at/showt...stertestangabe

[SoulRiser]

/**
     * Fügt eine Playlist unter dem durch name bestimmten 
     * Namen zur Bibliothek hinzu. Falls bereits eine 
     * Playlist mit dem Namen existiert, soll die bestehende 
     * durch die neue (durch playlist angegebene) Playlist 
     * ersetzt werden. 
     *
     * @param name Der Name, unter dem die 
     *     Playlist hinzugefügt werden soll 
     * @param playlist Die Playlist, welche 
     *     hinzugefügt werden soll 
     */
     public void addPlaylist(String name, Playlist playlist) {

das is ne fangfrage oda? ^^

[Neveragain]

Warum ist das eine Fangfrage?
Das ist der Tipp, dass sich eine Hashmap als Datenstruktur eignet

[SoulRiser]

Warum ist das eine Fangfrage?
Das ist der Tipp, dass sich eine Hashmap als Datenstruktur eignet

aso naja weil halt values mit gleichen keys überschrieben werden ^^
ok naja das mit der hashmap war mir schon vorher klar ^^

[Dory]

weils keine dummen fragen gibt..:

weiß wer wie die im repetitorium gezeigte methode:

public Playlist(Playlist playlist1, Playlist playlist2) {
		//ab hier erinner ich mich leider nicht mehr genau.. 
		this(playlist1);
		for(Song s : playlist1){
			playlist.addSong(s);
		}
		
		this(playlist2);
		for(Song s : playlist2){
			playlist.addSong(s);
	        }
		


}

wirklich ausgesehn hat bzw kann mir wer erklären was es mit dem this.() auf sich hat?

[Neveragain]

Du rufst damit den Kopierkonstruktor auf.

public Playlist(Playlist original)

this bezieht sich auf die Instanz von Playlist, in der du dich gerade befindest.

[Dory]

ajaaaa... *auf den kopf greif*

danke.

[mautsch]

Warum funktioniert dies als Kopierkonstruktor nicht irgendwie hänge ich grad komplett

public Playlist(Playlist original) {


playlist =new ArrayList<Song>();

for(Song s : original){
playlist.addSong(s);
}


}

[Neveragain]

Was funktioniert daran nicht?
Wenn du den Aufruf von oben meinst: Du erzeugst eine neue Playlist, daher sind immer nur die Songs aus original enthalten.

[mautsch]

Wenn ich es über die Konsole aufrufe kommt die fehler meldung " foreach not applicable to expression type"

[sutupud]

Warum funktioniert dies als Kopierkonstruktor nicht irgendwie hänge ich grad komplett

public Playlist(Playlist original) {


playlist =new ArrayList<Song>();

for(Song s : original){
playlist.addSong(s);
}


}

Weil du nicht über Playlist iterieren kannst, sondern - in deinem Fall - über "original.playlist" - also die konkrete Liste der Songs.

[Neveragain]

Oh, hups
Natürlich:
for (Song s: original.playlist)

[mautsch]

Ok danke

[Dory]

denselben fehler bekomm ich auch.. vlt ist der copy-konstruktor den ich dir geschickt hab doch nicht ganz richtig. :/

edit: zu langsam^^

[bernhard2202]

Mehrere Songs können zu einer Playlist (Playlist) zusammengefasst werden. Eine
Playlist hat eine bestimmte Reihenfolge, ein Song darf jedoch nur einmal in der Playlist
enthalten sein.

Sehe ich das richtig das ein Playlist dann einfach eine HashMap ist und wir halt die "getHashCode()" des songs so umschreiben müssen das alle eigenschaften drin vorkommen da ein song ja keine eindeutig identifizierende hat?.

[SoulRiser]

Sehe ich das richtig das ein Playlist dann einfach eine HashMap ist und wir halt die "getHashCode()" des songs so umschreiben müssen das alle eigenschaften drin vorkommen da ein song ja keine eindeutig identifizierende hat?.

für playlist eignet sich eine einfache arraylist besser, die songs beinhaltet

[bernhard2202]

für playlist eignet sich eine einfache arraylist besser, die songs beinhaltet

und du überprüfst einfach ob sich beim Hinzufügen schon ein solcher Song in der ArrayList befindet??

[SoulRiser]

und du überprüfst einfach ob sich beim Hinzufügen schon ein solcher Song in der ArrayList befindet??

jap, mit contains(Object elem)

[bernhard2202]

jap, mit contains(Object elem)

ich sollte wohl zuerst lesen steht eh im Kommentar das es reicht zu überprüfen ob GENAU DIE Instanz bereits vorkommt sorry.

[bernhard2202]

Eine dumme frage hab ich noch. Eine Plalist soll an ihrem Namen identifiziert werden aber weder in den Testfällen noch in dem Codegerüst ist ein Name vorhanden wo sollen wir den jetzt einbauen oder sollen wir einen namen generieren?

[Neveragain]

Schau dir mal Testfall 4 und 5 genauer an

[sutupud]

ich sollte wohl zuerst lesen steht eh im Kommentar das es reicht zu überprüfen ob GENAU DIE Instanz bereits vorkommt sorry.

... und auch wenn nicht wär's sinnvoll hier contains zu verwenden und dafür equals von Song zu überschreiben.

[Sorex1]

Gibt es irgendwo die Folien von Donnerstag letzter Woche bezgl. HasMaps und Listen ?
Ich hatte zum Zeitpunkt der Vorlesung FM Abgabegespräch

[Sorex1]

Oder irgendeine Seite wo ich das nachlesen kann

[0727384]

Hallo,

Songs und Playlist funktioniert ansich, aber ich habe ein seltsames Problem bei extractPlaylistByArtist

 public Playlist extractPlaylistByArtist(String artist) {
  Playlist newplaylist= new Playlist();
  System.out.println(artist);
  
  
  for (int x = 0;  x < this.playlist.size(); x++) {
   System.out.println(this.playlist.get(x).getArtist());
   
   if (this.playlist.get(x).getArtist().equals(artist)) {
     System.out.println("gleich")
  }
  }
return newplaylist; 
}

Ich muss ja eine Playlist zurückgeben, also hab ich eine Playlist newplaylist= new Playlist(); angelegt, stimmt das so???

Wenn ich das nun ausführe, so wird nur der System.out.println(artist); ausgeführt und die for schleife gar nicht.

Wenn ich das Playlist newplaylist= new Playlist(); lösche oder runter schiebe unter die for Schleife, wird es ausgeführt, ausgegeben und alle gleichen mit "gleich" markiert. Ich wollte die Playlist VORHER erstellen und dann die gleiche adden (statt gleich auszugeben (zum testen)).

Was mach ich hier falsch???

Ansonsten funktioniert Testfall 1 + 2 + 3 bis zu dem Zeitpunkt mit dem extractPlaylistByArtist normal.

Wäre für Tipps oder Hinweise dankbar.

[SoulRiser]

Oder irgendeine Seite wo ich das nachlesen kann

http://openbook.galileocomputing.de/javainsel/javainsel_13_002.html#dodtp5e6355b5-c483-4ca9-92f6-7bb32314150d
http://openbook.galileocomputing.de/javainsel/javainsel_13_004.html#dodtpa0fe5e88-64cd-4e41-9c55-fb9fd98e0b0c

gute lektüre, bündig und informativ

@0727384 post mal bitte deine methode ohne den prints, sondern so wie sie gehören sollte, dann kann ich dir vlt sagen was du falsch machst

[sutupud]

du musst natürlich den song zur neuen playlist hinzufügen, wenn der artist passt...
btw, wenn man den index nicht braucht, ist eine foreach-schleife fast immer eleganter:

for (Song s : playlist) {
  // do your stuff
}

edit: zum testen wäre es vielleicht gut, wenn du die playlists auch ausgibst...
frage: playlist ist nicht zufällig static?

[0727384]

OK, ist wohl ein Fehler von mir wo anders, playlist ist wirklich static da mir sonst eine NullPointerException kam/kommt. Das muss ich mir noch ansehen, was da los ist

danke trotzdem

[Mihai238]

... und auch wenn nicht wär's sinnvoll hier contains zu verwenden und dafür equals von Song zu überschreiben.

Ich habe versucht, equals von Song zu überschreiben, aber contains() gibt immer false aus. Hat jemand eine Idee wieso?

[sutupud]

Ich habe versucht, equals von Song zu überschreiben, aber contains() gibt immer false aus. Hat jemand eine Idee wieso?

schwierig ohne deine implementierung zu kennen...
sollte jedenfalls so ähnnlich aussehen:

@Override
public boolean equals(Object other) {
    if (other == null || !(other instanceof Song)) {
        return false;
    } else {
        Song s = (Song)other;
        return s.getArtist().equals(artist) && s.getTitle().equals(title) && s.getDuration() == duration;
    }
} 

[0727384]

So alles gelöst bis auf ein Problem mit public Library(Library libraryToImport, String[] listOfNames)

ich durchlaufe die HashMap mit
for (Map.Entry e : libraryToImport.lib.entrySet()) {

und möchte dann die passenden (also in der Array listOfNames vorgegebenen) Dinge hinzufügen zur Library.

Dachte so irgendwie:
addPlaylist(e.getKey (), e.getValue());

Aber leider ist dann e.getValue () ein String und keine Playlist?

Wie kann ich denn die Playlist dann hier ansprechen?


Danke im Voraus, tu mir da zeitweise schwer

@ TUTOR / LVA:
Wieso (oder wo?) gibts die Muster - Lösungen? Wär schon praktisch bzw. hilfreich. Zumindest kurz vor dem Test (damit man es auch selbst implementiert und nicht gleich aufgibt und nachsieht).

@ FORUM / BETREIBER:
Hatte ein Smilie am Ende des Beitrages und nach dem Edit konnte ich nur VOR den Smilie schreiben, aber nicht nachher den Cursor platzieren (weder mit Maus noch mit Tastatur), ist wohl ein Bug? Das hier ist aber grad IE8 Später mal testen wie es im FF aussieht.

[Neveragain]

So alles gelöst bis auf ein Problem mit public Library(Library libraryToImport, String[] listOfNames)

for (String s: listOfNames)
			addPlaylist(s,new Playlist(libraryToImport.playlists.get(s)));

[Mihai238]

@sutupud Vielen Dank für den Antwort. Ich habe mit Object_Name(z.B Person, Interval etc.) als Parameter implementiert, nicht Object.

[0727384]

Oh, danke @ Neveragain.

Ich habe allerdings
addPlaylist (s, new Playlist (libraryToImport.getPlaylist(s)));
schreiben müssen, damit es klappt (also getPlaylist() statt playlists.get).

Danke, nun klappts

[Neveragain]

Ich habe allerdings
addPlaylist (s, new Playlist (libraryToImport.getPlaylist(s)));
schreiben müssen, damit es klappt (also getPlaylist() statt playlists.get).

Das hängt davon ab, wie du es implementiert hast.
libraryToImport.playlists.get(s) sollte genauso funktionieren, wenn playlists deine HashMap ist.
Ich habe die Angabe gerade nicht vor mir, aber wenn es eine Methode getPlaylist(s) gibt, dann kann natürlich auch die verwendet werden.

[0727384]

OK, danke. Hast recht, funktioniert auch so wie du beschrieben hast.

[Black Sheep]

public Playlist(Playlist original) {

playlist =new ArrayList<Song>();

for(Song s : original){
playlist.addSong(s);
}
}

Bei mir kommt eine Fehlermeldung, wenn ich addSong verwende, mit add() funktionierts. Habe ich irgendwas falsch oder ist add auch ok?

Und den Kopierkonstruktor aus Post #5 verstehe ich auch nicht ganz, wahrscheinlich ist es ganz einfach, aber blick mich grad nicht durch :-( Mit this(playlist1) werden ja alle Songs aus der Playlist 1 in die Playlist übertragen, wenn ich das richtig verstanden habe? Aber wie kopiere ich dann die Playlist 2? Nochmal this kann ich nicht aufrufen, da kommt mir eine Fehlermeldung. Wieder mit einer for-Schleife oder gibt es eine andere Möglichkeit?

[Neveragain]

Bei mir kommt eine Fehlermeldung, wenn ich addSong verwende, mit add() funktionierts. Habe ich irgendwas falsch oder ist add auch ok?

Und den Kopierkonstruktor aus Post #5 verstehe ich auch nicht ganz, wahrscheinlich ist es ganz einfach, aber blick mich grad nicht durch :-( Mit this(playlist1) werden ja alle Songs aus der Playlist 1 in die Playlist übertragen, wenn ich das richtig verstanden habe? Aber wie kopiere ich dann die Playlist 2? Nochmal this kann ich nicht aufrufen, da kommt mir eine Fehlermeldung. Wieder mit einer for-Schleife oder gibt es eine andere Möglichkeit?

Ohne playlist dh nur addSong(s);
Ja, for Schleife passt.

[Black Sheep]

@ TUTOR / LVA:
Wieso (oder wo?) gibts die Muster - Lösungen? Wär schon praktisch bzw. hilfreich. Zumindest kurz vor dem Test (damit man es auch selbst implementiert und nicht gleich aufgibt und nachsieht).

Ich fände eine Musterlösung auch sehr hilfreich, kommt irgendwann eine online oder nicht?

[Neveragain]

Ich fände eine Musterlösung auch sehr hilfreich, kommt irgendwann eine online oder nicht?

Soweit ich weiß, ist nicht geplant eine online zu stellen.

[trambampolin]

Bei der zweiten Theoriefrage, ist es da nötig die Variablen final zu machen? Denn wenn bei der Addition der Vektor nicht mehr verändert werden würde, gäbe es eh keine Möglichkeit mehr, den Vektor zu ändern.

[Neveragain]

Ja, die müssen final sein, bloß funktioniert die Addition dann so nicht mehr

[Nayama]

Ja, die müssen final sein, bloß funktioniert die Addition dann so nicht mehr

private final double x;
private final double y;

public Vector add(Vector v) {
double i = this.x + v.x;
double j = this.y + v.y;
return new Vector(i,j);
}

Vector v = new Vector(3,4);     
v = v.add(new Vector(2,2));
System.out.println(v);

Würde das als Lösung zählen? So ist es zumindest für mich am sinnvollsten

[Neveragain]

private final double x;
private final double y;

public Vector add(Vector v) {
double i = this.x + v.x;
double j = this.y + v.y;
return new Vector(i,j);
}

Vector v = new Vector(3,4);     
v = v.add(new Vector(2,2));
System.out.println(v);

Würde das als Lösung zählen? So ist es zumindest für mich am sinnvollsten

ja. ist richtig.

[Vendredi]

Jedes mal der gleiche Mist, am Anfang läuft es total gut und dann kommt irgendwo drinnen auf einmal ein blöder Fehler der mich Stunden kostet.

Fazit: in 70 Minuten nicht schaffbar. Hab die Theorie noch nicht angefangen, arbeite jetzt seit 40 Minuten dran und werd noch mindestens ne Stunde für Fehlerbehandlung brauchen ehe ich auch nur mit Library anfangen kann -.- (und das nur, weil ich eine blöde Methode nicht hinkriege, nämlich public Playlist(Playlist playlist1, Playlist playlist2) der Rest funktioniert ordnungsgemäß)

Die Zeit ist der größte Feind eines Programmieranfängers wie mir

[Neveragain]

Was ist genau dein Problem mit dieser einen Methode?
Simpel gelöst:

this(playlist1);
for (Song s: playlist2.list)
   addSong(s);

[Vendredi]

Gut, dann hat die Fehlereingrenzung nicht geklappt. Fehler ist anscheinend schon in der Methode Playlist(Playlist original).

Wenn ich genau diesen Testfall wo darauf zugegriffen werde ausgrenze, klappt nämlich alles.

Sitz jetzt 55 Minuten dran, den Test am Dienstag kann ich knicken.

[Neveragain]

Gut, dann hat die Fehlereingrenzung nicht geklappt. Fehler ist anscheinend schon in der Methode Playlist(Playlist original).

Hast du die ArrayList initialisiert?

[Vendredi]

Hast du die ArrayList initialisiert?

Wie meinen?

ArrayList<Song> cl = new ArrayList<Song>();

[Neveragain]

Wie meinen?

ArrayList<Song> cl = new ArrayList<Song>();

Mal davon abgesehen, dass man es so nicht machen sollte - daran liegt es offenbar nicht - du hast eine PN.
(man sollte Variablen erst initialisieren, sobald man diese braucht)

[Vendredi]

Mal davon abgesehen, dass man es so nicht machen sollte - daran liegt es offenbar nicht - du hast eine PN.
(man sollte Variablen erst initialisieren, sobald man diese braucht)

Mein Fehler war ein anderer. Ich hab's schon so:

private ArrayList<Song> pl;

und dann erst im Konstruktor initialisiert. Die cl war ein Denkfehler meinerseits... brauch ja nur this() aufrufen. Da lag der Fehler

[Neveragain]

Also doch das was ich dachte

[manipuppee]

/edit: Hat sich erledigt es geht!!!

[vinkulja6]

gibt es am montag wieder ein repititorium?

[1student]

gibt es am montag wieder ein repititorium?

Am Montag hat es doch noch nie ein Repetitorium gegeben. Vor dem Test gibt es nur diese Q&A-Stunde.
https://pk.inflab.tuwien.ac.at/pp/uebung/tutorien/

[1student]

Am Montag hat es doch noch nie ein Repetitorium gegeben. Vor dem Test gibt es nur diese Q&A-Stunde. https://pk.inflab.tuwien.ac.at/pp/uebung/tutorien/

Edit: mein "nie" bezieht sich nur auf dieses Semester

Edit2: Gestern hat es ein Repetitorium gegeben und es wurde gesagt, dass Angabe und Lösung noch vor dem Test online gestellt werden.

[vinkulja6]

ich meinte natürlich das tutorium zu den tests.


meine extractbyartist methode macht nicht was sie soll, also das programm überspringt die foreach schleife

 public Playlist extractPlaylistByArtist(String artist)
{        // FILL IN        

 Playlist extractBy = new Playlist();
                
  for(Song s : extractBy.playlist)
  {
     if(s.artist.equals(artist))
        addSong(s);
 }
 return extractBy;
}

EDIT: meine fresse problem gelöst

[Eagleeye]

also ich hätte jetzt testfall 1 fertig

wenn ich nun Jukebox.java ausführe
terminiert er mir sofort das ganze ohne ein ergebnis zu liefern
muss ich in Jukebox was ändern??

[Eagleeye]

ps: ich arbeite mit eclipse

wie kann ich nun testfall 1 aufrufen?

[Vendredi]

edit: Hier stand Blödsinn.

(Hab für den Test keine IDE verwendet.)

Mein Tipp wäre in dem Fall als Alternativlösung einfach mit der Konsole in den Ordner manövrieren und die Testfälle mit "java Jukebox x" aufzurufen.

Ist zwar nur ein Workaround aber zur Not...

[emptyvi]

ps: ich arbeite mit eclipse

wie kann ich nun testfall 1 aufrufen?

Vorweg der übliche Hinweis, dass das Arbeiten mit Eclipse für Anfänger nicht ratsam ist.

Ansonsten wird der jeweilige Testfall deinem Programm als Argument beim Start übergeben. Ein solches kannst du in Eclipse unter Run/Run Configurations im Reiter Arguments setzen (kann sein, dass dein Eclipse eingedeutscht ist, ka.). Dort wäre dann einfach "1" einzutragen.

Liebe Grüße,
emptyvi

[Eagleeye]

thx an beide vorposter, hat beides geklappt


2 neue fragen


am Ende der Playlist Klasse, soll ich die to-String Methode schreiben
was auch nicht allzuschwer war, jedoch die "GesamtMinuten" ? Soll ich da eine eigene Methode dafür schreiben?
ich arbeite in der ganzen Klasse nur mit Arraylists

tipps tricks hinweise?

[Neveragain]

Einfach in der Schleife wo du die Songdaten rausliest die Dauer eines jeden Songs zu einer Summenvariable addieren.

[Eagleeye]

ich lese die einzelnen Songdate in der to-stringmethode aus der Arraylist mittels Iterator ab
wie sollte man da gezielt auf die duration kommen? oder meinst du ich soll die gesamt zeit schon in der klasse songs berechnen

[emptyvi]

Du gehst in deiner toString() Methode in der Playlistdie ArrayList mittels Iterator durch (nachdem du den eh schon benützt ^^). Du holst dir die Referenz vom Iterator (zum Beispiel: Song song = it.next()). Dann kannst du mit song.toString() deine Songdaten bekommen, und mit song.getDuration() die jeweilige Dauer (die du nur noch aufsummieren musst).

Liebe Grüße,
emptyvi

[Eagleeye]

dankee (: auf euch kann man sich verlassen als java noob

[Eagleeye]

Ich komm einfach bei der public Library(Library libraryToImport, String[] listOfNames) {

Methode nicht mehr weiter. Ich weiß zwar dass ich mit ner HashMap arbeiten soll, jedoch weiß ich nicht, wie ich die Playlists aus "libraryToImport" rausnehmen und zeitgleich mit deren namen in die HM einfügen soll

[jimb0]

also ich würde für alle strings im array listofnames die methode librarytoimport.getplaylist aufrufen, falls diese != null ist playlist zu eigener lib hinzufügen....

[Eagleeye]

Wie schaut bei euch die getPlayList Methode aus-

ich erstelle eine neue Playlist, nur wie vergleicht man und filtert aus der "libraryToImportEntry" die playlist raus?
um die hashmap zu erstellen brauch ich doch ne hashmap .....


hashception -.-

[jimb0]

library.get(name)... library ist hier die hashmap (key=string, value=playlist)
durchs aufrufen bekommst du nun von der library die playlist...
jetzt musst du in diese library kopie von dieser playlist (mit dem vorher implementierten kopierkonstruktor) hinzufügen...
also quasi:
Playlist tmp = libtoimport.getPlaylist(string..blabla);
addPlaylist(stringblablaba, newPlaylist(tmp));

[patrick02]

ne kurze Frage: warum muss ich die equal()-Methode überschreiben? Denn Song erbt ja sowieso von Object und Object implementiert ja equal() als Vergleich der Identitäten, was ja gebraucht wird. Bei mir hätten alle Testfälle funktioniert.

[Eagleeye]

@jimbo thx, jetzt funzt alles perfekt

[Smilem]

Hi,

Was ist der Fehler jetzt? Kennt jemand das aus?

[Neveragain]

Hi,

Was ist der Fehler jetzt? Kennt jemand das aus?

In deiner Jukebox.java steht statt "public class Jukebox {" irgendwas anderes...

[Smilem]

Ich dachte auch so. Schon kontrolliert. Nicht so..

Besser wäre vielleicht noch mal zu speichern :Q

[Neveragain]

Nachdem dein Screenshot abgeschnitten ist, kann ich nur mutmaßen, dass du dort "JukeBox" stehen hast statt "Jukebox".

edit: Da sollte vermutlich Jukebox und nicht JukeBox stehen - bin gerade in der Arbeit und kann nicht nachsehen.

[Smilem]

Es müsste schon funktionieren. Muss nicht nur Class Name und Kompilieren Name gleich sein.? Ich verwende sowieso nicht otherwhere Name "Jukebox".

Aber schon die Datei wurde geändert :Q Jetzt kommt nicht dieser Fehler. Aber gibts kein Ausgabe

Muss ich nur in der "testing()" Methode aufrufen und mit System.out ausgeben, was ich in anderen Klassen gescherieben hab..?

[Neveragain]

Jetzt kommt nicht dieser Fehler. Aber gibts kein Ausgabe

Muss ich nur in der "testing()" Methode aufrufen und mit System.out ausgeben, was ich in anderen Klassen gescherieben hab..?

java Jukebox -> Es wird die testing() ausgeführt.
java Jukebox 1
java Jukebox 2
...
führt den Testfall 1,2.... aus.

[Salmir]

Ich konnte mir jetzt nicht jede Frage/Antwort durchlesen... würd nur durcheinander kommen.

Frage: Kann mir einer sagen warum ich dauernd "incompatible types" bekomme beim foreach?

edit: war zu blöd um die ArrayList richtig zu schreiben und hatte "object vs Song" als types.. was natürlich schwachsinnig ist.

lg

[rock_n_roll]

Beim Kopieren von der Klasse "Library" erlebe ich Folgendes Problem: Ich habe beim Kopieren folgende Methode erstellt:

public Library(Library libraryToImport, String[] listOfNames) {
for(String names: listOfNames) {
String st=names;
Playlist p3=libraryToImport.getPlaylist(st);
addPlaylist(st,new Playlist(p3));
}

}
Aber es kommt immer noch einen Nullpointer Exception heraus. Kann mir bitte jemand helfen?
Danke im Voraus