Erste Schritte Was denkt ihr über eines meiner ersten Javaprogramme?

[19Beat99]

 package Test;

public class Main {

    public static void main(String[] args) {
        
        int iScan = 0;
        int iCalculation = 0;
        
        
        int indexOperator = 0;
        
        int lengthNumber = 0;
        int numberNumbers = 0;
        int number = 0;
        
        System.out.println("Enter your calculation.");
        
        String calculation = new java.util.Scanner(System.in).nextLine().toString();
        int lengthCalculation =calculation.length();
        
        char operations[] = new char[lengthCalculation];
        int calculateNumber[] = new int[lengthCalculation];
        int numbers[] = new int[lengthCalculation];

        
        while(iScan < lengthCalculation) {
            if(calculation.charAt(iScan) >= '0' && calculation.charAt(iScan) <= '9' ){
                
                calculateNumber[lengthNumber] = calculation.charAt(iScan) - '0';
                lengthNumber++;
                
                }
            else if(calculation.charAt(iScan) == '+' || calculation.charAt(iScan) == '-' || calculation.charAt(iScan) == '*' || calculation.charAt(iScan) == '/') {
                
        
                operations[indexOperator] = calculation.charAt(iScan);
                indexOperator++;
            
                    for(int i = 0;i<lengthNumber;i++) {
                    number = number + (int) (calculateNumber[lengthNumber - i-1] * Math.pow(10, i));
                    }
                
                numbers[numberNumbers] = number;
                lengthNumber = 0;
                numberNumbers++;
                number = 0;
                
                }
            else {
                System.err.println("Something is wrong!");
                }
            
        iScan++;
        }

        for(int i = 0;i<lengthNumber;i++) {
            number = (int) (number + calculateNumber[lengthNumber - i-1] * Math.pow(10, i));
            }
        
        numbers[numberNumbers] = number;
    
        

        
        
        
        while(iCalculation < indexOperator) {
            
            switch(operations[iCalculation]) {
            
            case '+':
                numbers[iCalculation+1] = numbers[iCalculation] + numbers[iCalculation+1];
                break;
                
            case '-':
                numbers[iCalculation+1] = numbers[iCalculation] - numbers[iCalculation+1];
                break;
                
            case '*':
                numbers[iCalculation+1] = numbers[iCalculation] * numbers[iCalculation+1];
                break;
                
            case '/':
                numbers[iCalculation+1] = numbers[iCalculation] / numbers[iCalculation+1];
                break;
            
            default:
                System.err.println("Error");
            
            }
            
            iCalculation++;
            
        }
        
        int result = numbers[indexOperator];
        System.out.println(result);
                
        }
    }

 

[mihe7]

Hm... was soll man dazu schreiben? Auf den ersten Blick scheint mir das ein überlegter Ansatz zu sein, an der Form kannst Du noch arbeiten:

• Paketnamen bitte in Kleinbuchstaben
• import statt vollqualifizierender Klassenname (java.util.Scanner) verwenden
• numberNumbers?!?
• einheitliche Verwendung von Variablennamen (iCalculation/indexOperator -> wenn das i für Index steht, dann hast Du einmal "i" und einmal "index" für Index. Dann besser "index" oder kurz "ix" für beide Fälle verwenden. Sollte das i dagegen für int stehen -> bitte nicht)
• einheitliche Einrückungen, das sieht ja aus wie Kraut und Rüben
• Leerzeilen so einsetzen, dass sie den Code gruppieren: nicht einmal drei, einmal eine, dann wieder zwei oder keine
• Leerzeichen ebenso einheitlich verwenden.

Die letzten drei Punkte sind reine Formatierung - das sollte jede IDE hinbiegen können.

Insgesamt würde ich aber sagen, dass das für ein erstes Programm schon sehr gut ist.

 

[temi]

• Variablen so spät wie möglich deklarieren (iScan, iCalculation, usw.)