rosserial(Wi-Fi)の委譲(delegation)はポリモーフィズムの罠があった。

# はじめに
個人的に、M5Stack公式のロボットである、[M5BALA \- ミニバランスカー](https://www.switch-science.com/catalog/3995/)のROS化を進めている。（[fudekun/ros\_m5bala: ROS driver for M5Bala](https://github.com/fudekun/ros_m5bala)）M5Balaは手のひらサイズでとっても「かわいいやつ」である。それでいて、価格はM5Stack Fireと合わせても1.3万円程度とオモチャとしては最高のコスパ。是非とも布教したい。
 
![IMG_3718.jpg](https://qiita-image-store.s3.ap-northeast-1.amazonaws.com/0/96992/2594d8ad-f5b8-4895-1e04-64fc7c59d209.jpeg)

このロボットの実装に関しては、後々詳細をまとめようと思うのだが、思いっきり引っ掛かったポイントがあったのでその点について先にまとめることとする。

## 結論から書くと、、、
__いっぱいVoteされている記事だからといって、鵜呑みにしてコピペせずによく見て使うこと。__
まぁ、とりあえず急ぐ人のために。rosserial(Wi-Fi)の委譲(delegation)はWiFiHardware実装は次のようにするのが良さそうであるということを最初に書いておく。

```cpp
class WiFiHardware {
  public:
  // 中略
  void write(uint8_t* data, int length) {
    client.write(data, length);    // After
    // for(int i=0; i<length; i++)    Before
    //  client.write(data[i]);
  }
  // 中略
};
```

rosserial(Wi-Fi)の実装を解説しながら、順を追って見ていく。

# 事の経緯

<blockquote class="twitter-tweet"><p lang="ja" dir="ltr">本日のros_m5bala進捗<br>xacroを作って、odomとtfを <a href="https://twitter.com/hashtag/M5stack?src=hash&amp;ref_src=twsrc%5Etfw">#M5stack</a> で計算してpubするようにした。大体合ってる。<br>気になるのが、odomのpub周期が1Hzくらいしか出ないのはそういうものなのか、はたまた遅い原因があるのか、、、要調査。<a href="https://t.co/TfBPvxlIB7">https://t.co/TfBPvxlIB7</a> <a href="https://t.co/eB2LmLx8Qj">pic.twitter.com/eB2LmLx8Qj</a></p>&mdash; T.Fukuta(fudekun) (@fudekunJP) <a href="https://twitter.com/fudekunJP/status/1135148690857897984?ref_src=twsrc%5Etfw">June 2, 2019</a></blockquote> <script async src="https://platform.twitter.com/widgets.js" charset="utf-8"></script>

自分の引用したツイートの通り、「odomのpub周期が1Hzくらいしか出ない」という問題にぶち当たった。
さすがに、M5Stackほどの高機能Arduino機で`1Hz、1KB/s`しか出ないのはおかしいと思い、いろいろ見直していくと、あやしげな点を発見した。

<blockquote class="twitter-tweet"><p lang="ja" dir="ltr">WiFiHardwareの送信処理が気になる。これは、データ量が増えると遅そう。。。<br>void write(uint8_t* data, int length) {<br> for(int i=0; i&lt;length; i++)<br> client.write(data[i]);<br>}</p>&mdash; T.Fukuta(fudekun) (@fudekunJP) <a href="https://twitter.com/fudekunJP/status/1135357098835886080?ref_src=twsrc%5Etfw">June 3, 2019</a></blockquote> <script async src="https://platform.twitter.com/widgets.js" charset="utf-8"></script>

そう、コピペした部分に怪しい実装があったのだ。


# rosserial(Wi-Fi)の実装
esp32やM5Stackのような、rosserial(Wi-Fi)を使ってやり取りする場合のコード記述について改めて振り返ってみる。実装方法についてググるとだいたい[この記事](http://answers.ros.org/question/232774/launch-a-ros-node-over-wifi/)にたどり着くと思う。以下では、odomに何も値を突っ込んでいないが、loopなり、別コアで値を突っ込む実装を追加するぐらいでだいたいこんな感じ。

```cpp
#include <ros.h>
#include <nav_msgs/Odometry.h>
#include <WiFi.h>

IPAddress server(192, 168, 0, 10);
WiFiClient client;
char ssid[] = "ssid";
char pass[] = "password";

class WiFiHardware {
  public:
  WiFiHardware() {};
  void init() {
    client.connect(server, 11411);
  }
  int read() {
    return client.read();         //will return -1 when it will works
  }
  void write(uint8_t* data, int length) {
    for(int i=0; i<length; i++)
      client.write(data[i]);
  }
  unsigned long time() {
     return millis(); // easy; did this one for you
  }
};

void setupWiFi() {
  WiFi.begin(ssid, password);
  uint8_t i = 0;
  while (WiFi.status() != WL_CONNECTED && i++ < 20) delay(500);
  if (i == 21) {
    while (1) delay(500);
  }
}

nav_msgs::Odometry msg;
ros::Publisher pub("odom", &msg);
ros::NodeHandle_<WiFiHardware> nh;

void setup() {
  setupWiFi();
  nh.initNode();
}

void loop() {
  pub.publish(&msg);
  nh.spinOnce();
  delay(500);
}
```

今回注目するのは、`WiFiHardware`クラス。

`NodeHandle_`（rosserialのPub/Sub通信を担う）ハンドラーに、「Wi-Fi処理を記述したクラス」をテンプレートを使って__委譲(delegation)__することで、いろんな通信規格に対応しているわけである。
esp32のWiFiClientクラス（[arduino\-esp32/WiFiClient\.h at master · espressif/arduino\-esp32](https://github.com/espressif/arduino-esp32/blob/master/libraries/WiFi/src/WiFiClient.h)）を使って、「Wi-Fi処理を記述したクラス」実装である`WiFiHardware`クラスを書いていく。client.writeが送信、client.readが受信に関する処理である。


# 今回問題になったところ
そう、ループでデータを送り出すここが問題である。この実装は広く普及しているようであるが、uint8_tの8ビット表現のバイトデータ（結構よく使う表現）をぶつ切りにして送り出している。送信データが小さいうちは問題ないのであるが、odomデータのようなそこそこのペイロード（1KB程度）のデータでは送信回数が1000回を超えてくる。これは遅いはずである。

```cpp
  void write(uint8_t* data, int length) {
    for(int i=0; i<length; i++)
      client.write(data[i]);
  }
```

# WiFiClientの実装を見直そう
WiFiClient.hについて見てみると、client.write処理には4種類の実装が存在する（ポリモーフィズム）ことに気がついた。
当初の実装は1番上の実装である。2番目の実装`write(const uint8_t *buf, size_t size);`をcpp含めて見てみると、複数バイトのデータを指定して送信できることが分かった。（そもそも1番目の実装が、2番目の実装を1ブロック（8bit）で送信するという実装になっていた。。。）

```cpp
size_t write(uint8_t data);
size_t write(const uint8_t *buf, size_t size);
size_t write_P(PGM_P buf, size_t size);
size_t write(Stream &stream);
```

```cpp
size_t WiFiClient::write(uint8_t data)
{
    return write(&data, 1);
}

// 中略

size_t WiFiClient::write(const uint8_t *buf, size_t size) 
{
  // 中略
  res = send(socketFileDescriptor, (void*) buf, bytesRemaining, MSG_DONTWAIT); 
  // 中略
}
```

# バルク送信を使った結果
というわけで、従来型実装の8bit毎送信モードと、バルク送信モードを使った結果を比較。
<blockquote class="twitter-tweet"><p lang="ja" dir="ltr">バルクの口があったので試してみたら、およそ150倍ほど爆速になった旨、報告しておきます。後から試す人のために、どこかにまとめておかなくては。<br><br>[before]<br>for(int i=0; i&lt;length; i++)<br> client.write(data[i]);<br>1Hz、1KB/s<br><br>[after]<br>client.write(data, length);<br>270Hz、150KB/s <a href="https://t.co/I3teOi1Oor">pic.twitter.com/I3teOi1Oor</a></p>&mdash; T.Fukuta(fudekun) (@fudekunJP) <a href="https://twitter.com/fudekunJP/status/1135368648502128640?ref_src=twsrc%5Etfw">June 3, 2019</a></blockquote> <script async src="https://platform.twitter.com/widgets.js" charset="utf-8"></script>
周波数では`270倍程度`、送信速度では`150倍程度`の高速化に成功した。もはや270Hzでodomの計算は無理なので、delayを入れてpubする頻度を調整する必要さえ出てくる。画像の送信もこの様子では問題がなさそうであるという手応えを得た。これは嬉しい。


# 最後に
esp32は、リファレンスが無い？のが気になる。おそらく、楽せずソースを読めということでしょう。
ソースと同じくらい、英語も読めるようになりたい。