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 

[widmo_91]

2. To że nie wymagają nie znaczy, że nie warto tego robić bo później klasa się rozrośnie jak to zwykle bywa i będziesz szukał w której metodzie została zadeklarowana taka zmienna i co robi a jak zadeklarujesz to wszystkie będziesz miał w jednym miejscu
5. Ale po co Ci ona skoro i tak ją nadpisujesz później:
if($connection)
{
$this->msg = "Server is UP";
}
else
{
$this->msg = "Server is DOWN";
}

Wywal: global $msg; i $this->msg = $msg;

Na początkowym etapie nauki radzę na siłę unikać zmiennych globalnych.