Java-Programm

[J /-\ \/ /-\]

Bin noch relativ neu in Java und habe auch nicht das hintergrundwissen, wie man Computerprogramme aufbaut. Ich habe zwar Erfahrung mit (Objektorientierter)programmierung in php, aber das kann man sicher nicht vergleichen. Es wäre nett, wenn sich jemand mal dieses .. "Programm" anschauen könnte und mir Kritik dazu geben könnte. Ich möchte gerne wissen, was ich verbessern kann und was ich mir für die zukunft merken soll.

Hier gibt es das ganze auch nochmal zum Download. Natürlich inklusive der kompilierten Dateien.​

Danke an jeden, der sich Zeit für mich nimmt

umrechner.java

package umrechner;

public class umrechner
{
    public static void main( String[] args )
    {
        boolean runAgain = true;
        String input;
        while( runAgain != false )
        {
            showMenü();
            input = getInput();
            switch(input)
            {
                case "a":
                    dateigrößen();
                    break;
                case "b":
                    break;
                case "c":
                    break;
                case "x":
                    runAgain = false;
                    break;
            }
        }
        System.out.println( "Programm beendet." );
    }
    
    public static void showMenü()
    {
        System.out.println("### Phips Umrechner ###");
        System.out.println("a - Dateigroessen");
        System.out.println("b - noch ausstehend");
        System.out.println("c - noch ausstehend");
        System.out.println("x - Programm beenden");
        System.out.println("### Version 0.1 ###");
        System.out.println("-------------------");
    }
    
    public static String getInput()
    {
        scannerInput wayne = new scannerInput();
        String input;
        System.out.print("Ihre Eingabe: ");
            input = wayne.newString();
        System.out.println("-------------------");
        return( input );
    }
    
    public static void dateigrößen()
    {
        String aE, zE;// aE = Ausgangseinheit; zE = Zieleinheit
        int a = 1, b = 1, zahl = 0;
        scannerInput x = new scannerInput();
        System.out.println( "\"b\"  --> byte" );
        System.out.println( "\"kb\" --> kilobyte" );
        System.out.println( "\"mb\" --> megabyte" );
        System.out.println( "\"gb\" --> gigabyte" );
        System.out.println( "\"tb\" --> terabyte" );
        System.out.print("Ausgangseinheit: ");aE= x.newString();
        System.out.print("Zielseinheit: ");zE= x.newString();
        
        switch(aE)
        {
            case "b":
                a = 1;
                break;
            case "kb":
                a = 2;
                break;
            case "mb":
                a = 3;
                break;
            case "gb":
                a = 4;
                break;
            case "tb":
                a = 5;
                break;
        }
        
        switch(zE)
        {
            case "b":
                b = 1;
                break;
            case "kb":
                b = 2;
                break;
            case "mb":
                b = 3;
                break;
            case "gb":
                b = 4;
                break;
            case "tb":
                b = 5;
                break;
        }
        System.out.print("Zahl: ");zahl = x.newInt();
        System.out.println( "Ergebnis: " + calc(a, b, 1024, zahl) );
    }
    
    public static int calc( int a, int b, int umrechnungszahl, int x )
    {
        if( a > b )
        {
            while( a > b )
            {
                x = x / umrechnungszahl;
                a--;
            }
        }
        else if( a < b )
        {
            while( a < b )
            {
                x = x * umrechnungszahl;
                b--;
            }
        }
        return x;
    }
}

scannerInput.java

package umrechner;

public class scannerInput
{
    java.util.Scanner input = new java.util.Scanner( System.in );
    
    public String newString()
    {
        return input.next();
    }
    
    public int newInt()
    {
        return input.nextInt();
    }
    
}

 

[TKausL]

Hallo,
ich versuche mal FeedBack zu geben:

#1: Klassen fangen mit einem GROßbuchstaben an (sollten sie)
#2: Nutz bitte KEIN Denglisch (showMenü, ...)
#3:

while( runAgain != false ) //Macht man nicht!
while( runAgain == true ) //Auch nicht!
while( runAgain ) //So siehts schon besser aus

#4: Default in die switch rein.

Ist so das, was mir auf Anhieb aufgefallen ist...

 

[faetzminator]

Ich habe zwar Erfahrung mit (Objektorientierter)programmierung [...]

Es ist aber überhaupt kein objektorientiertes Programm Hast du Erfahrung mit dem Zeugs von PHP 5 (eben OO~)?
Ansonsten, auch wenn inzwischen möglich, dass man Sonderzeichen und was weiss ich in die Programme einbringen kann, bitte tu das einfach nicht. Nur schon wegen den Charsets. Und Tastaturen ohne Umlaute.
Aber wenn du gem. TKausL auf Denglisch verzichtest, wird das auch kein Problem mehr sein

 

[Gast2]

Moin,

#5 benutze keine Deutschen Umlaute für Methoden/Variablen/Klassen/etc.

public void 綜藝節目() {
    // deshalb
}

nimm bitte einfach ASCII, sonnst fangen wir irgend wann an den Quellcode umzukopieren weil keiner mehr die Methoden lesen kann - weder Europäer noch Asiaten.

hand, mogel

Edith mein "zu Spät"

 

[Tomate_Salat]

a,b,c,wayne, aE, zE sind keine sinnvollen Variablen-/Parameternamen. Benenne diese sinnvoll, sonst weißt du hinterher nicht mehr, was diese bewirken. Gestalte deinen Code immer so, dass du beim Überfliegen schon erkennen kannst, was gerade passiert. Wenn dem mal nicht der Fall ist, dann helfen dir:
a) Kommentare (nicht nur über Methoden/Klassen was die machen, sondern auch im Code die einen Abschnitt kurz beschreiben)

// Hier wird der Benutzer geprüft
... x zeilen code
// Hier wird einen Aktion mit dem Benutzer ausgeführt

b) ggf. Abschnitte in der Methode in weitere Methoden(oder gar Klassen) auslagern. Eine Methode (Beispiel) [c]validateInputData()[/c] sagt aus was gerade passiert und man muss nicht erst (übertrieben) 500Zeilen Code anschauen, bei dem Daten behandelt werden.

In unserem Programm sind viele Methoden (viel zu) groß. Da bin ich immer froh wenn soetwas wie oben beschreiben gemacht wird/wurde.

 

[Landei]

Die Berechnung sollte mit Fließkomma-Zahlen erfolgen: Wenn ich 512 byte in kByte umwandle, möchte ich 0.5 und nicht 0 sehen.

 

[Spacerat]

PHP ist so objektorientiert, wie Wale Fische sind. Eigentlich kann man auch in Java nicht von Objektorientiert sprechen, wenn man sich nicht aus dem statischen Kontext der main löst. Richtig objektorientiert entwickelt man, wenn es ausser der main nichts anderes statisches mehr gibt aber das ist natürlich utopisch, Utility-Klassen sind ja nicht verboten. Je weniger man das Schlüsselwort "static" verwendet, um so objektorientierter programmiert man.
Sicher, in PHP ist seit Version 5 afaik auch ein OOP Ansatz da (mit Veerbung und Zeugs), aber wer nutzt den schon, wenn es nach wie vor um so vieles einfacher ist, die in PHP gebundelten Funktionen zu verwenden. PHP-Programmierer sind im Allgemeinen deswegen weniger mit OOP vertraut, als sie von sich aus behaupten, selbst wenn sie PHP 5 nutzen (welches diesen OOP Ansatz bietet) aber dennoch mehr oder weniger Code produzieren, der auch in PHP 4 noch lauffähig wäre.

 

[J /-\ \/ /-\]

Vielen Dank, dass ihr euch Zeit für mich genommen habt. Ich habe nun glaube ich alle Verbesserungsvorschläge eingebaut und den Code ein wenig aufgeräumt bzw. aufgesplittet.

Hier gibte s das ganze nochmal als Download inklusive der kompilierten Dateien

Umrechner.java

package umrechner;

public class Umrechner
{
    public static void main( String[] args )
    {
        boolean runAgain = true;
        String selectedMenuItem;
        while( runAgain )
        {
            showMainMenu();
            selectedMenuItem = selectMenuItem();
            switch( selectedMenuItem )
            {
                case "a":
                    FileSizes FileSizes = new FileSizes();
                    FileSizes.run();
                    break;
                case "b":
                    break;
                case "c":
                    break;
                case "x":
                    runAgain = false;
                    break;
                default:
                    break;
            }
        }
        System.out.println( "Programm beendet." );
    }
    
    public static void showMainMenu()
    {
        System.out.println("### Phips Umrechner ###");
        System.out.println("a - Dateigroessen");
        System.out.println("b - noch ausstehend");
        System.out.println("c - noch ausstehend");
        System.out.println("x - Programm beenden");
        System.out.println("### Version 0.1 ###");
        System.out.println("-------------------");
    }
    
    public static String selectMenuItem()
    {
        java.util.Scanner input;input = new java.util.Scanner( System.in );
        String selectedMenuItem;
        System.out.print("Ihre Eingabe: ");
            selectedMenuItem = input.next();
        System.out.println("-------------------");
        return( selectedMenuItem );
    }
}

Calc.java

package umrechner;

public class Calc
{
    public float execute( float a, float b, float conversionNumber, float x )
    /*
     * a und b bilden das Verhältnis
     * conversionNumber ist die Umrechnungszahl
     * x ist die zu berechnende Zahl
     */
    {
        if( a > b )//Einheit wird verkleinert
        {
            while( a > b )
            {
                x = x * conversionNumber;
                a--;
            }
        }
        else if( a < b )//Einheit wird vergrößert
        {
            while( a < b )
            {
                x = x / conversionNumber;
                b--;
            }
        }
        return x;
    }
}

Filesizes.java

package umrechner;

public class FileSizes
{
    public void run()
    {
        String from, to;//Ausgangseinheit und Zieleinheit
        float a = 1, b = 1, x = 0;//a und b bilden das Verhältnis zwischen Ausgangseinheit und Zieleinheit
        java.util.Scanner input = new java.util.Scanner( System.in );
        System.out.println( "\"b\"  --> byte" );
        System.out.println( "\"kb\" --> kilobyte" );
        System.out.println( "\"mb\" --> megabyte" );
        System.out.println( "\"gb\" --> gigabyte" );
        System.out.println( "\"tb\" --> terabyte" );
        System.out.print("Ausgangseinheit: ");from = input.next();
        System.out.print("Zielseinheit: ");   to   = input.next();
        
        switch( from )
        {
            case "b":
                a = 1;
                break;
            case "kb":
                a = 2;
                break;
            case "mb":
                a = 3;
                break;
            case "gb":
                a = 4;
                break;
            case "tb":
                a = 5;
                break;
            default:
                break;
        }
        
        switch( to )
        {
            case "b":
                b = 1;
                break;
            case "kb":
                b = 2;
                break;
            case "mb":
                b = 3;
                break;
            case "gb":
                b = 4;
                break;
            case "tb":
                b = 5;
                break;
            default:
                break;
        }
        System.out.print("Zahl: ");x = input.nextFloat();
        Calc Calc = new Calc();
        x = Calc.execute(a, b, 1024, x);
        System.out.println( "Ergebnis: " + x );
    }
}

 

[J /-\ \/ /-\]

Es ist aber überhaupt kein objektorientiertes Programm

Ich weiß. Ich wollte nur darauf hinaus, dass ich erfahrungen mit Objektorientierter Programmierung in PHP habe, abe rkeinen Plan habe wie man (Objektorientierte) computerprogramme aufbaut!

 

[Spacerat]

Ich wollte nur darauf hinaus, dass ich erfahrungen mit Objektorientierter Programmierung in PHP habe...

Äehm... noch mal... Du hast möglicherweise Erfahrung in PHP und im Zweifelsfalle evtl. schon länger also ungefähr seit PHP 4 wo an Objektorientiert noch überhaupt nicht bzw. nur schwer zu denken war. Das macht dich jetzt nicht unbedingt schlechter, aber merken kann man sich's trotzdem: OOP ist ab PHP 5 nur in der Dokumentation vorhanden. Okay, implementiert ist's wohl auch, aber scheinbar macht nicht ein PHP-Programmierer davon gebrauch. Will sagen: OOP-Erfahrungen sehen anders aus.

 

[J /-\ \/ /-\]

Ok . Aber wollen wir nun zu meinem Programm-Code zurückkommen?

 

[Spacerat]

Aber wieso denn? Was die OOP angeht, sieht das doch schon gut aus... schön viele Klassen... und... naja okay.

Kürzere Methoden könnte man noch schreiben. Evtl. fällt da einem schon bei den relativ identischen switch-Anweisungen mit "from" und "to" ein, z.B. eine Methode "getFactor(String einheit)".

 

[faetzminator]

Ich beginne meistens bei den Datenklassen - also Beans. Schreib mir einfach nur eine Struktur, die objektorientiert alle Daten mit Gettern und Settern zur Verfügung stellt - je nach dem noch kleine Zusatzmethoden. Also so was wie:

public class Human {
    private final String name;
    // Mann <-> Mann, Frau <-> Frau muss heutzutags auch erlaubt sein, deswegen keine Subklasse Man und Woman, in welche nur das Gegengeschlecht rein kann :P
    private Human marriagePartner; 

    public Human(String name) {
        this.name = name;
    }

    public String getName() {
        return name;
    }

    public void setMarriagePartner(Human marriagePartner) {
        this.marriagePartner = marriagePartner;
    }

    public Human getMarriagePartner() {
        return marriagePartner;
    }

    public boolean isWedded() {
        return marriagePartner != null; // wenn marriagePartner vorhanden -> verheiratet
    }
}

Und danach brauchst du noch irgendwelche Controller, Parser, Converter und ggf. irgendwo eine UI (egal ob Rich oder Thin client, ob Web oder Console, ...).
Joa, so einfach gehts eigentlich - fast
}

 

[Spacerat]

Ich hab' mir dein Programm, trotz langer Weile zwar nicht angesehen, kann mir aber gut vorstellen, dass meines ähnliches tut.

import java.util.Scanner;

public class MemEngineer {
	public enum Unit {
		BYTE("b", "Byte"),
		KBYTE("kB", "Kilobyte"),
		MBYTE("MB", "Megabyte"),
		GBYTE("GB", "Gigabyte"),
		TBYTE("TB", "Terabyte"),
		;

		private final String unit, name, toString;

		private Unit(String unit, String name) {
			this.unit = unit;
			this.name = name;
			toString = ordinal() + ". " + unit + " (" + name + ")";
		}

		public String unit() {
			return unit;
		}

		public String unitName() {
			return name;
		}

		public static Unit getByOrdinal(int ordinal) {
			for(Unit u : values()) {
				if(u.ordinal() == ordinal) {
					return u;
				}
			}
			return null;
		}

		public static Unit getByUnit(String unit) {
			for(Unit u : values()) {
				if(u.unit.equals(unit)) {
					return u;
				}
			}
			return null;
		}

		public static Unit getByName(String name) {
			for(Unit u : values()) {
				if(u.name.equals(name)) {
					return u;
				}
			}
			return null;
		}

		public static double claculate(double value, Unit fromUnit, Unit toUnit) {
			if(fromUnit == toUnit) {
				return value;
			}
			value *= Math.pow(1024.0, toUnit.ordinal() - fromUnit.ordinal());
			return value;
		}

		@Override
		public String toString() {
			return toString;
		}
	}

	public static void main(String[] args) {
		Scanner sc = new Scanner(System.in);
		for(Unit u : Unit.values()) {
			System.out.println(u);
		}
		System.out.print("Ausgangseinheit: ");
		Unit ae = null;
		while(ae == null) {
			ae = Unit.getByOrdinal(sc.nextInt());
		}
		System.out.print("Zieleinheit: ");
		Unit ze = null;
		while(ze == null) {
			ze = Unit.getByOrdinal(sc.nextInt());
		}
		System.out.print("Wieviel: ");
		double value = sc.nextDouble();
		System.out.println("Ergebnis: " + Unit.claculate(value, ae, ze) + ze.unit());
		sc.close(); // Wer hat sich das blos ausgedacht. Ich koennt' ... aber ehrlich jetze.
	}
}