3

我正在检查信用卡字段以确保它们每个都包含四位数字。

function creditcard ($creditCard1,$creditCard2,$creditCard3,$creditCard4)
{
    $regexp="/^[0-9]{4}$/";

    if (preg_match($regexp,$creditCard1,$creditCard2,$creditCard3,$creditCard4))//line 18
    {
        return TRUE;
    }
    else
    {
        echo'<script type="text/javascript">alert("Enter a valid credit card number.")</script>';
        return FALSE;
    }
}

    $creditCard1=$_POST['creditCard1'];
    $creditCard2=$_POST['creditCard2'];
    $creditCard3=$_POST['creditCard3'];
    $creditCard4=$_POST['creditCard4'];
    $creditcard=creditcard($creditCard1,$creditCard2,$creditCard3,$creditCard4);

我收到这个错误。

Warning: preg_match() expects parameter 4 to be long, string given in C:\xampp\htdocs\SFASC2\aa.php on line 18

我无法弄清楚我做错了什么，谢谢。有很多这样的问题，但没有一个给我解决方案..而且我一直在用头撞墙，毫无疑问，忽略了显而易见的事情。

4

2 回答 2

5

preg_match一次只检查一个主题。

试试下面的代码：

function creditcard($nums)
{
    $regexp="/^[0-9]{4}$/";

    for ($i = 0; $i < 4; $i++)
        if (! preg_match($regexp, $nums[$i]))
            return FALSE;
    return TRUE;
}

$creditCard1 = $_POST['creditCard1'];
$creditCard2 = $_POST['creditCard2'];
$creditCard3 = $_POST['creditCard3'];
$creditCard4 = $_POST['creditCard4'];

$ok = creditcard([$creditCard1, $creditCard2, $creditCard3, $creditCard4]);
if (! $ok)
    echo '<script type="text/javascript">alert("Enter a valid credit card number.")</script>';
于 2013-08-04T05:44:28.730 回答
4

最好使用preg_grep而不是preg_match这个用例，因为您正在对输入数组应用单个正则表达式。使用的主要好处preg_grep是您可以避免循环输入。考虑下面的代码：

function creditcard($arr) {
    static $re="/^\d{4}$/";
    return (preg_grep($re, $arr) === $nums);
}

$result = creditcard(['5423', '1234', '1234', '1234']);
// returns true
$result = creditcard(['5423', 'A234', '1234', '1234']);
// returns false

PS：我声明$re因为static该变量应该只声明和初始化一次。

于 2013-08-04T06:46:29.700 回答