PHP代码默认为不应该出错

时间:2014-11-10 18:10:16

标签: php maxmind

我有一个PHP开发人员创建一个重定向脚本,将特定状态的用户重定向到另一个URL,同时让其他人访问该网站。

问题是,它应该让他们访问该网站时,将与列出的状态不匹配的所有人重定向到错误URL。

我认为有回归失踪?你们觉得怎么样?

<?php
ini_set('display_errors',1);
ini_set('display_startup_errors',1);
error_reporting(-1);

require_once '/vendor/autoload.php';

use MaxMind\Db\Reader;

$databaseFile = '/geoip/GeoIP2-City.mmdb';
$ipWhiteList = ['123', '321'];

if(!in_array($_SERVER["REMOTE_ADDR"], $ipWhiteList)) {
    $reader = new Reader($databaseFile);
    $iso_code = $reader->get($_SERVER["REMOTE_ADDR"])['subdivisions'][0]['iso_code'];
    if (!isset($_REQUEST['HTTP_REFERER'])) {
        switch($iso_code) {
            case NJ:
                $url = 'http://example.com';
                break;
            case DE:
                $url = 'http://example.com';
                break;
            default:
                $url = 'http://www.example.com/?=error';
                break;
        }
        $reader->close();
        header('Location: '.$url);
        die();
    } else {
        if(strpos($_SERVER['HTTP_REFERER'], "example2.com") > -1) {
            echo "You were redirected from ".urldecode($_REQUEST['referer']).", but it is not available in your area (".$iso_code.").";
            break;
        } else {
            echo "Welcome!";
            break;
        }
    }
}
?>

2 个答案:

答案 0 :(得分:0)

也许尝试加载所有有效值,并检查以确保它有效,即使它不是NJ / DE。

if (in_array($state, array('NJ', 'DE'))) {
    // Redirect
} elseif (!in_array($states, $all_states_array)) {
    // Go to error.
}

其他是隐含的,因为脚本将继续工作。你可以用几种不同的方式构建它们,具体取决于你需要扩展它的程度:

if (!in_array($state, $all_states_array)) {
    // Error
}

if (in_array($state, array('NJ', 'DE'))) {
   // Redirect
}

您还可以添加所有案例:

case 'DE':
    // Do something;
    break;
case 'NJ':
    // Do soemthing;
    break;
case 'PA':
case 'AL':
case 'NY':
case 'OK':
case 'TX':
...
    // Valid, but not the right target.
    break;
default:
    // show error

答案 1 :(得分:0)

试试这个。如果州不是来自新泽西州或德国,那么什么也不做。

<?php
ini_set('display_errors',1);
ini_set('display_startup_errors',1);
error_reporting(-1);

require_once '/vendor/autoload.php';

use MaxMind\Db\Reader;

$databaseFile = '/geoip/GeoIP2-City.mmdb';
$ipWhiteList = ['123', '321'];

if(!in_array($_SERVER["REMOTE_ADDR"], $ipWhiteList)) {
    $reader = new Reader($databaseFile);
    $iso_code = $reader->get($_SERVER["REMOTE_ADDR"])['subdivisions'][0]['iso_code'];
    if (!isset($_REQUEST['HTTP_REFERER'])) {
        switch($iso_code) {
            case NJ:
                $url = 'http://example.com';
                header('Location: '.$url);
                                die();
                break;
            case DE:
                $url = 'http://example.com';
                header('Location: '.$url);
                die();
                break;
            default:
        }
        $reader->close();
    } else {
        if(strpos($_SERVER['HTTP_REFERER'], "example2.com") > -1) {
            echo "You were redirected from ".urldecode($_REQUEST['referer']).", but it is not available in your area (".$iso_code.").";
            break;
        } else {
            echo "Welcome!";
            break;
        }
    }
}
?>