arc4random实现的Power随机生成函数重复执行问题排查求助

Hey there, let's figure out why your function is firing multiple times unexpectedly! Since you mentioned it calls itself after triggering the Power function, the most likely issues tie back to how you're managing timing or recursive calls. Here are key areas to check:

1. Accidental multiple initial invocations #

It’s possible your function is being started more than once (e.g., repeated page load triggers, duplicate event bindings). This creates multiple independent timer chains that run in parallel, leading to overlapping "double/triple" triggers.

Add a simple guard clause to prevent concurrent runs:

let isFunctionRunning = false;

function generatePower() {
  if (isFunctionRunning) return;
  isFunctionRunning = true;

  // Execute your Power function here
  triggerPower();

  // Generate random 30-90 second delay (converted to ms)
  const randomDelay = Math.floor(Math.random() * 60000) + 30000;
  setTimeout(() => {
    isFunctionRunning = false;
    generatePower();
  }, randomDelay);
}

2. Leaking or unmanaged timers #

If you’re not clearing old timers before starting new ones, leftover timers can keep running and trigger additional calls. Store the timer ID to clean up prior to each new run:

let activeTimerId = null;

function generatePower() {
  // Clear any existing timer to avoid duplicate triggers
  if (activeTimerId) clearTimeout(activeTimerId);

  triggerPower();

  const randomDelay = Math.floor(Math.random() * 60000) + 30000;
  activeTimerId = setTimeout(() => {
    generatePower();
  }, randomDelay);
}

3. Async side effects in triggerPower() #

If your triggerPower function is asynchronous (e.g., uses promises, API calls), make sure you’re not accidentally re-invoking generatePower before the async operation finishes. For example, if an error in triggerPower triggers a retry that restarts the function, you could end up with multiple chains running.

Add logging to debug the flow:

let callCounter = 0;

function generatePower() {
  callCounter++;
  console.log(`Power trigger #${callCounter} initiated at ${new Date().toLocaleTimeString()}`);
  
  triggerPower()
    .then(() => {
      const randomDelay = Math.floor(Math.random() * 60000) + 30000;
      setTimeout(generatePower, randomDelay);
    })
    .catch(err => {
      console.error("Power trigger failed:", err);
      // Only restart if intended - avoid accidental duplicate restarts here
      const randomDelay = Math.floor(Math.random() * 60000) + 30000;
      setTimeout(generatePower, randomDelay);
    });
}

4. Duplicate event bindings #

If this function is tied to an event (e.g., button clicks, component mounts), ensure you’re not binding the event multiple times. For example, in frameworks like React, binding events in a component’s render method can create duplicate listeners on re-renders.

内容的提问来源于stack exchange，提问作者user9611921