値と紐づけられた動作が連動しない

Question

いつもお世話になっております。

前提・実現したいこと

このような動作を期待しています。

発生している問題・エラーメッセージ

現状、キーインの入力→動作の変更に至っていません。

例えば、ノミナル（0の状態）から1をキーインして点滅へ動作を変更させようとしても、
1を長押ししないと点滅してくれません。

どこが悪さをしているのか、見当がつきません。

該当のソースコード

SubThread sub = null;

float G = 255;

// 今の状態
int keyNum = 0;
int keyNumBefore = 0;

//変化開始FLG
boolean   cngAlpha;

void setup() {
  frameRate(60);
  size(500, 500, FX2D);
  textSize(16);
  textAlign(LEFT, TOP);

  // サブスレッドクラスを生成
  sub = new SubThread();
  //Threadクラスを生成
  Thread subThread = new Thread(sub);
  //実行開始
  subThread.start();
}

void draw() {
  background(0, G, 0);
  fill(0);
  text("I am Main Thread.", 8, 0);  

  if ( sub != null && sub.running ) {
    // サブスレッドが実行中
    text("Sub Thread is Running.", 8, 20);
  } else {
    // サブスレッドは停止中
    sub = null;
  }

  // 値を更新
  keyNum = keyNumBefore;
}

void keyPressed() {
  if (key == '0') {
    keyNum = 0;
    println("0 pressed");
  } else if (key == '1') {
    keyNum = 1;
    println("1 pressed");
  } else if (key == '2') {
    keyNum = 2;
    println("2 pressed");
  }
}

//フェードイン処理関数
void fadeIn() {
  //徐々に濃くする
  G = G + 4f;

  //不透明になったら変化終了
  if ( G > 255f ) { 
    G = 255f; 
    cngAlpha = false;
  }
}

//フェードアウト処理関数
void fadeOut() {
  //徐々に薄くする
  G = G - 4f;

  //透明になったら変化終了
  if ( G < 0f ) {
    G = 0f;
    cngAlpha = false;
  }
}



class SubThread implements Runnable {
  // 実行許可FLG
  private boolean running = true;

  // ここが実行される
  @Override
    public void run() {

    boolean toggle = true;

    while (running) {
      if (keyNum == 0) {// 今の状態が0
        if (keyNumBefore == 0) {// 1つ前の状態が0（常時点灯）の時｜継続
          // 常時緑
          G = 255;
        } else if (keyNumBefore == 2) {// 1つ前の状態が2（常時消灯）の時｜フェードイン
          // 徐々にGを0から255へ
          fadeIn();
        }
        println("Now keyNum is 0");
      } else if (keyNum == 1) {// 今の状態が1
        try {
          // 点滅間隔｜0.5 [sec]
          Thread.sleep(500);
        }
        catch (InterruptedException ex) {
          ex.printStackTrace();
        }
        if (toggle) {
          G = 255;
          println("Now G is 255");
        } else {
          G = 0;
          println("Now G is 0");
        }
        toggle = !toggle;
      } else if (keyNum == 2) {// 今の状態が2
        println("Now keyNum is 2");
        if (keyNumBefore == 1) {// 1つ前の状態が1（点滅）の時｜フェードアウト
          // 徐々に黒へ
          fadeOut();
        } else if (keyNumBefore == 2) {//1つ前の状態が2（常時消灯）の時｜継続
          // 常時黒
          G = 0;
        }
      }
    }
    G = 255;
    println("Sub Thread exit");
  }

  public void stopRunning() {
    running = false;
  }
}

補足情報（FW/ツールのバージョンなど）

processing 3.4

Comments

[dodox86]

> どこが悪さをしているのか、見当がつきません。
（コードがまだ未完成だと思いましたので、回答もそれ未満です）少なくとも、keyNumBefore変数は0のままdraw()内でkeyNum = keyNumBefore;と上書きしつづけていますから、keyNumはすぐに0になってしまいます。keyNumBefore変数を更新するコードも存在しませんので、0のままです。これではままだご要望の動きにはならないと思います。

[P5_USER]

dodox86さんご指摘ありがとうございます。

よくよく見ると、40行目が可笑しかったです。
keyNumBefore = keyNum;
とするべきでした。

Answer 1

40行目の

keyNum = keyNumBefore;

を

keyNumBefore = keyNum;

に変更しました。

初歩的なミスでした。

Comments

[thkana]

keyNum変数を受けての処理はsubThread.run()で行っているのですから、keyNumBefore = keyNum;の処理はsubThread.run()でやるべきかと思いますが如何?
そうでないと、subThread.run()でkeyNumが切り替わったことを検知する前にkeyNumBeforeが書き換えられてしまう可能性があります。

[P5_USER]

其の通りですね。

ご提案ありがとうございます！