Java - когда я запускаю программу калькулятора формул, по умолчанию задается вопрос «каков радиус круга», даже если выбрана другая фигура - PullRequest
0 голосов
/ 22 мая 2018
package messingaround;
import java.awt.*;
import javax.swing.*;
import messingaround.Secondary;
public class Main {

    public static void main (String[] args){
        Runnable r = new Runnable() {

            @Override
            public void run() {
                Object[] options = {
                    "Circle Area",
                    "Triangle Area",
                    "Square/Rectangle Area",
                    "Circle Circumference",
                    "Cube/Rectangular Prism Volume",
                    "Cylinder Volume",
                    "Pyramid Volume"
                };
                JComboBox shapePicker = new JComboBox(options);

ниже - фактический графический интерфейс пользователя с выпадающим меню

                String shapechoice = (String) JOptionPane.showInputDialog(
                        null,
                        "Choose from the following options...", 
                        "Formula Calculator",
                        JOptionPane.QUESTION_MESSAGE, null,
                        options, // Array of menuChoices
                        options[0]); // Initial choice

                    int shapepick = shapePicker.getSelectedIndex();

                Secondary CalculationsObject = new Secondary();

Здесь у меня проблемы, оператор switch всегда по умолчанию равен 0;

                switch(shapepick) {

Всегдапо умолчанию используется случай 0

                case 0:
                    CalculationsObject.CircleArea();
                    break;
                case 1:
                    CalculationsObject.TriangleArea();
                    break;
                case 2:
                    CalculationsObject.RectangleArea();
                    break;
                case 3:
                    CalculationsObject.CircleCircumference();
                    break;
                case 4:
                    CalculationsObject.CubeVolume();
                    break;
                case 5:
                    CalculationsObject.CylinderVolume();
                    break;
                case 6:
                    CalculationsObject.PyramidVolume();
                    break;
                default:
                    break;

                }
            }
        };

Sidenote: если речь идет о вещи invokeLater, мне потребуется объяснение того, что фактически делает invokeLater, потому что я не слишком знаком с ним.

        SwingUtilities.invokeLater(r);   
}

}

Это следующий класс ниже.Этот класс рассчитывает площадь / объем фигур.Я удалил некоторые методы, так как они не нужны или не связаны с проблемой.

package messingaround;
import javax.swing.JOptionPane;
public class Secondary {

public static void CircleArea(){

    String arearadiusdefine;    
    double arearadius, circlearea, arearadiussquared;

    arearadiusdefine = JOptionPane.showInputDialog("What is the radius of the circle?");
    arearadius = Double.parseDouble(arearadiusdefine);

    arearadiussquared = arearadius * arearadius;
    circlearea = arearadiussquared * 3.14159265359;

    JOptionPane.showMessageDialog(null, "The area of the circle is: " +circlearea, "Formula Calculator", JOptionPane.PLAIN_MESSAGE);
}
public static void TriangleArea(){

    String trianglebasedefine, triangleheightdefine;        
    double trianglebase, triangleheight, trianglearea;

    trianglebasedefine = JOptionPane.showInputDialog("What is the base");       
    triangleheightdefine = JOptionPane.showInputDialog("What is the height?");
    triangleheight = Double.parseDouble(triangleheightdefine);
    trianglebase = Double.parseDouble(trianglebasedefine);  

    trianglearea = trianglebase * triangleheight * 0.5;

    JOptionPane.showMessageDialog(null, "The area of the triangle is: " 
 +trianglearea, "Formula Calculator", JOptionPane.PLAIN_MESSAGE);
}

1 Ответ

0 голосов
/ 22 мая 2018

Поскольку вы еще не прикрепили обратный вызов к вашему JComboBox, ваш код switch запускается только один раз при запуске.

Первый рефакторинг вашего switch в функцию для повторного использования:

public void onOptionChange(int newOption) {
     switch(newOption) {
            case 0:
                CalculationsObject.CircleArea();
                break;
            case 1:
                CalculationsObject.TriangleArea();
                break;
            case 2:
                CalculationsObject.RectangleArea();
                break;
            case 3:
                CalculationsObject.CircleCircumference();
                break;
            case 4:
                CalculationsObject.CubeVolume();
                break;
            case 5:
                CalculationsObject.CylinderVolume();
                break;
            case 6:
                CalculationsObject.PyramidVolume();
                break;
            default:
                break;

            }
}

Определите прослушиватель "onOptionChange":

    public void optionChanged(ItemEvent event) {
       if (event.getStateChange() == ItemEvent.SELECTED) {
           // For each item change, you got two events: One for Deselect and the other for DESELECT
           int option = shapePicker.getSelectedIndex();
           // Invoke change
           onOptionChange(option);
       }
    } 

Затем присоедините прослушиватель событий:

JComboBox shapePicker = new JComboBox(options);
onOptionChange(0); // Default first option select.
shapePicker.addItemListener(new ItemListener() {
        @Override
        public void itemStateChanged(ItemEvent e) {
            optionChanged(e);
        }
    });
...