[PHP] Księga gości OOP

[PHP] Księga gości OOP, Proszę o analizę i błędy

malymakar Zobacz profil	22.07.2010, 21:38:44 Post #1
Grupa: Zarejestrowani Postów: 14 Pomógł: 0 Dołączył: 5.07.2009 Ostrzeżenie: (0%)	[PHP] pobierz, plaintext <?php class Ksiega { public function zapis() { if(!empty($_POST['nick']) && !empty($_POST['tresc']) && !empty($_POST['email'])) { if(preg_match('@^[a-zA-Z1-9]{2,30}\@[a-zA-Z1-9]{2,10}.[a-z]{2,3}$@',$_POST['email'])) { $plik = "ksiegagosci.txt"; $bufor = file_get_contents('ksiegagosci.txt'); $f = fopen($plik, "w"); $tresc = date('d M y H:i:s')." ".$_POST['nick']." napisał: ".$_POST['tresc']."<br/>"; fputs($f, $tresc); fputs($f, $bufor); fclose($f); $_SESSION['koment']=$_POST['hash']; } else echo "Nie poprawny adres email!<br/><br/>"; } else echo "Wypełnij poprawnie wszystkie pola!<br/><br/>"; } public function odczyt() { $plik = "ksiegagosci.txt"; $hak = fopen($plik, "r"); $tresc = fread($hak, filesize($plik)); echo $tresc; fclose($hak); } public function generateHash() { return md5(time() * rand()); } } ?> [PHP] pobierz, plaintext [HTML] pobierz, plaintext <?php session_start(); include("klasa_ksiega.php"); $ksiega = new Ksiega; ?> <html> <head> <link rel="stylesheet" href="style.css" type="text/css" /> </head> <body> <div id="formularz"> <table> <form action="ksiegagosci.php" method="post"> <tr> <td><label>Nick: </label></td> <td><input type="text" name="nick" size="10" maxlength="15"/></td> </tr> <tr> <td><label>Email: </label></td> <td><input type="text" name="email" size="15" maxlength="20"/></td> </tr> <tr> <td><label>Tresc: </label></td> <td><textarea name="tresc" size="30" maxlength="64" rows="3" cols="50">Tresc komentarza</textarea></td> <input type="hidden" id="hash" name="hash" value="<?=$ksiega->generateHash();?>"> </tr> <tr> <td id="wyslij" colspan="2"><input type="submit" value="Wyslij"/></td> </tr> </form> </table> </div> <?php if($_SESSION['koment']==$_POST['hash']) { $ksiega->odczyt();} else { $ksiega->zapis(); $ksiega->odczyt(); } ?> </body> </html> [HTML] pobierz, plaintext Witam, to jest mój pierwszy projekt na obiektach w php i chciałbym prosić o analizę oraz błędy jakie popełniłem przy klasie. Znajomość php jest u mnie na poziomie przedszkola, a z programowaniem obiektowym jest jeszcze słabiej, ale od czegoś trzeba zaczynać, dlatego proszę o jakieś sugestie. Co do samej księgi gości to zdaję sobie sprawę, że jest kiepska, bo niema blokady pliku przy zapisie, blokady antyfloodowej, itp., ale to tylko na potrzeby nauki pracy na klasach. ps. jeżeli chodzi o działanie, to działa bez zarzutu.

Odpowiedzi

slightyboy Zobacz profil	22.07.2010, 22:10:07 Post #2
Grupa: Zarejestrowani Postów: 52 Pomógł: 12 Dołączył: 12.07.2007 Ostrzeżenie: (0%)	Póki co jedynie opakowałeś proceduralny kod klasą i jej metodami, a nie o tym w OOP chodzi. Jak powinno to wyglądać? przyjęło się, iż m.in. nazwy klas i metod pisane są w języku angielskim, metoda "zapis" powinna zostać zastąpiona przez metodę np. "addEntry", która dodaje wpis do księgi gości - przekazywane do niej parametry to dane o wpisie (odwołując się do Twojego kodu, kolejno: $nick, $content, $email - wówczas przy wywołaniu przekazujesz dane z tablicy POST), jeśli ma to być OOP, skorzystaj z wyjątków ("throw Exception...", a następnie "try{ } catch.."), które będą "wyrzucane" w momencie kiedy pojawi się błąd (np. walidacji), w metodach zwracaj wartości, nie używaj echo, błędnie dopisujesz komentarze do pliku (skorzystaj z fopen [w odpowiednim trybie] oraz fwrite), co więcej zamiast wrzucać już sformatowany tekst (ze znacznikami HTML) lepiej stwórz coś na kształt bazy danych opartej na pliku (kolumny wierszy oddzielane np. znakiem "\|" - wówczas pobierasz wiersze pliku, rozbijasz każdy z nich funkcją explode i otrzymujesz wszystkie wpisy w postaci tablicy, którą możesz bezproblemowo zarządzać), jeśli już decydujesz się na bezpośrednie wyświetlanie zawartości pliku skorzystaj z file_get_contents, a nie fread; To tylko kilka z błędów (tych najbardziej rażących), które można znaleźć w Twoim kodzie. Musisz skupić się na poznaniu OOP w teorii, a dopiero potem stopniowo próbować ową wiedzę przekładać na praktykę. Warto też przeanalizować kody innych osób, spróbować je zrozumieć, a może nawet poszukać w nich błędów. (IMG:style_emoticons/default/smile.gif)