node シリアル書き出の順番

Question

var sp = new serialport.SerialPort(portName, {
    baudRate: 9600,
    dataBits: 8,
    parity: 'none',
    stopBits: 1,
    flowControl: false,
		parser: serialport.parsers.readline('\r')
    });

sp.write( 'command1\r', function(err, bytesWritten) {
		console.log("send command1");
				setTimeout(function(){
						sp.write( 'command2\r', function(err, bytesWritten) {
								console.log("send command2");
							});
				} ,100);
	});

nodeで、こんな感じのコードを書いて見たのですが
ときどきcommand2の方がcommand1より先に出力されてしまうことがあります
何故でしょう？必ず順番を守れるようにならないのでしょうか？

Answer 1

Promiseを使って解決出来ます。
https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Promise
http://qiita.com/koki_cheese/items/c559da338a3d307c9d88
http://junmt.hatenablog.com/entry/2015/12/11/171045

手元にnode.js環境がないので、動作検証できていないのですが、元コードを置き換えるとおおよそ以下のような感じになると思います。

var sp = new serialport.SerialPort(portName, {
  baudRate: 9600,
  dataBits: 8,
  parity: 'none',
  stopBits: 1,
  flowControl: false,
    parser: serialport.parsers.readline('\r')
  });

var spWrite = function (portName) {
  return new Promise(function(resolve, reject) {
    sp.write(portName + '\r', function (err, bytesWritten) {
      resolve('send: ' + portName);
    });
  });
};

spWrite('command1')
  .then(function (result) {
    console.log(result);
    return spWrite('command2');
  })
  .then(function (result) {
    console.log(result);
  });

他、Promise.allやasync/awaitもあるので合わせて、試してみてください。

Comments

[apple2]

ありがとうございます
難しそうでうですけど、試して見ます

[apple2]

promiseが使えないバージョンだったので、nodeのバージョンを4.8.2にあげたのですが
serialportが使えなくなり
npm install serialportでインストールしようとするとエラーになり
つまづいています

[退会済みユーザー]

すでに試されていると思われますが、環境による「npm install serialport」ではNGな場合もあるので確認してみてください。
https://github.com/EmergingTechnologyAdvisors/node-serialport/blob/master/README.md#installation-special-cases

どうしても解決出来ない場合、新たに質問を立てた方が回答を得やすいと思います。

[apple2]

nodeのバージョンをv0.11.14 にして無事にserialportがインストールでき、教えてもらったPromiseを試してみたのですが、以前と同じように逆転現象が起きました

受信する側の都合もあってインターバルを置きたく
var spWrite = function (portName) {
return new Promise(function(resolve, reject) {
setTimeout(function(){
sp.write(portName + '\r', function (err, bytesWritten) {
resolve( portName);
});
},400);
} );
};
setTimeoutを追加しました

[apple2]

私のインターバルの取り方がまずいのでしょうかね

[退会済みユーザー]

よくよくapple2さんが質問で記述されているコードを見直してみると、処理の流れとしては「command1」->「command2」となるはずですね（setTimeoutで処理をずらさなくても）。
同様の環境が用意できないので、ちょっと手詰まりな感じです。
引っ掻き回すだけとなってしまい、申し訳ありませんでした。

[apple2]

本当はシリアルの接続先では送ったコマンドがループバックされる仕組みになっていて、送ったコマンドが帰ってから次のコマンドを送るということがしたいのです。
今は受信の部分で帰ってきたdataとportNameを比較して合否を出すというようなことをやっているのですが、私のやり方が根本的に間違っているのでしょうか