What am I missing here?

[PuppetJacks]

Ok, hey, - I'm hoping someone can asist me with this? - I'm essentially writing a music library database, whereby all the tracks(with artist & title details in local variables) are loaded into one single musicLibrary collection.

When I want to update the JLists on the interface I'm using the below method to retrieve the full catalogue of tracks, then separate them into a TreeMap using the artist as the key & the associated tracks into an arraylist as the value.

for some blasted reason though it's writing the tracks multiple times. i.e, ACDC only has 3 songs, yet it presents the 3 songs 3 times, (so theres 9 songs!, 3 copies of each in the order 1,2,3,1,2,3,1,2,3) and if i click on different artists, that number seems to increase leading me to believe it's something to do with my gui calling the setListData() method.

some Println statements have revealed that the ArrayList being passed to the gui starts off too big, so it's obviously something to do with my iteration in the setListData() method. - If anyone can clearly see what I'm buggering up, please let me know as I'm going blind looking at this.

Thanks in advance

Here's my setListData() method within my mainLibrary Class.

    public TreeMap<String, ArrayList<Track>> getListData()
    {
    TreeMap<String, ArrayList<Track>> temp = guiTracks;
    ArrayList<Track> trackList = new ArrayList<Track>();
        for(Track t : mainLibrary)
        {
            if(temp.containsKey(t.getArtist()))
            {
                trackList = temp.get(t.getArtist());
                trackList.add(t);
            }
            else
            {
                trackList = new ArrayList<Track>();
                System.out.println(t.getArtist());
                trackList.add(t);
            }

        temp.put(t.getArtist(), trackList);
        }
    guiTracks = temp;
    return guiTracks;
    }

And here's the method in my gui which requests the TreeMap & updates the JLists..

    public void setTrackList()
    {
        TreeMap<String, ArrayList<Track>> listItems = library.getListData();
        ArrayList<Track> tracks = listItems.get(artistList.getSelectedValue());
        ArrayList<String> allTracks = new ArrayList<String>();
        for (Track t : tracks)
        {
            allTracks.add(t.getTitle());
        }
        trackList.setListData(allTracks.toArray());
        Track nextOne = library.getNextTrack();
        String nextFull = nextOne.getArtist() + " - " + nextOne.getTitle();
        lblNextTrack.setText(nextFull);
        this.getContentPane().repaint();
    }

The creation of guiTracks (used in getListData) is in the below method, which sets up the main Arraylist of all tracks.

    private void addToLibrary(Track thisTrack)
    {
    mainLibrary.add(thisTrack);
    Set<String> tempArray = guiTracks.keySet();
    if(tempArray.contains(thisTrack.getArtist()))
        {
        ArrayList tracks = guiTracks.get(thisTrack.getArtist());
        tracks.add(thisTrack);
        guiTracks.put(thisTrack.artist, tracks);
        }
    else
    {
        ArrayList<Track> tracks = new ArrayList();
        tracks.add(thisTrack);
        guiTracks.put(thisTrack.artist, tracks);
    }
    }

Any help greatly appreciated.

- PJ.

[Norm]

Continue adding println statements so you see what the computer sees and also so you see the results of what the computer is doing.

Hard to test without code that compiles, executes and shows the problem.

[PuppetJacks]

cheers, - I'm trying to suss it out, it's just excrutiatingly hard to do. - I appreciate it was a long shot with just those methods posted, but I guess I was hoping someone might spot some fundamental issue with my for loops that was causing the recurring Track submissions into my list.

No problem. - I shall continue with my search. thanks for your help.

[DarrylBurke]

What am I missing here?

A decent subject line. Please go through the Forum Rules -- particularly the third paragraph.

db

[PuppetJacks]

My apologies Master.