1

このコードスニペットのカスタマイズされたバージョンをjQueryベースのレンタル予約スクリプトに使用しています。このスクリプトに問題が見つかりました。問題を解決するために何時間も費やしましたが、修正できません。

最終レンタル日がシーズンの最終日でもある場合、問題が発生します。これが発生した場合、その日はカウントおよび計算されません。最初に私はそれを認識していません。なぜなら、最終日がシーズンの最終日より遅い場合、スクリプトは正常に機能するからです。

誰かが私の問題を理解するのを手伝ってくれるといいですね。

jsFiddleデモ

これがコードです。

var MILLI_PER_DAY = 86400000;
/**
 * identifier is a string that identify the season. like 'summer'
 * start and end must be string with date pattern: yyyy/MM/dd
 */
var Season = function(identifier, start, end) {
        this.id = identifier
        this.start = new Date(start);
        this.end = new Date(end);
    }
    /**
     * name is the product name
     * prices is an object that defines the price of each season.
     * e.g. {'summer' : 29.9, 'winter' : 35}
     */
var Product = function(name, prices) {
        this.name = name;
        this.prices = prices;
    }
var seasons = [
new Season('s1', '2012-01-01', '2012-02-28'),
new Season('s2', '2012-03-01', '2012-05-31')];
var products = [
new Product('single-room', {
    's1': 16,
    's2': 12
})];
/**
 * productName is the product name to be bought
 * dateStart and dateEnd is the range that productName will be used and
 * they should be a string representing a date with pattern: yyyy/MM/dd
 */
function calculatePrice(productName, dateStart, dateEnd) {
    var start = new Date(dateStart);
    var end = new Date(dateEnd);
    //finding product
    var product = null;
    for (var i = 0; i < products.length; i++) {
        var p = products[i]
        if (p.name == productName) {
            product = p;
            break;
        }
    }
    if (product != null) {
        var totalPrice = 0;
        for (var i = 0; i < seasons.length; i++) {
            var s = seasons[i]
            //if this range contains parts or all the season range
            if (start < s.end && end > s.start) {
                var seasonRange = Math.min(s.end, end) - Math.max(s.start, start);
                //due to the start day must count
                var seasonDays = 1 + (seasonRange / MILLI_PER_DAY);
                totalPrice += product.prices[s.id] * seasonDays;
            }
        }
        alert(product.name + " cost " + totalPrice + " in dates from " + dateStart + " to " + dateEnd);
    }
}
calculatePrice('single-room', '2012-02-08', '2012-03-10');
calculatePrice('single-room', '2012-03-05', '2012-05-10');
calculatePrice('single-room', '2012-01-05', '2012-02-10');

編集: 最初にあなたのクイック返信に感謝します:

Iamが問題を表示できる新しいフィドルを作成しました。返信が遅くなってすみませんが、料理をしていました;-)

最初の事実:シーズン1(s1)は2012/01/01に開始し、2012/07/26に終了しますシーズン2(s2)は2012/07/26に開始し、2012/12/31に終了します

シーズン1の料金は1泊1ユーロですシーズン1の料金は1泊2ユーロです

最初と最後の日は1としてカウントされます(したがって、最初の日はカウントされません)。

テストA:(終了日がシーズンの終了日よりも低い)要約:範囲(および価格)は正しく計算されます:

フィドルアラートを参照してください:日数の範囲:9 | カテゴリ:シングルルーム| コスト9| から:2012-07-17 | 宛先:2012-07-26

テストB:(終了日はシーズンの終了日と同じです)要約:最後の日付はカウントされません。範囲(および価格)は正しく計算されません:

フィドルアラートを参照してください:日数の範囲:9 | カテゴリ:シングルルーム| コスト9| から:2012-07-17 | 宛先:2012-07-27

テストC:(終了日がシーズンの終了日よりも高い)要約:最終日に等しい日はカウントされません。28日はRange(およびPrice)が正しく計算されません。

フィドルアラートを参照してください:日数の範囲:10 | カテゴリ:シングルルーム| コスト11| から:2012-07-17 | 宛先:2012-07-28

フィドル


くそー、今私は本当の問題を抱えています。Sarafiでは動作せず(Macバージョンをまだ試しました)、iPhoneでさえ価格を計算できません。FirefoxとChromeはOSXLionでうまく機能します。

価格が計算されていないため、スクリプトは次の行で停止しているようです。

 if ( product != null ) {
    var totalPrice = 0;
    for ( var i=0; i < seasons.length; i++ ) {
      var s = seasons[i]
      //if this range contains parts or all the season range
      if ( start < s.end && end > s.start ) {
        var seasonRange = Math.min(s.end,end) - Math.max(s.start,start);
        //due to the start day must count
        var seasonDays = 1 + (seasonRange/MILLI_PER_DAY);
        totalPrice += product.prices[s.id]*seasonDays;
      }
    }
    alert(product.name + " cost " + totalPrice + " in dates from " + dateStart + " to " + dateEnd);
  }
}

すでにsafarijavascriptエラーログを確認しましたが、問題は見つかりませんでした。

もう一度見てもらえますか?それは素晴らしいことだ ;-)

UPDATE 26.7.2012 この問題はSafari6.0(本日リリース)では発生しませんでした。

4

1 に答える 1

0

文字列の日付が誤って日付にキャストされていると思います。
すべての日付が 1 日早すぎます。
コンソールで実行する場合: new Date('2012-05-25'); 得られる値が表示されます。
これを次のように変更しました: new Date('2012/05/25'); 正しく動作しているように見えました。

calculatePrice('シングルルーム','2012/05/25','2012/05/30');

編集: これが私のコンソール出力です:
new Date('2012-03-10')
Fri Mar 09 2012 18:00:00 GMT-0600 (Central Standard Time)
new Date('2012/03/10')
Sat Mar 10 2012 00:00:00 GMT-0600 (中央標準時)

-編集-:
ロジックが間違っているようです。
提出する場合: calculatePrice('single-room', '2012-05-31', '2012-05-31'); 価格に対して 0 を取得します。
これは、開始がシーズン終了よりも遅くないためです。シーズン終了まで==です。まだ有効な時間です。
if ( start <= s.end && end > s.start ) {
したがって、次のようになります。
if ( start <= s.end && end > s.start ) {

于 2012-07-17T15:31:29.843 に答える