Forum PHP.pl > [PHP] Księga gości OOP

Pomoc - Szukaj - Użytkownicy - Kalendarz

malymakar

22.07.2010, 21:38:44

[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.

slightyboy

22.07.2010, 22:10:07

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.

malymakar

22.07.2010, 22:29:51

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.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.