Многопоточность (мониторы)

Спасибо большое! Подскажите, может знаете ответ на эти вопросы, нигде не смог найти ответа: 1) Когда лучше использовать Synchronized метод, а когда блок ? 2) Какие операции в Synchronized блоке не стоит выполнять ? Спасибо!

1) тут пусть подскажут другие, рельной практики не было у меня, предполагаю что блок стоит использовать, если код должен выполняться единоразово, метод если он вызывается несколько раз (а в принципе работы они равнозначны) 2) в этом блоке стоит выполнять только операции с монитором, остальное лучше вынести за

1. решаешь сам :) если у тебя метод, где выполняется куча всего, и ты планируешь синхронизироваться по методу целиком - то это получится "узкое место". потому что та куча работы, которая могла бы выполняться другими потоками параллельно - не будет так выполняться. ведь мы заблокировали доступ на выполнение этого метода целиком. на самом деле, когда ты помечаешь метод словом synchronized - в байткоде сам метод не синхронизирован, а просто внутри него от его начала и до самого конца создается огромный такой блок синхронизации на весь код метода. для примера, у тебя есть такой вот метод:

public void myMethod() {
    // тут делается что-то очень долгое
    // например, идем в базу за данными
    // или вообще ждем пока пользователь что-то на клавиатуре наклацает
    ...
    // тут именно та часть программы, которая должна быть синхронизирована
    ...
    // тут снова же делается что-то очень длительное
}

если ты тут пометишь весь метод myMethod() целиком ключевым словом synchronized - то у тебя программа будет тупить и медленно работать, потому что все треды будут ждать пока первый сделает длинную операцию, короткую и снова длинную. потом второй тред делает длинную операцию, короткую и снова длинную, итд. хотя на самом деле те две длинные операции и не требуют быть синхронизированными, а можно было бы их делать параллельно (все треды одновременно ждут данных каждый сам для себя). вот в таком случае, конечно, нет смысла целый метод помечать как synchronized, а достаточно поместить только ту короткую часть в синхронизированный блок. но когда у тебя, например, простой геттер, и ты не хочешь париться писать в нем еще блок синхронизации отдельно - тогда да, лупанул synchronized на метод - и все :)

2. те, которые не требуют синхронизации, которые не надо выполнять по очереди. чаще всего, будет проще понять какие операции должны быть синхронизированы. так вот все остальные - не надо помещать в синхронизированный блок :)