txtファイルの特定の行だけ削除したい

Question

やりたいこと

JavaScriptで、main.txtを参照して指定した行（今回はテステスが書いてある1行目）のみを削除させたいです。
main.txtの中身は、

寿司食べたい
テステス
ワーワー
おなかすいたー
ヒューヒュー
...

このように適当な日本語の羅列があるだけです。

###書いてみたコード

    const fs = require("fs");
    fs.readFile("main.txt", "utf8", function(err, data) {
      const ward = ("テステス");//指定するワード
      const lines = data.split("\n");//行で分ける
      
      const index = lines.findIndex(line => ~line.indexOf(ward)); //そのワードがある行（１）
      console.log(index + "行目にある");//(1行目にある)
      
      const linesderlete = data//以下、いろんなとこから漁ったものなので正直解りきってない
        .split("\n")
        .slice(index)
        .join("\n");
      fs.writeFile("main.txt", linesExceptFirst, "utf8", function(err) {
        if (err) return console.log(err);
      });
    });

###起きている問題

1. 最初はワードがある行を教えてくれるのだが、もう一度やると0と言われてしまいます。本当に行を削除したはずなら見つからずに-1をいうはずです

2. エラーメッセージは出ていません。

3. 動作環境的に、txtへの編集がすぐに反映されないので、何がその時起こっているのかを目視する方法がありません。

###動作環境
glitchというサイトを使って動かしています。
そのため、上記に記した通りmain.txtに書き込んだ内容がすぐに反映されず難航しています。

###追記：解決方法
2KOHさん、AkitoshiManabeさん、そしてshinji709さん、ありがとうございました！

fs.readFile("main.txt", "utf8", function(err, data) {
 var ward = "テステス";
 const lines = data.split("\n");

 console.log(lines); //元の文章

 function filterWards(arr, query) {
   return arr.filter(function(el) {
     return el.indexOf(query) == -1;
   });
 }

 const henkou = filterWards(lines, ward); //目指す文章
 console.log(henkou);//"テステス"が消え、かつ改行されている

 fs.writeFile("main.txt", henkou.join("\n"), function(err) {});
});

おそらくこれは最適解かと言われればそうではないと思いますが、手助けをしてもらい最終的に自分で書けたので個人的には満足です。重ね重ねありがとうございましたっ

Answer 1

私も環境整えたほうがいいと思いますけど、ロジックの間違いなのでさしあたっては以下のようにブラウザのコンソールで確認できる形にしてデバッグしてはどうですか？

var text =
`寿司食べたい
テステス
ワーワー
おなかすいたー
ヒューヒュー
`;
var word = "テステス";
var lines = text.split("\n");
var index = lines.findIndex(line => ~line.indexOf(word));
console.log(index + "行目にある");
var linesRemained = text
  .split("\n")
  .slice(index)
  .join("\n");
text = linesRemained;
console.log(text);

コンソール出力:

1行目にある
テステス
ワーワー
おなかすいたー
ヒューヒュー

テステスじゃなくて寿司食べたいが消えてますね。テステスは0行目に残ってますから-1じゃなくて0が返ってくるんですね。

原因は2KOHさん回答の通り。sliceでどうにかしようとしないでfilter使うといいと思いますよ。

Comments

[siranui]

解決できました、ありがとうございました！

Answer 2

指定した行の削除を行っているのが

//以下、いろんなとこから漁ったものなので正直解りきってない

の部分のようですが、指定した行の削除が正しく行えていません。
slice について調べてみてください。

それと、もっとまともな動作環境を整えたほうがいいです。
Node.js をインストールするだけでいいと思うのですが、できない理由でもあるのでしょうか。

Comments

[siranui]

解決できました、ありがとうございました！

Answer 3

特定の行だけ削除したい

「特定の行だけ削除」を「特定の行以外を残す」という視点に変えてみては？

回答は出ていますので、割愛します。

---

main.txtに書き込んだ内容がすぐに反映されず難航しています

すぐに反映する目的で 上書き保存するようなファイル操作 を書く場合でも、意図する更新処理ができるまでは 保存処理(fs.writeFile()) をコメントアウトする 事を視野に入れる。「上書き保存＝元のファイルは失われる」という事実を軽視すると、意図しないファイルが出来上がってしまう度に ソースファイルの準備作業を繰り返す 事になるため。

以下、いろんなとこから漁ったものなので正直解りきってない

メソッドチェーン（メソッドを繋ぐコード）は、すべてのメソッドについて、２点に着目する。

1. どんなオブジェクトのメソッドか
2. 返り値はどんなオブジェクトになるのか

const linesderlete = data.split("\n")
        .slice(index)
        .join("\n");　// メソッドチェーン

// 分解する
var lines = data.split("\n"); // 文字列 -> 配列
var sliced = lines.slice(index); // 配列 -> 新しい配列
var linesderlete = sliced.join("\n"); // 配列 -> 文字列

Comments

[siranui]

解決できました、ありがとうございました！