自作キーボードに同時押しと、単体押しの区別をさせたいです！

【相談内容】自作キーボードに同時押しと、単体押しの区別をさせたい！

現在HID機能のついたPro Microと画像の自作キーマトリクスを用いて自作キーボードを作っています。
作っている目的は、GarageBandというパソコンのキーボードで演奏できる音楽ソフトに使う専用キーボードを作るためなのですが、
今はとりあえず「キーaのみ長押しすると『a』、キーdのみ長押しすると『d』、キーaとキーdを同時に長押しすると『s』のみが連打される」というコードをに作ったのですが、同時押しする際にうまく動いてくれません。

【具体的な問題】同時押しすると『a』『d』『s』全て反応する

具体的には、もう一つの画像のように、「キーaとキーdを同時に押すと、『a』『d』『s』が同時に全て反応」してしまいます。

これを「同時押しの時は『s』のみ反応、『a』『d』は打たれない」ようにしたいです！

自分なりに考えてみたこと

多分後半のif分の作り方が悪いとは思っているのですが解決策がよくわかりません...
知識のある方、アドバイスお願いします！

イメージ説明


#include <Keypad.h>
#include <Keyboard.h>

const byte ROWS = 4; //four rows
const byte COLS = 6; //four columns
//define the cymbols on the buttons of the keypads
char hexaKeys[ROWS][COLS] =
{
  {'S', 'T', 'U', 'V', 'W', 'X'},
  {'t', 'g', 'y', 'h', 'u', 'j'},
  {'M', 'N', 'O', 'O', 'Q', 'R'},
  {'a', 's', 'd', 'f', 'g', 'h'},
};

byte rowPins[ROWS] = {10, 16, 14, 15}; //connect to the row pinouts of the keypad
byte colPins[COLS] = {9, 8, 7, 6, 5, 4}; //connect to the column pinouts of the keypad

Keypad kpd = Keypad( makeKeymap(hexaKeys), rowPins, colPins, ROWS, COLS );

unsigned long loopCount;
unsigned long startTime;
String msg;

bool atest = false;
bool dtest = false;


void setup() {
  Serial.begin(9600);
  loopCount = 0;
  startTime = millis();
  msg = "";
  Keyboard.begin();

}


void loop() {
  loopCount++;
  if ( (millis() - startTime) > 5000 ) {
    Serial.print("Average loops per second = ");
    Serial.println(loopCount / 5);
    startTime = millis();
    loopCount = 0;
  }

  // Fills kpd.key[ ] array with up-to 10 active keys.
  // Returns true if there are ANY active keys.
  if (kpd.getKeys())
  {
    for (int i = 0; i < LIST_MAX; i++) // Scan the whole key list.
    {
      if ( kpd.key[i].stateChanged )   // Only find keys that have changed state.
      {
        switch (kpd.key[i].kstate) {  // Report active key state : IDLE, PRESSED, HOLD, or RELEASED
          case PRESSED:
            msg = " PRESSED.";
            break;
          case HOLD:
            msg = " HOLD.";
            //                    Keyboard.press(kpd.key[i].kchar);
            break;
          case RELEASED:
            msg = " RELEASED.";
            Keyboard.releaseAll();
            //              atest,dtest = false;
            atest = false;
            dtest = false;
            //                    Keyboard.release(kpd.key[i].kchar);
            break;
          case IDLE:
            msg = " IDLE.";
        }
      }
      
      if (kpd.isPressed('a'))  {
        atest = true;
        Serial.println(atest);
      }
      if (kpd.isPressed('d'))  {
        dtest = true;
        Serial.println(dtest);
      }


      if (atest && dtest) {
        Keyboard.press('s');
      }
      if (atest) {
        Keyboard.press('a');
      }
      if (dtest) {
        Keyboard.press('d');
      }


      Serial.print("Key");
      Serial.print(kpd.key[i].kchar);
      Serial.println(msg);


    }
  }
}

thkana

2022/01/22 23:36

ライブラリを使用しているのなら、ライブラリの出自を記載してください。内容を精査していませんが、この問題はライブラリの内部にかかわりそうな気がします。ライブラリ(ヘッダ)名が同じでも内容の異なる別のライブラリがいくつかあって使い方や挙動が違う、などというのも時々あることですので、あなたが使ったものを明記してください。

tmp

2022/01/26 08:40

同時押しの目的も書くと手段の説明不足でも推測されやすくなり、あなたの求める回答も出やすいと思いますが、目的はなんですか？ PRESSEDではなくHOLDなのでコメント化前のソースは、長押し対応していたのに修正後は、そういった考慮もありません。長押しは不要ですか？同時押しは、aとdがまさにOFF->ONに同時に変化することを示していますか？それとも、aがONの状態であってさらにdがOFF->ONになることも同時押しに含まれますか？同時に変化することを示す場合、物理的に同時に押せてもキーのスキャンのタイミングでソフト的に同時に変化とならない場合もありますが、それらは考慮すべきですか？それとも勉強なので理想的な信号と考えて考慮は不要ですか？目的がわからないのでどういった動作が妥当か推測しにくいです。修正後のソースがキーのリリースが適当なので、その音楽ソフトは、キーを押している状態は不要なのですか？目的が書いてないので、音楽ソフトの知識のない私が考えるとキーを押したとき、離したとき、それぞれ通知するコメント化前のもともとのソースの方が、音楽用キーボードとしては妥当な動作だと思ってしまいます。

回答3件

「キーaのみ長押しすると『a』、キーdのみ長押しすると『d』、キーaとキーdを同時に長押しすると『s』のみが連打される」というコード

キーボード（楽器）なら押した瞬間に鳴らないとおかしいのでは？
つまり、押した瞬間に音が鳴る仕様になるでしょう。
そして、『長押しとみなす時間を経過した途端に』勝手に異なる音に変わったら、よりおかしくなるのでは？

※そもそも、現在のスケッチでは「長押し」の処理は何もしていませんよ。間違えて「長」を付けてしまっただけでしょうか？

現在、

・どれかのキーが離れたら、(PCで)全部のキーを離したことにする
・aとdの両方が押されていればsをPCへ送る
・aが押されていればaをPCへ送る
・bが押されていればbをPCへ送る

しかしていません。

「長押し」を別にすれば

Arduino
1if (atest && dtest) {
2	Keyboard.releaseAll();
3	Keyboard.press('s');
4}else if (atest) {
5	Keyboard.press('a');
6}else (dtest) {
7	Keyboard.press('d');
8}

とすれば多少近づくのかも知れません。
（けど楽器のキーボードの仕様としてどうなのか？？）

投稿2022/01/23 04:13

nac_tnk

総合スコア461

キーaとキーdを同時に押すと、『a』『d』『s』が同時に全て反応」してしまいます。

のようになるのは、判定が以下のようになっているからです。

c
1      if (atest && dtest) {
2        Keyboard.press('s');
3      }
4      if (atest) {
5        Keyboard.press('a');
6      }
7      if (dtest) {
8        Keyboard.press('d');
9      }

これは、

aとdが押されていたらsを出力
aが押されていたらaを出力
dが押されていたらdを出力

をそれぞれ判定しているからです。

ここを排他的、要するに、前の条件を満たしていたら残りはチェックしない、という動きにすればよさそうです。
これで上手く行くのかどうかは微妙ですが、

c
1      if (atest && dtest) {
2        Keyboard.press('s');
3      }
4      else if (atest) {
5        Keyboard.press('a');
6      }
7      else if (dtest) {
8        Keyboard.press('d');
9      }