Question

我是PHP的新手，特别是面向对象的php，我正在研究一些简单的代码，而且我不太清楚到目前为止我写的是什么问题。我确定这很简单，但我已经在这墙上撞了一下头，我想在这里问。

class primes
{
$TestValues = array(0, 1, 2, 3, 4);
function IsPrime($number)
{

    //if number is a number, perform the rest of the tests, else, return -1 (error)
    if(is_numeric($number))
    {

        //if number is less than 0, return -1 (error)
        if($number < 0)
            return -1;
        //if number is 0, then return 0 (false, not prime)
        if($number == 0)
            return 0;
        //if number is greater than 1024, return -1 (error)
        if($number > 1024)
            return -1;
        //if number is 1, return 0 (false, not prime)
        if($number == 1)
            return 0;
        //if number is 2, return 1 (true, is prime)
        if($number == 2)
            return 1;
        //if number mod 2 is 0, then it is even, and no even number is prime except 2, which is handled above. so return 0 (false, not prime)
        if($number % 2 == 0)
            return 0;

        //if number has passed all previous tests, mod it by all odd numbers from 3 to its square root rounded up.
        for($i = 3; $i <= ceil(sqrt($number)); $i = $i +2)
        {
            //if any numbers mod 3 to its square root equal 0, return 0, (false, not prime)
            if($number % $i == 0)
                return 0;
        }
        //if the number has passed all above requirements, then it is a prime number below 1024.
        return 1;
    else
    {
        return -1;
    }
    }
}

function TestIsPrime()
{
    foreach($TestValues as $value)
        IsPrime($value);
        if(IsPrime() == 0)
            echo($value . "=> Is not Prime");
        elseif(IsPrime() == 1)   
            echo($value . "=> Is Prime");
        elseif(IsPrime() == -1)
            echo($value . "=> Is an Error");

}

 function main()
 {
     TestIsPrime();

  }

}
main();

我收到一条错误消息，说我不会将我的数组放在当前的位置。我不太确定php代码的结构应该如何与一个类一起工作，所以我不知道在哪里放置$TestValues数组，所以我尝试了几个地方，没有人会被接受。另外，我在连接到第一个if(is_numeric($number))的else语句中收到错误，但我无法确定该错误不是由我收到的另一个小错误引起的。最后一个错误是我不确定在这个单页代码中调用类中的函数。任何帮助，将不胜感激。再一次，我是新手在PHP中做任何有用的东西，但到目前为止我都喜欢它。谢谢，

Answer 1

您的代码缺少几个大括号，您$TestValues也是类的一部分，而不是全局变量。

我修复了代码，但你必须检查数学并阅读php网站上的oop tutorial。

<?php
class Primes {
    public $TestValues = array(0, 1, 2, 3, 4);
    function IsPrime($number) {

        //if number is a number, perform the rest of the tests, else, return -1 (error)
        if(is_numeric($number)) {

            if($number < 0 || $number > 1024)
                return -1;
            if($number === 0 || $number === 1 || $number % 2 === 0)
                return 0;

            //if number has passed all previous tests, mod it by all odd numbers from 3 to its square root rounded up.
            for($i = 3; $i <= ceil(sqrt($number)); $i = $i + 2) {
                //if any numbers mod 3 to its square root equal 0, return 0, (false, not prime)
                if($number % $i == 0)
                    return 0;
            }
            //if the number has passed all above requirements, then it is a prime number below 1024.
            return 1;
         } else {
            return -1;
        }
    }

    function test() {
        foreach($this->TestValues as $value) {
            $t = $this->IsPrime($value);
            if($t === 0) {
                echo($value . "=> Is not Prime");
            } elseif($t === 1) {   
                echo($value . "=> Is Prime");
            } elseif(IsPrime() == -1) {
                echo($value . "=> Is an Error");
            }
            echo "\n";
        }
    }
}

function main() {
    $prime = new Primes();
    $prime->test();
}

main();

Answer 2

您的代码中存在一些语法错误。我现在对你的代码进行了一些修改：

<?php

// Check the number to be prime or not
function IsPrime($number) 
{
    // If the variable is not numeric, negative or greater than 1024, exit
    if (!is_numeric($number) || $number < 0 || $number > 1024)
    {
    return -1;
    }
    // Perform the tests
    switch($number)
    {
    //if number is 0 or 1, then return 0 (false, not prime)
        case 0 :
        case 1 :
            return 0;
        break;
    //if number is 2, return 1 (true, is prime)
        case 2 :
            return 1;
        break;
    }

    //if number mod 2 is 0, then it is even, and no even number is prime except 2, return 0 (false, not prime)
    if($number % 2 == 0) 
    { 
    return 0;
    }

    //if number has passed all previous tests, mod it by all odd numbers from 3 to its square root rounded up.
    for($i = 3; $i <= ceil(sqrt($number)); $i = $i +2)
    {
        if($number % $i == 0) 
        {
            return 0;
        }
    }

    //if the number has passed all above requirements, then it is a prime number below 1024.
    return 1;
} // End of isprime function

// create an array of numbers
$testvalues = range(-2, 100);

foreach($testvalues as $value) 
{
    switch(isprime($value))
    {
        case 0 :
            echo("<p style='color:gainsboro;'>" . $value . "=> Is not Prime</p>");
            break;
        case 1 :
            echo("<p style='color:green;'>" . $value . "=> Is Prime</p>");
            break;
        case -1 :
            echo("<p style='color:red;'>" . $value . "=> Is an Error</p>");
            break;
    }
}
?>

注意：

我觉得你这里不需要上课。这是一个简单的程序和使用类将使它有点复杂，PHP以其为人所知简单。所以保持简单。
在某些情况下，切换比if更好，更易读，所以如果使用它，请使用它可能http://php.net/manual/en/control-structures.switch.php
每种语言都有首选语法。在PHP中，通常不会在命名变量（类除外）中使用大写字母不同于其他语言
php中没有main()

希望这有帮助，请原谅我的英语，如果我错了，请纠正我

Answer 3

你收到else语句错误的原因是因为}语句的关闭大括号 if，请注意你的{{{}后面有一个右大括号1}}条件，而不是这样做，你必须首先关闭else语句然后执行if语句，在你的情况下else表示else然后做某事，你也有这么多if语句看起来有点多余。你能做的就是把它们放在像if is not numeric

这样的条件下

你无法获得$ Testvalues的原因是你需要将数组传递给你的函数然后你可以使用它，你在哪里调用你的TestIsPrime函数？当你打电话给它时，做某事 TestIsPrime（$ TestValues）;

顺便说一下，最好不要将大写字母用于变量...

在哪里放置数组和其他问题

3 个答案: