joptionpane quizz , improve my code ....

[pipikrk]

i want to make kind of quizz with joption pane,
can someone give me tips , how to improve my code??

import javax.swing.JOptionPane;

public class Main {


    public static void main(String[] args) {

String no = null;
no = JOptionPane.showInputDialog("sum of 229+566=");
if (    no == null ? "sum" == null : no.equals("795")) {
JOptionPane.showMessageDialog(null, "correct  " + no);
} else
{JOptionPane.showMessageDialog(null, "wrong " + no);}

String nom = null;
nom = JOptionPane.showInputDialog("mul of 229x566=");
if (    nom == null ? "sum" == null : nom.equals("129614")) {
JOptionPane.showMessageDialog(null, "correct  " + nom);
} else
{JOptionPane.showMessageDialog(null, "wrong " + nom);}


    }

[JosAH]

Does your code work as you want it to work? Yes? There's a saying: "if it ain't broke, don't fix it".

kind regards,

Jos

[pipikrk]

it is working, but i want to put 50 questions , so i have to do this all the time!
there is no better way i can fix it???

[JosAH]

it is working, but i want to put 50 questions , so i have to do this all the time!
there is no better way i can fix it???

Sure there is: build a method for it and pass parameters for the part(s) that change per call of the method.

kind regards,

Jos

[pipikrk]

can u please give a quick example ??

many thanks

[JosAH]

Sure, this is a quick example:

boolean sumQuestion(int x, int y) {
   return (""+(x+y)).equals(JOptionPane.showInputDialog("sum of "+x+"+"+y+"=");
}

Because it's spoonfeeding, you have to figure out its details.

kind regards,

Jos

[Junky]

how to improve my code??

Fix the gawd awful if statements.

if ("795".equals(no)) {