[java applet] mooie "else and if"

[Ruler-Nrg]

eigenlijk een simple vraag, ik ben totaal nieuw met java. heb voor school een mooie apllet geschreven, alleen met gruwelijk lelijk programmeer werk.
Mijn vraag is simpel: kan onderstaande code mooier/ overzichtelijker

code:

    }                      
    public void commandAction(Command command, Displayable displayable) {                    
         if (displayable == Kaatsheuvel) {                     
            if (command == backCommand2) {                   
                getDisplay().setCurrent(get_stadkeuze());                        
            } else if (command == screenCommand3) {                      
                getDisplay().setCurrent(get_kh1());                        
            } else if (command == screenCommand4) {                      
            }                       
        } else if (displayable == Breda) {
            if (command == backCommand1) {                     
                getDisplay().setCurrent(get_stadkeuze());                        
            } else if (command == screenCommand5) {                      
                getDisplay().setCurrent(get_moerwijk());                        
            } else if (command == screenCommand7) {                      
                getDisplay().setCurrent(get_deburgt());                        
            } else if (command == screenCommand8) {                      
                getDisplay().setCurrent(get_bischopshoeve());                        
            } else if (command == screenCommand6) {                      
                getDisplay().setCurrent(get_ginnekenweg());                        
            }                       
        } else if (displayable == stadkeuze) {
            if (command == screenCommand1) {                     
                getDisplay().setCurrent(get_Kaatsheuvel());                       
            } else if (command == screenCommand2) {                     
                getDisplay().setCurrent(get_Breda());                        
            } else if (command == exitCommand1) {                      
                exitMIDlet();                        
            }                       
        } else if (displayable == deburgt) {
            if (command == deburgt.DISMISS_COMMAND) {                     
                getDisplay().setCurrent(get_Breda(), get_deburgt());                        
            }                       
        } else if (displayable == ginnekenweg) {
            if (command == ginnekenweg.DISMISS_COMMAND) {                     
                getDisplay().setCurrent(get_Breda(), get_ginnekenweg());                        
            }                       
        } else if (displayable == kh1) {
            if (command == kh1.DISMISS_COMMAND) {                     
                getDisplay().setCurrent(get_Kaatsheuvel(), get_kh1());                        
            }                       
        } else if (displayable == moerwijk) {
            if (command == moerwijk.DISMISS_COMMAND) {                     
                getDisplay().setCurrent(get_Breda(), get_moerwijk());                        
            }                       
        } else if (displayable == bischopshoeve) {
            if (command == bischopshoeve.DISMISS_COMMAND) {                     
                getDisplay().setCurrent(get_Breda(), get_bischopshoeve());                        
            }                       
        }

edit: code erbij is handig

[ Voor 63% gewijzigd door Ruler-Nrg op 30-10-2006 10:32 ]

[intoxicated]

http://java.sun.com/docs/.../nutsandbolts/switch.html

[Ruler-Nrg]

verry nice . .maar niet echt makkelijk.. maar ik ga men best even doen

[Janoz]

Je zou de boel al een stuk duidelijker maken wanener je fatsoenlijke naamgeving zou gebruiken. screenCommand1 is nu niet echt een handige naamgeving. Daarnaast zie ik dat je data opslaat als programma code ipv datum. Waarom heb je een variabele die moerwijk of khl heet? Tot slot is het in java veel gebruikelijker om cammelcase te gebruiken. Dus ipv get_moerwijk() gewoon getMoerwijk().

[sig69]

A switch works with the byte, short, char, and int primitive data types.

Gaat in dit geval niet werken dus.

Wat me opvalt is dat je eigenlijk maar een paar verschillende dingen doet (vooral getDisplay().setCurrent), alleen dan met andere plaatsen/wijken(?). Dit moet vast wel op een generieke manier te implementeren zijn.

[ Voor 5% gewijzigd door sig69 op 30-10-2006 10:51 ]

[Ruler-Nrg]

switch lijkt me inderdaad niet te werken.

het programma is een winkel vinder. voor een mobiele telefoon. het doel is om per stad, en dan per straat. de winkels te vinden .

ik weet dat de code echt een rotzooi is. het meeste is namenlijk automatiesch gegenereerde code.
Hiervoor heb ik netbeans gerbuikt. het nadeel is alleen dat de apllicatie en het idee zijn goedgekeurd.
Maar dat de code netter en overzichterlijker moet. en dan vooral het gedeelte wat hierboven staat.

[4VAlien]

Maak eerst een boolean of integer variabele om aan te geven of je stadkeuze wil laten zien of een stad.

Dan krijg je
if [stadkeuze] stadkeuzefunctie()
else displaystad()

in displaystad() heb je dan ook nog maar 1x getDisplay().setCurrent(get_stadkeuze()); nodig. Veel meer is er niet te redden denk ik

ps: applet applicatie (dubbel l != dubbel p)

[BalusC]

Wat doen die get_ methoden eigenlijk?

Ik zou zelf denken om de sleutels en de waarden richting een (multidimensionele) List of Map te refactoren. Bijvoorbeeld:

Java:

private static Map displayables;

private void loadCommands() { // Roep dit ergens eenmalig aan. Constructor, (static) init blok, ofzo.
    displayables = new HashMap();
    Map kaatsheuvelCommands = new HashMap();
    kaatsheuvelCommands.put(backCommand2, get_stadskeuze());
    kaatsheuvelCommands.put(screenCommand3, get_kh1());
    ...
    displayables.put(Kaatsheuvel, kaatsheuvelCommands);
    ...
}

public void commandAction(Command command, Displayable displayable) {                    
    getDisplay().setCurrent(((Map) displayables.get(displayable)).get(command));
}    

Misschien niet de juiste oplossing in jouw geval, aangezien ik geen idee heb wat die get_ functies nou doen, hoe je aan de variabelen als 'Kaatsheuvel' komt en wat verder de project eisen zijn (kan 'static' in jouw geval kwaad?). Maar dit geeft je wel een idee denk ik Je zou die get_ zooi wellicht ook weer kunnen refactoren naar getKeuze(String straatnaam) {}.

[ Voor 8% gewijzigd door BalusC op 30-10-2006 12:04 . Reden: blaat ]

[JKVA]

BalusC schreef op maandag 30 oktober 2006 @ 11:57:
Wat doen die get_ methoden eigenlijk?

Ik zou zelf denken om de sleutels en de waarden richting een (multidimensionele) List of Map te refactoren. Bijvoorbeeld:

Java:

private static Map displayables;

private void loadCommands() { // Roep dit ergens eenmalig aan. Constructor, (static) init blok, ofzo.
    displayables = new HashMap();
    Map kaatsheuvelCommands = new HashMap();
    kaatsheuvelCommands.put(backCommand2, get_stadskeuze());
    kaatsheuvelCommands.put(screenCommand3, get_kh1());
    ...
    displayables.put(Kaatsheuvel, kaatsheuvelCommands);
    ...
}

public void commandAction(Command command, Displayable displayable) {                    
    getDisplay().setCurrent(((Map) displayables.get(displayable)).get(command));
}    

Misschien niet de juiste oplossing in jouw geval, aangezien ik geen idee heb wat die get_ functies nou doen, hoe je aan de variabelen als 'Kaatsheuvel' komt en wat verder de project eisen zijn (kan 'static' in jouw geval kwaad?). Maar dit geeft je wel een idee denk ik Je zou die get_ zooi wellicht ook weer kunnen refactoren naar getKeuze(String straatnaam) {}.

Ik zat idd aan dezelfde soort oplossing te denken. Je wil niet steeds die gevallen uitprogrammeren, tenzij je per geval heel specifieke logica wil doen. In de andere gevallen is opzoeken in een Map vaak gemakkelijker, zeker als het dynamisch moet zijn en/of je heel veel gevallen hebt/verwacht.

[Verwijderd]

Het lijkt er op / zou het kunenn zijn dat je een vrij vaste verzameling van plaatsen hebt? dan kan je een "enum" gebruiken. Hier kan je wel switch mee doen.

[Ruler-Nrg]

Dank voor al uw reacties. Ik ga er mee stoeien. Ik laat wel weten hoe het afloopt