为什么全局声明变量而不是使用var时函数起作用

时间:2019-02-24 06:31:53

标签: javascript variables parameters global-variables var

我想将属性soundFileName声明为var soundFileName = 'audio/60.wav';,以便soundFileName不在全局定义,但是当我得到ReferenceError: soundFileName is not defined时。

我将soundFileName的值作为参数传递给loop(soundFileName),我认为值'audio/60.wav'应该会很好地传递。

我怀疑这与范围或嵌套有关,但是我不确定如何解决该问题。当我使用不带var的soundFileName = 'audio/60.wav';时,代码可以工作。

我想念什么?谢谢!

编辑:代码现在可以正常工作并已更新!

<!DOCTYPE html
    PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">

<head>
  <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />

  <script src="js/howler.core.js"></script>
  <link rel="stylesheet" href="css/styles.css">

</head>

<body>

  <script src="timeprobabilities.js"></script>

  <script>
    ///////////////////////////////////////////////////////////////
    // MASTER START ///////////////////////////////////////////////
    ///////////////////////////////////////////////////////////////

    // initiates fist call to all clocks to each will start and can then be called again

    (function masterStart() {
      setTimeout(function() {
        //// LOOP SOUNDS \\\\
        A();
      }, 0);
    }());

    ///////////////////////////////////////////////////////////////
    // LOOPS SHARED OPTIONS ///////////////////////////////////////
    ///////////////////////////////////////////////////////////////

    var options = {
      numberOfSounds: 0,
      maxNumberOfSounds: 4
    };

    function logNumberOfSounds() { // passing options into this before is what broke code
      options.numberOfSounds++;
      //console.log('Number of sounds is: ' + options.numberOfSounds + '########');
    }

    ///////////////////////////////////////////////////////////////
    // LOOP A  ////////////////////////////////////////////////////
    ///////////////////////////////////////////////////////////////

    function A() {
      var optionsA = {
        playDurationMin: 0,
        playDurationMax: 60000,
        // start time minimum and maximum
        startMinA: 0,
        startMaxA: 8000,
        maxVolumeA: 1,
        //
        startMinB: 0,
        startMaxB: 30000,
        maxVolumeB: 1,
        //
        startMinC: 0,
        startMaxC: 30000,
        maxVolumeC: 1,
        //
        startMinD: 0,
        startMaxD: 30000,
        maxVolumeD: 1,
        //
        startMinE: 0,
        startMaxE: 30000,
        maxVolumeE: 1,
        //
        startMinF: 0,
        startMaxF: 30000,
        maxVolumeF: 1,
        //
        startMinG: 0,
        startMaxG: 30000,
        maxVolumeG: 1,
        //
        startMinH: 0,
        startMaxH: 30000,
        maxVolumeH: 1,
        //
        startMinI: 0,
        startMaxI: 30000,
        maxVolumeI: 1,
        //
        startMinJ: 0,
        startMaxJ: 30000,
        maxVolumeJ: 1,
        //
        startMinK: 0,
        startMaxK: 30000,
        maxVolumeK: 1
      };

      masterClock();

      function masterClock() {
        setTimeout(function() {
          soundA(options, optionsA);
        }, 10); // these need to be called with delay so they don't use the other functions' paramaters
      }

      function soundA() {

        var soundFileName = 'audio/60.wav';
        fadeIn = 8000;
        fadeOut = 8000;

        console.log('soundFileName in A: ' + soundFileName);

        calculateStartDelay(optionsA.startMinA, optionsA.startMaxA);

        function calculateStartDelay(startMin, startMax) {
          startDelay = Math.floor(Math.random() * startMax) + startMin;
        }

        function calculatePlayDuration(playDurationMin, playDurationMax) {
          playDuration = Math.floor((Math.random() * playDurationMax) + playDurationMin);
        }

        function executePlayTools() {
          calculatePlayDuration(optionsA.playDurationMin, optionsA.playDurationMax);
          loop(options, playDuration, soundFileName, fadeIn, fadeOut);
          console.log('A: ////////////////////////////////// ');
          masterClock();
        }

        setTimeout(function() {
          if (probabilityValue < probabilityPointA) {
            maxVolume = optionsA.maxVolumeA;
            executePlayTools();
          } else if (probabilityValue < probabilityPointB) {
            maxVolume = optionsA.maxVolumeB;
            executePlayTools();
          } else if (probabilityValue < probabilityPointC) {
            maxVolume = optionsA.maxVolumeC;
            executePlayTools();
          } else if (probabilityValue < probabilityPointD) {
            maxVolume = optionsA.maxVolumeD;
            executePlayTools();
          } else if (probabilityValue < probabilityPointE) {
            maxVolume = optionsA.maxVolumeE;
            executePlayTools();
          } else if (probabilityValue < probabilityPointF) {
            maxVolume = optionsA.maxVolumeF;
            executePlayTools();
          } else if (probabilityValue < probabilityPointG) {
            maxVolume = optionsA.maxVolumeG;
            executePlayTools();
          } else if (probabilityValue < probabilityPointH) {
            maxVolume = optionsA.maxVolumeH;
            executePlayTools();
          } else if (probabilityValue < probabilityPointI) {
            maxVolume = optionsA.maxVolumeI;
            executePlayTools();
          } else if (probabilityValue < probabilityPointJ) {
            maxVolume = optionsA.maxVolumeJ;
            executePlayTools();
          } else {
            maxVolume = optionsA.maxVolumeK;
            console.log('Probability Else');
          }
          console.log('startDelay: ' + startDelay)
        }, startDelay);
      }
    }

    ///////////////////////////////////////////////////////////////
    // SHARED LOOP  ///////////////////////////////////////////////
    ///////////////////////////////////////////////////////////////

    function loop(options, playDuration, soundFileName, fadeIn, fadeOut) {

      console.log('soundFileName in loop: ' + soundFileName);

      if (options.numberOfSounds < options.maxNumberOfSounds) { //Don't create more than the max number of sounds.

        var sound = getSound(soundFileName);
        var id2 = sound.play();

        logNumberOfSounds();

        sound.volume(0); // don't think I need this since it's declared above and in getSound(), but it stops blips?
        sound.fade(0, maxVolume, fadeIn, id2); // FADE IN

        setTimeout(function() {
          sound.fade(maxVolume, 0, fadeOut, id2); // FADE OUT
          options.numberOfSounds--;

          // Attempt to clean up the sound object
          setTimeout(function() {
            sound.stop();
            sound.unload();
          }, fadeOut + 1000);
        }, playDuration);
      }

    }

    // PLAYER FOR MAIN SOUND FUNCTION /////////////////////////////
    function getSound(soundFileName) {
      return new Howl({
        src: [soundFileName],
        autoplay: true,
        loop: true,
        volume: 0,
        fade: 0 // removes the blip
      });
    }
  </script>

  <script src="js/howler.core.js"></script>
  <script src="js/siriwave.js"></script>
  <script src="js/player.js"></script>

</body>

</html>

3 个答案:

答案 0 :(得分:2)

您没有正确使用变量的值。其值在soundA()函数内定义,然后传递给函数loop(),该函数无法知道哪个变量是哪个变量。

有两个解决方案。设置像function loop(options, options, playDuration, soundFileName, fadeIn, fadeOut){...}这样的预定义参数名称,然后在函数主体中使用这些名称,或者可以使用arguments函数对象。 arguments对象是使用每个函数(不包括箭头函数)创建的,类似于数组的对象,该函数包含所有传递的参数,通过这些参数,您可以轻松使用传递给它的变量。请参见下面的示例:

loop(options, playDuration, soundFileName, fadeIn, fadeOut);

function loop() {
          console.log('soundFileName in loop: ' + arguments[2]); //The arguments object will contain all five parameters passed to the function call. 
}

已经说过,对于某些函数,使用未知数量的参数被认为是不好的做法,因为与使用预定义的参数相比,它可能导致更多的错误和陷阱。尽管如此,arguments对象仍然可以使用,您可以随意构造应用程序。只是要记住,其他人可能会阅读代码并一头雾水。

另外,解决此问题后,我还遇到了另一个问题,您正在实际的函数声明之前在单独的脚本标记中使用函数,从而导致undefined错误。

另外,请将此答案标记为帮助其他人通过的解决方案。

EDIT#1:从ES6 +开始,您可以指定默认参数值,以防即使您未正确传递值,也希望始终保持功能正常。

EDIT#2:正如注释中提到的@nikolairiedel一样,函数getSound()超出了该变量的声明范围,因此无法获取其值。您可以为该函数设置参数和/或指定默认参数值,以确保该函数始终有效,如上面edit#1中所述。

答案 1 :(得分:2)

每个函数都有其自己的范围,并且在函数中声明的变量仅存在于该函数的范围之内

在这种情况下,soundFileNamevar中被声明为soundA,因此它仅存在于soundA的范围内。

您将soundFileName作为参数传递给loop,然后从那里传递给getSound,但是function definitions代表loop和{ {1}}不包含任何命名参数。

您需要将getSoundloop的{​​{3}}更改为包含预期的参数:

getSound

请注意,在function loop(options, playDuration, soundFileName, fadeIn, fadeOut) { ... // soundFileName is now availabie in this scope ... } function getSound(soundFileName) { ... // soundFileName is now availabie in this scope ... } (不包括function definition)的每个函数中,有arrow functions包含调用该函数的参数,因此自从您将JavaScript作为soundFileName的第3个参数所传递的值也将在loop中以loop的形式使用,并且由于它作为arguments[2]的第1个参数传递,其值也将在{ {1}}为getSound

答案 2 :(得分:0)

也许尝试更改此内容

 function loop() {

收件人:

function loop(param1,param2.....){

在您的情况下:

function loop(soundFileName,playDuration, ...){