如何通过fs.readFile存储变量？解析非标准JSON格式TXT文件问题

嘿，我看了你写的这个解析非标准JSON文件的函数，思路是对的——把每行的对象拼接成标准JSON数组格式，但这里有几个可以优化的点，能避免潜在问题，还能让函数更实用，我给你梳理下：

现有代码的潜在问题 #

1. 空行处理漏洞：如果文件末尾有空行，或者中间存在全空白的行，slice(0, -1)会错误地删掉最后一个有效对象的部分字符，导致JSON解析失败。
2. 异步逻辑不完整：当前函数仅打印处理后的字符串，调用方无法获取解析后的JSON对象；而且直接throw err在异步回调里可能会导致未捕获的异常。
3. 缺少JSON解析步骤：你只生成了JSON格式的字符串，但没有把它解析成可直接使用的JavaScript对象。

改进后的代码（现代异步/await风格） #

用fs.promises配合异步函数，代码更易读，错误处理也更清晰：

const fs = require('fs').promises;

module.exports = {
  bot_detection: async function(input_file_path) {
    try {
      // 读取文件内容
      const rawData = await fs.readFile(input_file_path, 'utf-8');
      // 分割行并过滤空行（包括全空白行）
      const validLines = rawData.split(/\r?\n/).filter(line => line.trim() !== '');
      // 拼接成标准JSON数组字符串
      const jsonString = `[${validLines.join(',')}]`;
      // 解析为JavaScript对象
      const parsedData = JSON.parse(jsonString);
      
      console.log('解析完成:', parsedData);
      return parsedData; // 返回结果给调用方
    } catch (error) {
      console.error('文件读取或解析失败:', error);
      throw error; // 抛出错误让调用方自行处理
    }
  }
};

如果你偏好回调风格的实现 #

如果项目还在使用传统回调模式，也可以这样写：

const fs = require('fs');

module.exports = {
  bot_detection: function(input_file_path, callback) {
    fs.readFile(input_file_path, 'utf-8', function(readErr, rawData){
      if (readErr) {
        return callback(readErr); // 将读取错误传递给回调
      }
      
      try {
        const validLines = rawData.split(/\r?\n/).filter(line => line.trim() !== '');
        const jsonString = `[${validLines.join(',')}]`;
        const parsedData = JSON.parse(jsonString);
        
        callback(null, parsedData); // 成功时传递解析结果
      } catch (parseErr) {
        callback(parseErr); // 将解析错误传递给回调
      }
    });
  }
};

额外说明 #

如果你的文件中每个JSON对象本身是标准格式（比如示例里的{"timestamp": 1487184625, "user": "Eric",...}），上面的方法完全适用。但如果对象内部包含换行（比如字符串里的换行），按行分割就会出问题——不过从你的描述来看，应该是每行一个完整对象，这个场景下没问题。

内容的提问来源于stack exchange，提问作者user7496931