Intelligently generate file structure as xml

Question

So I have about a g'zillion songs on my personal server. (well maybe not a g'zillion, but close)
I recently wrote a simple java class to recursively run through and print the filenames under the directory name.
What i get is a nice xml document that I can render using xls. This all works very except when I hit the last folder. I can not seem to get the class to realize that it is the last folder, and insert the closing tag.
Usually I close the previous dir when i make a new one, since it was the easiest way to tell. but now I get something like this;
<Collection>
<Directory>
<Artist>Aerosmith</Artist>
<Song>Love in an Elavator</Song>
<Song>Crazy</Song>
</Directory>
<Directory>
<Artist>Afrika</Artist>
<Song>Flight of the Cosmic Hippo</Song>
missing tag
</Collection>
Since there is not another directory my app never knows to close the last. Here is the logic for improvement.
private static void compileResults() {
String lastDir="";
boolean lastSong=false;
// TODO Auto-generated method stub
for (Iterator iter = SONGS.iterator(); iter.hasNext();) {
String fileName = (String) iter.next();
System.out.println(fileName);
//get the path of current file
String tempPath=fileName.substring(0,fileName.lastIndexOf("\\"));
if(fileName.contains(".")){     
// file found - check location
if((PATH.equals(tempPath))&&(lastSong)){
// we have finished folders and are back in parent directory
closeDirectory();
System.out.println(xmlOutput.toString());
// we want to ignore files in parent folder, exit loop.
break;
// song is in child folder, append to output
appendSong(fileName.substring((fileName.lastIndexOf("\\")+1),fileName.lastIndexOf(".")));
lastSong=true;
} else {
// directory found, close previous if existed
if(lastSong) {
closeDirectory();                         
// directory listing appended to output
appendArtist(fileName);
lastSong=false;
lastDir=fileName;
}// end for
// close string buffer          
finishOutput();
Edited by: Masterkeedu on Sep 25, 2007 9:39 AM

Answer 1

well, if it's always the last one because of the algorithm you're doing, why not just append a </directory> tag at the end along with your collection? If it's consistently going to be the last one that isn't tagged every time (and based on looking at your code that will be the case), that would be an easier solution.