勝手ながら添削

[yosuke-furukawa]

これだけのサンプル作成は本当に素晴らしいですね。

ただ、いくつか勘違いされている所もありそうなのと、サンプルとしてどうなのかなと思ったところがあったので、コメントだけ残しておきます。

■1. IPが192.168.1.202に固定はどうなんでしょう。ローカルでは動かない時も多いとおもいますし、ローカルで動作させようと思ったらlocalhostに書き換えるか127.0.0.1とかを指定しなくてはいけなくて初心者は困惑するのでは。IPを固定する意味はないのであればlocalhostにしておいたほうが無難な気がします。

■2. sysモジュールは古いのでutilモジュール使いましょう。
昔のサンプルから取ってきたのか、sysを使っている所があって、古い気がしました。utilを使いましょう。

■3. listenの後、Serverが起動したことを表すログを毎回console.logを書いてるんですが、listenの成功失敗に関わらず出るし、listenのタイミングじゃないので微妙です。

listenはコールバックを受け付けるのでそこで出す方が一般的ですね。

var http = require("http");
http.createServer(function (req, res) {
    res.writeHead(200, {"Content-Type": "text/plain"});
    res.end("hello, world!\n");
}).listen(1337, "192.168.1.202", function() {
  console.log("Server running at http://192.168.1.202:1337/");
});

■4. 043_socket_ioでクライアント(自分を含めた全て)にイベント送信
知っているかもしれませんが、socket.broadcast.emitして、socket.emitしてますが、io.sockets.emitの一回でできるのでサンプルとしてはそれでやった方がいいですね。
自分のsocketの場合にだけ処理を変えたい時は分けておいていいんですが。043は特に変えているわけでは無さそうなので。他にも044とかでも同じことしてますがこういう場合はio.sockets.emitの方がいいです。

あとcliant.html(client.htmlのtypoっぽい?)の中のscriptタグ内でfuncメソッドをクライアントで実行してますが関数が引数に渡されていないので、エラーになります。

■5. あと、ブログのタイトルにwebsocket編、と書いているんですが、socket.io = websocketだと勘違いされているケースが多いので、どちらかというとsocket.io編にするか、別にsocket.ioに限ってない気がするのでnode.js TIPSにしては如何でしょう。
まぁ細かい話なのですが、本当にこの勘違い多いんです。

6以降はさらに微妙な話ですが。。

■6. requestからのデータをdataとendのイベントで受ける、Stream1で記述されているので、少し古いですね、これからはreadableとendで受けるStream2っぽく記述しておくと今風かもしれません。ただ、コレに関してはnode v0.10以上でしか使えないので古いnodeも対象とするのであれば要らないと思います。

■7. cliant.htmlにsocket.io.jsのURLが記述されているんですが、IPがhtml内にベタ書きで残念ですね。せっかくIPを外部ファイル化しているのに。 src = "/socket.io/socket.io.js" としておけばIPは不要かと思いますが、明示したいのであればfs.readFileのコールバックでdataの中身を差し替える方法が良いかもしれませんね。

こんな感じ?

// 外部のHTMLデータを読み込み
    fs.readFile(
        "./043_cliant.html",
        function (err, data) {
                        if (err) { throw err; }
       data = "" + data;
      data = data.replace('</body>', '<script type="text/javascript" src="http://' + setting.IP + ':' + setting.PORT+ '/socket.io/socket.io.js"></script></body>');
            // HTTPレスポンスヘッダを作成・送信(200:OK,500:ServerError,404:NotFound)
            res.writeHead(200, {"Content-Type": "text/html; charset=UTF-8"});
            res.write(data);
            res.end();

        }
    );

ひとまずこんな所でしょうか、全部見たわけじゃないので他にもあるかもしれませんが、

[omatoro]

質問です！

「■4. 043_socket_ioでクライアント(自分を含めた全て)にイベント送信」にて以下の記述の意味がよく分かりません…。

あとcliant.html(client.htmlのtypoっぽい?)の中のscriptタグ内でfuncメソッドをクライアントで実行してますが
関数が引数に渡されていないので、エラーになります。

io.sockets.emitには3つめの引数であるコールバック関数が使えないよ、という意味でしょうか？

[yosuke-furukawa]

む。
これは僕のミスですね。io.sockets.emitはコールバックを受け付けないようですね。
どうもsocket.io側のバグのようです。 socketio/socket.io#673

なので043でio.sockets.emitをやると送信元のコールバックは読まれません。
コールバックを出力させるのであれば基の方法でやるしかなさそうですね。

[omatoro]

なるほど！バグだったのですね。
ありがとうございます！

コールバックを使わないようにio.sockets.emitを実行するように修正しています。
ただ申し訳ないのが、私の不手際によるものなのかcommitのログがおかしくなっています。
差分が上手く表示されず、全て修正したような表示になっています。

すみません、web上で確認するときちんと表示されています。
お騒がせしました。

全て修正し終えたら、その分の内容をissueにて記載しますので、確認をよろしくお願いします。

[yosuke-furukawa]

ここまで意欲的にやってくれると思ってなかったので、素晴らしいと思います。
ここまでやってくれるのであれば、もう少し気づいた所があるので入れていこうかと。

■1. package.jsonに入れて、npm installで環境構築できるようにしておいて、README.mdを置いてそれを説明した方が親切かもしれませんね。
プロジェクトルートにpackage.jsonとnode_modulesを置いておけば001とか002の下のフォルダには不要なので。

.
├── node_modules
├── package.json
├── 001
  ├── //jsファイル郡

■2. 例外処理をどうするかは考えたほうがいいかもしれません。
こんな感じにthrow errで書いてるんですが、これ、誰もキャッチしてないからファイルに関するエラーが発生したらnodeが死にます。

server.on("request", function(req, res) {
    // 外部のHTMLデータを読み込み
    fs.readFile(
        "./044_client.html",
        function (err, data) {
            if (err) { throw err; }

            // HTTPレスポンスヘッダを作成・送信(200:OK,500:ServerError,404:NotFound)
            res.writeHead(200, {"Content-Type": "text/html; charset=UTF-8"});
            res.end(data);
        }
    );
});

サンプルなのでそれでもいいんですけど、本当ならlogに残した上で適切なエラーを返す気がしますね。

server.on("request", function(req, res) {
    // 外部のHTMLデータを読み込み
    fs.readFile(
        "./044_client.html",
        function (err, data) {
            if (err) {
              //なんかログに残す、ここではとりあえずconsole.logで。
              console.log(err);
              res.writeHead(500);
              res.end("Server error : " + err);
            }

            // HTTPレスポンスヘッダを作成・送信(200:OK,500:ServerError,404:NotFound)
            res.writeHead(200, {"Content-Type": "text/html; charset=UTF-8"});
            res.end(data);
        }
    );
});

■3. 044でconnectedイベントを発行してますけど、特に何か送信しないのであればconnectイベントをクライアントで受信すれば十分ですね。connectedをわざわざ発行する必要はありません。

■4. 超微妙なんですけど、044と045が配列を毎回繰り返してて惜しい感じですね。
node.jsはその性質上、シングルスレッドなのでCPU処理に弱いのです。ループが頻繁にあると他の処理が止まります。可能であればデータ構造をハッシュで持ち、idをkeyとして、名前をvalueで持ったほうが良いですね。

var MEMBER = {};

MEMBER[id] = name;

みたいな。

五月雨ですいません。。。気づいた時に対応してくれれば良いです。