Использование лямбда для выполнения пустоты: Это правильно?

Question

Является ли это правильным? Я в основном хочу выполнить определенный метод void.

public abstract class Uniforms { 
    public static void setUniformMatrix4(final UniformLocation uniformLocation, final boolean transpose, final Matrix4f matrix4f) { 
     setUniformImpl(uniformLocation, u -> GL20.glUniformMatrix4(uniformLocation.getLocation(), transpose, matrix4f.asFloatBuffer())); 
    } 

    public static void setUniformMatrix4(final UniformLocation uniformLocation, final boolean transpose, final FloatBuffer matrix4fFloatBuffer) { 
     setUniformImpl(uniformLocation, u -> GL20.glUniformMatrix4(uniformLocation.getLocation(), transpose, matrix4fFloatBuffer)); 
    } 

    public static void setUniform(final UniformLocation uniformLocation, final float value) { 
     setUniformImpl(uniformLocation, u -> GL20.glUniform1f(uniformLocation.getLocation(), value)); 
    } 

    public static void setUniform(final UniformLocation uniformLocation, final int value) { 
     setUniformImpl(uniformLocation, u -> GL20.glUniform1i(uniformLocation.getLocation(), value)); 
    } 

    private static void setUniformImpl(final UniformLocation uniformLocation, final Consumer<UniformLocation> consumer) { 
     Program oldProgram = Program.getUsing(); 
     for (Program program : Program.getPrograms()) { 
      if (program.usesUniform(uniformLocation)) { 
       program.use(); 
       consumer.accept(uniformLocation); 
      } 
     } 
     if (oldProgram != null) { 
      oldProgram.use(); 
     }   
    } 
} 

Answer 1

Если ваша программа компилируется и дает вывод, который вы ожидаете, то в этом смысле это правильно. Однако это можно было бы прояснить.

Четыре общедоступных перегрузки все принимают параметр UniformLocation, который затем передается методу setUniformImpl. Затем этот метод вызывает потребителя и передает ему uniformLocation ... который затем игнорируется. Призывы к GL2.glUniform* методам в lambdas все захватuniformLocation вместо использования параметра u. Вы можете заменить Consumer на Runnable.

В качестве альтернативы, вы можете изменить тип Consumer к тому, что возвращаемое значение Uniform.getLocation() есть и извлечь каждый из этих вызовов в setUniformImpl и передать результат в качестве арг к consumer.accept. Это упростит ситуацию. У вас должно получиться что-то вроде:

public static void setUniform(final UniformLocation uniformLocation, final int value) { 
    setUniformImpl(uniformLocation, u -> GL20.glUniform1i(u, value)); 
} 

private static void setUniformImpl(final UniformLocation uniformLocation, final Consumer<Location> consumer) { 
    ... 
    consumer.accept(uniformLocation.getLocation()); 
    ... 
} 

Answer 2

Если он скомпилирован, тогда проблем нет.

Давайте предположим, что метод ожидает:

void m(Consumer<SomeEvent> e); 

Тогда в Java 7 вы могли бы написать:

m(new Consumer<SomeEvent>() { void accept(SomeEvent e) { /* do nothing */ } }); 

где потребитель вы предоставите игнорирует аргумент e.

Аналогично, с лямбды, вы можете игнорировать аргументы и написать:

m(e -> { /* do nothing */ });