🚀 go-pugleaf

Scott wrote:

> Hi,
> 
> I'm using an MP3 class library from http://www.vdheide.de/projects.html.  It
> is the same library in the J-Jukebox project.
> 
> The library offers a set of classes to read MP3 files.  The main class is
> MP3File, which extends File.
> 
> I've got code that scans all of my MP3 and looks for those that have ID3
> tags (the older way to mark the artist, song title, etc. in the MP3 file).
> The code appears below.
> 
> I've got about 1800 files to scan.  At file # 897, I get a
> java.io.IOException: Too many open files.
> 
> Can anyone suggest how to avoid this problem (aside from only handling 897
> files at a time)?  If there was a File.close() command, I'd happily execute
> it.

Your MP3 library is flawed in this respect.  In the ID3 class, the 
checkForTag() method opens a file and fails to close it.  You'll need to 
fix that method if you want to get past this problem.

Since you are new to programming, I'll give you a hint for making this 
fix simpler ... it is considered bad programming practice to have 
multiple return statements within a method.

Poking thru the source also reveals that new MP3File() will create a new 
ID3 object.  So you probably don't need to create one yourself.

-- 

Regards,

Lori Olson (TeamB)

------------

Save yourself, and everyone else, some time and search the
newsgroups and the FAQ-O-Matic before posting your next
question.

Google Advanced Newsgroup Search
     http://www.google.ca/advanced_group_search
Other Newsgroup Searches:
     http://www.borland.com/newsgroups/ngsearch.html
Joi Ellis's FAQ-O-Matic:
     http://www.visi.com/~gyles19/fom-serve/cache/1.html