Esse erro se deve a tentativa de acessar o primeiro elemento (Index 0
) da arr2
, sendo que arr2
não possui nenhum elemento (length 0
) pois você declarou ela com int[] arr2= {}
, sem nenhum elemento entre as chaves.
No loop você está declarando b
como a.length-1
, o que faz com que o loop nunca verifique o último elemento de a
, pois ele está no índice a-length-1
, e a condição é i < b
, por exemplo:
a.length: 5;
b: a.length-1 = 4;
while (i < b); // o while só passará pelas posições 0, 1, 2 e 3, pois são menores que b (4)
Outra coisa que notei foi que você atribui arr2[i] = a[i]
, deixando espaços vazios caso haja números pares antes de um ímpar, pois arr2 será atribuído na posição i
, por exemplo:
// Usando o seu código, porém assumindo que arr2 possui um tamanho suficiente
a: [0, 4, 2, 87, 6, 9, 3]
arr2: [ , , , 87, , 9, 3]
Para corrigir isso, utilize uma variável que guarda o índice atual de arr2
:
public static void maiorValorImpar(int[] a) {
int b = a.length; // Correção da não verificação do último elemento
int[] arr2 = new int[b];
int i = 0, j = 0; // j guarda onde será colocado o próximo número em arr2
while(i < b) {
if(a[i]%2 != 0) {
arr2[j]= a[i];
System.out.println(arr2[j]);
j++;
}
i++;
}
}
Quando trabalhar com arrays que você não sabe inicialmente o tamanho, recomendo utilizar o ArrayList (documentação) (w3schools), assim você não precisa alocar espaços na array que talvez não utilize, como na linha 3 do meu exemplo (int[] arr2 = new int[b];
).
maiorValorImpar
, por exemplo, pode confundir outra pessoa (ou até vc mesma), já que o código não está verificando o maior valor ímpar, e sim copiando todos os ímpares para outro array (algo comocopiarImpares
seria melhor, por exemplo). O mesmo vale para as variáveis:impares
seria bem melhor quearr2
,numeros
seria melhor quea
, etc. Pode parecer um detalhe bobo, mas nomes significativos podem ajudar inclusive no raciocínio quando vc está montando a lógica...