有没有更有效的编码方法？

Question

有问题的代码是第二个 foreach 循环中包含的代码，其目的是防止精确重复的纬度和经度。

foreach($aUsers as $k => $v)
{
    // generate address
    $aAddress = array();
    if(!empty($v['city_location']))
        $aAddress[] = $v['city_location'];
    if(!empty($v['country_child_id']))
    {
        $aRow = $oDb->select('name')
             ->from(getT('country_child'))
             ->where('child_id = \''.$v['country_child_id'].'\'')
             ->execute('getRow');
        $aAddress[] = $aRow['name'];
    }
    if(!empty($v['postal_code']))
        $aAddress[] = $v['postal_code'];
    if(!empty($v['country_iso']))
    {
        $aRow = $oDb->select('name')
             ->from(getT('country'))
             ->where('country_iso = \''.$v['country_iso'].'\'')
             ->execute('getRow');
        $aAddress[] = $aRow['name'];
    }
    $sAddress = implode(', ',$aAddress);
    /// get location
    $aLatLon = $oGeoMap->getLatLon($v['user_id'],1,$sAddress);
    if($aLatLon['success'] === true)
    {
        foreach($aUsers as $k2 => $v2)
        {
            $iAdd = .01;
            $iAttempts = 0;
            while($v2['latitude'] == $aLatLon['latitude'] && $v2['longitude'] == $aLatLon['longitude'])
            {
                $iAttempts++;
                switch($iAttempts){
                    case 1:
                        $aLatLon['latitude'] += $iAdd;
                        break;
                    case 2:
                        $aLatLon['longitude'] += $iAdd;
                        break;
                    case 3:
                        $aLatLon['latitude'] += $iAdd;
                        $aLatLon['longitude'] += $iAdd;
                        break;
                    case 4:
                        $aLatLon['latitude'] -= $iAdd;
                        $aLatLon['longitude'] -= $iAdd;
                        break;
                    case 5:
                        $aLatLon['latitude'] += $iAdd;
                        $aLatLon['longitude'] -= $iAdd;
                        break;
                    case 6:
                        $aLatLon['latitude'] -= $iAdd;
                        $aLatLon['longitude'] += $iAdd;
                        break;
                    case 7:
                        $iAdd += .01;
                        $iAttempts = 0;
                        break;
                }
            }
        }
        $aUsers[$k]['latitude'] = $aLatLon['latitude'];
        $aUsers[$k]['longitude'] = $aLatLon['longitude'];
        $aUsers[$k]['address'] = $sAddress;
    }
    else
        unset($aUsers[$k]);
}

Answer 1

我强烈同意弗兰克·法默的评论，但如果您坚持这种做事方式，请尝试按照以下方式进行操作：

$lls = array();
static $offs = array( array(1,0), array(0,1), array(1,1),
    array(-1,-1), array(1,-1), array(-1,1) );

foreach($aUsers as $k=>&$v) {
    $lla=$la=somelat; $llo=$lo=somelong;
    for($add=.01;isset($lls["{$lla}x$llo"]);$add+=.01) {
        foreach($offs as $o) {
            $lla=$la+$o[0]*$add; $llo=$lo+$o[1]*$add;
            if(!isset($lls["{$lla}x$llo"])) break;
        }
    }
    $lls["{$lla}x$llo"] = true;
    $v['latitiude']=$lla; $v['longtitude']=$llo;
    $v['address'] = $sAddress;
}

也就是说，如果我正确理解您的意图。并更改变量名称以符合您自己的标准；-)

代码未经测试。

不，我不会重新阅读完整的代码;-)

Answer 2

您可以尝试使用第二种数据结构来保存现有的纬度+经度位置，例如按纬度然后经度索引的二维用户数组。您无需循环遍历 $aUsers 数组，而是检查另一个数据结构中是否存在给定的纬度和经度。

请注意，由于您只能使用整数和字符串作为数组键，并且将浮点数转换为整数或字符串将导致精度损失，因此如果您使用纬度和，则应将浮点数转换为其位字符串等效项经度作为键。一个简单的 pack('d', $l) 应该可以工作，或者您可以使用：

function fl2hex($l, $fmt='d') {
    $hex = unpack('H*', pack($fmt, $l));
    return $hex[1];
}

如果您知道您的纬度和经度存储为浮点数而不是双精度数（尽管我相当确定 PHP 只使用双打），您可以使用“f”作为包格式而不是“d”。

为了提高可读性，您可以将部分代码（例如 switch($iAttempts)）重构为附加函数。

至于多人在一个位置的问题，另一种解决方案（假设地图显示在屏幕上，而不是打印出来）是为一组人使用不同的指示器。当用户将鼠标悬停在组图标上时，它可以“爆炸”成星形的单个图标（如果有意义的话），每个图标都有一条线将图标连接到人员的位置。我在几个应用程序中看到过类似的内容，但现在无法想到是哪些。

Answer 3

您可以使用带有分号的 switch() ...例如还

switch($attempts) {
    case 1;
    case 3;
    case 5;
        $aLatLon['latitude'] += $iAdd;
    break;
}

没有测试过这个；但我确信如果你继续这样做或类似的事情，它会起作用。

希望这有帮助

Answer 4

建议：

这更多的是对所有代码的一般建议，而不仅仅是内部循环。我会更改代码以使用专用结构而不是数组。这是一个基本示例：

foreach($aUsers as $k => $v)
{
    $userInfo = new UserInfo($v);

    // generate address
    $sAddress = $userInfo->getAddress();

    /// get location
    $latLonResult = $oGeoMap->getLatLon($userInfo->user_id, 1, $sAddress);
    ...
}

实际答案：

就优化内部循环而言，我只需删除该代码。要么提出一个新的 UI 元素，让多个用户位于同一位置，要么忽略用户（如果他们的纬度/经度已添加到地图中）。

如果你必须保留它，迈克尔的解决方案看起来不错。