PHP - 仅当余数为 0 时才执行（和回显）操作 - 进入无限循环

Question

让我从结尾和实际问题开始：

我正在尝试编写一个 PHP 脚本，只有我们输出并解析两个随机数 $x 和 $y当 modulo == 0 （它应该始终回显有效函数）

我的意思是：

脚本回显：

4 / 2 =>好的！

脚本回显：

4 / 3 =>坏的！

什么都没有=>坏的！

---

我会尽力引导您完成我的推理，但请注意：

我是新手，所以：我学到了一些东西，提高了赌注，并尝试方便地应用它。

---

首先我生成随机数，简单：

$x = mt_rand(1,10);
$y = mt_rand(1,10);

现在我说，除法后有两种可能性，余数为 0 或不为 0，因此 if() 语句看起来是个好主意，所以我这样做：

if($x % $y == 0){

echo $x 。 “/”。 $y;

}else{

echo "余数不为0";

。

这按预期工作，当然，我总是想生成这个余数不是 0 除法

所以我的方法如下，为什么不使用用户生成的函数（以前从未使用过但可以派上用场）所以，如果我知道这个 工作得很好：

<?php
 
function hello(){   
      echo "Hello!";  
}
 
hello();
?>

为什么不像我之前所说的那样提高赌注并使用它，所以我混合了：

<?php
$x = mt_rand(1,10);
$y = mt_rand(1,10);
 
function foo(){
 
if($x % $y == 0){
 
echo $x . "/" . $y;
 
}else{
        
foo();  
        
}
 
}
 
?>

它不起作用，我得到一个空白屏幕。

我对这个脚本的期望是什么：

• 创建一个带有 if() 语句的函数
• 如果余数为 0，很好，回声。
• 否则，从头开始，一次又一次，直到得到一个没有余数的除法。

这是行不通的。

所以我基本上会问，我做错了什么？这后面的线有什么问题吗？

这可以通过微妙的改变起作用吗？

---

对于经验丰富的编码人员，我知道你们都可以使用 MS PAINT 作为 IDE 制作一个脚本来执行此操作（但与我的无关），但是：

我对代码本身不感兴趣，因为这仅用于学习目的。

这不是我第一次遇到这种情况，我经常因为无限循环而碰壁，所以变化越小（解释越大）越好，并且将非常感激。

提前致谢！

---

现在完成后，我刚刚意识到问题甚至可能不是无限循环，即使问题不再存在，您对我的方法有何看法？ （当然我也会编辑并删除问题中的无限循环）

Answer 1

在我看来，您只设置了 $x 和 $y 一次。

这意味着当您递归调用 foo() 时它们不会改变。我首先尝试这样的事情：

<?php
    function foo() {
        $x = mt_rand(1,10);
        $y = mt_rand(1,10);
        if ($x % $y == 0) {
            echo $x . "/" . $y;
        } else {
            $x = mt_rand(1,10);    # need to recalculate them here.
            $y = mt_rand(1,10);
            foo();  
        }
    }
?>

不过我会检查该代码。我了解 PHP 和了解阿拉姆语 :-) 我的分析更多地基于一般语言知识，因此语法可能是假的。

当然，在这种情况下，您可能需要考虑完全避免递归。确实没有必要。当递归使您的代码更加优雅时，它就很好，但我不认为这里是这种情况。我会尝试这样的事情：

<?php
    function foo() {
        $x = mt_rand(1,10);
        $y = mt_rand(1,10);
        while ($x % $y != 0) {
            $x = mt_rand(1,10);
            $y = mt_rand(1,10);
        }
        echo $x . "/" . $y;
    }
?>

Answer 2

好吧，让我们从基础开始。

您的应用程序不显示任何内容的原因是您没有告诉它。您已经定义了一个将显示某些内容的函数，但您从未真正调用过它。

基本上，您需要执行以下操作：

function foo() {
  // foo logic
}


foo();

注意如何在应用程序的主线逻辑中在函数本身之外对 foo 进行调用。

接下来，您需要将对 mt_rand() 的调用移至 foo() 函数内部，以便生成新的随机数每次通话时。

最后，您不需要从foo()内部调用foo()。这样做是允许的，并且称为“递归”，但这里确实不需要。您应该做的是设置您的函数，以便在函数完成时返回一个布尔值（即 true/false），让您知道它有效。然后，您可以设置一个 while 循环来重复执行该函数，直到它正确完成。

这是一个例子：

function foo() {
  //define the random values
  $x = mt_rand(1,10);
  $y = mt_rand(1,10);

  //if they match, echo the statement and return true so the loop ends
  //otherwise, return false so the loop repeats
  if (($x % $y) == 0) {
    echo $x . "/" . $y;
    return true;
  } else {
    return false;
  }
}

//we're now *outside* of the function, and in mainline logic

//setup a variable to hold the return value of the function
//initialize it to false so that the first loop executes
$foundEquation = false;


//execute the function in a loop until we find
//the right equation
while(!$foundEquation) { 
  $foundEquation = foo();
}

这能回答你的问题吗？如果您在理解这里发生的事情时遇到任何困难，请告诉我。我很高兴澄清有关此代码的任何内容。

Answer 3

你的问题是你没有重新声明 $x 和 $y 。假设，这个应该进入无限递归。如果你想递归地执行此操作，则必须在 foo() 函数中定义 $x 和 $y 。确保变量是在 if() 语句之外和之前声明的。

祝你好运！
丹尼斯·M.

Answer 4

在您发布的“它不起作用”链接中，您没有调用函数 foo()。

一些注意事项：

• y 可能为零，因此您可能会收到“除以零”错误。
• 您正在使用递归，并且大多数时候您的代码都会进入 else 情况。如果嵌套深度变大，则可能会遇到堆栈溢出。

Answer 5

Paxdiablo 走在正确的轨道上，但这可能是您想要的：

<?php
    function foo() {
        do
        {
            $x = mt_rand(1,10);
            $y = mt_rand(1,10);
        }
        while ($x % $y != 0);

        echo $x . "/" . $y;
    }
?>

这将循环执行 do { } while 内的代码，直到 while 条件为 false。 while 和 do while 之间的区别在于，代码在检查条件之前执行。