JLayer всегда выбрасывает ArrayIndexOutOfBounds Exception

Question

Я использую JLayer для декодирования MP3 и сохраняю его как данные PCM. Однако, независимо от того, какой файл MP3 я использую, я всегда получаю исключение ArrayIndexOutOfBoudnsException с индексом 443. Я слышал, что это происходит только с некоторыми MP3-файлами, но это происходит в каждом кадре с каждым воспроизведением MP3. Вот мой код:

private short[] getPCM(Header frameHeader, Bitstream bs) { 
    short[] pcm = null; 
    try { 
     Decoder d = new Decoder(); 
     SampleBuffer buffer = (SampleBuffer) d.decodeFrame(frameHeader, bs); 
     pcm = buffer.getBuffer(); 
    } catch (ArrayIndexOutOfBoundsException | DecoderException e) { 
     System.err.println("JLayer, stap it"); 
    } 
    return pcm; 
} 

И метод вызывается с этим кодом:

while ((frameHeader = bs.readFrame()) != null) { 
    short[] pcm = getPCM(frameHeader, bs); 

    for(short i : pcm){ 
     try { 
      os.write(i); 
     } catch (IOException e) { 
      e.printStackTrace(); 
     } 
    }  
} 

трассировки стека:

java.lang.ArrayIndexOutOfBoundsException: 433 
at javazoom.jl.decoder.Bitstream.get_bits(Unknown Source) 
at javazoom.jl.decoder.LayerIIIDecoder.decode(Unknown Source) 
at javazoom.jl.decoder.LayerIIIDecoder.decodeFrame(Unknown Source) 
at javazoom.jl.decoder.Decoder.decodeFrame(Unknown Source) 
at com.dentonposs.Canvas.getPCM(Canvas.java:70) 
at com.dentonposs.Canvas.paintComponent(Canvas.java:48) 
at javax.swing.JComponent.paint(Unknown Source) 
at javax.swing.JComponent.paintChildren(Unknown Source) 
at javax.swing.JComponent.paint(Unknown Source) 
at javax.swing.JLayeredPane.paint(Unknown Source) 
at javax.swing.JComponent.paintChildren(Unknown Source) 
at javax.swing.JComponent.paintToOffscreen(Unknown Source) 
at javax.swing.RepaintManager$PaintManager.paintDoubleBuffered(Unknown Source) 
at javax.swing.RepaintManager$PaintManager.paint(Unknown Source) 
at javax.swing.RepaintManager.paint(Unknown Source) 
at javax.swing.JComponent.paint(Unknown Source) 
at java.awt.GraphicsCallback$PaintCallback.run(Unknown Source) 
at sun.awt.SunGraphicsCallback.runOneComponent(Unknown Source) 
at sun.awt.SunGraphicsCallback.runComponents(Unknown Source) 
at java.awt.Container.paint(Unknown Source) 
at java.awt.Window.paint(Unknown Source) 
at javax.swing.RepaintManager.paintDirtyRegions(Unknown Source) 
at javax.swing.RepaintManager.paintDirtyRegions(Unknown Source) 
at javax.swing.RepaintManager.prePaintDirtyRegions(Unknown Source) 
at javax.swing.RepaintManager.access$700(Unknown Source) 
at javax.swing.RepaintManager$ProcessingRunnable.run(Unknown Source) 
at java.awt.event.InvocationEvent.dispatch(Unknown Source) 
at java.awt.EventQueue.dispatchEventImpl(Unknown Source) 
at java.awt.EventQueue.access$000(Unknown Source) 
at java.awt.EventQueue$3.run(Unknown Source) 
at java.awt.EventQueue$3.run(Unknown Source) 
at java.security.AccessController.doPrivileged(Native Method) 
at java.security.ProtectionDomain$1.doIntersectionPrivilege(Unknown Source) 
at java.awt.EventQueue.dispatchEvent(Unknown Source) 
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source) 
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source) 
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source) 
at java.awt.EventDispatchThread.pumpEvents(Unknown Source) 
at java.awt.EventDispatchThread.pumpEvents(Unknown Source) 
at java.awt.EventDispatchThread.run(Unknown Source) 

Answer 1

Вам необходимо позвонить Bitstream.closeFrame() между каждым вызовом readFrame() - так

while ((frameHeader = bs.readFrame()) != null) { 
    short[] pcm = getPCM(frameHeader, bs); 

    bs.closeFrame(); 

    .. more .. 

Answer 2

судя по ней, вы тр чтобы декодер декодировал тот же кадр снова и снова. Это не сработает, BitStream имеет внутреннее состояние (указатель на следующий бит для чтения), который становится недействительным после декодирования кадра. Исключение указывает, что он пытается прочитать пропускную способность буферов.

Для каждого кадра необходимо вызывать decodeFrame() только один раз.

Редактировать: Ваш стек не показывает, где вы действительно читаете данные. Поскольку исключение находится внутри метода компоновки компонентов, я предполагаю, что вы читаете кадр в другом потоке или в другом месте.

Также, глядя на ваш метод getPCM(): Этот код будет никогда работы. Вы не можете создать новый экземпляр декодера для каждого кадра - это просто не сработает, потому что декодер имеет много внутреннего состояния, которое необходимо и зависит от предыдущего кадра (кадров). Вы должны создать декодер только один раз и повторно использовать его для всех кадров.

Посмотрите на источник javazoom.jl.player.Player - он показывает правильный игровой цикл в методе воспроизведения (int) (его несколько строк длиной, но вы должны придерживаться порядка вызова, показанного там , иначе вы уничтожите правильное состояние с декодером и объектами, удерживаемыми декодером).