Java if语句不与变量进行比较

时间:2013-04-11 00:31:45

标签: java if-statement

这是另一个学校项目,其中主类生成两个随机数,然后将它们显示为添加问题。然后用户必须输入两个数字的总和。程序应该将输入与正确的答案进行比较,但每次输入正确的答案时,它都说错了吗?

主类:

import java.util.Random;

public class MathQuiz
{
private int answer1, answer2, answer3, answer4, answer5, n1, n2, sum, score,           scoreFinal;
private String message, m2, m3, m4, m5, finalPage;
public MathQuiz()
{
}

Random dice1 = new Random();

public void setN1()
{
    n1 = dice1.nextInt(100);
}

public void setN2()
{
    n2 = dice1.nextInt(100);
}

public void setSum()
{
    sum = n1+ n2;
}

public int getN1()
{
    return n1;
}

public int getN2()
{
    return n2;
}

public int getSum()
{
    return sum;
}



public String getAnswer1(String a1)
{
    if (a1.equals(getSum()))
    {
        message = "Correct";
        score++;
    }

    else 
    {
        message = "Incorrect";
    }

    return message;
}

public String getAnswer2(String a2)
{
    if (a2.equals(getSum()))
    {
        m2 = "Correct";
        score++;
    }

    else
    {
        m2 = "Incorrect";
    }

    return m2;
}

public String getAnswer3(String a3)
{
    if (a3.equals(getSum()))
    {
        m3 = "Correct";
        score++;
    }

    else
    {
        m3 = "Incorrect";
    }

    return m3;
}

public String getAnswer4(String a4)
{
    if (a4.equals(getSum()))
    {
        m4 = "Correct";
        score++;
    }

    else
    {
        m4 = "Incorrect";
    }

    return m4;
}

public String getAnswer5(String a5)
{
    if (a5.equals(getSum()))
    {
        m5 = "Correct";
        score++;
    }

    else
    {
        m5 = "Incorrect";
    }

    return m5;
}

public String getLast()
{
    scoreFinal = score * 20;
    finalPage = "Finished! Your score is " + scoreFinal + "%"; 
    return finalPage;
}
}

的JOptionPane:

import javax.swing.JOptionPane;

public class Dialog
{
public static void main(String[] args)
{
    String A1, A2, A3, A4, A5;
    int a1, a2, a3, a4, a5;

    MathQuiz quiz = new MathQuiz();

    JOptionPane.showMessageDialog(null, "Answer these 5 math questions...");

    quiz.setN1();
    quiz.setN2();
    quiz.setSum();
    A1 = JOptionPane.showInputDialog(quiz.getN1() + " + " + quiz.getN2());

    JOptionPane.showMessageDialog(null, quiz.getAnswer1(A1));

    quiz.setN1();
    quiz.setN2();
    quiz.setSum();
    A2 = JOptionPane.showInputDialog(quiz.getN1() + " + " + quiz.getN2());

    JOptionPane.showMessageDialog(null, quiz.getAnswer1(A2));

    quiz.setN1();
    quiz.setN2();
    quiz.setSum();
    A3 = JOptionPane.showInputDialog(quiz.getN1() + " + " + quiz.getN2());

    JOptionPane.showMessageDialog(null, quiz.getAnswer1(A3));

    quiz.setN1();
    quiz.setN2();
    quiz.setSum();
    A4 = JOptionPane.showInputDialog(quiz.getN1() + " + " + quiz.getN2());

    JOptionPane.showMessageDialog(null, quiz.getAnswer1(A4));

    quiz.setN1();
    quiz.setN2();
    quiz.setSum();
    A5 = JOptionPane.showInputDialog(quiz.getN1() + " + " + quiz.getN2());

    JOptionPane.showMessageDialog(null, quiz.getAnswer1(A5));

    JOptionPane.showMessageDialog(null, quiz.getLast());
}
}

5 个答案:

答案 0 :(得分:4)

传递给getAnswerX()方法的答案是字符串,你将它们与整数进行比较。 String类'.equals()方法返回false,因为参数是一个不同的类型,一个int原语。您可以使用Integer.parseInt()将字符串转换为整数,或者在比较它们之前使用Integer.toString()将整数转换为字符串。

这样的事情会更好:

public String getAnswer1(String a1)
{
    if (Integer.parseInt(a1) == getSum())
    {
        message = "Correct";
        score++;
    }

    else 
    {
        message = "Incorrect";
    }

    return message;
}

答案 1 :(得分:0)

您正在将代码中的Stringint进行比较,这些不等于。尝试更改您的比较

a1.equals(getSum())

a1.equals(getSum().toString())

以便将字符串与字符串进行比较。

答案 2 :(得分:0)

您正在尝试将整数与字符串进行比较

尝试改变

a1.equals(getSum())

a1.equals(String.valueOf(getSum()))

http://www.tutorialspoint.com/java/java_string_valueof.htm

答案 3 :(得分:0)

步骤1:不要让用户询问一组五个问题,而是向他们提出一个问题。         这将极大地加快调试速度。一旦您的代码有效,请提出五个问题。

步骤2:使用代码只询问一个问题,然后您可以进行更简单的更改,         正如Asaph所说,Integer.toString()并计算答案。         你不会觉得性能受到了打击。

答案 4 :(得分:0)

在撰写本文时,有两个答案,两个都是正确的。尝试某事的第一个好工作。很多人来这里寻找专业人士为他们做功课。

以下是一些需要考虑改进代码的事项:

1)一致的缩进将帮助您阅读代码。 2)为什么你需要这么多的getAnswer方法,你只需要一个,而不是1-5。 3)你应该检查你的getAnswer方法的String参数是否为null,否则你将得到一个空指针异常。

还有其他一些东西,但现在可以使用。