[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

malymakar Zobacz profil	22.07.2010, 22:29:51 Post #2
Grupa: Zarejestrowani Postów: 14 Pomógł: 0 Dołączył: 5.07.2009 Ostrzeżenie: (0%)	Dzieki bardzo za tą obszerną odpowiedź... dała mi ona dużo do myślenia. Prosiłbym o nie zamykanie tematu, ponieważ po dogłębnym przeanalizowaniu wszystkich punktów i zaleceń postaram sie umieścić kolejną wersję tego kodu... Pewnie nie nastąpi to zbyt szybko, ale napewno kiedyś będzie. Ten post edytował malymakar 22.07.2010, 22:30:29