O problema é esse seu if (i == 0) sc.nextLine();
que está no lugar errado.
Observe que isso está depois da linha anterior que tem o pessoa[i] = sc.nextLine();
. Assim, ele não atribuir uma string vazia ao nome de pessoa[0]
e vai pular o nome que deveria ser da primeira pessoa. Assim, quando ele não mostra nada é porque a primeira pessoa havia sido sorteado.
A solução é retirar o if
e mover esse sc.nextLine();
para logo após o sc.nextInt();
:
import java.util.Scanner;
import java.util.Random;
class Sorteio {
public static void main(String[] args) {
Scanner sc = new Scanner(System.in);
System.out.println("Informe o numero de pessoas no sorteio: ");
int n = sc.nextInt();
sc.nextLine();
String[] pessoas = new String[n];
for (int i = 0; i < n; i++) {
System.out.println("Informe a " + (i + 1) + "a pessoa: ");
pessoas[i] = sc.nextLine();
}
Random rd = new Random();
for (int j = 0; j < 200; j++) {
System.out.println("O vencedor do sorteio e: " + pessoas[rd.nextInt(n)]);
}
}
}
Observe que coloquei o sorteio sendo efetuado 200 vezes para te mostrar que sempre dá uma resposta que seja um dos nomes e nunca dá uma exceção.
Veja aqui funcionando no ideone.
A necessidade de utilizar esse nextLine()
para esvaziar o buffer é meio ruim. A solução é eliminar o nextInt()
e só usar o nextLine()
:
import java.util.Scanner;
import java.util.Random;
class Sorteio {
public static void main(String[] args) {
Scanner sc = new Scanner(System.in);
System.out.println("Informe o numero de pessoas no sorteio: ");
int n = Integer.parseInt(sc.nextLine());
String[] pessoas = new String[n];
for (int i = 0; i < n; i++) {
System.out.println("Informe a " + (i + 1) + "a pessoa: ");
pessoas[i] = sc.nextLine();
}
Random rd = new Random();
for (int j = 0; j < 200; j++) {
System.out.println("O vencedor do sorteio e: " + pessoas[rd.nextInt(n)]);
}
}
}
Veja aqui funcionndo o ideone.
Esse comportamento estranho ocorre com o sc.nextInt()
e com o sc.nextLine()
por causa desses motivos:
O System.in
é o objeto que fornece a outros objetos aquilo que o usuário digitou.
O System.in
armazena o que o usuário digitou em uma área de memória denominada de buffer.
Outros objetos (no caso o Scanner
) podem solicitar ao System.in
para fornecer alguns dos caracteres ou bytes digitados pelo usuário. (Ele trabalha com bytes, mas esses são interpretados/convertidos em caracteres pelo Scanner
).
Quando existe alguma coisa no buffer e o System.in
é solicitado a fornecer alguns bytes, ele irá fornecê-los ao retirá-los do início do buffer. Se sobrarem bytes excedentes no buffer, eles continuarão lá.
Quando o buffer estiver vazio e o System.in
for solicitado a fornecer alguns bytes, ele vai travar e esperar o usuário digitar algum texto terminado com enter. Esse texto, inclusive o enter é armazenado no buffer e então segue-se o que está no item anterior: alguns bytes são retirados do início do buffer e entregues. Quaisquer bytes excedentes permanecem lá.
Quando o método nextLine()
do Scanner
for chamado, ele vai consumir todos os bytes que encontrar até encontrar o enter e devolver isso que ele consumiu sem o enter. Ou seja, ele vai consumir a primeira linha do que estiver no System.in
e descartar o enter.
Já o método nextInt()
funciona de uma forma diferente, ele vai consumir apenas os bytes que correspondam a dígitos, deixando o enter para trás dentro do buffer.
Desta forma, temos uma diferença importante: O nextLine()
consome e descarta o enter no final da linha, mas o nextInt()
o deixa lá. É por isso que o nextLine()
após o nextInt()
é necessário e ele devolve uma string vazia. A função dele é retirar o enter que foi largado no começo do buffer pelo nextInt()
.
Esse comportamento incoerente é resolvido ao usar-se apenas o nextLine()
e abandonar o nextInt()
. O Integer.parseInt(String)
é usado para transformar a linha lida em um número.