Использование множественного наследования в Java 8

Я использую функцию Java 8 или неправильно ее использую?

Обратитесь к приведенному ниже коду и пояснению, чтобы узнать, почему он был выбран таким образом.

public interface Drawable {
    public void compileProgram();

    public Program getProgram();

    default public boolean isTessellated() {
        return false;
    }

    default public boolean isInstanced() {
        return false;
    }

    default public int getInstancesCount() {
        return 0;
    }

    public int getDataSize();

    public FloatBuffer putData(final FloatBuffer dataBuffer);

    public int getDataMode();

    public boolean isShadowReceiver();

    public boolean isShadowCaster();    //TODO use for AABB calculations

    default public void drawDepthPass(final int offset, final Program depthNormalProgram, final Program depthTessellationProgram) {
        Program depthProgram = (isTessellated()) ? depthTessellationProgram : depthNormalProgram;
        if (isInstanced()) {
            depthProgram.use().drawArraysInstanced(getDataMode(), offset, getDataSize(), getInstancesCount());
        }
        else {
            depthProgram.use().drawArrays(getDataMode(), offset, getDataSize());
        }
    }

    default public void draw(final int offset) {
        if (isInstanced()) {
            getProgram().use().drawArraysInstanced(getDataMode(), offset, getDataSize(), getInstancesCount());
        }
        else {
            getProgram().use().drawArrays(getDataMode(), offset, getDataSize());
        }
    }

    default public void delete() {
        getProgram().delete();
    }

    public static int countDataSize(final Collection<Drawable> drawables) {
        return drawables.stream()
                .mapToInt(Drawable::getDataSize)
                .sum();
    }

    public static FloatBuffer putAllData(final List<Drawable> drawables) {
        FloatBuffer dataBuffer = BufferUtils.createFloatBuffer(countDataSize(drawables) * 3);
        drawables.stream().forEachOrdered(drawable -> drawable.putData(dataBuffer));
        return (FloatBuffer)dataBuffer.clear();
    }

    public static void drawAllDepthPass(final List<Drawable> drawables, final Program depthNormalProgram, final Program depthTessellationProgram) {
        int offset = 0;
        for (Drawable drawable : drawables) {
            if (drawable.isShadowReceiver()) {
                drawable.drawDepthPass(offset, depthNormalProgram, depthTessellationProgram);
            }
            offset += drawable.getDataSize();   //TODO count offset only if not shadow receiver?
        }
    }

    public static void drawAll(final List<Drawable> drawables) {
        int offset = 0;
        for (Drawable drawable : drawables) {
            drawable.draw(offset);
            offset += drawable.getDataSize();
        }
    }

    public static void deleteAll(final List<Drawable> drawables) {
        drawables.stream().forEach(Drawable::delete);
    }
}

---

public interface TessellatedDrawable extends Drawable {
    @Override
    default public boolean isTessellated() {
        return true;
    }
}

---

public interface InstancedDrawable extends Drawable {
    @Override
    default public boolean isInstanced() {
        return true;
    }

    @Override
    public int getInstancesCount();
}

---

public class Box implements TessellatedDrawable, InstancedDrawable {
    //<editor-fold defaultstate="collapsed" desc="keep-imports">
    static {
        int KEEP_LWJGL_IMPORTS = GL_2_BYTES | GL_ALIASED_LINE_WIDTH_RANGE | GL_ACTIVE_TEXTURE | GL_BLEND_COLOR | GL_ARRAY_BUFFER | GL_ACTIVE_ATTRIBUTE_MAX_LENGTH | GL_COMPRESSED_SLUMINANCE | GL_ALPHA_INTEGER | GL_ACTIVE_UNIFORM_BLOCK_MAX_NAME_LENGTH | GL_ALREADY_SIGNALED | GL_ANY_SAMPLES_PASSED | GL_ACTIVE_SUBROUTINE_UNIFORM_MAX_LENGTH | GL_ACTIVE_PROGRAM | GL_ACTIVE_ATOMIC_COUNTER_BUFFERS | GL_ACTIVE_RESOURCES | GL_BUFFER_IMMUTABLE_STORAGE;
        int KEEP_OWN_IMPORTS = UNIFORM_PROJECTION_MATRIX.getLocation() | VS_POSITION.getLocation();
    }
//</editor-fold>
    private FloatBuffer data;
    private Program program;

    private final float width, height, depth;

    public Box(final float width, final float height, final float depth) {
        this.width = width;
        this.height = height;
        this.depth = depth;
        data = generateBox();
        data.clear();
    }

    @Override
    public void compileProgram() {
        program = new Program(
                new VertexShader("data/shaders/box.vs.glsl").compile(),
                new FragmentShader("data/shaders/box.fs.glsl").compile()
        ).compile().usingUniforms(
                        UNIFORM_MODEL_MATRIX,
                        UNIFORM_VIEW_MATRIX,
                        UNIFORM_PROJECTION_MATRIX,
                        UNIFORM_SHADOW_MATRIX
                        );
    }

    @Override
    public int getInstancesCount() {
        return 100;
    }

    @Override
    public Program getProgram() {
        return program;
    }

    @Override
    public int getDataSize() {
        return 6 * 6;
    }

    @Override
    public FloatBuffer putData(final FloatBuffer dataBuffer) {
        FloatBuffer returnData = dataBuffer.put(data);
        data.clear();   //clear to reset data state
        return returnData;
    }

    @Override
    public int getDataMode() {
        return GL_TRIANGLES;
    }

    @Override
    public boolean isShadowReceiver() {
        return true;
    }

    @Override
    public boolean isShadowCaster() {
        return true;
    }

    private FloatBuffer generateBox() {
        FloatBuffer boxData = BufferUtils.createFloatBuffer(6 * 6 * 3);

        //put data into boxData

        return (FloatBuffer)boxData.clear();
    }
}

Сначала выполните шаги по тому, как я пришел к этому коду:

• Я начал с интерфейса Drawable, и каждая реализация имела свои собственные методы drawDepthPass, draw и delete.

• Рефакторинг delete к методу default был простым, тривиальным и не должен быть неправильным.

• Однако, чтобы иметь возможность рефакторирования drawDepthPass и draw, мне нужен был доступ к тому, был ли тег Drawable тесселирован и/или установлен, поэтому я добавил публикацию (не по умолчанию) методы isTessellated(), isInstanced() и getInstancesCount().

• Затем я понял, что это будет немного громоздко, поскольку мы, программисты, ленивы, чтобы реализовать их в каждом Drawable.

• Как следствие, я добавил методы default в Drawable, давая поведение самого базового Drawable.

• Затем я подумал, что я все еще ленив и не хочу вручную реализовывать его для тесселлированных и инстантивных вариантов.

• Итак, я создал TessellatedDrawable и InstancedDrawable, которые обеспечивают default isTessellated() и isInstanced() соответственно. И в InstancedDrawable я отменил реализацию default getInstancesCount().

В результате я могу иметь следующее:

• Обычный Drawable: public class A implements Drawable
• Тесселированный Drawable: public class A implements TessellatedDrawable
• Экранный Drawable: public class A implements InstancedDrawable
• Tessellated и instanced Drawable: public class A implements InstancedDrawable, TessellatedDrawable.

Просто, чтобы убедиться, что все это компилируется и работает отлично, implements InstancedDrawable, TessellatedDrawable отлично обрабатывается Java 8, поскольку в какой-то степени не существует двусмысленности, с которой интерфейс должен функционировать.

Теперь на мою собственную оценку дизайна ООП:

• Каждый Drawable на самом деле является Drawable, поэтому Collection<Drawable> не сломается.
• Можно группировать все TessellatedDrawable и/или InstancedDrawable, не имеющие отношения к тому, как именно оно реализуется.

Другие мысли, которые у меня были:

• Используйте более обычный, многоуровневый подход, однако я проигнорировал это, так как это закончилось бы:

• abstract class AbstractDrawable

• class Drawable extends AbstractDrawable
• class TessellatedDrawable extends AbstractDrawable
• class InstancedDrawable extends AbstractDrawable
• class InstancedTessellatedDrawable extends AbstractDrawable

Я также рассмотрел шаблон Builder, однако это шаблон, который будет использоваться, когда вы создаете множество уникальных экземпляров определенного объекта, и это не то, что мы здесь делаем, и это не о конструкторе объект.

Итак, первый и последний вопрос: я использую функцию Java 8 или неправильно ее использую?