[PHP]problem z księgą gości Opcje

[maniek30dg]

Witam, mam pytanie jak zrobić żeby wpisy oddzielała pozioma kreska i pod imię była data wpisu z mniejszą czcionką? jeszcze jest jeden poważny problem, po wpisaniu dłuższego tekstu i wysłaniu go wychodzi on poza wymiary strony, proszę o pomoc.
kod:

[PHP] pobierz, plaintext 
<form action="index.php" method="post">
<input name="imie" type="text" value="imie">
<br><input name="email" type="text" value="e-mail">
<br><input name="www" type="text" value="http://">
<br><textarea name="wpis" rows="3" cols="30">wpis</textarea>
<br><input type=submit value=" dodaj! ">
</form>
 
<?
$ksiega = "ksiega.txt";
 
$imie=$_POST["imie"];
$email=$_POST["email"];
$www=$_POST["www"];
$wpis=$_POST["wpis"];
 
 
if (strlen($wpis)>5) {
  $f=fopen($ksiega,"a");
  $rekord="`$imie`$email`$www`$wpis`";
  $rekord=ereg_replace(10," ",$rekord);
  $rekord=ereg_replace(13," ",$rekord);
  fputs($f,"$rekord\n"); fclose($f);
}
 
$wpisy=file($ksiega);
$wpisy=array_reverse($wpisy);
$ile=count($wpisy);
for ($i=0;$i<$ile;$i++) {
  $wpis = explode("`",$wpisy[$i]);
  echo "<p>$wpis[1] - $wpis[2] - $wpis[3]<br>$wpis[4]<p>";
}
?>
[PHP] pobierz, plaintext 

[BigPig]

1) Funkcja ereg_replace jest przestarzała co widać w manualu:
http://www.php.net/manual/en/function.ereg-replace.php

Jej działanie możesz zastąpić funkcją preg_replace albo str_replace.

2) Jeśli chodzi o kreskę to najbardziej prowizorycznie można tak to zrobić, że mając ten kod:

[PHP] pobierz, plaintext 
$wpisy=file($ksiega);
$wpisy=array_reverse($wpisy);
$ile=count($wpisy);
for ($i=0;$i<$ile;$i++) {
$wpis = explode("`",$wpisy[$i]);
echo "<p>$wpis[1] - $wpis[2] - $wpis[3]<br>$wpis[4]<p>";
}
?>
[PHP] pobierz, plaintext 

Dodajemy np. zmienną $kreska = "_______________________"; i dodajesz ją np. tu:

[PHP] pobierz, plaintext 
 echo "<p>$wpis[1] - $wpis[2] - $wpis[3]<br>$wpis[4]<p> $kreska";
[PHP] pobierz, plaintext 

Z tego co pamiętam to <p> narzuca nową linię, więc powinno być ok. Jeśli ma to wyglądać o wiele lepiej, to trzeba już się pobawić stylami CSS.

3) Jeśli chodzi o date to na początku sprawdź funkcję "date" i zrób mniej więcej tak:

[PHP] pobierz, plaintext 
$imie=$_POST["imie"];
$email=$_POST["email"];
$www=$_POST["www"];
$wpis=$_POST["wpis"];
$data = date()//w date uzupełniasz nawias tak jak chcesz(oczywiście zgodnie ze standardem, poczytaj więcej w manualu).
 
if (strlen($wpis)>5) {
$f=fopen($ksiega,"a");
$rekord="`$imie`$email`$www`$wpis`$data`";
$rekord=ereg_replace(10," ",$rekord);
$rekord=ereg_replace(13," ",$rekord);
fputs($f,"$rekord\n"); fclose($f);
}
[PHP] pobierz, plaintext 

Potem, gdy chcesz wyświetlić date to normalnie tu ją wypisujesz oznaczająć jako $wpis[5].(nie analizowałem dokładnie kodu, więc może być to inny element tablicy, ogólnie tak to ma być)

[PHP] pobierz, plaintext 
 echo "<p>$wpis[1] - $wpis[2] - $wpis[3]<br>$wpis[4]<p> $kreska";
[PHP] pobierz, plaintext 

4) Jeśli chodzi o ostatni problem, z długością tekstu. Tutaj też byś się musiał pobawić ze stylami CSS, żeby np. wpakować to co wyrzuca nam pętla for w trakcie odczytywania pliku w DIV'y, a potem poprzez CSS jakoś ładnie to ułożyć w całość.


Na koniec takie trzy rady.
a) Widzę, że raczej na pewno uczysz się PHP z jakiejś książki. To dobrze, ale pamiętaj o tym, że jak książka jest stara, to część przykładów też powinna być inaczej przedstawiona. Np. jak tu widać, autor użył funkcji ereg_replace, której już nie powinno się używać. Bądź wyczulony na takie rzeczy.
b )

[PHP] pobierz, plaintext 
$imie=$_POST["imie"];
$email=$_POST["email"];
$www=$_POST["www"];
$wpis=$_POST["wpis"];
[PHP] pobierz, plaintext 

Nie musisz tak koniecznie obsługiwać zmiennych formularza. Równie dobrze od razu możesz podawać dane poprzez POST bez wcześniejszego deklarowania zmiennych o tych samych nazwach. Kiedyś miałem podobny nawyk co Ty, było to spowodowane faktem, że autor pewnej książki, robił tak samo (IMG:style_emoticons/default/smile.gif) Praktycznie to raczej nic nie dawało.
c)Kod jest o wiele bardziej czytelny, gdy zamiast układać klamry w ten sposób:

[PHP] pobierz, plaintext 
if (strlen($wpis)>5) {
$f=fopen($ksiega,"a");
$rekord="`$imie`$email`$www`$wpis`";
$rekord=ereg_replace(10," ",$rekord);
$rekord=ereg_replace(13," ",$rekord);
fputs($f,"$rekord\n"); fclose($f);
}
[PHP] pobierz, plaintext 

Układamy tak:

[PHP] pobierz, plaintext 
if (strlen($wpis)>5) 
{
   $f=fopen($ksiega,"a");
   $rekord="`$imie`$email`$www`$wpis`";
   $rekord=ereg_replace(10," ",$rekord);
   $rekord=ereg_replace(13," ",$rekord);
   fputs($f,"$rekord\n"); fclose($f);
}
[PHP] pobierz, plaintext 

Jakie to ma znaczenie? Takie, gdy się przegląda kod, który jest bardziej skomplikowany to nie musimy zaznaczać klamer by zobaczyć, które ze sobą są połączone. Wszystko widać na pierwszy rzut oka. Zdecydowanie łatwiej jest się połapać w takim czymś:

[PHP] pobierz, plaintext 
if (strlen($wpis)>5) 
{
   $f=fopen($ksiega,"a");
   $rekord="`$imie`$email`$www`$wpis`";
   $rekord=ereg_replace(10," ",$rekord);
   $rekord=ereg_replace(13," ",$rekord);
   fputs($f,"$rekord\n"); fclose($f);
   if()
   {
      switch()
      {
       }
    }
}
[PHP] pobierz, plaintext 

Niż w tym:

[PHP] pobierz, plaintext 
if (strlen($wpis)>5) 
{
   $f=fopen($ksiega,"a");
   $rekord="`$imie`$email`$www`$wpis`";
   $rekord=ereg_replace(10," ",$rekord);
   $rekord=ereg_replace(13," ",$rekord);
   fputs($f,"$rekord\n"); fclose($f);
   if(){ 
      switch(){      
       }
    }
}
[PHP] pobierz, plaintext 

Te ostatnie trzy punkty to takie moje drobne rady (IMG:style_emoticons/default/smile.gif)

Ten post edytował BigPig 14.09.2013, 19:22:32