[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

[amii]

Dodam jeszcze taką ciekawostkę kiedy używasz:

[PHP] pobierz, plaintext 
echo "$podpis";
echo "$komentarz";
[PHP] pobierz, plaintext 

masz niewielki narzut spowodowany tym, że parser odczytuje kod z cudzysłowów i szuka znaków specjalnych i zmiennych w środku. Lepiej w takim przypadku dać bez cudzysłowów.

Dla czystych zmiennych tekstowych lepsze są apostrofy. Cudzysłów jest dobry (wygodniejszy) kiedy w środku masz zmienne lub znaki specjalne i tekst.

[PHP] pobierz, plaintext 
echo "Jakiś tam sobie tekst w zmiennej $zmienna a tu jeszcze znak specjalny \n";
[PHP] pobierz, plaintext 

Ten post edytował amii 25.05.2012, 19:44:08