Пишу очередной тетрис, просто для себя. После Мартина и Макконела хочется простой и красивой архитектуры. До некоторого момента все удавалось, но начали выползать костыли.
Есть следующий упрощенный кусок архитектуры:
Фигура на поле:
class Figure{ List<Shape> shapes; Shape currentShape; }
Форма фигуры:
class Shape{ private int[][] model; }
Нужен класс, который создает различные фигуры («T», «J», «L» и прочие). Пока фантазии хватило только на это:
public class FigureFactory {
private static List< Figure > figures = init();
private static List< Figure > init() {
ArrayList< Figure > figures = new ArrayList<>();
Figure J = null, L = null, O = null, S = null, T = null, Z = null;
J = new Figure( new Position( 0, 0 ), new ArrayList< Figure.Shape >() {{
add( new Figure.Shape( new int[][]{ { 0, 1 }, { 0, 1 }, { 1, 1 } } ) );
add( new Figure.Shape( new int[][]{ { 1, 0, 0 }, { 1, 1, 1 }, } ) );
add( new Figure.Shape( new int[][]{ { 1, 1 }, { 1, 0 }, { 1, 0 } } ) );
add( new Figure.Shape( new int[][]{ { 1, 1, 1 }, { 0, 0, 1 }, } ) );
}} );
...
return Arrays.asList( J, L, O, S, T, Z );
}
public static Figure getJ() {
return figures.get( 0 );
}
public static Figure getL() {
return figures.get( 1 );
}
...
public static Figure getRandomFigure() {
return figures.get( ( (int) ( Math.random() * figures.size() ) ) );
}
Проблемы и вопросы следующие:
Название класса. Суффикс
Factory
, как подсказывает гугл, применяется при использовании паттерна «Абстрактная фабрика», а здесь его явно нет. Какое лучше всего подойдет?Builder
не подходит по той же причине. Есть еще какой-то паттерн для подобного?Иммутабельность. Основная проблема —
Figure
не неизменяемый (immutable), так как нужно переключать текущие состояния (Shape
), поворачивая фигуру. Сейчас каждый последующий вызовgetL()
возвращает ссылку на один и тот же объект, изменение которого испортит его для последующих вызовов. Можно переместить вытащить создание каждой конкретной фигуры изinit()
вgetL()
/getlJ()
и возвращать новые объекты, но когда тогда быть сgetRandomFigure()
? Нужен метод, возвращающий любую случайную фигуру из всех возможных, но без списка возможных не понятно, как это сделать. Хотелось бы без рефлексии. Вопрос не в том, как сделать, чтобы работало, а как сделать красиво и аккуратно.Резонно ли здесь применить наследование, реализовав группу классов наподобие
JFigure
,TFigure
,LFigure
и спрятав всё создание туда?
Буду благодарен за любые ссылки\книжки и особенно за критику.