I had actually wanted to write things in subprocedures, as it's the way I usually do things when programming, because it makes for more readable code, however I was then having problems with printing. I found that if I created an "artistBreak" sub and "albumBreak" sub, and then tried to print from within those subprocedures, I got errors trying to print. I was a little confused if I should open the artist page again within the subprocedure, and then seek to the end and print, or if I should somehow be able to declare an open on the global level, and then print normally. I was also debating if it would be more "proper" to do all "print ALBUMMASTER 'blah blah'" or the way I did, doing a select, and then switching when needed. I agree with you though, it should be cleaned up, and broken into smaller pieces.