1

これがJSフィドルです。スクリプトはそれ自体を物語っています。私はそれが機能しないことを指摘したいだけです。見て、違うことを教えてください。前もって感謝します。私はjavascriptプログラミングのすべてのルールであると信じていることに従いましたが、どこかで何かを見落としていたに違いありません。また、PHPのスクリプトで実際に動作するバージョンを作成しました。動作するPHPは、この投稿の2番目のスクリプトです。PHPは最後の数字で文字列を分割し、余分な文字列を挿入して、新しい文字列をマージします

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か所で特定しようとします。

  1. 誤った時間計算

    1日は86400秒、1年は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は、科学的記数法と一致しない場合は-1になります。考慮される唯一の整数値falseは0です。したがって、-1の値は常に真と見なされ、科学的記数法の形式であるかどうかに関係なく、年、日、および時間を常に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. 時間配列全体を結合してから、配列の最後の項目を再度タックします

    これを解決させてあげましょう。最後の項目を複製したくはありません。また、時間がカテゴリの1つに均等に分割される場合も処理する必要があります。

これらは、スクリプトの技術的な問題です。別の完全な答えは、あなたがやろうとしていることを達成するためのよりエレガントな方法で書くことができます。

于 2013-02-02T20:17:20.297 に答える
1

すでに与えられた答えに追加するために、コードの主なスタイルの問題は、ループや関数を使用して取り除くことができる不必要な繰り返しがたくさんあることです。

そうは言っても、計算は大幅に簡略化でき、「E」または「。」を検索するすべてのものは必要ないようです。この場合、ループを使用する価値はないでしょう。以下の提案では、ラベルを追加するためだけにループを使用しています。もう1つのヒントは、コードを読みやすくするために、可能な場合は説明的な変数名( hoursminutesの代わりに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の文字列のメソッドです。

これを修正する1つの方法は、次のように数値を文字列に変換することです。

a = a + "";

これにより、indexOf()エラーが削除されます。

残りのエラーについては、関数も誤用していると思います。たとえば、JavaScriptには存在しないように見えるcount()メソッドを使用します。

于 2013-02-02T19:03:04.910 に答える