Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

> [PHP]skrypt w OOP, Czy dobrze??
marcio
post 8.03.2008, 19:56:46
Post #1





Grupa: Zarejestrowani
Postów: 2 291
Pomógł: 156
Dołączył: 23.09.2007
Skąd: ITALY-MILAN

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

Witam tak jak kiedys napisalem chce sie nauczyc OOP i robie sa prosta ksiege

OTO KOD:
[PHP] pobierz, plaintext 
  1. <?php
  2.  
  3. ob_start();
  4. include('config.php');
  5.  
  6. class Ksiega {
  7.  
  8.   public $data, $ip, $host, $nick, $message;
  9.   private $pasy, $action, $method, $db;
  10.  
  11. public function __construct($nick, $message, $action, $method) {
  12.  
  13.   global $pasy;
  14.  
  15.   $this -> action = $action;
  16.   $this -> method = $method;
  17.   $this -> data = date('j/n/Y H:i:s');
  18.   $this -> nick = htmlspecialchars(addslashes($nick));
  19.   $this -> message = htmlspecialchars(addslashes($message));
  20.   $this -> ip = $_SERVER['REMOTE_ADDR'];
  21.   $this -> host = gethostbyaddr($_SERVER['REMOTE_ADDR']);
  22.   $this -> pasy = $pasy;
  23. }
  24.  
  25.  
  26. public function db_connect() {
  27.  
  28.   if($this -> db = mysql_connect($this->pasy['host'], $this->pasy['login'], $this->pasy['haslo'])) {
  29.   mysql_select_db($this->pasy['baza']);
  30.   return true;
  31.   }
  32.  
  33.   else return false;
  34.  
  35.   }
  36.  
  37.  
  38. public function db_close() {
  39.  
  40.   mysql_close($this -> db);
  41.  
  42.   }
  43.  
  44.  
  45. public function new_query($query) {
  46.  
  47.   $this -> db_connect();
  48.   if($result = mysql_query($query))
  49.   return $result;
  50.  
  51.   else return die(mysql_error());
  52.   $this -> db_close();
  53.  
  54.   }
  55.  
  56.  
  57. public function show_form() {
  58.  
  59. echo('<html>
  60. <head><meta http-equiv="content-type" charset="utf-8" /><title>GuEsT bY MARCIO</title>
  61. <style type="text/css">
  62. a {color:#FDEDF2}
  63. body {color:#FDEDF2}
  64. </style>
  65. </head>
  66. <body bgcolor="#000000">
  67. <center><img src="gora.jpg" width="1240" height="100" alt="obrazek"/></center>
  68. <table align="center" width="75%">
  69. <form action="'.$this -> action.'" method="'.$this -> method.'">
  70. <tr><td align="center"><input type="text" name="nick" size="71" value="Nick" style="font-style:italic; background-color:#000000; color:#00A54E; border-color: #000000; border: #000000;"/></td></tr>
  71. <tr><td align="center"><textarea name="message" rows="3" cols="70" style="font-style:italic; background-color:#000000; color:#00A54E; border-color: #000000; border: #000000;">Tutaj wpisz wiadomosc</textarea></td></tr>
  72. <tr><td align="center"><input type="submit" name="dodaj" value="Dodaj wpis" style="font-style:italic; background-color:#000000; color:#00A54E; border-color: #000000; border: #000000;" /></td></tr>
  73. </form></table>
  74. </body><html>'); 
  75. }
  76.  
  77.  
  78. public function add_reply() {
  79.  
  80.    $this -> show_form();
  81.  
  82.    if(!empty($_POST['nick']) && !empty($_POST['message']) && isset($_POST['dodaj'])) {
  83.  
  84.    setcookie("Spamer", "Spam", time() + 300);
  85.    $zapytanie = ("insert into ksiega_gosci(nick,message,data,ip,kod) values('$this->nick', '$this->message', '$this->data', '$this->ip', '')");
  86.  
  87. 	if($_COOKIE['Spamer'] != "Spam") {
  88.  
  89. 	$this -> new_query($zapytanie);
  90. 	$host = $_SERVER['HTTP_HOST'];
  91. 	$uri = rtrim(dirname($_SERVER['PHP_SELF']), '/');
  92.  
  93. 	}
  94.  
  95. 	else {
  96.  
  97. 	echo('<script>alert("Spadaj spamerze musisz odczekac 5min :)");</script>');
  98.    
  99. 		}
  100. 	}
  101.  
  102. 	 else echo('<script>alert("Prosze wypelnic formularz");</script>'); 
  103.  
  104. 	 }
  105.  
  106.  
  107. public function show_reply($liczba = 30) {
  108.  
  109. 	$query = ("select * from ksiega_gosci");
  110. 	$wynik = $this->new_query($query);
  111. 	$num = mysql_num_rows($wynik);
  112. 	$ile = ceil($num/30);
  113.  
  114. 	if(!isset($_GET['strona'])) $_GET['strona'] = 0;
  115.  
  116. 	$strona = (int)$_GET['strona'];
  117. 	$id = (int)$_GET['id'];
  118. 	$query1 = ("select * from ksiega_gosci order by id desc limit ".$strona*$liczba.",".$liczba);
  119. 	$wynik1 = $this->new_query($query1);
  120. 	while($tab = mysql_fetch_assoc($wynik1)) {
  121.  
  122. 	echo('<table border="1" width="75%" align="center" style="border: 1px dotted red;"><tr><td width="40%">Dodal: '.stripslashes($tab['nick']).'</td><td width="26%">Dnia:'.$tab['data'].'</td></tr>
  123. <tr><td><b>'.stripslashes($tab['message']).'</b></td></tr></table></form>');
  124.  
  125. 	}
  126.  
  127. 	if(mysql_num_rows($wynik) == 0) {
  128.  
  129. 	echo('<script>alert("Nie ma wpisow");</script>');
  130. 	}
  131.  
  132. 	echo('<Br><center><b>Strona: '.($strona).' z '.($ile).'</b></center>');
  133.  
  134. 	if(is_numeric($strona) && !empty($strona) && $strona > 0) {
  135.  
  136. 	echo('<center><a href="?strona='.($strona-1).'"><img src=\'rev.gif' /></a>  ');
  137.    
  138. 	}
  139.  
  140.  
  141. 	if(is_numeric($strona) && !empty($strona) && $strona < $ile) {
  142.    
  143. 	echo('<a href="?strona='.($strona+1).'"><img src='next.gif' /></a></center>');
  144.  
  145. 	}
  146.  
  147. 	else {
  148.  
  149. 	echo('<center><a href="?strona='.($strona+1).'"><img src='next.gif' /></a></center>');
  150.  
  151. 	  }
  152.  
  153. 	echo('<table align="center" style="border: 1px dotted red;"><tr><td align="center">');
  154.  
  155. 	for($i = 0; $i < $ile; $i++) 
  156. 	echo('<a href="?strona='.$i.'">['.$i.']</a> ');
  157. 	echo('</td></tr></table>');
  158.  
  159.   }
  160. };
  161.  
  162. $ksiega = new Ksiega($_POST['nick'], $_POST['message'], 'new_klasa.php', 'post');
  163. $ksiega -> add_reply();
  164. $ksiega -> show_reply();
  165.  
  166. echo('<Br><div style="background:#FFFFFF;color:#000000;text-align:center;border:2px solid:white;"><code><b>Powered by Marcio. GG(4854727)</b><img src="http://www.gadu-gadu.pl/users/status.asp?id=4854727&styl=1"/><Br><a style="color:#000000" href="http://ekmll.com" TARGET="_blank">My site</a></code></div>');
  167. ?>
[PHP] pobierz, plaintext

NIe chce zebys ktos z was analizowal kod linijka po linicje po kod dziala lecz tak patrzac na oko czy kod jest zgdony z OOP czyli czy nie jest to tak jak wy to nazywacie pseudo OOP jesli tak to bylbym wdzeczny jak ktos by pokazal jak by to mialo wygladac w czystym OOP nie zeby mi pisal na nowo kod ale zeby pokazal np jak powinna wygldac funkcja add_reply() albo ktora z nich

Oprocz tego mam jeszcze kilka pytan bylbym wdzieczny jak by ktos na nie odpowiedzial nie musi odpowiadac na wszystkie

1.DO czego sluzy klasa interface??BO nie zrozumialem
2.Do czego sluzy abstract??
3.Do czego sluzy static??

I potem mam jeszcze jedno pytanie czytalem kurs na temat OOP
Czyli w OOP do obiektow mozna sie odwolywac za pomoca pol i metod jednak gdy zmienne klasowe sa publiczne gdy sa typu private albo protected to tylko za pomoca metod dobrze mowie??
Jednak chcialbym sie zapytac czy ktos z was bedzie tak mily i by mi pokazal roznice pomiedzy odwolywaniem sie do obiektow za pomoca pol i metod smile.gif bo pomieszalo mi sie teraz wszystko i juz nie wiem co jest czym...

Ten post edytował marcio 8.03.2008, 20:00:57


--------------------
Zainteresowania: XML | PHP | MY(SQL)| C# for .NET | PYTHON
http://code.google.com/p/form-builider/
Moj blog
Go to the top of the page
+Quote Post
 
 Start new topic
Odpowiedzi
marcio
post 10.03.2008, 23:16:09
Post #2





Grupa: Zarejestrowani
Postów: 2 291
Pomógł: 156
Dołączył: 23.09.2007
Skąd: ITALY-MILAN

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

W sumie masz racje ale dzieki smile.gif no coz poucze sie jeszcze i zobacze co z tego bedzie mam nadzieje ze wkrotce zrozumie

P.S mam jeszcze jedno male pytanie czy bylbys tak wdzieczny i molgbys mi dac jakies twoje przykladowe klasy co kiedys napisales nie zadlugie i nie skomplikowane sciaglem pager'a Nospor'a ale jest zatrudnu i zadlugi a inne kody w sieci idem moze masz cos ciekawego??

Napisalem to na nowo na 3 klasach + system szablonow jak zwykle nie jest to OOP ale zawsze cos dziala dzialac jednak moze ktos pokazac co by zmienil

Mam jeszcze jedno pytanie probowalem to zrobic na interfejsach zeby nie robic tych wszystkich klas i nie dawac extends i robil np tak
Plik interface.php
[PHP] pobierz, plaintext 
  1. <?php
  2. interface Db {
  3.  
  4. public function db_connect();
  5. public funcion db_close();
  6. public function new_query();
  7. //etc wszystkie metody potrzebne do sql
  8. }
  9.  
  10. interface Html_code {
  11.  
  12. public function html();
  13. public function button();
  14. public function text_input();
  15. //etc wszystkie metody do szablonu html
  16. }
  17. ?>
[PHP] pobierz, plaintext

Potem plik interface.php includowalem do new_klasa.php(glowna klasa) i robilem cos takiego
[PHP] pobierz, plaintext 
  1. <?php
  2. class Guest implements Db, Html_code {
  3. //tu wszystkie metodu pola etc...
  4. }
  5. ?>
[PHP] pobierz, plaintext

I wywalalo jakis blad a nie wiem dlaczego tzn ze za db_connect prawidlo jest sie odwolywac Db::db_connect a nie Guest::db_connect i tak samo z innymi metodami wiec co zle robilem??

Czy piszac to na interfejsach kod bylby krotszy skoro nie musial bym pisac w kazdel klasie metody lecz tylko w klasie w ktorej je potrzebowalem

P.S i jeszcze jedno to co ja robie to jest dziedziczenie
Db
|
Html_code
|
Guest

Jednak dla kadzej klasu musialem pisac wlasne metody jednak stosujac interface wychodzi na wielodziedziczenie
czyli Klasa Guest dziedziczy od Db i Html_code naraz nawet ze wyczytalem ze nie jest to jednak takie wielodziedziczenie jak w C++/Java ale czy za pomoca interface nie bylo by mniej kodu i czy idzie tak zrobic bo wtedy pisze tylko metody gdzie mi sa potrzebne

Tu jest link klasy:http://rafb.net/p/IOORnq87.html


--------------------
Zainteresowania: XML | PHP | MY(SQL)| C# for .NET | PYTHON
http://code.google.com/p/form-builider/
Moj blog
Go to the top of the page
+Quote Post

Posty w temacie
- marcio   [PHP]skrypt w OOP   8.03.2008, 19:56:46
- - Piniek   po pierwsze nie umieszczaj kodu html w klasie i w...   8.03.2008, 19:58:56
- - marcio   P.S co sie stalo z bbcode'em pierwszy raz mi s...   8.03.2008, 20:02:22
- - Jarod   Ehh... global..   8.03.2008, 20:38:25
- - marcio   To moze ktos pokarze dlaczeg jest to tylko skladni...   8.03.2008, 20:50:26
- - Cysiaczek   Obejrzyj kod jakiegoś frameworka, albo zobacz...   8.03.2008, 21:03:59
- - marcio   o0..trudne to jest wiesz mysle ze odrazu nie napis...   8.03.2008, 21:13:18
- - Cysiaczek   Tylko, że tego się nie pisze, tylko generuje. Samo...   8.03.2008, 21:27:27
- - marcio   No ok dzieki ale to mi w niczym nie pomaga bo klas...   8.03.2008, 21:38:22
- - Cysiaczek   Nie da się tego kodu zmienić. należy go napisać zu...   8.03.2008, 22:01:42
- - marcio   @Cysiaczek na ksiazki sknera nie jestem mam w domu...   8.03.2008, 22:27:27
|- - zlw   Cytat(marcio @ 8.03.2008, 22:27:27 ) ...   9.03.2008, 10:28:32
- - marcio   No ok @zlw wiec poczytalem troche kurs'ow i pr...   9.03.2008, 14:28:39
- - Cysiaczek   [PHP] pobierz, plaintext <?phppublic function g...   9.03.2008, 15:09:54
- - marcio   Powiem tak ksiazki o OOP nie mam tzn mam ksiazki d...   9.03.2008, 15:20:52
- - Piniek   dam ci link do artykułu http://kavuka.pl/blog/show...   9.03.2008, 15:40:17
- - Cysiaczek   eheheh. To samo próbuję wytłumaczyć tutaj, w temac...   9.03.2008, 15:46:41
- - marcio   Czy jestem chociaz na taki etapie?? CytatUważam (p...   9.03.2008, 16:31:08
- - Cysiaczek   Masz, ale teraz to już naprawdę marsz do książek K...   9.03.2008, 16:36:40
- - marcio   Czyli ogolnie powinno sie tworzyc klasy dla kazdej...   9.03.2008, 16:52:23
- - Cysiaczek   Bo nie będę Ci przecież wszystkich funkcji pisał. ...   9.03.2008, 17:17:25
- - marcio   Wykombinowalem cos takiego bylbym wdzieczny jak kt...   9.03.2008, 18:29:30
- - l0ud   marcio, ale dlaczego tym kodem robisz właściwie ws...   9.03.2008, 18:46:58
- - marcio   @l0ud a moglbys pokazac mniej wiecej jak ma wyglad...   9.03.2008, 19:01:37
- - Xniver   Zobacz sobie Open Power Template albo Smarty Cyta...   9.03.2008, 23:30:52
- - marcio   OK Xniver wielkie dzieki zaraz lukne co robi ten s...   10.03.2008, 00:03:43
- - Xniver   IMHO gorzej tego zrobić nie mogłeś....   10.03.2008, 00:27:53
- - marcio   Wiem ze tego najlepiej nie napisalem ale czy mozes...   10.03.2008, 00:37:22
- - Xniver   Jeśli chodzi o ten twój poprzedni kod to tam ...   10.03.2008, 00:39:15
- - marcio   O lol az mi siara probowalem dac global ale do glo...   10.03.2008, 00:49:42
- - Xniver   [PHP] pobierz, plaintext <?phppublic function _...   10.03.2008, 01:08:07
- - marcio   no tak to jak zrobic zebym nie mysial uzywac globa...   10.03.2008, 01:10:33
- - Xniver   [PHP] pobierz, plaintext <?phppublic function _...   10.03.2008, 01:12:35
- - marcio   AHa tak patrzac na ten twoj kod zastanawialem sie ...   10.03.2008, 01:23:43
- - Cysiaczek   @marcio - Pokazałem Ci, jak wyglądają klasy księgi...   10.03.2008, 06:56:44
- - marcio   @Cysiaczek staram sie jak moge ale...jakos trudno ...   10.03.2008, 16:24:15
- - Cysiaczek   Nie, nie posiadam e-booków, ani adresów, bo na łam...   10.03.2008, 16:27:37
- - marcio   W sumie masz racje ale dzieki no coz poucze sie j...   10.03.2008, 23:16:09
- - Xniver   Czemu cały czas dziedziczysz/implementujesz i...   11.03.2008, 10:35:04
- - marcio   CytatNie możesz po prostu użyć jej jako zewnętrzne...   11.03.2008, 15:07:57
- - Xniver   Nie chodziło o to: [PHP] pobierz, plaintext ...   11.03.2008, 15:24:24
- - marcio   Cytat(Xniver @ 11.03.2008, 15:24:24 )...   11.03.2008, 16:35:28
- - carbolymer   Widze tu braki w podstawach OOP. Extends (jak sama...   11.03.2008, 21:56:24
- - marcio   Dzieki czyli operator :: mozna powiedziec ze zaste...   11.03.2008, 22:53:38
- - Xniver   CytatDzieki czyli operator :: mozna powiedziec ze ...   11.03.2008, 23:00:25
- - marcio   No troche sie uparlem nie wiem czemu jakos tak w...   11.03.2008, 23:13:27
- - Xniver   Bo $this wywołuje funkcje znajdujące się w ak...   11.03.2008, 23:20:39
- - marcio   No tak ale ta funckja new_query znajduje sie w kla...   11.03.2008, 23:32:43
- - specialplan   marcio, a moze zacznij od: http://pl.wikibooks.or...   12.03.2008, 12:00:28
- - Noctrum   Cysiaczek moge prosic Ciebie o wyjasnienie tej li...   14.03.2008, 18:56:53
- - Cysiaczek   Hm, to jest pseudokod, rozbudowany, ale nadal pseu...   14.03.2008, 19:26:49
- - marcio   Sorki ze nie odpowiadalem ale nie bylo mnie... Dz...   16.03.2008, 17:11:29
- - tomahawk   A powiedzcie mi jedną rzecz... Czemu w OOP używa ...   16.03.2008, 18:23:25
|- - shine   Cytat(tomahawk @ 16.03.2008, 18:23:25...   20.03.2008, 12:22:50
|- - mosqi83   Cytat(tomahawk @ 16.03.2008, 18:23:25...   22.03.2008, 22:44:51
- - marcio   Chyba dla przejzystosci...   16.03.2008, 18:27:29
- - tomahawk   Ale czemu akurat w ten sposób? Możne np. dać twar...   16.03.2008, 18:29:29
|- - Sedziwoj   Cytat(tomahawk @ 16.03.2008, 18:29:29...   16.03.2008, 18:48:04
- - itsme   temat ma duzo wspolnego z OOP ale nie na poziomie ...   16.03.2008, 18:48:02
- - tomahawk   To dlaczego w php jedne funkcję są s twardymi spac...   17.03.2008, 17:21:58
- - l0ud   CytatTo dlaczego w php jedne funkcję są s twardymi...   17.03.2008, 20:44:27
- - Sedziwoj   Cytat(l0ud @ 17.03.2008, 20:44:27 ) C...   17.03.2008, 23:12:51

« Następny starszy · PHP · Następny nowszy »
 

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

 

Tryb wyświetlania: Przełącz na: Standardowy · Przełącz na: Linearny+ · Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 26.06.2025 - 06:24
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn