check_the_ping()

Complexity

Conditions	8
Paths	10

Size

Total Lines	52
Code Lines	30

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function check_the_ping($request_id)
{
    if (empty($request_id)) {
        return "Request ID not found!";
    }

    $url = "https://check-host.net/check-result/" . $request_id;

    $ch = curl_init();
    curl_setopt_array($ch, [
        CURLOPT_URL => $url,
        CURLOPT_RETURNTRANSFER => true,
        CURLOPT_HTTPHEADER => ["Accept: application/json"],
    ]);

    $response = curl_exec($ch);

    if (curl_errno($ch)) {
        echo "cURL error: " . curl_error($ch);
        return null;
    }

    $decoded_response = json_decode($response, true);

It seems like $response can also be of type true; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

    $pings = [];
    $nodes = ["ir1.node.check-host.net", "ir3.node.check-host.net", "ir4.node.check-host.net"];

    foreach ($nodes as $node) {
        if (empty($decoded_response[$node])) {
            continue;
        }

        $count = 0;
        $ping_sum = 0;
        foreach ($decoded_response[$node][0] as $value) {
            if (@$value[0] == "OK") {
                $count++;
                $ping_sum += $value[1];
            }
        }

        if ($count !== 0) {
            $ping_avg = (@$ping_sum / $count) * 1000;
            $pings[$node] = $ping_avg;
        }
    }

    $json_pings = json_encode($pings, JSON_PRETTY_PRINT);

    curl_close($ch);

    return $json_pings;
}