4

このコードを書き直して、プロセッサでより適切に動作するようにすることはできますか? 別のスレッドで一定の周期でいくつかのタスクを実行するクラスがあります。場合によっては、このプロセスを一時停止して再開することができます。現在、一時停止用のフラグを使用していますが、正常に動作しますが、この方法でのループは、プロセスが一時停止しているときにプロセッサをロードします。これを修正することは可能ですか?

private boolean mIsCanceled = false;
private boolean mIsPaused = true; // TODO more efficient for processor way of pausing is required
private final Thread mTimerThread = new Thread(new Runnable() {
    @Override
    public void run() {
        while(!mIsCanceled){
            try {
                Thread.sleep(UPDATE_PERIOD);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
            if (!mIsPaused){
                doStep();
            }
        }
    }
});

public MyClass(){
    mTimerThread.start();
}

private void pause(){
    mIsPaused = true;
}

private void resume(){
    mIsPaused = false;
}

private void doStep(){
    // Some code
}

私のコードの代替実装を提供してください。

PS環境はAndroid OS 2.2+です

4

4 に答える 4

5

利用可能なツールは次のとおりです。

wait/ notify- 私たちは皆、この古風なシステムから逃れようとしています。

Semaphores - スレッドがそれをつかんだら、解放するまでそれを保持するので、再度つかんでもブロックされません。これは、自分のスレッド内から一時停止できないことを意味します。

CyclicBarrier- 使用するたびに新規作成する必要があります。

ReadWriteLock- お気に入り。好きなだけスレッドを一時停止することができ、すべてのスレッドが を呼び出したときにのみ再開されresumeます。必要に応じて一時停止することもできます。

import java.util.concurrent.locks.ReadWriteLock;
import java.util.concurrent.locks.ReentrantReadWriteLock;

/**
 * PauseableThread is a Thread with pause/resume and cancel methods.
 *
 * The meat of the process must implement `step`.
 *
 * You can either extend this and implement `step` or use the factory.
 *
 * Note that I cannot extend Thread because my resume will clash with Thread's deprecated one. 
 *
 * Usage: Either write a `Stepper` and run it in a `PausableThread` or extend `PausableThread` and call `blockIfPaused()` at appropriate points.
 */
public abstract class PauseableThread implements Runnable {
  // The lock.
  // We'll hold a read lock on it to pause the thread.
  // The thread will momentarily grab a write lock on it to pause.
  // This way you can have multiple pausers using normal locks.
  private final ReadWriteLock pause = new ReentrantReadWriteLock();
  // Flag to cancel the wholeprocess.
  private volatile boolean cancelled = false;
  // The exception that caused it to finish.
  private Exception thrown = null;

  @Override
  // The core run mechanism.
  public void run() {
    try {
      while (!cancelled) {
        // Block here if we're paused.
        blockIfPaused();
        // Do my work.
        step();
      }
    } catch (Exception ex) {
      // Just fall out when exception is thrown.
      thrown = ex;
    }
  }

  // Block if pause has been called without a matching resume.
  private void blockIfPaused() throws InterruptedException {
    try {
      // Grab a write lock. Will block if a read lock has been taken.
      pause.writeLock().lockInterruptibly();
    } finally {
      // Release the lock immediately to avoid blocking when pause is called.
      pause.writeLock().unlock();
    }

  }

  // Pause the work. NB: MUST be balanced by a resume.
  public void pause() {
    // We can wait for a lock here.
    pause.readLock().lock();
  }

  // Resume the work. NB: MUST be balanced by a pause.
  public void resume() {
    // Release the lock.
    pause.readLock().unlock();
  }

  // Stop.
  public void cancel() {
    // Stop everything.
    cancelled = true;
  }

  // start - like a thread.
  public void start() {
    // Wrap it in a thread.
    new Thread(this).start();
  }

  // Get the exceptuion that was thrown to stop the thread or null if the thread was cancelled.
  public Exception getThrown() {
    return thrown;
  }

  // Create this method to do stuff. 
  // Calls to this method will stop when pause is called.
  // Any thrown exception stops the whole process.
  public abstract void step() throws Exception;

  // Factory to wrap a Stepper in a PauseableThread
  public static PauseableThread make(Stepper stepper) {
    StepperThread pauseableStepper = new StepperThread(stepper);
    // That's the thread they can pause/resume.
    return pauseableStepper;
  }

  // One of these must be used.
  public interface Stepper {
    // A Stepper has a step method.
    // Any exception thrown causes the enclosing thread to stop.
    public void step() throws Exception;
  }

  // Holder for a Stepper.
  private static class StepperThread extends PauseableThread {
    private final Stepper stepper;

    StepperThread(Stepper stepper) {
      this.stepper = stepper;
    }

    @Override
    public void step() throws Exception {
      stepper.step();
    }
  }

  // My test counter.
  static int n = 0;

  // Test/demo.
  public static void main(String[] args) throws InterruptedException {

    try {
      // Simple stepper that just increments n.
      Stepper s = new Stepper() {
        @Override
        public void step() throws Exception {
          n += 1;
          Thread.sleep(10);
        }
      };
      PauseableThread t = PauseableThread.make(s);
      // Start it up.
      t.start();
      Thread.sleep(1000);
      t.pause();
      System.out.println("Paused: " + n);
      Thread.sleep(1000);
      System.out.println("Resuminng: " + n);
      t.resume();
      Thread.sleep(1000);
      t.cancel();
    } catch (Exception e) {
    }
  }
}

編集:より一般的に使用できるようにコードが変更されました。

于 2012-05-19T23:33:36.243 に答える
1

最良のオプションは、wait()/notify() を使用するか、単にScheduledExecutorServiceに切り替えることです。

適切な wait()/notify() の使用には注意が必要です。スレッドの詳細については、「Java Concurrency in Practice」を強くお勧めします。

于 2012-05-19T16:07:08.930 に答える
0

ここでの最善の方法はThread.wait、スリープする代わりに待機中Thread.notifyのスレッドに使用し、待機中のスレッドで使用することだと思います。詳細はこちら: http://www.javamex.com/tutorials/synchronization_wait_notify.shtml

于 2012-05-19T14:18:26.233 に答える
0

スレッドをスリープ状態にする代わりにモニターを使用することで、効率を大幅に向上させることができます。コード内でキーワード synchronized を使用してブロックを作成するだけです。そして、モニターとして機能する最後のオブジェクトです。モニターの API でさらに調べてください。

于 2012-05-19T14:19:55.583 に答える