[PHP]Mój pierwszy skrypt, proszę o sugestie Opcje

[robert-s]

No dobra, dzisiaj zacząłem nauke PHP i udało mi się po całym dniu wypocić jeden skrypt. Mógłby ktoś mnie zmiażdżyć i wytknąć wszystkie błędy, żebym nie popełniał ich na przyszłość? Generalnie robi to co chcę, czyli odczytuje, wysyła i zapisuje komentarze do pliku csv, jednak może coś da się zrobić w zgrabniejszy sposób? Mam jedną stronę na CMS nieobsługującym komentarzy i chcę skrypt wstawić na jedną podstronę tylko, żeby user mógł sobie wyrazić opinię.

[PHP] pobierz, plaintext 
<?php
 
$wczytaj = fopen("komentarze.csv", "r");	
while ($tablica = fgetcsv($wczytaj, 1000, ","))
{
echo "<div id='data'>$tablica[0]</div>";
echo "<div id='imie'>$tablica[1] pisze:</div>";
echo "<div id='tresc'>$tablica[2]</div>";
}
fclose ($wczytaj);		
?>
 
<form id="commentform" method="post" action="wyslij.php">
 
<div>
<label for="podpis">Podpis</label>
<input id="podpis" type="text" name="podpis">
</div>
 
<div>
<label for="komentarz">Komentarz</label>
<textarea id="komentarz" type="text" name="komentarz"></textarea>
</div>
 
<div>
<input id="przycisk" type="submit" value="Opublikuj komentarz" name="przycisk">
</div>
 
</form>
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
<?php
 
$podpis = $_POST['podpis'];
$komentarz = $_POST['komentarz'];
 
echo "$podpis";
echo "$komentarz";
echo date('j.m.Y');
 
$czas = date('j.m.Y');
 
$list = array($czas, $podpis, $komentarz);
 
$wczytaj = fopen('komentarze.csv', 'a');
 
fputcsv($wczytaj, $list);
 
fclose($wczytaj);
?>
[PHP] pobierz, plaintext 

Teraz planuję jeszcze dorobić małe zabezpieczenie w postaci pytania i zliczanie znaków. Gdy będzie mniej niż ileś, komentarz się nie zapisze. Aha. I powrót na poprzednią stronę po wysłaniu. Dałoby radę dać jakąś podpowiedż jakie funkcje mi w tym pomogą?

Ten post edytował robert-s 24.05.2012, 23:45:12

[Crozin]

1. Kod wstawiaj w BBCode PHP: [ php ] ... [ / php ], nie cytaty.
2. Jeżeli jako drugi argument fgetcsv ustawiony na 1000, to wypadałoby albo upewnić się przy dodawaniu nowego wpisu, że jego łączna długość w pliku nie przekroczy tych 1000 znaków, albo ustawić ten argument za zero by funkcja zawsze odczytywała cały wiersz.
3. Nigdy nie używaj jakiś magicznych, nic nie znaczących stałych w kodzie (patrz: $tablica[0], $tablica[1], $tablica[2]). W pętli wyświetlającej dane, od razu przypisz sobie poszczególne elementy tablicy do normalnych zmiennych (list).

CSV to fatalny format do przechowywania komentarzy. Znacznie lepszym rozwiązaniem byłoby wykorzystanie np. XML-a, który nie będzie miał problemów, gdy ktoś wrzuci w treść komentarza przecinek (znak separatora) albo nową linię.