arduinoにPCからシリアル通信で続けてデータを送りたい

Question

前提・実現したいこと

シリアル通信で周波数と長さを送って曲を鳴らしたいです。
440,1000,e,800,1000,e,600,500,e・・・
と続けて音を鳴らしたいです。

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

440,1000,e
とシリアル通信で送ると、440Hzの音が１秒間流れるようにはなりましたが、次の音をうまく鳴らせません。

440,1000,e,800,1000,e
の場合だと440Hzの音が一秒間鳴らずに、一瞬で次の音に移ってしまいます。

該当のソースコード

#define size 6

int i = 0;
char buf[size];
int data[size];

void setup() {
  Serial.begin(9600);
  pinMode(12,OUTPUT);
}

void loop() {
 if (Serial.available()>0) {
    buf[i] = Serial.read();
    if (buf[i] == 'e') {
      for(i=0;i<size;i++){
      data[i] = atoi(strtok(buf, ","));
      }
      i = 0;
    }
      else {
     i++;
    }
    tone(12,data[0],data[1]);
  }
}

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

arduino

雑な質問で申し訳ないのですが、教えていただけると有難いです。
宜しくお願い致します。

Answer 1

tone()の説明
には、

同じピンに対してtone()を実行した場合は周波数が変化します。

とありますから、1秒間鳴らしたいなら、1秒待ってから次の音を送る必要があります。
「自分で時計を見て1秒待つ」、「1秒の割り込みを使う」などの方法をとる必要があります。

Answer 2

いろいろ意図通りではないソースコードと思われます。まずは、tone()に渡している引数を確認(表示)してみてはどうでしょうか? 意図した数値が入っていますか?

#define size 6

int i = 0;
char buf[size];
int data[size];

buf[]は、6バイトしかないので、"440,1000,e" すら受け取ることができません。うまく動いているとしたら、たまたまですね。必要なだけ領域を確保しましょう。

  for(i=0;i<size;i++){
    data[i] = atoi(strtok(buf, ","));
  }

sizeは6なので、必ず6回実行します。これは、意図したことですか? 現状のソースだと、'e'を受け付けるたびに、bufの処理を行っているので、この時点ではデータ一組分だけがbufに入ってると思います。

また、strtok()の使い方も間違っていそうです。通常、1回目はbufを第一引数に渡しますが、2回目以降はNULLを第一引数に渡します。

あと、Serial.available() > 0が成立するたびに、tone()を呼び出しているのも、意図通りではない気がします。必要な情報を組み立ててから、tone()を呼び出すのがいいと思います。

繰り返しになりますが、tone()を呼び出す前に、tone()の引数を表示して、内容を確認してみてはどうでしょうか?

それから、tone()で音を鳴らしているときに、次のtone()を実行すると、最初のtone()は終了します。

If the tone is playing on the same pin, the call will set its frequency.

一瞬で次の音に移るのは、このためと思います。