Refactorización para mover un campo privado de una clase a su clase auxiliar?

Question

EDIT: Envié un Eclipse enhancement request for this refactoring.

¿Hay alguna forma de mover un campo privado de una clase a su clase auxiliar? El siguiente UML de rayas de pollo muestra lo que estoy haciendo manualmente ahora. La clase C1 tiene field privado y una referencia final privada a un objeto Helper antes de la refactorización.

Después de la refactorización, todas las referencias en C1' a field se cambian a helper.getField() y helper.setfield() según corresponda.

class Field {} 

class C1 { 
    final private Field field; 
    final private Helper helper; 

    public Field getField() { 
     return field; 
    } 

    public C1() { 
     helper = new Helper(); 
     field = new Field(); 
    } 
} 

class Helper {} 

class C1Prime { 
    final private HelperPrime helper; 

    public Field getField() { 
     return helper.getField(); 
    } 

    public C1Prime() { 
     helper = new HelperPrime(); 
    } 
} 

class HelperPrime { 
    final private Field field; 
    public HelperPrime() { 
     field = new Field(); 
    } 
    public Field getField() { 
     return field; 
    } 
} 

He utilizado las capacidades de refactorización de Eclipse un poco, pero no puedo encontrar una manera de automatizar este.

Por ejemplo, idealmente arrastraría el campo/atributo/miembro privado de una clase a otra y espero que Eclipse me pregunte cómo quiero manejar las referencias no resueltas. No ofrece sugerencias y rompe todas las referencias.

La operación que he estado repitiendo es separar el conocimiento y el comportamiento que realmente no pertenecen a la clase actual. Estoy moviendo atributos y comportamiento que hace referencia a ciertos campos fuera de la clase original en una nueva clase de "ayuda".

El primer paso en mi refactorización es mover los campos. Una referencia a la clase auxiliar existe como un campo en la clase de la que estoy refactorizando. Para no romper C1 durante la refactorización, creo que sería bueno si Eclipse ofreciera generar getters y setters en Helper' y actualizar las referencias en C1 para usar los getters/setters en la nueva clase.

Answer 1

El asunto es que su campo debe ser privado (ni siquiera sé por qué existe la posibilidad de crear campos públicos no finales). Entonces, ¿cómo podría acceder a él desde otra clase?

Si no desea romperlo durante un refactorio, le puedo dar un pequeño truco que utilizo que a veces me ayuda.

Cuando crea su clase, conviértala ya sea en una clase interna de la clase existente o en una segunda clase en el mismo archivo.

Si la convierte en una clase interna, puede copiar los métodos primero y pueden hacer referencia al miembro de la otra clase. Una vez que obtenga toda la funcionalidad transferida, puede mover la variable. En este punto, ninguna de las referencias a la variable debe actualizarse ya que accede a ella de la misma manera, independientemente de la clase en la que se encuentre.

Hacer una clase en una segunda clase en el mismo archivo puede ser agradable si se está separando funcionalidad también: le permite acceder a todo a la vez sin pasar el ratón entre las ventanas. Cuando hayas terminado, simplemente arrastra la nueva clase en su propio archivo java, vuelve a calcular las importaciones, vuelve a formatear para que sea público y guárdalo.

Casi siempre uso una de estas metodologías cuando creo una nueva clase que interactúa con clases existentes.

Answer 2

Para una función de Java, puede simplemente hacer clic derecho en su nombre, luego seleccionar Refactor > Move. En el asistente, seleccione la nueva clase que ahora administrará este método.

Creo que esto le puede ayudar en su tarea, incluso si no se puede elegir varios elementos a la vez que se debe mover ...

Answer 3

¿No será right click the field > Refactoring > Move hacer?

Sí, no actualiza las referencias, pero imagine lo que tendría que hacer: en todos los lugares donde se hace referencia a su campo, tendrá que crear una instancia de la nueva clase. Eso no es práctico.

Answer 4

La operación de movimiento en un campo no funcionará correctamente. Puedes mover un campo pero eclipse no moverá los getters y setters. Es una pregunta interesante si esto es posible, no creo que sea así.

Answer 5

Si la clase de destino no existe todavía, puede usar Refactor > Extract Class. Esto creará una nueva clase que contendrá los campos que seleccione, y agregará un campo de este tipo a su clase original.

No estoy seguro de si puedes hacerlo para una clase que sí existe, pero siempre puedes usar Extract Class y luego cortar el contenido en tu nueva clase.

Answer 6

Bueno, simplemente no tiene sentido que esto funcione en general. Semánticamente, es una operación extraña.Si mueve el campo a una clase nueva, disjunta (digamos de Cadena a Entero), el código que hace referencia a ella no tendrá una instancia de la nueva clase para usar para obtener el campo de instancia.

Así que solo hay casos especiales en los que tiene sentido: cuando el campo es un miembro estático, o lo está moviendo a una clase principal.

Para los miembros estáticos, funciona bien hacer clic con el botón derecho en el campo que desea mover (el nombre de la variable) y hacer clic en Refactorizar -> Mover. Elija el nuevo tipo. Las referencias se actualizarán automáticamente (pruébelo usted mismo y vea)

Para cuando lo mueva a/desde una clase para padres, puede usar Refactor-> Levantar o Empujar hacia abajo, pero esto no cambiará automáticamente referencias para usted (solo un problema con Push Down; con el polimorfismo Pull Up dicta que las referencias siguen siendo correctas).