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]

1. Daj @ przed fsockopen bo jak sie nie uda polączenie to przewie Ci skypt
2. Zadeklaruj zmienne ip, port, msg w klasie przed ich użyciem.
3. Zamiast Get bardziej by pasowała nazwa Set
4. Co to za zwyczaj nazywania metod z wielkich liter?
5. Co to za instrukcja global $msg bo nie rozumiem inicjatywy

[Dominator]

1. Daj @ przed fsockopen bo jak sie nie uda polączenie to przewie Ci skypt
2. Zadeklaruj zmienne ip, port, msg w klasie przed ich użyciem.
3. Zamiast Get bardziej by pasowała nazwa Set
4. Co to za zwyczaj nazywania metod z wielkich liter?
5. Co to za instrukcja global $msg bo nie rozumiem inicjatywy

1. Zrobione
2. Czemu? W PHP na wiki jest napisane "Zmienne w PHP nie wymagają deklaracji"
3. Zrobione
4. A nie wiem, z tutoriali pewnie (IMG:style_emoticons/default/biggrin.gif)
5. Gdy nie dam global $msg to będę miał Notice: Undefined variable: msg in bla bla