Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> Klasa do sprawdzania statusu serwera
Dominator
post
Post #1





Grupa: Zarejestrowani
Postów: 565
Pomógł: 15
Dołączył: 11.10.2010

Ostrzeżenie: (20%)
X----


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

  1. <?php
  2.  
  3. Class Status
  4. {
  5. public function Get($ip, $port)
  6. {
  7. global $msg;
  8. $this->ip = $ip;
  9. $this->port = $port;
  10. $this->msg = $msg;
  11. $connection = fsockopen($this->ip, $this->port, $errorNo, $errorStr, 3);
  12. if($connection)
  13. {
  14. $this->msg = "Server is UP";
  15. }
  16. else
  17. {
  18. $this->msg = "Server is DOWN";
  19. }
  20. }
  21. public function Display()
  22. {
  23. echo $this->msg;
  24. }
  25. }
  26.  
  27. $pub = new Status();
  28. $pub->Get("25.135.219.119","25565");
  29. $pub->Display();
  30.  
  31. ?>
Go to the top of the page
+Quote Post
 
Start new topic
Odpowiedzi
widmo_91
post
Post #2





Grupa: Zarejestrowani
Postów: 109
Pomógł: 25
Dołączył: 10.08.2009

Ostrzeżenie: (0%)
-----


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
Go to the top of the page
+Quote Post

Posty w temacie


Reply to this topicStart new topic
2 Użytkowników czyta ten temat (2 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Aktualny czas: 4.10.2025 - 21:45