Algoritmo de perguntas, mas com respostas do tipo string pelo teclado no java

[Antoniorr]

Olá a todos, gostaria de uma ajuda para concluir esse código abaixo. Não sei onde está o erro.
 

package teste;import java.util.ArrayList;import java.util.Collections;import java.util.List;import java.util.Random;import java.util.Scanner;public class Teste {private static final int LIMITE = 3;private static final int QUANT = 3;String resp;public static void main(String[] args) {metodo1();System.out.println();}private static void metodo1() {// 1 - Crie uma lista e insira os numeros de 1 a 20 nessa lista.List<Integer> lista = new ArrayList<Integer>();for (int i = 1; i <= LIMITE; i++) {lista.add(i);}// 2 - Sorteie um numero entre a primeira e a ultima posição da lista.Random random = new Random();for (int i = 0; i < QUANT; i++) { // 4 - repita os passos 2 e 3 19 vezesint max = lista.size();int indice = random.nextInt(max);// 3 - Remova o numero daquela posição, esse é o seu numero sorteado// ou outra operação conforme necessárioint valor = lista.remove(indice);if (valor == 1){pergunta1();}if (valor == 2){pergunta2();}if (valor == 3){pergunta3();}// 4 - repita os passos 2 e 3 50 vezes, como voce sempre remove o numero sorteado, não tem como repetir o valor sorteado.}}public static void pergunta1(){Scanner ler = new Scanner();System.out.println("Quanto é um + um?");resp = ler.nextLine();if (resp == "dois"){System.out.println("Correto");}else{System.out.println("Errado");}}}

Acredito que haver com a classe Scanner. Estou tentando implementar um código que faz uma pergunta aleatória e recebe como resposta do tipo string e diga se está errado ou certo.

[psykotico]

sem testar, o primeiro ponto que peguei foi nessa linha:

if (resp == "dois"){

para comparação de Strings, utilize o método "equals" ou "equalsIgnoreCase" caso não queira diferenciação entre maiúsculas e minúsculas ... procure no Javadoc String que vai ter a descrição do método.

 

...

 

Uma dica. Se você já está utilizando Collections (List, Map, Set), vale mais a pena utilizar um método mais generalizado, que você possa passar a pergunta e a resposta como parâmetro, ao invés de um método para cada questão ...

[Armando Shimokawa]

Outra dica: talvez fechar o scanner cada vez que ele é usado:

scanner.close();

link onde achei, no StackOverflow:
http://stackoverflow.com/questions/15613626/scanner-is-never-closed