Java - Является ли делегирование хорошим решением в этом случае?

Question

вот мой вопрос:

У меня огромный класс (HugeClass), и я хотел разбить его на несколько небольших классов (LittleClass1, LittleClass2, ...). Я слышал о делегации. Звучит неплохо, но я думаю, что это не может работать в моем случае. Действительно, мои маленькие классы нужны некоторые из атрибутов HugeClass:

public class HugeClass 
{ 
    // Attributes 
    private Object object1; 
    private Object object2; 
    private Object object3; 
    ... 
    private Object objectN; 

    // Delegation 1 
    private LittleClass1 little1 = new LittleClass1(); 

    // Function delegated in the LittleClass1 
    public void delegated1() 
    { 
     little1.delegated1(); 
    } 

} 

Вот пример класса Делегирование:

public class LittleClass1 
{ 
    public LittleClass1() 
    { 

    } 

    public void delegated1() 
    { 
     // Here, I need object1, object3, and more to work ! 
    } 

} 

Количество атрибутов, необходимых для функции delegated1 может быть большим. Поэтому я считаю, что использование конструктора LittleClass1 не очень удобно.

И поскольку LittleClass1 переопределяет только один метод HugeClass, я не думаю, что LittleClass1 должен расширять HugeClass.

У вас есть идеи по решению? Использование другого шаблона?

Спасибо!

Update

делегированные функции может понадобиться не только переменные экземпляра, но и функции экземпляра:

public class LittleClass2 
{ 
    public LittleClass2() 
    { 

    } 

    public void delegated2() 
    { 
     // I need object2 and delegated1 to work ! 
    } 

} 

Давать HugeClass конструктору может решить эту проблему. Но это хорошее решение?

Answer 1

Разрушение огромного класса на более мелкие классы, как правило, полезно для улучшения ремонтопригодности, тестируемости и общего качества кода. Меньшие куски должны быть легче рассуждать изолированно. Вы хотите искать семантически различные функции огромного класса и разделить их на части, сначала на извлечение методов, а затем на извлечение классов. Вы не обязательно ищете шаблон, столько, сколько ищете методы рефакторинга, как в книгах Рефакторинг и Working Effectively with Legacy Code.

Теперь, если ваши маленькие классы, по-видимому, разделяют слишком много переменных экземпляра огромного класса, возможно, вам следует сделать шаг назад и начать с некоторых плотных подвешивающих плодов, таких как код, который не зависит от слишком большого числа переменных ; или попытайтесь найти группировку этих переменных, которые семантически имеют смысл для инкапсулирования в новый объект, что уменьшит количество этих свободных переменных и повысит качество дизайна, так как поиск скрытых концепций в дизайне делает код более явный и понятный.

ОБНОВЛЕНИЕ: Кстати, наследование действительно не очень хорошая идея. Вы хотите отделить проблемы, а наследование - еще один более тонкий способ общения.

Answer 2

Похоже, вам просто нужно LittleClass1 подкласс HugeClass и его поля protected:

public abstract class HugeClass { 
    // Attributes - now protected instead of private 
    protected Object object1; 
    protected Object object2; 
    protected Object object3; 
    ... 
    protected Object objectN; 

    -- Note: No delegation, no reference to LittleClass1 here  

    public abstract void delegated() 

} 

public class LittleClass1 extends HugeClass { 

    public void delegated() { 
     // Here, you have access to object1, object2 etc 
    } 
}