JSでfsを使い、辞書の内容を追加する方法について

Question

前提・実現したいこと

配列の中にオブジェクトを追加していくテキストファイルを実装したいと思っています。
具体例は

[
 {
   "private key": private_key,
   "public key" : public_key
  }
 {
   "private key": private_key,
   "public key" : public_key
  }
  :
]

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

しかし、実際のテキストファイルの出力は

{"private key":4jt4q..... ,"public key":54321f4ddg.....}

となり、ファイルを書きこむと辞書の内容が更新されてしまい、別の辞書を追加することが出来ていません。

該当のソースコード

const bitcoinjs = require("bitcoinjs-lib")
const crypto = require("crypto")
const base58Check = require("bs58check")
const fs = require("fs")

let random1 = crypto.randomBytes(32)
let ECPair = bitcoinjs.ECPair.fromPrivateKey(random1,{compressed:false})

let private_key = ECPair.privateKey.toString("hex")
let public_key = ECPair.publicKey.toString("hex")

function Key_file(private_key,public_key) {
    const key_list = {
        "Private Key": private_key,
        "Public Key": public_key,
    }

    const jsonString = JSON.stringify(key_list)
    const jsonBuffer = Buffer.from(jsonString,"utf8")
    fs.writeFileSync("key.txt",jsonBuffer)
    
    
}

Key_file(private_key,public_key)

試したこと

fs.writeFileSync("key.txt",jsonBuffer)の後ろに辞書を再び作ったり、fs.readFileSyncを追加したりしました。

よろしくお願い致します。

Answer 1

ファイルを書きこむと辞書の内容が更新されてしまい、別の辞書を追加することが出来ていません。

常に上書き保存するコードだからです。
ファイル操作の基本は「読み、書き、保存」です。

// NodeJS

const fs = require("fs")
const file = "key.txt";

function loadJson( filepath ) {
  let jsonText;
  if( fs.statSync( filepath ).isFile() ) {
    jsonText = fs.readFileSync( filepath, "utf8" );
  }
  return jsonText ? JSON.parse(jsonText) : "";
}

function saveJson( filepath, json ) {
  let jsonText = JSON.stringify(json);
  fs.writeFileSync( filepath, jsonText, "utf8" );
}

function Key_file(private_key,public_key) {
  // 読み：ファイルを読んで内容を取得
  let ary = loadJson( file )||[];

  // 書き：内容を更新
  ary.push({ /* omitted */ });

  // 保存：更新内容を上書き保存
  saveJson( file, ary );
}

追記）上記コードは NodeJS の以下の 同期関数で表現してみました。

• fs.statSync()
• fs.readFileSync()
• fs.writeFileSync()

なお、パフォーマンスを求めるなら、更に考える必要があります。

• 他のロジックを考える
• ファイルの代わりに DB にする。
• RAMドライブを導入し高速化＆定期バックアップする。

（揮発性のメインメモリの一部をドライブ化するとデータ喪失しやすい）

• etc

Comments

[ryoooos]

丁寧にコードまで書いていただき、ありがとうございます！
ブロックチェーンに興味があり、Node.jsをやり始めましたが、やはり体系的に学ばなければダメですね。
ファイルの使い方も含め、Node.jsの基礎本を読み、勉強していきます。
また、機会があればよろしくお願いします！

Answer 2

fs.readFileを用いてファイルを取得し
その取得したデーターに新しいデーターを追加(push)してから保存するとできると思います

Comments

[ryoooos]

ご回答ありがとうございます
pushを使用して修正しました。

配列(key_list)を定義しておかないと、undifinedとなり、エラーになりますが、
```
fs.writeFileSync("key.txt",key_list)

key_list.push({
"private" : private_key,
"public": public_key
})
fs.readFileSync("key.txt",key_list)

```
事前に配列を定義しておくと、配列が空の状態がファイルに書き込まれて、pushが機能しません。
```
key_list=[]
fs.writeFileSync("key.txt",key_list)

key_list.push({
"private" : private_key,
"public": public_key
})
fs.readFileSync("key.txt",key_list)
```

この点はどう対処すればよろしいでしょうか？

[mouse_484]

配列であればpushですが単純に前の文字列+新しいをファイルに保存で良いとおもいます(txtファイルなようなので）

[ryoooos]

ありがとうございます。

文字列がテキストファイルに保存されません。
以下のコードの理解に誤りがあれば教えてください。

```
fs.writeFileSync("key.txt")
ここでテキストファイルを作成
const key_list = []
空の配列を作成
key_list.push({
"private" : private_key,
"public": public_key
})
配列に辞書を追加
fs.readFileSync("key.txt",key_list)
key_listが追加された状態でファイルの保存
```

[mouse_484]

readは読む方ですよ？
writeが書き込む方で逆です

[ryoooos]

ありがとうございます。
writeとreadを入れ換えて、出力すると、[object object]となりました。
配列をバイナリデータに変換する必要があると思い、以下のように変更しましたが、出力は変わりませんでした。何が原因でしょうか。

```
fs.readFileSync("key.txt")
const key_list = []
Buffer.from(JSON.stringify(key_list.push({
"private" : private_key,
"public": public_key
})))
fs.writeFileSync("key.txt",key_list)

```

[mouse_484]

そのコードから先程まで言っていた事が何も反映されていないのですが...?

またjson形式で保存するならkey.jsonなどにして
fsでjsonを扱う方法を調べて実装してください

[ryoooos]

もう少しfsについて調べてみようと思います。
この度は、ありがとうございました。