[Java] Refactoring methode

[Cuball]

Waarschijnlijk zie ik hier iets simpel over het hoofd, maar gewoon eventjes een voorbeeld:

Stel ik heb volgende code

Java:

Integer amount = null;
    if (type == REALISATION_TYPE) {
            amount = reward.getRealisationAmount(i);
        } else if (type == TIP_TYPE) {
            amount = reward.getTipAmount(i);
        } else { amount = reward.getEmptyAmount(i);
          }
//doe iets verder met amount

als ik dan via eclipse refactor en extract method op los laat krijg ik het volgende

Java:

Integer amount = null;
amount = getAmount(reward, type, amount, i);
//doe iets verder met amount

nu vroeg ik me af of deze amount parameter wel meegegeven moet worden ? en dat ik in getAmount methode gewoon een lokale Integer amount kan aanmaken en deze returnen? volgens mij blijft dit hetzelfde.

[Salandur]

die amount parameter is inderdaad redelijk nuttigloos.

[Janoz]

In principe is het hetzelfde. Sterker nog, ik heb je code gekopieerd en zelf extract method gebruikt, maar bij mij wordt de amount niet eens meer meegegeven.

Wat ik me nog wel kan voorstellen is dat er tussen regel 1 en regel 2 in je orginele code al iets gebeurt met amount. In dat geval kan amount niet binnen de methode aangemaakt worden omdat er eventueel al acties op zijn uitgevoerd.

[Nick_S]

Dan heb je in Eclipse je code niet genoeg geselecteerd. Je moet hem selecteren vanaf het aanmaken van je Integer declaratie.

Java:

//Begin selectie
        Integer amount = null;

        if (type == REALISATION_TYPE) {
            amount = new Integer(10);
        } else if (type == TIP_TYPE) {
            amount = new Integer(15);
        } else {
            amount = new Integer(20);
        }
        // doe iets verder met amount
//Eind selectie

En daarna refactor, extract method.

[Janoz]

Zelfs toen ik de declaratie niet meenam werd deze niet als parameter meegegeven. Er bleef wel een apparte declaratie en toekenning staan.