node.jsとjsonファイルとの連携がうまくいきません

Question

独学で勉強しています。小規模システムをイメージして、チャットアプリを作ろうとしています。メッセージ等をDBに保存したいのですが、mongoDB等の前にjsonファイルへの書き込み、呼び出しをしようと試行しています。

NewChat
|_node_modulesフォルダ
|_chat.js
|_index.html
|_package-lock.json
|_database.json
チャット機能自体は正常に動いています。そこでユーザーが入室時にユーザーネームを入力するので、まずはその名前をdatabase.jsonにpushしようと考えました。以下はテンプレートだけ作成したdatabase.json

	{
	"id":"あんじ",	
	},

正常に動いていたファイルに追記したのは、上部の
var fs = require("fs");
var database = fs.readFileSync("database.json" , "utf-8");
に加え、「ここから追記　ここまで追記」の箇所になります。

var app = require('express')();//expressを使うため
var http = require('http').Server(app);//expressを使って通信を扱う
var io = require('socket.io')(http);//socketを使うため
var POST = 1234;//localhost:3000
var fs = require("fs");
var database = fs.readFileSync("database.json" , "utf-8");


//ルートディレクトリにアクセスした時に動く処理
app.get('/', function(req, res) {
  //index.htmlにリダイレクトする
  res.sendFile(__dirname + '/index.html');
});

//socket.ioに接続された時に動く処理
io.on('connection', function(socket) {


	socket.on('setUserName', function (userName) {
		if(!userName) userName = '匿名';
	 
		socket.userName = userName;
		io.emit('message', socket.userName  + 'さんが入室しました！', 'System');
	});
  console.log('入室したユーザー : %s', socket.userName);
　　//ここから追記
  database = JSON.parse(database);
  database.push({id:"socket.userName"});
  fs.writeFileSync(database.json,JSON.stringify(database));
  //ここまで追記
  
  //接続時に振られた一意のIDをコンソールに表示
  console.log('入室したID : %s', socket.id);
  //接続時に全員にIDを表示（messageというイベントでクライアント側とやりとりする）
  
  
  //messageイベントで動く
  //全員に取得したメッセージとIDを表示
  socket.on('message', function(msj) {
      io.emit('message', msj, socket.userName);
  });

  //接続が切れた時に動く
  //接続が切れたIDを全員に表示
  socket.on('disconnect', function(e) {
    console.log('接続が切れたID : %s', socket.userName);
  });
});

//接続待ち状態になる
http.listen(POST, function() {
  console.log('接続開始', POST);
});

ユーザーに名前を問うウィンドウが出されるタイミングでコンソールに以下が表示されます。

接続開始 1234
入室したユーザー : undefined
undefined:4
        },
        ^

SyntaxError: Unexpected token } in JSON at position 22
    at JSON.parse (<anonymous>)
    at Namespace.<anonymous> (C:\Users\annji\Desktop\NewChat\chat.js:27:19)
    at Namespace.emit (events.js:189:13)
    at Namespace.emit (C:\Users\annji\Desktop\NewChat\node_modules\socket.io\lib\namespace.js:213:10)
    at C:\Users\annji\Desktop\NewChat\node_modules\socket.io\lib\namespace.js:181:14
    at process._tickCallback (internal/process/next_tick.js:61:11)

参考になせていただいたのは
https://ex-script.com/?p=377
こちらで、｝系の綴り間違いかと何度か確かめましたが該当しないように思われます。

どなたか間違いをご教示いただけると助かります。

---

お答えいただきました方、ご回答ありがとうございます。
投稿直後に私も思いまして、コンマを外しましたが違うエラーが出てしまいました。

	{
	"id":"りんご"
	}

接続開始 1234
入室したユーザー : undefined
C:\Users\annji\Desktop\NewChat\chat.js:28
  database.push({id:"userName"});
           ^

TypeError: database.push is not a function
    at Namespace.<anonymous> (C:\Users\annji\Desktop\NewChat\chat.js:28:12)
    at Namespace.emit (events.js:189:13)
    at Namespace.emit (C:\Users\annji\Desktop\NewChat\node_modules\socket.io\lib\namespace.js:213:10)
    at C:\Users\annji\Desktop\NewChat\node_modules\socket.io\lib\namespace.js:181:14
    at process._tickCallback (internal/process/next_tick.js:61:11)
コード

---

何度もすみません。
fs.writeFileSync('database.json',JSON.stringify(database));
ファイルのコーテーションも試しましたが同様でした。

Answer 1

TypeError: database.push is not a function

database.jsonは配列ではなく普通のオブジェクトです。よって変数databaseも普通のオブジェクトなので、pushメソッドはありません。

database.push({id:"socket.userName"});

とありますが、これはどういう意図でしょうか？databaseに"id":"りんご"というプロパティと、"id":"socket.userName"というプロパティが両方存在してほしいということでしょうか？

また、予想ですが、"socket.userName"という文字列を指定したいわけではなく、ユーザー名を指定したいと思うので、{id:"socket.userName"}ではなく{id: socket.userName}だと思います。

Comments

[Izumo1101]

ご回答ありがとうございます。
オブジェクトからデータを取り出し、ここにプッシュする。
{
"items" : [
{"id":"りんご"},
　　{"id":"みかん"}
]
}
ここにユーザーネームをpushする。

var obj = JSON.parse(database);
var items = obj.items;
var i;
var item;
var id;
for (i in items) {
item = items[i];
id = item.id;
console.log(id);
}
items.push({id: socket.userName});
console.log(items);

確かにここでいうitemsに追記されました。
もしよければ・・・ですが、これをまたjsonに返す方法を(方向性だけでも）教えていただけると嬉しいです。

[karamarimo]

そのまま
fs.writeFileSync('database.json',JSON.stringify(database));
ではうまくいきませんでしたか？
修正: クォーテーション

[Izumo1101]

obj.items.push({id: socket.userName});
fs.writeFileSync('database.json', JSON.stringify(obj));

読み込みのテストのためにitemsという余計な見出しを付けちゃいまして。勉強のためにはそっちが良いかと思い試行していました。
解決できました。ありがとうございます。

Answer 2

JSON ファイルの記述が間違っています。

コンマ（,）があると、必ず何かしらのデータが続くものとして解釈するノーテーションの仕様があります。

database.json

{
    "id":"あんじ"
}

Comments

[Izumo1101]

ありがとうございます。
投稿直後に思い当って試行したのですが今度は違うエラーが出てしまいまいした。