Comment passer d'une méthode à partir d'une classe à une autre classe lorsque les deux classes ne sont pas du tout liées

Je voudrais faire de cette façon:

1. Assurer que vos tests travail et le code pour être re-factorisation est couvert. Si vous n'avez pas de tests d'écrire des tests. Ils sont votre corde de sécurité.
2. Utiliser le re-factoring modèle extrait de la superclasse pour créer la nouvelle classe que vous souhaitez déplacer certaines méthodes.
3. Utiliser le re-factoring modèle tirer vers le haut les méthodes pour déplacer les méthodes avec les variables qu'ils ont besoin de la super-classe. Maintenant, vous allez voir si les méthodes que vous souhaitez déplacer et les variables d'instance ont des dépendances à d'autres méthodes que vous ne souhaitez pas déplacer. Si oui, vous devez d'abord briser cette dépendance.
4. Trouver tous les code client qui doit utiliser le nouvel extrait de classe au lieu de la "vieille" classe et de le réécrire pour le nouvel extrait de classe.
5. Supprimer le "s'étend" de la relation entre les deux classes. Maintenant, le code client doit travailler ou que vous avez raté quelque chose.

Aussi un bon livre pour apprendre à appliquer re-factoring modèles est Travailler Efficacement avec le Code existant

OriginalL'auteur René Link

Je vais vous montrer le processus que j'ai suivi.
Considérer ce type de code:

public class GodClass {
    public someInstanceMethodToMove() {
         //some code 1
    }

    public static someStaticMethodToMove() {
         //some code 2
    }

    public static void main(String[] args) {
        GodClass c = ...;
        c.someInstanceMethodToMove();
        GodClass.someStaticMethodToMove();
    }
}

Créer la nouvelle classe:

public class SingleResponsibilityClass {
}

La méthode statique peut être directement placé dans le SingleResponsibilityClass par l'utilisation d'Eclipse est Refactor > Move... refactoring comme décrit par Prabhakaran:

public class GodClass {
    public someInstanceMethodToMove() {
         //some code 1
    }

    public static void main(String[] args) {
        GodClass c = ...;
        c.someInstanceMethodToMove();
        SingleResponsibilityClass.someStaticMethodToMove();
    }
}

public class SingleResponsibilityClass {
    public static someStaticMethodToMove() {
         //some code 2
    }
}

Pour la méthode d'instance, le processus est un peu plus complexe. Lire ci-dessous.

Extraire une méthode de someInstanceMethodToMove() et laissez-les nom, il someInstanceMethodToMove2():

public class GodClass {
    public someInstanceMethodToMove() {
        someInstanceMethodToMove2();
    }

    private someInstanceMethodToMove2() {
         //some code 1
    }

    //...
}

Utiliser le SingleResponsibilityClass dans la méthode originale:

public class GodClass {
    public someInstanceMethodToMove() {
        someInstanceMethodToMove2(new SingleResponsibilityClass());
    }

    private someInstanceMethodToMove2(SingleResponsibilityClass obj) {
         //some code 1
    }

    //...
}

Remarque: il est important que SingleResponsibilityClass est un paramètre de la méthode d'instance pour se déplacer, sinon Eclipse ne pas bouger de ce type.
À partir de là, cliquez-droit sur someInstanceMethodToMove2(), et sélectionnez Refactor > Move..., sélectionnez SingleResponsibilityClass type dans l'assistant, puis appliquer:

public class GodClass {
    public someInstanceMethodToMove() {
        new SingleResponsibilityClass().someInstanceMethodToMove2();
    }

    //...
}

public class SingleResponsibilityClass {
    private someInstanceMethodToMove2() {
         //some code 1
    }

    public static someStaticMethodToMove() {
         //some code 2
    }
}

Puis cliquez droit sur SingleResponsibilityClass' someInstanceMethodToMove2() méthode et Refactor > Rename à someInstanceMethodToMove():
public class GodClass {
public someInstanceMethodToMove() {
nouveau SingleResponsibilityClass().someInstanceMethodToMove();
}

    //...
}

public class SingleResponsibilityClass {
    private someInstanceMethodToMove() {
         //some code 1
    }

    public static someStaticMethodToMove() {
         //some code 2
    }
}

Puis cliquez droit sur GodClass' someInstanceMethodToMove() méthode et Refactor > Inline:

public class GodClass {
    public static void main(String[] args) {
        GodClass c = ...;
        new SingleResponsibilityClass().someInstanceMethodToMove();
        SingleResponsibilityClass.someStaticMethodToMove();
    }
}

public class SingleResponsibilityClass {
    private someInstanceMethodToMove() {
         //some code 1
    }

    public static someStaticMethodToMove() {
         //some code 2
    }
}

OriginalL'auteur JnRouvignac

a-t-elle tout de votre satisfaction

package com.hussi.stackOverFlow;
class ClassOne {

    public void methodInClassOne(String stringParam)
    {
        ClassTwo classTwoObj = new ClassTwo();
        classTwoObj.methodInClassTwo(stringParam);
    }

}


class ClassTwo {

    public void methodInClassTwo(String stringParam)
    {
        System.out.println(stringParam);
    }

}


public class ClassThree {

    public static void main(String[] args) 
    {
        ClassOne objClassOne = new ClassOne();
        //calling method of class two in class one
        objClassOne.methodInClassOne("pass this String value");

    }

}

OriginalL'auteur Hussain Akhtar Wahid 'Ghouri'