Exception und Ausgabe der Duplikate

[Gast1800]

Hallo,
wir haben in der Uni die Aufgabe bekommen eine Exception-Klasse zu schreiben und die vorhandene Main-Klasse mit try/catch umzuformen.
Wenn z.B. ein Composer 2x eingefügt wird soll eine ComposerAlreadyContainedException geschmissen werden und zusätzlich soll der Name des doppelt eingefügten Componisten ausgegeben werden.
Bei der Ausgabe steht bei mir jedoch "Duplikat: null"
Würde gerne wissen, wo mein Denkfehler liegt

ExceptionKlasse:
public class ComposerAlreadyContainedException extends Exception{

private String composer;

ComposerAlreadyContainedException(){

}
ComposerAlreadyContainedException(String s, String composer){
super(s);
this.composer = composer;
}
String getComposer() {
return composer;
}
}

und ein Teilausschnitt der Main:

public class Main {

public static void main(String[] args) {
MusicDataStore music = new MusicDataStore();
Composer mozart = new Composer("Mozart, Wolfgang Amadeus", 1756, 1791);
Composer hiasl = new Composer("Hintermoser, Hiasl", 1926);


try
{
music.addComposer(mozart);
music.addComposer(mozart);
music.addComposer(hiasl);
throw new ComposerAlreadyContainedException();
}
catch(ComposerAlreadyContainedException e){
System.out.println("Duplikat: " + e.getComposer());
}

 

[Tarrew]

Das Thema kommt mir bekannt vor:
https://www.java-forum.org/thema/exception-in-main-abfangen-oder-in-der-methode.181730/

Konkret zu deinem Code:

Du musst nicht manuell im try-Block die Exception werfen. Deine addComposer-Methode muss eine Exception werfen, wenn ein Composer eingefügt werden soll, der schon vorhanden ist. Dabei fügste dann den Composer in die Exception ein.

 

[Gast1800]

Okay, meine Exception im try-Block habe ich gelöscht, in meiner addComposer-Methode werfe ich auch eine Exception, aber wahrscheinlich habe ich hierbei auch noch einen Fehler.
Meine addComposer-Methode sieht momentan so aus:

MusicDataStore addComposer(Composer composer)throws Exception{
Composer.add(composer);
if(this.equals(getComposers())){
throw new ComposerAlreadyContainedException();
}
return composer;
}

Denke, dass mein Fehler hierbei ab der Abfrage mit if beginnt.

 

[Tarrew]

Ich gehe mal davon aus, dass deine Einträge in einer Liste gespeichert sind?

Guck dir mal die contains-Methode an.
https://docs.oracle.com/javase/8/docs/api/java/util/List.html#contains-java.lang.Object-

 

[Gast1800]

Achso ja soweit hab ichs verstanden, vielen Dank .
Eine letzte Frage dazu hab ich noch. Wenn ich jetzt
try
{
music.addComposer(mozart);
music.addComposer(hiasl);
music.addComposer(hiasl);
habe kommt dabei richtig heraus Duplikat: Hintermoser, Hiasl

Wenn ich allerdings nun


try
{
music.addComposer(mozart);
music.addComposer(mozart);
music.addComposer(hiasl);
music.addComposer(hiasl);

mache, bringt er dabei nur das erste Duplikat, wie gibt man denn mehrere Duplikate aus?

Meine add-Methode habe ich so umgeschrieben:
MusicDataStore addComposer(Composer composer)throws Exception{


if(Composer.contains(composer) == Composer.add(composer)){
throw new ComposerAlreadyContainedException(composer.getName(),composer.getName());
}

return composer;
}

 

[Tarrew]

if(Composer.contains(composer) == Composer.add(composer)){

So ist das eigentlich nicht gedacht, weil du zwei Sachen vergleichst, die eigentlich nicht zusammenhängen.
Dass das jetzt klappt, ist eher zufall.

Du solltest das eher schreiben wie:

if(list.contains(deinElement))
     wirfException
else
    fuegeElementHinzu

Mehrere Duplikate kannst du nur ausgeben, indem du mehrere try-catch Blöcke um jede deiner add-Methodenaufrufe machst.

 

[Gast1800]

Achso okay vielen Dank, es funktioniert jetzt .