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.

[Dominator]

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.

2. Zrobione, przemyślałem sobie i doszedłem do wniosku, że masz rację. Dzięki (IMG:style_emoticons/default/wink.gif)
5. Usunąłem global + $this->msg = $msg i działa pięknie, dzięki jeszcze raz! (IMG:style_emoticons/default/biggrin.gif)

Ogólnie rzecz biorąc, napisałem sobie drugi, taki sam skrypt. Według was - który lepszy?
Robiłem benchmark i różnica pomiędzy skryptami wynosi 0,000061.

Wersja 1:

[PHP] pobierz, plaintext 
Class Status
{
	public function Set($ip, $port = 80)
	{
		$this->ip   = $ip;
		$this->port = $port;
		$connection = @fsockopen($this->ip, $this->port, $errorNo, $errorStr, 3);
		if($connection)
		{
			$this->msg = "Server is UP";
		}
		else
		{
			$this->msg = "Server is DOWN";
		}
		fclose($connection);
	}
	public function Display()
	{
		echo $this->msg;
	}
	public static function getmicrotime()
	{
		return array_sum(explode(' ', microtime()));
	}
}
[PHP] pobierz, plaintext 

Wersja 2:

[PHP] pobierz, plaintext 
Class Status
{
	public function __construct($ip, $port = 80)
	{
		$this->Set($ip, $port);
		$this->Connect();
	}
	public function Set($ip, $port)
	{
		$this->ip   = $ip;
		$this->port = $port;
	}
	public function Connect()
	{
		$con = @fsockopen($this->ip, $this->port, $errorNo, $errorStr, 3);
		if($con)
		{
			$this->msg = "+";
		}
		else
		{
			$this->msg = "-";
		}
	}
	public function Display()
	{
		echo $this->msg;
	}
	public static function getmicrotime()
	{
		return array_sum(explode(' ', microtime()));
	}
}
[PHP] pobierz, plaintext 

Jakie macie uwagi?

Ten post edytował Dominator 23.07.2013, 23:32:38