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');

编辑: 首先感谢您的快速回复:

我制作了一个新的小提琴,我可以在其中展示这个问题。抱歉我回复晚了,但我在做饭;-)

首先是事实:第 1 季 (s1) 从 2012/01/01 开始,到 2012/07/26 结束 第 2 季 (s2) 从 2012/07/26 开始,到 2012/12/31 结束

第 1 季每晚 1 欧元 第 1 季每晚 2 欧元

第一天和最后一天算作一天(所以第一天不算数)。

测试 A:(结束日期低于季节结束日期)总结:范围(和价格)将被正确计算:

请参阅小提琴警报:以天为单位的范围:9 | 类别:单人间 | 成本 9 | 来自: 2012-07-17 | 至:2012-07-26

测试 B:(结束日期与季节结束日期相同) 摘要:最后一个日期不计算在内。范围(和价格)将无法正确计算:

请参阅小提琴警报:以天为单位的范围:9 | 类别:单人间 | 成本 9 | 来自: 2012-07-17 | 至:2012-07-27

测试 C:(结束日期高于季节结束日期) 摘要:等于最后一天的日期不计算在内。第 28 位将是范围(和价格)将无法正确计算:

请参阅小提琴警报:以天为单位的范围:10 | 类别:单人间 | 成本 11 | 来自: 2012-07-17 | 至: 2012-07-28

小提琴


该死,现在我有一个真正的问题。它在 Sarafi 上不起作用(尝试过 Mac 版本),甚至 iPhone 也无法计算价格。Firefox 和 Chrome 在 OSX Lion 下运行良好。

价格不会被计算,似乎脚本在这些行附近停止:

 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);
  }
}

我已经检查了 safari javascript 错误日志,但我发现那里没有问题。

小伙伴们可以再看一遍吗?那会很好 ;-)

更新 26.7.2012 Safari 6.0(今天发布)上没有出现该问题。

4

1 回答 1

0

我认为您的字符串日期被错误地转换为日期。
你所有的约会都早了一天。
如果您在控制台中运行: 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(中部标准时间)

-edit-:
逻辑看起来不对。
如果提交: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 回答