Ayuda con PHP

#9

El primer else no tiene ningún sentido, podrías quitarlo y funcionaría igual

El problema que tienes con esa forma de hacer ping es que estás confundiendo lo que es un ping mediante ICMP que abrir una conexión mediante un socket al puerto 80 de una máquina.

Te resumo más o menos las condiciones que debe reunir una máquina para poder recibir ping según el método:

• ICMP: El firewall debe permitir tráfico ICMP.
• Puerto XX: El firewall debe permitir el tráfico entrante en el puerto XX, y detrás debe haber un servicio respondiendo a las conexiones entrantes.

El ICMP es el tipo de tráfico que se utiliza para medir el ping REAL, mientras que el método de abrir un socket a un puerto sirve para medir el tiempo de respuesta DE UN SERVICIO, con lo cual, si no hay un servicio detrás, NO HAY PING.

El tuyo es el segundo caso, si las máquinas a las que hagas ping no tienen detrás un servicio HTTP no te van a responder, no va a poder realizar la conexión, y esa función te va a devolver siempre -1, aunque sí puedas acceder a dicha máquina.

Voy a intentar explicar más o menos lo que hace el código:

$starttime = microtime(true); # Toma nota de la hora de inicio del script para medición del ping
$file      = @fsockopen ($domain, 80, $errno, $errstr, 10); # Abre un socket en el puerto 80 de la máquina objetivo
$stoptime  = microtime(true); # Toma nota de la hora de fin del script para medición del ping
$status    = 0; #Inicialización de la variable status (aunque podría haber sido FALSE directamente

Seguimos para bingo

if (!$file) $status = -1; # Si no ha podido abrir el socket, el resultado de fsockopen es "false". Si ocurre esto, significa que no se puede conectar con el servidor, y seteamos la variable a -1.
else { # Se ha podido abrir el socket
    fclose($file); # Cerramos el socket
    $status = ($stoptime - $starttime) * 1000; # Calculamos en milisegundos el ping
    $status = floor($status); # Redondeamos el ping para evitar cosas como 5.99999999999ms en lugar de 6ms o 5.99ms
}

Que es un código muy básico, no me jodas xD. Si ese código lo ejecutas desde un servidor propio o como script de línea de comandos, si lo ejecutas como administrador puedes utilizar esto: https://github.com/geerlingguy/Ping

Por último, cómo dejaría yo el código:

function pingDomain($domain, $port = 80){
    $starttime = microtime(true);
    $file      = @fsockopen ($domain, $port, $errno, $errstr, 10);
    $stoptime  = microtime(true);
    $status    = FALSE;

    if($file){
        fclose($file);
        $status = ($stoptime - $starttime) * 1000;
        $status = floor($status);
    }
    return $status;
}

$domain_ping = pingDomain("192.168.90.211");
if ($domain_ping !== FALSE) {
    echo 'Conectado | Ping: ' . $domain_ping . 'ms';
} else {
    echo 'No conectado';
}

#13 En mi script sí lo uso (que para algo haces ping), por eso lo he puesto:

echo 'Conectado | Ping: ' . $status . 'ms';

Es más, voy a corregirlo y ponerlo de otra forma xD.