Почему не работает код на php?

Question

[Anzor]

Есть вот такой код:

<?php
// Read the content of the json file
$json_file = file_get_contents('polygon.json');

// Decode the json content into an array of coordinates
$polygon = json_decode($json_file, true);

// Check if the request contains a coordinate to be verified
if(isset($_GET['coordinate'])){
    $coordinate = $_GET['coordinate'];
    $is_inside = false;

    // Loop through the array of coordinates
    foreach($polygon as $key => $point){

        // Check if the coordinate exists in the polygon
        if($point['lat'] == $coordinate['lat'] && $point['lng'] == $coordinate['lng']){
            $is_inside = true;
            break;
        }
    }

    // Output a positive or negative message
    if($is_inside){
        echo "The coordinate is inside the polygon";
    }else{
        echo "The coordinate is not inside the polygon";
    }
}
?>

И файл с координатами - https://pastebin.com/cBSbSYcS

Делаю запрос формата

http://mydomain.ru/check_coordinate.php?coordinate=41.063013254638385,57.266932308895065

И выдает The coordinate is not inside the polygon. При том, что координата проходит по границе полигона (первая точка в файле с полигоном).

Comments

[humoured]

С чего бы после кода $coordinate = $_GET['coordinate'];
в переменной $coordinate оказался бы массив с ключами $coordinate['lat'] и $coordinate['lng']? Передаётся строка, а не массив: ?coordinate=41.063013254638385,57.266932308895065.

Answer 1

[Rsa97]

А вы сделайте var_dump($_GET['coordinate']); и посмотрите, что вам приходит.

Comments

[Rsa97]

Anzor, Просто добавьте в начало PHP-скрипта эту команду и сделайте запрос.

[Anzor]

Rsa97,

string(38) "41.063013254638385, 57.266932308895065"
The coordinate is not inside the polygon

Хотя эта же координата в первой точке полигона

[aryzhanki]

Anzor, массив !== строка (с координатами через ЗАПЯТУЮ)

[Rsa97]

Anzor, А дальше вы пытаетесь работать со строкой как с массивом, используя индексы 'lon' и 'lat'.
У вас, кстати, и в JSON-файле нет таких индексов в массивах.

[Anzor]

Rsa97, не мог бы пожалуйста, привести пример формата записи координат полигона?

[Михаил Ливач]

Anzor, писал не глядя:

<?php
const LATITUDE = 0;
const LONGITUDE = 1;
// Read the content of the json file
$json_file = file_get_contents('polygon.json');

// Decode the json content into an array of coordinates
$polygon = json_decode($json_file, true);

// Check if the request contains a coordinate to be verified
if(isset($_GET['coordinate'])){
    $coordinate = explode(', ', $_GET['coordinate']); // запятая с пробелом
    $is_inside = false;

    // Loop through the array of coordinates
    foreach($polygon as $key => $point){

        // Check if the coordinate exists in the polygon
        // можно проще: if ($point == $coordinate) {
        if($point[LATITUDE] == $coordinate[LATITUDE] && $point[LONGITUDE] == $coordinate[LONGITUDE]){
            $is_inside = true;
            break;
        }
    }

    // Output a positive or negative message
    if($is_inside){
        echo "The coordinate is inside the polygon";
    }else{
        echo "The coordinate is not inside the polygon";
    }
}

[Rsa97]

Михаил Ливач, Почти. Только заменить foreach на

foreach ($polygon['geometry']['coordinates'] as $point)

[Anzor]

Михаил Ливач, пока что по прежнему The coordinate is not inside the polygon, координата в запросе взята из первой точки полигона.

Код целиком твой, с правкой Rsa97

[Anzor]

Михаил Ливач, запрос такой:

http://domain.ru/check_coordinate.php?coordinate=41.06000845574921,57.2617701871141

[Rsa97]

Anzor, Не стесняйтесь отлаживать свой код.
Включите вывод ошибок, добавив в начало файла

ini_set('display_errors', '1');
error_reporting(E_ALL);

Выводите через var_dump переменные и смотрите, соответствует ли их содержимое вашим ожиданиям. Если нет, то разбирайтесь почему.

Сейчас, например, странно выглядит отсутствие пробела после запятой в запросе и его наличие в $_GET['coordinate']. Такое может быть только если вы сами добавили этот пробел в запрос. При этом, explode ожидает, что пробел там будет. И, если вы сделаете запрос без этого пробела, то explode не сможет правильно разбить строку.

[Anzor]

Rsa97,

string(38) "41.063013254638385, 57.266932308895065"
<br />
<b>Notice</b>:  Undefined index: coordinates in <b>/home/c/co03498/rdcntnt/public_html/test/check_coordinate.php</b> on line <b>19</b><br />
<br />
<b>Warning</b>:  Invalid argument supplied for foreach() in <b>/home/c/co03498/rdcntnt/public_html/test/check_coordinate.php</b> on line <b>19</b><br />
The coordinate is not inside the polygon

[Anzor]

Rsa97, текущий вид кода такой:

<?php
const LATITUDE = 0;
const LONGITUDE = 1;
var_dump($_GET['coordinate']);
ini_set('display_errors', '1');
error_reporting(E_ALL);
// Read the content of the json file
$json_file = file_get_contents('polygon.json');

// Decode the json content into an array of coordinates
$polygon = json_decode($json_file, true);

// Check if the request contains a coordinate to be verified
if(isset($_GET['coordinate'])){
    $coordinate = explode(', ', $_GET['coordinate']); // запятая с пробелом
    $is_inside = false;

    // Loop through the array of coordinates
    foreach ($polygon['geometry']['coordinates'] as $point){

        // Check if the coordinate exists in the polygon
        // можно проще: if ($point == $coordinate) {
        if($point[LATITUDE] == $coordinate[LATITUDE] && $point[LONGITUDE] == $coordinate[LONGITUDE]){
            $is_inside = true;
            break;
        }
    }

    // Output a positive or negative message
    if($is_inside){
        echo "The coordinate is inside the polygon";
    }else{
        echo "The coordinate is not inside the polygon";
    }
}

[Михаил Ливач]

Anzor, пришлось всё же запустить код. Оказывается, в файле с json ещё уровень вложенности.
Ниже исправленный код, но хочу высказать замечание:
как понимаю, json описывает набор многоугольников ( на текущий момент один), каждый многоугольник задан координатами вершин. Текущий код только проверяет совпадение заданной точки с одной из вершин многоугольника. Как проверить попадание точки внутрь многоугольника - не помню совершенно.

Теперь к коду. Арифметика с плавающей точкой не допускает точных сравнений, так как само представление числа - неточное. Выхода есть два:
1) умножение величин на кратное 10 число, чтобы перейти к целочисленной арифметике. Так поступают, например, с деньгами, когда соответствующего типа в языке нет ( в PHP нет );
2) проверка, что разница ниже некоторой граничной величины, обычно называемой "эпсилон". Классический подход.

Я выбрал второй вариант.

<?php
const LATITUDE = 0;
const LONGITUDE = 1;
const EPSILON = 0.000000000001;

function isEquals( $a, $b ) {
    return abs( $a - $b ) < EPSILON;
}
// Read the content of the json file
$json_file = file_get_contents('polygon.json');

// Decode the json content into an array of coordinates
$polygon = json_decode($json_file, true);

// Check if the request contains a coordinate to be verified
if(isset($_GET['coordinate'])){
    $coordinate = explode(', ', $_GET['coordinate']); // запятая с пробелом
    $is_inside = false;

    // Loop through the array of coordinates
    foreach ($polygon['geometry']['coordinates'] as $polygonPoints){
        foreach ( $polygonPoints as $point ) {
            // Check if the coordinate exists in the polygon
            if( isEquals( $point[LATITUDE], $coordinate[LATITUDE] )
                && isEquals( $point[LONGITUDE], $coordinate[LONGITUDE] )
            ) {
                $is_inside = true;
                break;
            }
        }
    }

    // Output a positive or negative message
    if($is_inside){
        echo "The coordinate is inside the polygon";
    }else{
        echo "The coordinate is not inside the polygon";
    }
}

[Михаил Ливач]

кстати, спасибо, что напомнили про работу с плавающими числами