-1

这是一个基本的 JavaScript 代码，用于计算给定值的“提示”。我在这里做错什么了吗？它给了我null数组或NaN代替我的数组内容。

var john = {

  fullName: 'john smith',
  bills: [124, 48, 32, 268, 180, 42],

  calcTips: function () {
    this.tips = [0];
    this.finalvalue = [0];

    for (var i = 0; i < this.bills.length; i += 1) {
      var percentage;

      if (this.bills[i] < 50) {
        percentage: .2;
      }
      else if (this.bills[i] >= 50 && this.bills[i] < 200) {
        percentage: .15;
      }
      else {
        percentage: .1;
      }

      this.tips[i] = this.bills[i] * percentage;
      this.finalvalue[i] = this.bills[i] + this.bills[i] * percentage;
    }
  }
}
4

3 回答 3

1

这是假设数组中已经存在tips等于数组中值数量的值bills，并更新其中的相应值tips（不正确，我可能会添加。稍后介绍）。

但是，由于您仅使用一个值 0 进行了初始化tips，因此循环在第一次迭代之后没有任何要更新的内容。

相反，我建议：


var john = {
    fullName: 'john smith',
    bills: [124, 48, 32, 268, 180, 42],

    calcTips: function() {

        /* I'm assuming you initialized these with one value of 0 because you're doing 
        a calculation later using these, so we'll leave them be */
        this.tips = [0];
        this.finalvalue = [0];

        for ( var i = 0; i < this.bills.length; i+= 1 ) { 
            var percentage;
            /* Reassigning variables requires the equals sign (=) rather than 
            a colon (:) */

            if (this.bills[i] < 50) {
               percentage = .2;
            } else if (this.bills[i] >= 50 && this.bills[i] < 200 ) {
                percentage = .15; 
            } else {
                percentage = .1;
            }

            /* If i is 0, only update the first value of tips (using splice()), past 
            that, push in a new value */
            if (i === 0) {
               this.tips.splice(i, 1, this.bills[i] * percentage ;
            } else {
                this.tips.push(this.bills[i] * percentage);
            } 

            this.finalvalue[i] = this.bills[i] + this.bills[i] * percentage;
        }
    }
}
于 2020-04-09T21:55:30.517 回答
1

正如其他答案所指出的那样，根本问题是percentage:线条。将它们更改为变量赋值percentage = 0.2;将解决它。

您可能会问自己“为什么这没有引发错误？”。

此语法用于定义“标签”。标签用于标识使用breakor的语句continue。基本上是一种识别循环的方法，允许您命名您具体中断或继续的循环。MDN 有一些很好的标签使用示例。虽然很少使用，但它可以放在你的后兜里 :)

于 2020-04-09T22:17:24.533 回答
0

你只需要使用percentage = value而不是那些:

var john = {
  fullName: 'john smith',
  bills: [124, 48, 32, 268, 180, 42],


  calcTips: function() {

      this.tips = [0];
      this.finalvalue = [0];

  for ( var i = 0; i < this.bills.length; i+= 1 ) { 
      var percentage = 0;

      if (this.bills[i] < 50) {
          percentage= .2;
      } else if (this.bills[i] >= 50 && this.bills[i] < 200 ) {

          percentage= .15; 
      }

          else {
          percentage= .1;
      }

      this.tips[i] = this.bills[i] * percentage ;
      this.finalvalue[i] = this.bills[i] + this.bills[i] * percentage;
    }
  }
}
于 2020-04-09T21:59:38.070 回答