О стиле кодирования

Метод должен помещаться на экране. Обязательно должны быть коментарии.

/**
* sets bla
/

pubplic void setBla(someType bla){
    this.bla = bla
}

/**
* gets bla
*/

pubplic someType getBla(){
    return this.bla
}

не смешно ли?

@param, @return, @see и прочие забыл.

Если же не доводить идею до маразма, то да, средняя функция должна помещаться на экране и делать что-то законченное (и имя иметь говорящее).

> Метод должен помещаться на экране. Обязательно должны быть коментарии.

При этом монитор можно поворачивать вертикально :-)

ИМХО, 100 строк (без учета объявления переменных и т.д. в начале п/п).

Длина фигня. Главное, чтобы функция выполняла свою частную задачу. Гораздо бОльшее значение имеет глубина (вложенность условий), говорят лучше всего держать это значение в пределах 4-х.. тем более, что иначе код может начать уползать вправо.

неудобно читать, когда не понимаешь что делает код

все остальное - частные случаи этого