Methoden Methode mit Parametern

[Alina_C]

Hallo, ich benötige Hilfe bei OOP im Allgemeinen.

Ich habe eine Klasse mit diversen Parametern die in einer anderen Klasse mit getter und setter Methoden.
Mein Problem ist jetzt, dass ich in einer dritten Klasse mit der main Methode nun ein Objekt erstellen will.
Das Objekt soll in etwa so erstellt werden.
Konkreter: Es gibt die Klasse Album.java mit den privaten Variablen long id, String artist, String title, Vector<String> songs, Integer year (für das Erscheinungsjahr).
Dementsprechende getter und setter Methoden gibt es in der Klasse, wobei jedes Objekt mit der id instanziiert wird.

In der Klasse AlbumControl soll ein neues Album-Objekt mittels der Methode addAlbum erstellen.
Sie Soll die obigen Variablen als Parameter haben.

Hier mein Code von der Klasse Album.
Ist es dabei richtig den Vector für die Songs schon bei den Klassenvariablen zu erstellen (Zeile 7)?

package randompackage;
import java.util.*;

public class Album {
	private long id;
	private String artist;
	private String title;
	private Vector<String> songs = new Vector<String>();
	private Integer year;

	
	public Album(long id){
		this.id = id;
		this.artist = "";
		this.title = "";
		this.year = null;
	}
	
	public long getId(){
		return id;	
	}
	
	public void setTitle(String title){
		this.title = title;
	}
	
	public String getTitle(){
		return title;
	}
		
	public Vector<String> getSongs(){
		return songs;
	}
	
	public void setYear(int year){
		this.year = year;
	}
	
	public int getYear(){
		return year;
	}
	
	public void setArtist(String artist){
		this.artist = artist;
	}
	
	public String getArtist(){
		return artist;
	}
	
	
}

Nun will ich in der Klasse AlbumControl nur die Methode addAlbum(long id, String artist, String title, Vector<String> songs, Integer year)
Ist das wirklich möglich dies über die ständig selbe Album Instanz test zu machen?
Auch ist mir nicht klar wie ich den Vector nun hinkriege. test.addSong(songs); scheint da nicht zu klappen -> 'not applicable'

package justapackage;
import java.util.*;

public class AdminControl {
	private Vector<Album> albums;
	
	public AdminControl(){
		albums = new Vector<Album>();
	}
	
	public boolean addAlbum(long id, String artist, String title, Vector<String> songs, Integer year){
		Album test = new Album(id);
		test.setArtist(title);
		test.addSong(songs);
		test.setTitle(title);
		test.setYear(year);
	}
}

In der "Testlauf"-Klasse mit der main habe ich mir sowas wie das hier vorgestellt:

package justapackage;
import java.util.*;

public class Main {
	public static void main (String[] args){
		Vector<String> song = new Vector();
		song.addElement("'Smells Like Teen Spirit");
		AlbumControl test = new AlbumControl();
		System.out.println((test.addAlbum(0, "Nirvana", song, 1991)));
	}
}

Ausgegeben wird aber nur "[]" - was mache ich alles falsch? :bahnhof:

 

[Gast2]

Dein Code kann nicht kompilieren. Die Methode addAlbum muss einen boolean zurückgeben. Außerdem sollte das neu erstelle Album objekt in den Vector (besser List verwenden) gelegt werden. Du solltest dann noch eine Methode getAlbums() anbieten die dir den Vector zurückgibt, den kannst du dann ausgeben.

 

[Kowalski]

Gegen Ende von der addAlbum()-Methode musst du dein Album auch an den Vektor einfügen.

public boolean addAlbum(long id, String artist, String title, Vector<String> songs, Integer year){
        Album test = new Album(id);
        test.setArtist(title);
        test.addSong(songs);
        test.setTitle(title);
        test.setYear(year);


        return albums.add(test);

    }

Edit: Einen hauch zu langsam...

 

[HimBromBeere]

Also erstens:

Ist das wirklich möglich dies über die ständig selbe Album Instanz test zu machen?

Du meintest sicherlich die selbe Instanz der AlbumControl, dann heißt die Antwort natürlich ja. Smells like teenspirit ist ja sicherlich ein anderes Album als Nevermind oder ... also brauchst du für jedes Album auch eine eigene Album-Instanz.
Warum sollte addAlbum() eine Bool zurückgeben? add-Methoden haben normalerweise keinen Rückgabewert.
Zweitens: Die Klasse Vector gilt als veraltet, nimm stattdessen lieber eine ArrayList o.ä.


EDIT: Wenn addAlbum() einen Fehler verursachen kann, dann schmeiß eine Exception...

 

[Alina_C]

Dein Code kann nicht kompilieren. Die Methode addAlbum muss einen boolean zurückgeben. Außerdem sollte das neu erstelle Album objekt in den Vector (besser List verwenden) gelegt werden. Du solltest dann noch eine Methode getAlbums() anbieten die dir den Vector zurückgibt, den kannst du dann ausgeben.

Ach stimmt, ich wollte irgendwie überprüfen lassen ob das Objekt tatsächlich aufgenommen wurde. Daher rührt die boolean. Wie könnte man das mit true bzw false tatsächlich überprüfen?
Gibt es da für Vector eine spezielle Methode? Die Überprüfung sollte dabei nicht von Main ausgehen sondern von AlbumControl - dann bringt mir ja ein try/catch nicht viel oder?

Edit: Wow so schnell so viele Antworten, das Forum (also ihr) seid echt top :toll:

 

[Alina_C]

Ich komme leider nicht weiter.

Die Album Klasse:

package test;
import java.util.*;

public class Album {
    private long id;
    private String artist;
    private String title;
    private Vector<String> songs = new Vector<String>();
    private Integer year;
 
    
    public Album(long id){
        this.id = id;
        this.artist = "";
        this.title = "";
        this.year = null;
    }
    
    public long getId(){
        return id;  
    }
    
    public void setTitle(String title){
        this.title = title;
    }
    
    public void addSong(String song){
    	this.songs.addElement(song);
    }
    
    public String getTitle(){
        return title;
    }
        
    public Vector<String> getSongs(){
        return songs;
    }
    
    public void setYear(int year){
        this.year = year;
    }
    
    public int getYear(){
        return year;
    }
    
    public void setArtist(String artist){
        this.artist = artist;
    }
    
    public String getArtist(){
        return artist;
    }
}

Die AlbumControl, mittels der man dann die Alben speichern und ausgeben kann.

package test;
import java.util.*;

public class AlbumControl {
    private Vector<Album> albums;
    
    public void AdminControl(){
        albums = new Vector<Album>();
    }
    
    public boolean addAlbum(long id, String artist, String title, Vector<String> songs, Integer year){
        Album test = new Album(id);
        test.setArtist(title);
        test.addSong(songs.toString());
        test.setTitle(title);
        test.setYear(year);
        return albums.add(test);
    }
}

Und die Test-Klasse, in der ich ein neues AlbumControl anlege. Nun will er mir das "Test" nicht hinzufügen.
Das heist ich muss das zuerst in einen String Vector packen und diesen dann übergeben?

package test;

public class AlbumControlTestDrive {
	
	public static void main (String[] args){
		AlbumControl albumConTest = new AlbumControl();
		Vector<String> temp = new Vector<String>();
		temp.add("Smells Like Teen Spirit");
		albumConTest.addAlbum(0, "Nirvana", "Nevermind", temp, 1991);
	}
}

Edit:
Ach ich hab da einen Fehler in der addAlbum() geschrieben, sie erwartete fälschlicherweise einen Album-Vector. Hab das nun in einen String Vector geändert. Ausführen lässt sich das ganze aber nicht
->

Exception in thread "main" java.lang.NullPointerException
	at test.AlbumControl.addAlbum(AlbumControl.java:17)
	at test.AlbumControlTestDrive.main(AlbumControlTestDrive.java:10)

 

[SlaterB]

wenn addAlbum u.a. einen Parameter Vector verlangt, dannn musst du auch Vector übergeben,
wie kann dabei Unklarkeit bestehen?

es ist allerdings wahrscheinlich falsch gebaut, zumindest klingt [c]Vector<Album> songs[/c] falsch, warum nicht Vector<String>?

> ich muss das zuerst in einen String Vector packen und diesen dann übergeben?

das wäre dann passend, ja,

 

[bygones]

du brauchst ja auch einen Vector, du gibts aber einen String.

Ok ein Album kann mehrere Songs haben, aber warum ist der Typ eines Songs "Album" ?

du kannst zb es doch so machen

public boolean addAlbum(long id, String artist, String title, Integer year, String... songs){
        Album test = new Album(id);
        test.setArtist(title);
        for(String each : songs) {
            test.addSong(each); 
        }
        test.setTitle(title);
        test.setYear(year);
        return albums.add(test);
    }

und

AlbumControl albumConTest = new AlbumControl();
albumConTest.addAlbum(0, "Nirvana", "Nevermind", 1991, "Song1", "Song2", "Song3")

dann wird wirklich jeder einzelne Song hinzugefuegt

 

[HimBromBeere]

Wenn du ein Album erstellst, dann brauchst du eine Liste von Songs (Strings) und nicht von Alben. Daher funktioniert

public boolean addAlbum(long id, String artist, String title, Vector<Album> songs, Integer year)

das nicht wie gewünscht.

Achja, wenn du übrigens den Vector mit toString() umwandelst, wirst du wahrscheinlich auch nicht eine Auflistiung aller Einträge bekommen, sondern nur einen kryptischen Text, der das Objekt Vector<Album> symbolisiert. Iteriere lieber durch alle Einträge dr Liste und füge jeden Eintrag mit .addSong() einzeln ein.

public boolean addAlbum(long id, String artist, String title, List<String> songs, Integer year){
        Album test = new Album(id);
        test.setArtist(title);
        Iterator<String> i = songs.iterator();
        while (i.hasNext() {
            test.addSong(i.next());
        }
        test.setTitle(title);
        test.setYear(year);
        return albums.add(test);
    }

Beachte aber bitte, dass ich - frech wie ich bin - aus deinem Vektor ´ne Liste gemacht habe (gemäß meinem Hinweis von gestern), da ich grade nicht im Kopf hab, wie man durch ´nen Vektor durchiteriert.

EDIT: Verdammt, zu spät...

 

[Voltaire81]

Beachte aber bitte, dass ich - frech wie ich bin - aus deinem Vektor ´ne Liste gemacht habe (gemäß meinem Hinweis von gestern), da ich grade nicht im Kopf hab, wie man durch ´nen Vektor durchiteriert.

Der Vollständigkeit halber :

        for (Iterator iterator = songs.iterator();iterator.hasNext(); ) {
            String song = (String)iterator.next();
            test.addSong(song);        
        }

Mir gefällt die Notation

String ... songs

allerdings auch besser.
Wofür dann oft nochmal nen Iterator draus gemacht wird ist eine Frage die oft gestellt und nie ausreichend beantwortet wurde. Reicht auch vollkommen.

        for (int i = 0; i < songs.length; i++) {
            test.addSong(songs[i]);
        }

Letztere Notation ist ein wenig zügiger (nicht das es bei dem Beispiel wichtig wäre).
Last but not least. Noch mal schneller wäre... :

        for (int i = 0, n = songs.length; i < n; i++) {
            test.addSong(songs[i]);
        }

 

[bygones]

Beachte aber bitte, dass ich - frech wie ich bin - aus deinem Vektor ´ne Liste gemacht habe (gemäß meinem Hinweis von gestern), da ich grade nicht im Kopf hab, wie man durch ´nen Vektor durchiteriert.

schau dir mal die Klasse Vector an, dann ist es mehr als offensichtlich ;-)

 

[AngryDeveloper]

Letztere Notation ist ein wenig zügiger (nicht das es bei dem Beispiel wichtig wäre).
Last but not least. Noch mal schneller wäre... :

        for (int i = 0, n = songs.length; i < n; i++) {
            test.addSong(songs[i]);
        }

Sagen wir doch gleich, dass dies bei keinem Beispiel jemals wichtig wäre.



Iterieren würde ich auch über Vector oder List mit der enhanced for, wie es auch bygones in seinem Beispiel macht.

 

[Alina_C]

Danke für die Antworten, denn durch jeden Denkanstoss lerne ich :rtfm:

Die Klasse AlbumControl sieht bei mir nun so aus

package test;
import java.util.*;

public class AlbumControl {
    private Vector<Album> albums;
    
    public void AdminControl(){
        albums = new Vector<Album>();
    }
    
    public boolean addAlbum(long id, String artist, String title, Vector<String> songs, Integer year){
        Album test = new Album(id);
        test.setArtist(title);
        for (Enumeration el = songs.elements(); el.hasMoreElements();){
        	test.addSong((String)el.nextElement());
        	System.out.println("\""+el.nextElement()+"\"" + " added");
        }
        test.setTitle(title);
        test.setYear(year);
        return albums.add(test);
    }
}

Und so die Testklasse

package test;
import java.util.*;

public class AlbumControlTestDrive {
	
	public static void main (String[] args){
		AlbumControl albumConTest = new AlbumControl();
		Vector<String> temp = new Vector<String>();
		temp.add("Smells Like Teen Spirit");
		albumConTest.addAlbum(0, "Nirvana", "Nevermind", temp, 1991);
	}
}

Das Ganze führt jedoch zu

Exception in thread "main" java.util.NoSuchElementException: Vector Enumeration
	at java.util.Vector$1.nextElement(Unknown Source)
	at test.AlbumControl.addAlbum(AlbumControl.java:16)
	at test.AlbumControlTestDrive.main(AlbumControlTestDrive.java:10)

Was mache ich denn noch alles falsch ;(

 

[Voltaire81]

In der Praxis ist es das aber. Daher auch die Vervollständigung.
Einen Grund warum man aus dem bestenenden String[] erst einen Iterator machen sollte nennst auch Du nicht.

 

[Alina_C]

In der Praxis ist es das aber. Daher auch die Vervollständigung.
Einen Grund warum man aus dem bestenenden String[] erst einen Iterator machen sollte nennst auch Du nicht.

Hat das was mit der Komplexitätsklasse zu tun, oder ist das so ne interne Javasache?

 

[AngryDeveloper]

In der Praxis ist es das aber.

for (int i = 0; i < array.lengt; i++) wird gegen for (int i = 0, n = array.length; i < n; i++) nie einen Unterschied machen. Unter Umständen sogar den JIT behindern.

Habe nirgends geschrieben, dass man daraus einen Iterator machen soll.
Für die enhanced for spricht aber allein schon sowas wie:

As ever, write the most readable code first

 

[HimBromBeere]

Wer hat die denn den Quatsch mit der Enumeration eingetrichtert? Du sollst doch bloß durch den Vektor durchiterieren (for int i = 0; i < songs.length; i++).

 

[Alina_C]

Wer hat die denn den Quatsch mit der Enumeration eingetrichtert? Du sollst doch bloß durch den Vektor durchiterieren (for int i = 0; i < songs.length; i++).

Oh, dachte das macht man so.

also eher sowas hier?

        for (int i = 0; i < songs.indexOf(songs.lastElement()); i++){
        	test.addSong(songs.toString());
        }

 

[Voltaire81]

for (int i = 0; i < array.lengt; i++) wird gegen for (int i = 0, n = array.length; i < n; i++) nie einen Unterschied machen. Unter Umständen sogar den JIT behindern.

Nun, in der Praxis macht es nen Unterschied.
Ich schreibe das selbst ungern so umständlich. Bei zeitkritischen Anwendungen kommt es aber genau auf soetwas an.

 

[HimBromBeere]

Nein, es steht doch genau in der Klammer, was du machen sollst:

public boolean addAlbum(long id, String artist, String title, Integer year, String... songs){
        Album test = new Album(id);
        test.setArtist(title);
        for(String each : songs) {
            test.addSong(each); 
        }
        test.setTitle(title);
        test.setYear(year);
        return albums.add(test);
    }

Oder eben, wie Voltaire schon sagte:

for (int i = 0; i < songs-length; i++) {
     test.addSong(songs[i]);
}

EDIT: @Voltaire könnten wir zum Thema zurück kehren und versuchen, die Fragen unseres lieben TOs zu klären?

EDIT: Den Rückgabetyp von addAlbum() solltest du übrigens weglassen, wirf im Zweifelsfall eher eine Exception.

 

[Voltaire81]

@Voltaire könnten wir zum Thema zurück kehren und versuchen, die Fragen unseres lieben TOs zu klären?

Du hast natürlich recht

 

[Alina_C]

Okay, da mach ich lieber wieder void rein.

for(String each : songs) {
            test.addSong(each);
            System.out.println(test.getSongs());
        }

Damit klappt es jetzt. Super

Mittels der System.out.println(test.getSongs()); und der restlichen getter könnte ich dann leicht eine getAlbum in AlbumControl.java schreiben.

Möglicherweise eine blöde Frage (auch wenn gemunkelt wird, dass es die garnicht gibt), aber wieso klappte da meine Iteration

                for (int i = 0; i < songs.indexOf(songs.lastElement()); i++){
        	test.addSong(songs.elementAt(i));
        	System.out.println(test.getSongs());
        }

nicht?

Edit:
eigentlich nicht so wichtig, denn es gibt ja die Methode size

 

[Voltaire81]

Möglicherweise eine blöde Frage (auch wenn gemunkelt wird, dass es die garnicht gibt), aber wieso klappte da meine Iteration

                for (int i = 0; i < songs.indexOf(songs.lastElement()); i++){
        	test.addSong(songs.elementAt(i));
        	System.out.println(test.getSongs());
        }

nicht?

Die Gerüchte stimmen. Es gibt keine blöden Fragen.
Also...

songs.indexOf(songs.lastElement())

liefert Dir wie es da steht den Index. Das erste Element hat den Index 0, das zweite den Index 1 etc...
Du brauchst aber die Länge des Vectors, also einen mehr.
Es hätte also auch

songs.indexOf(songs.lastElement())+1

funktioniert. Aber das ist ja unschön.

 

[bygones]

hab ich was verpasst... was soll der komplizierte ausdruck [c]songs.indexOf(songs.lastElement())[/c] denn ?

[c]songs.size()[/c] bekommt man die laenge der liste/des vectors

edit: ah gerade den edit von ihr gesehen...

 

[HimBromBeere]

Ja, bygones... das hier:

Edit:
eigentlich nicht so wichtig, denn es gibt ja die Methode size

EDIT: Oh verdammt... jetzt hast du´s editiert, wenn ich hier noch lange edititiere, kommt bestimmt eine Meldung mit "jaa, hab´s gesehen" rein... darum... IGNORE THIS

 

[bygones]

Ja, bygones... das hier:

EDIT: Oh verdammt... jetzt hast du´s editiert, wenn ich hier noch lange edititiere, kommt bestimmt eine Meldung mit "jaa, hab´s gesehen" rein... darum... IGNORE THIS

verdammtes unkontrolliertes Multithreading im Forum ;-)