为什么这个 if 语句不受我的输入影响？

Question

我想构建一个将 AM/PM 转换为 24 小时格式的算法。它还没有完成，但我到目前为止的代码表现得很奇怪。

当我输入"25:05:45PM" 时，它应该进入第一个if 语句的第一个分支，但应该不是输入第二个if 语句。我检查了条件，肯定是false。我的大脑正在融化。

这是代码：

function conversionTime(s) {
  if (s.includes('PM')) {
    let temp = s.slice(0, 8).split(':');
    if (temp[0] >= 01 && temp[0] <= 12); {
      temp[0] = Number(temp[0]) + 12;
      return temp.join(':')
    }
  } else if (s.includes('AM')) {
    let temp2 = s.slice(0, 8).split(':');
    return temp2
  }
}
console.log(conversionTime("25:05:45PM"))

Answer 1

Gotcha.

if (temp[0] >= 01 && temp[0] <= 12);

这个分号是罪魁祸首！它的意思是“if 语句已经结束，不需要做任何事情”，所以你的代码被解释为：

if (temp[0] >= 01 && temp[0] <= 12);

{
  temp[0] = Number(temp[0]) + 12;
  return temp.join(':');
}

块中的代码将始终运行。存在此功能，因此您可以充分利用 let 的作用域：

let x = "outside";
console.log(x);
{
  let x = "inside";
  console.log(x);
}
console.log(x);

出色地，真的它的存在是因为 C 语言就是这样工作的——它早于let 语句——但这就是它对这些日子有用的东西。

Answer 2

我会这样做..

function conversionTime(timeAPM)
  {
  let [h,m,s,apm] = timeAPM.match(/(d+)|(w+)/g);  

  if (timeAPM !== `${h}:${m}:${s}${apm}`
   || isNaN(h) || isNaN(m) || isNaN(s) 
   || (apm !== 'AM' && apm !== 'PM')    
   ) return undefined;

  if (apm === 'PM' && h <= 12 ) h = +h +12;
  return `${h}:${m}:${s}`;
  }

console.log(conversionTime("25:05:45PM"))

我不知道这是否是他故意的，但 PO 的代码也返回 undefined。除了他对AM/PM 标签的依附之外，OP 没有对条目的有效性做出解释。

所以第二个版本内容是检查确实有3 数值，用: 分隔，然后直接跟AM 或PM。

这些数字的位数未知：他的测试使用一个小时 = 25，而它是一个 AM/PM 演示文稿。那么为什么不用40 数字表示秒或分钟...

我并不是说我的正则表达式是宇宙中最好的。

我也可以在我的代码中添加：

console.log(conversionTime("10:31:25BAM")) // -> undefined
console.log(conversionTime("1:3:5abc"))    // -> undefined
console.log(conversionTime("1:3zzzz"))     // -> undefined
console.log(conversionTime("a:10:15PM"))   // -> undefined
console.log(conversionTime("285::4875PM")) // -> undefined
// or more absurd:
console.log(conversionTime("285:1505:4875PM")) // -> 285:1505:4875