Java Developer

Code style



Если блок if содержит в себе всю логику, то для лучшей читаемости инвертируйте if и переместите весь код из него.



Плохо

public void foobar(List<Bar> bars) { 

  if (bars.isEmpty()) {

    Bar bar = new Bar();

    bar.setA();

    bar.setB();

    bar.setC();

    bar.setD();

    repository.save(bar);

  }      

}

Хорошо

public void foobar(List<Bar> bars) { 

  if (!bars.isEmpty()) {

    return;

  }

  Bar bar = new Bar();

  bar.setA();

  bar.setB();

  bar.setC();

  bar.setD();

  repository.save(bar);

}