Одиночная строка, если содержит метод вызова

Question

У меня есть метод, который условно вызывает другого. Например:

public boolean contionallyDoSomething(boolean something){ 
    boolean didSomething = false; 
    if(something){ 
     theSomething(); 
     didSomething = true; 
    } 
    return didSomething; 
} 

public void theSomething(){ 
    //do something 
} 

Я полагал, что я мог бы переписать это:

public boolean contionallyDoSomething(boolean something){ 
    boolean didSomething =something && theSomething(); 
    return didSomething; 
} 

public boolean theSomething(){ 
    //do something 
    return true; 
} 

Второй метод чувствует себя намного аккуратнее, и я лично это нравится. Однако я немного устал писать код, как будто это может вернуться, чтобы укусить меня. Каковы нижние стороны? Каковы преимущества/недостатки?

Благодаря

Answer 1

Самый большой недостаток этого кода, особенно, если оно совершено в общее хранилище является читаемость. if statement branching легко отображает намерение, где, поскольку сжатый код требует второго взгляда (для простых смертных), чтобы расшифровать то, что он делает.

Что касается преимуществ, то вы пишете меньше кода и в некоторой степени компилятор оптимизирует его для работы быстрее.

Answer 2

public boolean contionallyDoSomething(boolean something) { 
    if (something) { 
     theSomething(); 
    } 
    return something; 
} 

Несомненно, это имеет смысл, вместо того, чтобы возвращать второе логическое значение, которое будет таким же, как и пройденное?

Answer 3

Почему бы вам не попробовать это ?:

http://en.wikipedia.org/wiki/Strategy_pattern

Так что не укусит вас в будущем :)

Answer 4

Вы пробираться в субъективную воду. Все плюсы и минусы связаны с читабельностью. Лично я сделаю еще один шаг и напишу его как:

public boolean contionallyDoSomething(boolean something) { 
    return something && theSomething(); 
} 

Но я считаю, что плотность в коде повышает читаемость. Другие могут (и будут) не соглашаться.