javascript 括弧の使い方がわからない。ターミナル上でのエラー「SyntaxError: Unexpected」どこが記述ミスなのか分からない

###前提・実現したいこと

nodeとsocketでチャットシステムを作りたいと思っています。

括弧の記述ミスがあると思われるのですが、
どこに、どういう括弧を記述すればいいのか、
ミスがあるのか、わかりません。経験不足です。
chat.js:25というエラー内容から、
chat.jsの25行目で括弧がなかったり
括弧が余分に多かったりするんじゃないかなと思うのですが・・・

わかりやすく説明できずに申し訳ありません。

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

このコードで、ターミナル上で ndoe chat.js と実行すると以下のようなエラーが発生します。


/Desktop/javascript_sample/チャットチャレンジ/chat.js:25
});
 ^

SyntaxError: Unexpected token )

t exports.runInThisContext (vm.js:53:16)
    at Module._compile (module.js:373:25)
    at Object.Module._extensions..js (module.js:416:10)
    at Module.load (module.js:343:32)
    at Function.Module._load (module.js:300:12)
    at Function.Module.runMain (module.js:441:10)
    at startup (node.js:139:18)
    at node.js:968:3

###該当のソースコード

chatt.js

var http = require('http');
var fs = require('fs');

var server = http.createServer();
server.on('request', doRequest);
server.listen(1234);
console.log('Server running!');

  // リクエストの処理
  function doRequest(req, res) {
      fs.readFile('./chat.html', 'UTF-8',
          function(err, data) {
              res.writeHead(200, {'Content-Type': 'text/html'});
              res.write(data);
              res.end();
          });

          
  io.socket.on('connection' , function(socket){
     socket.emit('greeting' , {message: 'hello'}, function (data){
       consle.log('result;' + data);
     });
});

chat.html


<html>
  <hrad>
     <title>チャットする</title>

     <script src="//ajax.googleapis.com/ajax/libs/jquery/1.8.0/jquery.min.js"></script>
         <script src="/socket.io/socket.io.js"></script>
         <script src="/javascript/chat.js"></script>
         <script>

<body>

    <!-- 送るボタンだよ -->
    <input type="text" id="msg_input" style="wibth:200px;" />
    <button onclick="publishMessage()">語る</button>

</body>

<script type= "text/javascript">

//イベントとcall backの定義
var socketio = io.connect('http://localhost:1234');

  socket.on('msg', function(socket){
    log('ここまでクリアだよ！');
    socket.emit(msg send,{message, 'data'});
    socket.on('msg push', function(msg){
      log(ここまでクリアだよ！);
    });
  });

</script>
</html>

###試したこと


nodeのソースコードを見て
括弧の記述方法のルールを探して

     }
});

を書いたり消したりした。

###補足情報(言語/FW/ツール等のバージョンなど


テキストエディタはAtomを使っています。

行動規範の内容に同意します

回答6件

chatt.jsの最後の

javascript
1});

を

javascript
1}

にするのはどうでしょうか。

投稿2016/05/30 05:00

編集2016/05/30 05:01

Lhankor_Mhy

総合スコア36074

Lhankor_Mhy

2016/05/30 05:02

ああ、間違えました……

行動規範の内容に同意します

まずはインデントを揃えるようにしましょう。
きれいに揃っていれば、括弧との対応がわかりやすくなります。

また、対応する括弧をハイライトしてくれるプラグインを活用するなど
コーディングミスの出にくい環境を作ることも重要かと思います。
Atomエディタだとbracket-matcherなどがあるようです。

↓インデントを整えてみました。
function doRequest(req, res) {に対応する閉じ括弧がないことがわかります。

javascript
1// リクエストの処理
2function doRequest(req, res) {
3	fs.readFile('./chat.html', 'UTF-8', function(err, data) {
4		res.writeHead(200, {'Content-Type': 'text/html'});
5		res.write(data);
6		res.end();
7	});
8
9	io.socket.on('connection', function(socket){
10		socket.emit('greeting', {message: 'hello'}, function (data){
11			consle.log('result;' + data);
12		});
13	});
14

投稿2016/05/30 05:10

rkojima

総合スコア421

function doRequestの波括弧の閉じがありませんね。res.end();の2行下に}を加えると動くかと思います。

JavaScriptでは、括弧の開きと閉じはかならずペアで存在します。ですから、ソースコードを書く時は先頭から順に打つのではなく、function hoge(){}というふうに先に閉じ括弧まで書いてしまってから中身を書く、そして括弧の階層ごとにインデント（行の左端のTabやスペース）を揃える、ということを意識すると、括弧の対応が崩れにくくなるかと思います。

Atomには、対応する括弧を表示する機能があります。開き括弧または閉じ括弧にカーソルを合わせると、Atomが対応すると判断した括弧に下線を表示してくれますから、それらも意識しながらコーディングをすると良いかもしれません。

ご参考になれば幸いです。

投稿2016/05/30 05:05

kxphotographer

総合スコア37

最後に「}」が足りてないように見えますが・・・

javascript
1var http = require('http');
2var fs = require('fs');
3
4var server = http.createServer();
5server.on('request', doRequest);
6server.listen(1234);
7console.log('Server running!');
8
9// リクエストの処理
10function doRequest(req, res) {
11    fs.readFile('./chat.html', 'UTF-8', function(err, data) {
12        res.writeHead(200, {'Content-Type': 'text/html'});
13        res.write(data);
14        res.end();
15    });
16
17    io.socket.on('connection' , function(socket){
18        socket.emit('greeting' , {message: 'hello'}, function (data){
19            consle.log('result;' + data);
20        });
21    });
22} // ここに「}」が足りてない

投稿2016/05/30 05:05

orange0190

総合スコア1698

関係ないかもしれませんが。

JavaScript
1// consle.log('result;' + data);
2// ↓
3console.log('result;' + data);

投稿2016/05/30 05:02

kei344

総合スコア69398

25行目の下に中括弧を閉じ忘れています。

初心者のうちはAtomよりお金払ってでもWebStormなどのIDEを使ったほうがいいと個人的には思います。
括弧を自動補完してくれたり、Syntaxエラーになるところに赤線が引かれたりするので便利です。
(Atomでもプラグインを入れれば結構いい環境作れると思いますが、そのプラグインを入れる作業が敷居が高いと思います)
https://www.jetbrains.com/webstorm/

投稿2016/05/30 05:00