Bug #556

Logic error in SentenceDetectorME class

Adicionado por Alessandro Kissaba mais de 5 anos atrás. Atualizado mais de 5 anos atrás.

Situação:Closed Início:26/03/2012
Prioridade:Normal Data prevista:
Atribuído para:William Colen % Terminado:

0%

Categoria:Base Tempo gasto: -
Versão:3.2.0

Descrição

An java.lang.ArrayIndexOutOfBoundsException always occurs when sentPosDetect method is invoked at line 97.

That line has the following code :

System.arraycopy (res,0,newRes,0,res.length + 1);

The reason of the error is because the last parameter expects the number of elements to be copied from variable res, in other words I think this line should be something like :

System.arraycopy (res,0,newRes,0,res.length);

I did some tests to clear my doubts about this issue :

import java.util.Arrays;

public class TesteArrayCopy
    {
    public static void main( String[] args )
        {
        int[] v = new int[]{ 1, 2, 3 }; // length : 3
        int[] nv = new int[ v.length + 1 ]; // length : 4

        System.arraycopy( v, 0, nv, 0, v.length ); // ok
        System.out.println( Arrays.toString( nv ) ); // [ 1, 2, 3, 0 ]

        System.arraycopy( v, 0, nv, 0, v.length + 1 ); // error - java.lang.ArrayIndexOutOfBoundsException
        System.out.println( Arrays.toString( nv ) );
        }
    }

Histórico

#1 Atualizado por William Colenmais de 5 anos

Thank you for reporting. We will fix it for the the next release.

#2 Atualizado por William Colenmais de 5 anos

  • Categoria ajustado para Base
  • Atribuído para ajustado para William Colen

#3 Atualizado por William Colenmais de 5 anos

  • Situação alterado de New para Resolved

It is fixed in the trunk and will be available in 3.1.3. Thank you again.

#4 Atualizado por Alessandro Kissabamais de 5 anos

William Colen escreveu:

It is fixed in the trunk and will be available in 3.1.3. Thank you again.

You are welcome, CogrOO is really great and I'm glad to help with the project.

#5 Atualizado por William Colenmais de 5 anos

  • Versão alterado de 3.1.2 para 3.2.0

#6 Atualizado por William Colenmais de 5 anos

  • Situação alterado de Resolved para Closed

Exportar para Atom PDF