Klasa do sprawdzania statusu serwera Opcje

[Dominator]

Witajcie
Napisałem sobie klasę do sprawdzania statusu serwera (włączony/wyłączony). Możecie go ocenić? (pod względem czy jest napisany poprawnie).

[PHP] pobierz, plaintext 
<?php
 
Class Status
{
	public function Get($ip, $port)
	{
		global $msg;
		$this->ip   = $ip;
		$this->port = $port;
		$this->msg  = $msg;
		$connection = fsockopen($this->ip, $this->port, $errorNo, $errorStr, 3);
		if($connection)
		{
			$this->msg = "Server is UP";
		}
		else
		{
			$this->msg = "Server is DOWN";
		}
	}
	public function Display()
	{
		echo $this->msg;
	}
}
 
$pub = new Status();
$pub->Get("25.135.219.119","25565");
$pub->Display();
 
?>
[PHP] pobierz, plaintext 

[rocktech.pl]

Witam.

1. Nazwa klasy jest zbyt ogólna (czy używając tej klasy zobaczę obciążenie procesora na serwerze?)
2. Metoda Set() (ale co ..), jak już to użyj setterów setIP setPort
3. Connect() owszem można i tak ale niech Connect() wykonuje tylko operacje łączenia
4. Własność msg ma się nijak + - i, wyświetlaj (plus,OK,COOL,NICE) lub (minus,FAIL,BAD,WRONG) w zależności do statusu tu jest to wręcz zakazane

[PHP] pobierz, plaintext 
<?php
 
class ServerConnectionStatus
{
 
    public $ip = null;
    public $port = null;
    public $active = false;
 
    public function __construct($ip, $port = 80)
    {
 
        $this->ip = $ip;
        $this->port = $port;
    }
 
    public function check()
    {
        $con = @fsockopen($this->ip, $this->port, $errorNo, $errorStr, 3);
        if ( is_resource($con) ) {
            $this->active = true;
        } else {
            $this->active = false;
            throw new RuntimeException($errorStr);
        }
    }
 
    public function isActive()
    {
        return $this->active;
    }
 
}
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$serverCStatus = new ServerConnectionStatus('localhost', 80);
try {
    $serverCStatus->check();
    echo $serverCStatus->isActive() ? '+' : '-';
} catch ( Exception $e ) {
    echo $e->getMessage();
}
[PHP] pobierz, plaintext