PHP质数判断脚本报错求助：现有代码运行异常

嘿，我看了你的代码，里面有几个明显的语法和逻辑问题导致运行报错，我给你修正一下，顺便解释问题出在哪：

首先，先看修正后的完整代码：

<html>
<body>
<form method="post">
    Number: <input type="text" name="prime">
    <input type="submit" value="Submit">
</form>

<?php
// 先检查表单是否提交，避免未定义索引错误
if ($_SERVER["REQUEST_METHOD"] == "POST") {
    $num = $_POST["prime"];
    // 处理非数字、小于2的情况
    if (!is_numeric($num) || $num < 2 || $num != (int)$num) {
        echo "请输入大于等于2的正整数";
    } else {
        $num = (int)$num;
        $flag = 0;
        // 优化循环：只需要遍历到平方根，提升效率
        for ($i = 2; $i <= sqrt($num); $i++) {
            if ($num % $i == 0) {
                $flag = 1;
                break;
            }
        }
        if ($flag == 0) {
            echo "$num is a prime number";
        } else {
            echo "$num is not a prime number";
        }
    }
}
?>
</body>
</html>

原代码的问题解析： #

• HTML与PHP的位置错误：你把Number : <input>这些HTML元素直接写在了<?php ?>标签内部，PHP解析器会把这些HTML文本当成PHP代码处理，导致语法错误。正确的做法是把HTML内容放在PHP标签外面，PHP代码单独处理逻辑。
• 未处理表单未提交的情况：页面首次加载时，$_REQUEST["prime"]是不存在的，会触发"未定义索引"的报错。我们需要通过$_SERVER["REQUEST_METHOD"] == "POST"来判断表单是否提交，只有提交后才处理数值。
• 多余的分号：你在<input>标签后面加了分号（比如<input type="submit"...>;），这是HTML语法错误，HTML标签不需要加分号。
• 边界情况未处理：原代码没有考虑输入不是数字、小于2、小数的情况，比如输入1、0、负数或者字符串时，会得到错误的结果，我们需要先做合法性校验。
• 循环效率问题：原代码循环到$num/2，其实只需要循环到sqrt($num)就足够判断质数了，因为如果$num有大于平方根的因数，那对应的另一个因数一定小于平方根，这样能减少循环次数。

这样修改后，脚本就能正常运行，并且处理各种异常输入啦。

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