基于外部JSON的Node.js化学元素状态展示功能优化方案问询

问题背景 #

我开发的Node.js应用需根据输入值展示各类化学元素的状态，相关元素信息存储于外部生成的不可修改JSON文件中，文件结构如下：

{
 "1":{
   "Symbol":"H",
   "MeltingPoint": 13.81,
   "BoilingPoint": 20.28,
    ...
  }
}

用户在输入框中输入目标温度后，应用需遍历该JSON文件，将元素与输入值对比后，把对应状态值存入数组并发送至电子显示单元；针对密度等其他属性，将采用逻辑相似的功能。

目前已实现核心功能代码如下：

const assumedInputTemperature = 20.0;
const display = new Array(119);
const color_solid = 1;
const color_liquid = 2;
const color_gas = 3;
const color_undefinded = 4;

for (const key in elements){
    const value = elements[key];
    const meltingPoint = value.MeltingPoint;
    const boilingPoint = value.BoilingPoint1;
    const number = parseInt(key)

    console.log(`${number}: ${meltingPoint} : ${boilingPoint}`)

    if(meltingPoint == "" && boilingPoint == ""){
        console.log("Unknown!!?")
        display[number] = color_undefinded;
    } else {
        if (meltingPoint > assumedInputTemperature) {
            display[number] = color_solid;
        } else if (meltingPoint < assumedInputTemperature && boilingPoint > assumedInputTemperature) {
            display[number] = color_liquid;
        } else if (meltingPoint < assumedInputTemperature && boilingPoint < assumedInputTemperature) {
            display[number] = color_gas;
        } else {
            console.log("ERROR!");
        }
    }
    console.log(display[number]);
}

该功能已可正常运行，但希望采用更优的实现方式，以下是具体改进建议：

改进建议 #

1. 用语义化常量替代魔法数字 #

把代表状态的数字定义成语义化对象，提升代码可读性和维护性，避免后续修改状态值时需要逐一查找：

const ElementState = {
  SOLID: 1,
  LIQUID: 2,
  GAS: 3,
  UNDEFINED: 4
};
// 后续使用直接调用ElementState.SOLID，无需记忆数字对应含义

2. 优化遍历方式与类型转换 #

使用Object.entries()遍历元素对象，避免手动解析键名；同时对熔点、沸点做类型转换，处理JSON中空字符串的情况，避免类型比较错误：

// 遍历优化
for (const [elementNumStr, element] of Object.entries(elements)) {
  const elementNum = Number(elementNumStr);
  // 空字符串转为undefined，统一处理缺失数据的情况
  const meltingPoint = element.MeltingPoint === "" ? undefined : Number(element.MeltingPoint);
  const boilingPoint = element.BoilingPoint1 === "" ? undefined : Number(element.BoilingPoint1);
  // ...后续逻辑
}

3. 完善边界条件处理 #

原代码未覆盖温度等于熔点/沸点、仅单个属性有值的情况，补充后逻辑更严谨：

// 先判断是否有有效数据
if (meltingPoint === undefined && boilingPoint === undefined) {
  display[elementNum] = ElementState.UNDEFINED;
} else if (meltingPoint === undefined || boilingPoint === undefined) {
  // 仅一个属性有值，无法准确判断状态，标记为未知
  display[elementNum] = ElementState.UNDEFINED;
} else {
  // 覆盖等于的情况，比如温度等于熔点时视为固体
  if (assumedInputTemperature <= meltingPoint) {
    display[elementNum] = ElementState.SOLID;
  } else if (assumedInputTemperature >= boilingPoint) {
    display[elementNum] = ElementState.GAS;
  } else {
    display[elementNum] = ElementState.LIQUID;
  }
}

4. 提取状态判断逻辑为独立函数 #

把状态判断逻辑抽成单独函数，提升代码模块化程度，方便后续复用或扩展密度等其他属性的判断逻辑：

function getElementState(meltingPoint, boilingPoint, inputTemp) {
  if (meltingPoint === undefined && boilingPoint === undefined) {
    return ElementState.UNDEFINED;
  }
  if (meltingPoint === undefined || boilingPoint === undefined) {
    return ElementState.UNDEFINED;
  }
  if (inputTemp <= meltingPoint) return ElementState.SOLID;
  if (inputTemp >= boilingPoint) return ElementState.GAS;
  return ElementState.LIQUID;
}

// 遍历中直接调用函数
display[elementNum] = getElementState(meltingPoint, boilingPoint, assumedInputTemperature);

5. 优化数组初始化 #

默认初始化display数组为未知状态，避免因遍历遗漏导致的空值问题：

// 初始化长度119的数组，所有元素默认设为未知状态
const display = Array.from({ length: 119 }, () => ElementState.UNDEFINED);

6. 移除冗余日志或添加调试开关 #

生产环境中频繁的console.log会影响性能，可直接移除或通过调试开关控制日志输出：

const isDebug = false; // 生产环境设为false，开发环境设为true
if (isDebug) {
  console.log(`${elementNum}: ${meltingPoint} : ${boilingPoint}`);
  console.log(display[elementNum]);
}

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