1

这是一个JS Fiddle。剧本不言自明。我只想指出它不起作用。请看一下并告诉我该怎么做。提前致谢。我遵循了我认为是 javascript 编程中的每条规则，但不知何故，我一定忽略了一些东西。我还制作了一个实际的工作版本，即 PHP 脚本。工作中的 PHP 是本文中的第二个脚本：PHP split string at last number, insert a 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);
4

3 回答 3

4

我将尝试在一处找出脚本在技术上存在的所有问题。

  1. 时间计算不正确

    一天有 86400 秒，一年有 365 天有 31536000 秒。如果他们不想担心价值观，你通常会看到人们做这样的事情：

    var minutes = 60;
var hours = 60 * 60;
var days = 24 * 60 * 60;
var years = 365 * 24 * 60 * 60;

  2. 使用indexOf()不支持该方法的对象（在本例中为数字）。

    其他人在评论和答案中指出了这一点，但基本上，如果您要在它们上调用字符串方法，请将您的数字转换为字符串：

    num = num + "";
numIndex = (num + "").indexOf("foo");

  3. 没有正确检查返回值indexOf()

    indexOf()返回字符串开始位置的索引（从 0 开始）。如果未找到该字符串，则返回 -1。在多个位置，您正在执行以下操作：

    var a2 = a.indexOf("E-");
if (a2) {
  a = 0;
}

    在这种情况下，如果 a2 与科学记数法不匹配，则 a2 将为 -1。唯一考虑的整数值false是 0。因此，-1 值始终被认为是 true，并且您始终将年、日和小时设置为 0，无论它们是否采用科学记数法格式。

  4. 不考虑科学记数法格式的大小写

    在我的浏览器中，非常小，接近于零的值如下所示：

    7.888609052210118e-31

    您的搜索与此值不匹配。考虑到您的逻辑，这甚至可能无关紧要。有什么理由不总是使用Math.floor()吗？您的 JS 浮点问题无论以何种方式切片都将是一个问题。

  5. 不使用 Math.floor() 的返回值

    在多个位置，您可以执行以下操作：

    Math.floor(a);

    然后你继续假设它a已经假设了它的底值。您需要执行以下操作才能做到这一点：

    a = Math.floor(a);

  6. 将所有时间组件设置为字符串，然后通过Number

    您在时间数组中显式存储字符串格式（例如 23 小时 6 分钟），但随后您将时间数组过滤为Number. 我认为您正在尝试过滤掉空白字符串，这是您将时间值设置为 0 时的值。传递一个函数filter()来过滤掉这些空白条目，如下所示：

    time.filter(function(x) { return x === "" ? false : true; });

  7. count()不是一种方法Array

    您可能正在寻找length，您实际上在下面正确使用了它。

  8. 您将整个时间数组连接在一起，然后再次添加数组中的最后一项

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

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

于 2013-02-02T20:17:20.297 回答
1

为了补充已经给出的答案，您的代码的主要样式问题是有很多不必要的重复，您可以使用循环和/或函数来摆脱这些重复。

话虽如此，数学可以大大简化，似乎不需要所有搜索“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);

这是一种更多地使用循环来进行数学运算的替代方法。

于 2013-02-02T21:58:47.680 回答
0

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

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

a = a + "";

它消除了 indexOf() 错误。

对于其余的错误，我认为您也在滥用功能。例如，您使用 count() 方法，该方法在 JavaScript 中似乎不存在。

于 2013-02-02T19:03:04.910 回答