Question

嘿伙计们在文本框中的jcombobox中显示所选项目有些问题，这是我的代码：

double Cubao = 0;
double Zambales = 0;
final int Tickets = 50;
jLabel1.setText(String.valueOf(Tickets));
final double discount = .8;
String selecteditem = jComboBox2.getSelectedItem().toString();
String selectitem = jComboBox1.getSelectedItem().toString();
int Passengers = Integer.parseInt(jTextField2.getText());
if (jComboBox2.getSelectedItem().equals("Deluxe AC")) {
if (jComboBox1.getSelectedItem().equals("Cubao_QuezonCity"))    
   Cubao = 780; 
}
if (jComboBox2.getSelectedItem().equals("Ordinary")) {
if (jComboBox1.getSelectedItem().equals("Cubao_QuezonCity"))    
   Cubao = 500; }
jTextField4.setText(String.valueOf(Cubao * Passengers));

if (jComboBox2.getSelectedItem().equals("Deluxe AC")) 
if (jComboBox1.getSelectedItem().equals("Iba_Zambales")) {
   Zambales = 530; 
if (jComboBox2.getSelectedItem().equals("Ordinary"))
if (jComboBox2.getSelectedItem().equals("Iba_Zambales"))
   Zambales = 300;
 {   

jTextField4.setText(String.valueOf(Zambales * Passengers));   
}
{


jTextField4.setEditable(false);
jTextField1.setEditable(false);

现在，当我将总线类型设置为Deluxe AC时，它将显示： http://tinypic.com/r/2s9zwcj/8 但是，当我将总线类型设置为普通时，它将显示： http://tinypic.com/r/29mu0so/8

我的代码似乎有什么问题？

Answer 1

您多次检查相同的字段，这意味着如果块很难读取代码，那么您将拥有大量的netsted。我建议这样的事情：

if (jComboBox1.getSelectedItem().equals("Cubao_QuezonCity")) {
    if (jComboBox2.getSelectedItem().equals("Deluxe AC")) {
        Cubao = 780;
    } else {
        Cubao = 500;
    }
    jTextField4.setText(String.valueOf(Cubao * Passengers));
    jTextField4.setEditable(false);
    jTextField1.setEditable(false);
}
if (jComboBox1.getSelectedItem().equals("Iba_Zambales")) {
    if (jComboBox2.getSelectedItem().equals("Deluxe AC")) {
        Zambales = 530;
    } else {
        Zambales = 300;
    }
    jTextField4.setText(String.valueOf(Zambales * Passengers));
    jTextField4.setEditable(false);
    jTextField1.setEditable(false);
}

jComboBox getSelected Item错误

1 个答案: