to see what s1 is. I suspect s1 may be .mp3 and not just mp3 so the if statement is never identified. I also see no way for s1 to ever be ogg based on this code since the value of s1 is dependent on song and the value of song never changes in the code.