Question

这是JS Fiddle。该剧本不言而喻。我只想指出它不起作用。请看一看，告诉我该做些什么。提前致谢。我已经遵循了我认为是javascript编程中的每个规则，但在某种程度上我肯定忽略了一些东西。我还制作了一个实际工作版本og脚本我PHP。工作的PHP是这篇文章中的第二个脚本：PHP split string at last number, insert an extra string and merge the new string。

function calTime(x) {

if (x === '') {
    x = 54098;
} // Time in seconds
var f = 31536000, // seconds in a year
    d = 86400, // seconds in a day
    h = 3600, // seconds in an hour
    m = 60, // seconds in a minute
    xa,
    xb,
    xc,
    xe,
    xq,
    string,
    lb_y = 'year',
    lb_ys = 'years',
    lb_d = 'day',
    lb_ds = 'days',
    lb_h = 'hour',
    lb_hs = 'hours',
    lb_m = 'minute',
    lb_ms = 'minutes',
    lb_s = 'second',
    lb_ss = 'seconds',
    lb_and = 'and';

// a = years
var a = x / f;

// To prevent complications using scientific numbers less than 0 ex 7.2341232E-23
var a1 = a.indexOf("E-");
if (a1) {
    a = 0;
}

// Split a so we only get the numbers before '.'
var a2 = a.indexOf(".");
if (a2) {
    Math.floor(a);
}

// if $a is less or equal to 0 - it is 0
if (a <= 0) {
    a = 0;
}

// b = days
var b = (x - (f * a)) / d;

// To prevent complications using scientific numbers less than 0 ex 7.2341232E-23
var b1 = b.indexOf("E-");
if (b1) {
    b = 0;
}

// Split b so we only get the numbers before '.'
var b2 = b.indexOf(".");
if (b2) {
    Math.floor(b);
}

// if $b is less or equal to 0 - it is 0
if (b <= 0) {
    b = 0;
}

// c = hours
var c = (x - (f * a) - (d * b)) / h;

// To prevent complications using scientific numbers less than 0 ex 7.2341232E-23
var c1 = c.indexOf("E-");
if (c1) {
    c = 0;
}

// Split c so we only get the numbers before '.'
var c2 = c.indexOf(".");
if (c2) {
    Math.floor(c);
}

// if $c is less or equal to 0 - it is 0
if (c <= 0) {
    c = 0;
}

// e = minutes
var e = (x - (f * a) - (d * b) - (h * c)) / m;

// Split $e so we only get the numbers before '.'
var e2 = e.indexOf(".");
if (e2) {
    Math.floor(e);
}

// if $e is less or equal to 0 - it is 0
if (e <= 0) {
    e = 0;
}

// $q = seconds
var q = (x - (f * a) - (d * b) - (h * c) - (m * e));

// Rewrite numbers if below 9
if (a <= 9) {
    xa = '0' + a;
} else {
    xa = a;
}
if (b <= 9) {
    xb = '0' + b;
} else {
    xb = b;
}
if (c <= 9) {
    xc = '0' + c;
} else {
    xc = c;
}
if (e <= 9) {
    xe = '0' + e;
} else {
    xe = e;
}
if (q <= 9) {
    xq = '0' + q;
} else {
    xq = q;
}

// Rewrite labels
if (a <= 1) {
    lb_ys = lb_y;
}
if (b <= 1) {
    lb_ds = lb_d;
}
if (c <= 1) {
    lb_hs = lb_h;
}
if (e <= 1) {
    lb_ms = lb_m;
}
if (q <= 1) {
    lb_ss = lb_s;
}

// if == 0 - do not show
if (a === 0) {
    a = '';
} else {
    a = a + ' ' + lb_ys;
}
if (b === 0) {
    b = '';
} else {
    b = b + ' ' + lb_ds;
}
if (c === 0) {
    c = '';
} else {
    c = c + ' ' + lb_hs;
}
if (e === 0) {
    e = '';
} else {
    e = e + ' ' + lb_ms;
}
if (q === 0) {
    q = '';
} else {
    q = q + ' ' + lb_ss;
}


var time = [a, b, c, e, q];

time = time.filter(Number);

var count = time.count();
var last = time[time.length - 1];

if (count == 1) {
    string = last;
} else if (count === 0) {
    string = '<i>No Time described</i>';
} else {
    string = time.join(', ') + ' ' + lb_and + ' ' + last;
}

return string;
}

document.getElementById("demo").innerHTML = calTime(83200);

Answer 1

我会尝试在一个地方识别脚本在技术上的错误。

时间计算不正确

一天有86400秒，365天有31536000秒。如果他们不想担心价值观，你通常会看到人们做这样的事情：
```
var minutes = 60;
var hours = 60 * 60;
var days = 24 * 60 * 60;
var years = 365 * 24 * 60 * 60;
```
在不支持该方法的对象（在本例中为数字）上使用indexOf()。

其他人已经在评论和答案中指出了这一点，但基本上，如果您打算在其上调用字符串方法，请将您的数字转换为字符串：
```
num = num + "";
numIndex = (num + "").indexOf("foo");
```
未正确检查indexOf()
的返回值
indexOf()返回字符串开始位置的索引（从0开始）。如果找不到该字符串，则返回-1。在几个地方，您正在做类似的事情：
```
var a2 = a.indexOf("E-");
if (a2) {
  a = 0;
}
```
在这种情况下，如果它与科学记数法不匹配，则
a2将为-1。唯一被视为false的整数值为0.因此，-1值始终被视为true，并且您始终将年，日和小时设置为0，无论它们是否采用科学记数法格式。
不考虑科学记谱法格式的案例

在我的浏览器中，非常小，接近于零的值看起来像这样：
```
7.888609052210118e-31
```
您的搜索与此值不匹配。考虑到你的逻辑，这甚至可能都不重要。有没有理由不总是使用Math.floor()？你的JS浮点问题无论如何都会成为问题。
未使用Math.floor（）的返回值

在多个地方，您可以执行以下操作：
```
Math.floor(a);
```
然后你继续假设a已经承担了它的地板价值。您需要执行以下操作才能实现此目的：
```
a = Math.floor(a);
```
将所有时间组件设置为字符串，然后按Number
过滤它们
您在时间数组中明确存储字符串格式（例如23小时，6分钟），然后按Number过滤时间数组。我认为你试图过滤掉空白字符串，这就是你将时间值设置为0时的函数。将函数传递给filter()以过滤掉那些空白条目，如下所示：
```
time.filter(function(x) { return x === "" ? false : true; });
```
count()不是Array 的方法

您可能正在寻找length，您实际上正确地使用了{{1>}。
您将整个时间阵列连接在一起，然后再次处理数组中的最后一项

我会让你解决这个问题。您不想复制最后一项，并且您可能还想处理时间平均分为您的某个类别的情况

这些是您脚本的技术问题。另一个完整的答案可以写在更优雅的方式来完成你想要做的事情。

Answer 2

要添加已经给出的答案，代码的主要样式问题是存在大量不必要的重复，您可以摆脱使用循环和/或函数。

话虽如此，数学可以大大简化，并且似乎不需要搜索“E”或“。”的所有东西，在这种情况下，它可能不值得使用循环;在下面的建议中，我只使用循环来添加标签。另一个提示是尽可能使用描述性变量名称（hours，minutes，而不是a，b），以使您的代码更具可读性。

http://jsfiddle.net/m54Du/16/

function calTime(seconds) {
    if (seconds === '') {
        seconds = 54098;
    } // Time in seconds
    seconds = Math.floor(seconds);
    if (isNaN(seconds) || seconds <= 0) {
        return '<i>No time described</i>';
    }
    var minutes = Math.floor(seconds / 60), 
        hours = Math.floor(minutes / 60),
        days = Math.floor(hours / 24), 
        years = Math.floor(days / 365), // assuming not leap!
        timeData = [years, days % 365, hours % 24, minutes % 60, seconds % 60],
        pluralLabels = ['years', 'days', 'hours', 'minutes' , 'seconds'],
        singularLabels = ['year', 'day', 'hour', 'minute', 'second'],
        time = [];
    for (var i = 0; i < timeData.length; i++) {
        if (timeData[i] > 1) {
            time.push(timeData[i] + ' ' + pluralLabels[i]);
        }
        else if (timeData[i] > 0) {
            time.push(timeData[i] + ' ' + singularLabels[i]);
        }
    }
    var last = time.pop();
    return time.length ? time.join(', ') + ' and ' + last : last;
}
document.getElementById("demo").innerHTML = calTime(83200);

Here's an alternative更多地使用循环来进行数学运算。

Answer 3

您的变量a，b等是数字。 indexOf是JavaScript中字符串的方法。

解决此问题的一种方法是将您的数字转换为字符串，例如：

a = a + "";

删除indexOf（）错误。

对于其他错误，我认为你也在滥用功能。例如，您使用count（）方法，该方法似乎不存在于JavaScript中。

Javascript计算功能不起作用

3 个答案: