M5StackとArduino間を用いて複数データのやり取りがしたい

Question

前提・実現したいこと

はじめまして、趣味の機械工作にM5StackとArduinoを組み合わせようとしたところ、プログラミングで行き詰ってしまい、その道の方にアイデアをもらうために今回質問をさせていただきました。

以前私はArduinoを用いた簡単な玩具を作ったことがあり、今回はその玩具にM5Stackを組み合わせ、画面にArduinoから受信した値を表示すれば面白くなるのではないかと考えました。

やり取りするデータが二種類あるため、混同を防ぐためにもググって出てきた、配列と疑似的なsplit関数を用いた方法を試したのですが、代入がうまくいかないのか、どちらのデータも最初に定義していた0が返ってくる形になりました。

いくつか試してみたところ、ググって持ってきたsplit関数が悪さをしているのではないかとまでは推測できたのですが、如何せん調べて出てきたものほぼそのままを使った状態なので、どう直せばいいのかも検討がつきません。
当方プログラミングの初心者のため覚束無い質問になっているかとは思いますが、どうかお知恵をお貸しください。

発生している問題

Arduinoから送った複数データを場合分けしてM5Stackの画面に表示したいが、場合分けと代入がうまくいかず両データ共に0が返ってくる。

該当のソースコード

POSX=1
POSY=2

//Arduino上の送信に関する部分
void loop(){
中略
　SendData(POSX, posX);
　SendData(POSY, posY);
　delay(10);
}

void SendData(int kind, int data){  
  Serial.write(kind);
  Serial.write(","); 
  Serial.write(int(data));
  Serial.write("\n");
}

//M5Stack上の疑似split関数など
int split(String data, char delimiter, int *dst){
    int index = 0;
    int arraySize = (sizeof(data)/sizeof((data)[0]));
    int datalength = data.length();
    for (int i = 0; i < datalength; i++) {
        char tmp = data.charAt(i);
        if ( tmp == delimiter ) {
            index++;
            if ( index > (arraySize - 1)) return -1;
        }
        else dst[index] += float(tmp);
    }
    return (index + 1);
}

void serialEvent(){
  String myString = Serial.readStringUntil('\n');
  int index = split(myString, ',', cmds);
  if(Serial.available() >= 2){
    switch((cmds[0])){
    case POSX:
    posX = cmds[1];
    break;
     
    case POSY:
    posY = cmds[1];
    break;
    }
  }
}

Answer 1

ググって持ってきたsplit関数が悪さをしている

いや、悪いですねぇ。でも、ネットにあったのを真似ただけ、ワタシは悪くない、とか言いたいのでしょうか。真似てはいけない間違ったモノは初心者だからといってやっぱり真似ちゃいけません。
なるほど、' Arduino split string 'でググると上位でヒットするこれですか。
これがマトモでないことは、最初の一行で
String cmds[2]={"\0"};
とでも変更してみれば破綻することで簡単にわかります。

String cmds[3] = {"\0"}; // 分割された文字列を格納する配列

int split(String data, char delimiter, String *dst, int arraySize) {
  int index = 0;
  int datalength = data.length();
  for (int i = 0; i < datalength; i++) {
    char tmp = data.charAt(i);
    if ( tmp == delimiter ) {
      index++;
      if ( index > (arraySize - 1)) return -1;
    }
    else dst[index] += tmp;
  }
  return (index + 1);
}

void setup() {
  // 文字列
  String cmd = "on,off,on";
  // 分割数 = 分割処理(文字列, 区切り文字, 配列)
  int index = split(cmd, ',', cmds, sizeof(cmds)/sizeof(cmds[0]));
  // 結果表示
  Serial.begin(9600);
  if(index<0){
    Serial.println("ERROR");
  }else
  for (int i = 0; i < index; i++) {
    Serial.println(cmds[i]);
  }
}

void loop() {
}

とでもすればいいでしょうか。

もちろん、引数をintの配列にして、エラーが出ないように適当に変更してもそれで動くというものではありません。
例えば、

int cmds[3] = {}; // 分割された数値を格納する配列

int split(String data, char delimiter, int *dst, int arraySize) {
  int index = 0;
  String val="";
  int datalength = data.length();
  for (int i = 0; i < datalength; i++) {
    char tmp = data.charAt(i);
    if ( tmp == delimiter ) {
      dst[index++]=val.toInt();
      val="";
      if ( index > (arraySize - 1)) return -1;
    }
    else val += tmp;
  }
  dst[index]=val.toInt();
  return (index + 1);
}

void setup() {
  // 文字列
  String cmd = "123, 456 , 78 ";
  // 分割数 = 分割処理(文字列, 区切り文字, 配列)
  int index = split(cmd, ',', cmds, sizeof(cmds)/sizeof(cmds[0]));
  // 結果表示
  Serial.begin(9600);
  if(index<0){
    Serial.println("ERROR");
  }else
  for (int i = 0; i < index; i++) {
    Serial.println(cmds[i]);
  }
}

void loop() {

}

ぐらいやってみて、数値の切り出しができるか確認してから自分のプログラムに取り込むべきでしょう。

で、serialEventの方はあなたのオリジナルでしょうか。
if(Serial.available() >= 2){
は何をしているのか、ちょっと説明してみて下さい。

Comments

[tau]

thkana様

まずは回答ありがとうございます。
if(Serial.available()>=2){
のくだりについてなのですが、実はthkana様の提示した件のサイトを元に、私のやろうとしていることに近いことを行おうとしたブログがあり、私が参考にしたのはそちらになります。
そちらで受信用の関数として掲載されたものに含まれていました。

初心者のためあまりよく考えずに入れてしまったのですが、2つ以上のデータが受信した時(今回の場合posXとposYを同時に受信することがあるため)、以下の処理を行え、という意味だと認識しています。

お手数をおかけしますが、もう少しご教授お願いできると幸いです。

[thkana]

たとえ趣味でも機械工作をする方なら、なにも考えずにあっちのものとこっちのものをセロハンテープでくっつけても動くわけがない、という感じ、わかりますよね。それとも、万が一動いちゃうかも知れないことを期待してやってみるクチですか?
機械(コンピュータも同じですけれど)は、こちらが初心者かベテランかなんて関係なく、ちゃんと作れば動きますし、ヘンなことやれば壊れます(コンピュータ壊すのは結構大変ですけれど)。

> 初心者のためあまりよく考えずに入れてしまった
取るべき行動は逆じゃないですか? 初心者なればこそ、よく考えるべきじゃないですか? ベテランなら経験だけでも相当判断できちゃいますけど、初心者なら一つ一つ吟味して組み合わせなきゃ。

if(Serial.available() >= 2){
で何が起こるのか、追ってみて下さい。

Answer 2

まず、シリアル受信側では、１行の文字列を受信して処理するようにしましょう
受信文字を、バッファにためていき、\nが来たら、そのバッファの文字列を処理するようにします。