Такая проблема: методы keyPressed и keyReleased устанавливают флаг, а в
отдельном треде уже в зависимости от кнопки просчитываются новые координаты и вызывается repaint().
     Проблема в том, что paint(Graphics g) выполняется намного дольше чем просчет новых координат,
поэтому пока отрисуется один кадр, уже просчитаны координаты далеко вперед и следующий кадр «перескакивает».
     Пробовал в методе paint сделать synchronized блок на переменную pressedMove - ничего не изменилось. Делал synchronized paint(...) тоже самое..
     Получилось только через еще один флаг paintInProgress.
     Как сделать синхронизацию через synchronized блок или метод?

Naik, а почему ты synchronized в paint ставишь? Ты же переменные в keyPressed и keyReleased изменяешь, вот там и пробуй. А, и в run надо, там тоже изменяется переменная.
Не совсем понял, архитектуру данного решения. Почему переменные устанавливаются в keyPressed, влияют на paint, а проверяются вообще в отдельном потоке?

Naik, попробывал уменьшить размер сохраняемой картинки- сохранило нормально! Спасибо.

Naik, вот пример синхронизированного метода:
synchronized void method1(){
}

aNNiMON, в keyPressed устанавливается код кнопки полю класса и вспомог. переменные. Нажатие обрабатывается в отдельном треде чтобы облегчить методы keyPressed, keyReleased. Получается в главный поток занят только отрисовкой, а в другом производятся все вычисления.

Вот здесь советуют писать как можно более быстровыполняемые методы, которые вызываются системой (keyPressed, etc) — http://www.mobilab.ru/articles/4/

Naik, там какая-то беда с системными вызовами. поставил небольшой сон в методе paint, мидлет поспал и потом вывел всё сразу, хотя должно было вылезти по очереди. надо поманипулировать с вызовами системнымных методов, тесты погонять.

Короче сделал без всяких synchronized. Синхронизирую вручную с пом. paintInProcess:

Открыть спойлер

Закрыть спойлер

[копировать] [скачать]

1. public void paint(Graphics g) {
2.         mapHolder.draw(g, mapX - curX, mapY - curY);
3.         g.drawImage(cursor, curX - 7, curY - 7, Graphics.LEFT | Graphics.TOP);
4.         paintInProcess = false;
5.     }
6.  
7.     public void keyPressed(int keyCodeNew) {
8.         keyCode = keyCodeNew;
9.         oneClick = true;
10.         pressedMove = true;
11.     }
12.  
13.     public void keyRepeated(int keyCodeNew) {
14.         keyCode = keyCodeNew;
15.         oneClick = false;
16.         pressedMove = true;
17.     }
18.  
19.     public void keyReleased(int keyCode) {
20.         pressedMove = false;
21.     }
22.  
23.     public void run() {
24.         while (running) {
25.             if (pressedMove && !paintInProcess) {
26.                 if (keyCode == KEY_NUM4) {
27.                     mapX = mapX <= 0 ? 0 : mapX - 1;
28.                     updateGraphics();
29.                 } else if (keyCode == KEY_NUM2) {
30.                     mapY = mapY <= 0 ? 0 : mapY - 1;
31.                     updateGraphics();
32.                 } else if (keyCode == KEY_NUM6) {
33.                     mapX = mapX >= w ? w : mapX + 1;
34.                     updateGraphics();
35.                 } else if (keyCode == KEY_NUM8) {
36.                     mapY = mapY >= h ? h : mapY + 1;
37.                     updateGraphics();
38.                 }
39.                 if (oneClick) {
40.                     pressedMove = false;
41.                 }
42.             } else {
43.                 try {
44.                     Thread.sleep(1);
45.                 } catch (InterruptedException ex) {}
46.             }
47.         }
48.     }
49.  
50.     private void updateGraphics() {
51.         curX = getXOnScreen(mapX);
52.         curY = getYOnScreen(mapY);
53.         paintInProcess = true;
54.         repaint();
55.     }

Теперь как и нужно на каждое изменение координат приходится одно выполнение paint.

Молодец!

Соберите плиз!

Nokia_372Oc, собрал, только исправил неправильный доступ к переменной ln из главного класса на такой:
Main.m.ln