Как я должен подходить к преломлению плохо написанного Java-кода обратной польской нотации в Eclipse IDE

Question

Мне трудно понять, с чего начать, когда дело доходит до рефакторинга следующей плохо написанной обратной польской нотации в Eclipse.И хотел бы знать, с чего начать вообще при попытке рефакторинга кода.

import java.util.Scanner;

class StackNode 
{
    public StackNode(double data, StackNode underneath) 
    {
        this.data = data;
        this.underneath = underneath;
    }

    public StackNode underneath;
    public double data;
}


class RPN 
{
    public void into(double new_data) 
    {
        StackNode new_node = new StackNode(new_data, top);
        top = new_node;
    }

    public double outof( ) 
    {
        double top_data = top.data;
        top = top.underneath;
        return top_data;
    }

    public RPN(String command) 
    {
        top = null;
        this.command = command;
    }

    public double get( ) 
    {
        double a, b;
        int j;

        for(int i = 0; i < command.length( ); i++) 
        {
            // if it's a digit
            if(Character.isDigit(command.charAt(i))) 
            {
                double number;

                // get a string of the number
                String temp = "";
                for(j = 0; (j < 100) && (Character.isDigit(command.charAt(i)) || (command.charAt(i) == '.')); j++, i++) 
                {
                    temp = temp + String.valueOf(command.charAt(i));
                }

                // convert to double and add to the stack
                number = Double.parseDouble(temp);
                into(number);
                } else if(command.charAt(i) == '+') {
                    b = outof( );
                    a = outof( );
                    into(a + b);
                } else if(command.charAt(i) == '-') {
                    b = outof( );
                    a = outof( );
                    into(a - b);
                } else if(command.charAt(i) == '*') {
                    b = outof( );
                    a = outof( );
                    into(a * b);
                } else if(command.charAt(i) == '/') {
                    b = outof( );
                    a = outof( );
                    into(a / b);
                } else if(command.charAt(i) == '^') {
                    b = outof( );
                    a = outof( );
                    into(Math.pow(a, b));
                } else if(command.charAt(i) != ' ') {
                    throw new IllegalArgumentException( );
            }
        }

        double val = outof( );
        if(top != null) 
        {
            throw new IllegalArgumentException( );
        }

        return val;
    }

    private String command;
    private StackNode top;

    /* main method */
    public static void main(String args[]) 
    {
        Scanner input = new Scanner(System.in);
        while(true) 
        {
            System.out.println("Enter RPN expression or \"quit\".");
            String line = input.nextLine( );
            if(line.equals("quit")) 
            {
                input.close();
                break;
            } else {
                RPN calc = new RPN(line);
                System.out.printf("Answer is %f\n", calc.get( ));
            }
        }
    }
}

Что-нибудь поможет, спасибо.PS Извините за длинный код, я не могу сделать его короче.

Answer 1

Вы можете использовать SonarLint (расширение Eclipse) , которое помогает обнаруживать и исправлять проблемы с качеством при написании кода. Вы можете скачать его с Eclipse Marketplace .