удалить дублирующий код в java

Question

class A extends ApiClass 
{ 
    public void duplicateMethod() 
    { 
    } 
}  


class B extends AnotherApiClass 
{ 
    public void duplicateMethod() 
    { 
    } 
} 

У меня есть два класса, которые расширяют различные классы api. У двух классов есть несколько повторяющихся методов (тот же метод повторяется в обоих классах) и как удалить это дублирование?

Редактировать

Оба ApiClass и AnotherApiClass не под моим контролем

Answer 1

Звучит как случай для «Стратегического узора».

class A extends ApiClass { 
    private ClassContainingDupMethod strategy; 
} 

class N extends AnotherApiClass { 
    private ClassContainingDupMethod strategy; 

    public methodCallingDupMethod(){ 
     strategy.dupMethod(); 
    } 
} 

class ClassContainingDupMethod{ 
    public dupMethod(){;} 
} 

Или это метод dupMethod, унаследованный от классов Api?

Answer 2

class BaseApiClass 
{ 
    public void duplicateMethod() 
    { 
    } 
} 

class ApiClass extends BaseApiClass 
{ 
} 
class AnotherApiClass extends BaseApiClass 
{ 
} 

class A extends ApiClass 
{ 
}  

class B extends AnotherApiClass 
{ 
} 

Answer 3

В зависимости от того, какой код вы могли бы сделать что-то вроде:

public class Util 
{ 
    public static void duplicateMethod() 
    { 
     // code goes here 
    } 
} 

, а затем просто другой два метода duplicateMethods называют это. Таким образом, код не будет дублироваться, но имя метода и вызов метода Util.duplicateMethod будут.

Если код в Util.duplicateMethod необходим для доступа к переменным класса или класса класса A и B, это не получится так хорошо, но это потенциально возможно (сообщите мне, если вам это нужно).

EDIT (на основе комментариев):

С переменными экземпляра становится менее довольно ... но может быть сделано. Что-то вроде:

interface X 
{ 
    int getVar(); 
    void setVar(A a); 
} 

class A 
    extends ApiClass 
    implements X 
{ 
} 

class B 
    extends AnotherApiClass 
    implements X 
{ 
} 

class Util 
{ 
    public static void duplicateMethod(X x) 
    { 
     int val = x.getVal(); 
     x.setVal(val + 1); 
    } 
} 

Таким образом, для каждой переменной вам нужно получить доступ вы бы способ получить (и установить в случае необходимости). Мне это не нравится, поскольку он делает методы get/set общедоступными, что может означать, что вы делаете доступными вещи, которые вы не хотите быть доступными. Альтернативой было бы сделать что-то с отражением, но я бы хотел этого еще меньше :-)

Answer 4

Дублирующие методы, которые полагаются на переменные-члены, также подразумевают повторяющиеся переменные-члены, и это начинает пахнуть как слишком большие классы. Как бы выглядели эти конкретные переменные-члены с помощью метода (-ов), если бы вы извлекли их в свой класс и затем составили этот класс в свои другие классы? Prefer composition over inheritance.

Answer 5

Вам необходимо объединить классы в один объект, а затем все классы, используя два других класса, изменить их код, чтобы использовать один класс.