[php] Księga gości Opcje

[_bezimienny]

Mam pewien problem. Chcę zrobić księgę gości opartą na plikach. Zatem...
Mam bazę danych w pliku baza.txt:

user1|user1@mail.pl|wpis1
user2|user2@mail.pl|wpis2

Nastęnie zrobiłem skrypt czytający poszczególne dane:

[PHP] pobierz, plaintext 
<?php
 
$j=1; // 1, bo są 2 wpisy
for($i=$j; $i>0;$i--) // $i jest inkrementowane, żeby najnowsze były pierwsze
{
$user = file("baza.txt");
$data = $user[$i];
list($nick, $email, $wpis) = explode("|", $data);
echo "$nick<br>";
echo "$email<br>";
echo "$wpis<br>";
}
 
?>
[PHP] pobierz, plaintext 

Chyba jest zrozumiały...

Teraz mój problem. Wyświetla mi tylko 1 wpis. Za pierwszym obiem $i=1, ok, skrypt działa, za drugim razem $i=0 - nie działa (IMG:http://forum.php.pl/style_emoticons/default/sad.gif)

Nie wiem też jaką użyć funkcję, aby zczytało ile jest linijek w pliku :/ Być może lepiej użyć jakąś inną pętlę :?:

Prosze o pomoc!
pozdr

[maverickkk]

No bo z tego co widze to działa w druga stronę (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif)

masz zwiększac a nie zmienszjać liczbe wyświetlanych wiadomości

Zresztą jak dasz $i=0 to przeanalizuj wygląd pętli $i=$j; $i>0;$i--) i co widzisz ?

pętla wykonuj się do puki $i jest większa od 0 to wiadomo oco chodzi

Co do liczby wierszy w pliku:

echo count(file('plik.php'));


... aha i $i jest Dekrementowana nie inkrementowana (IMG:http://forum.php.pl/style_emoticons/default/guitar.gif)

Ten post edytował maverickkk 19.04.2006, 18:10:27

[siemakuba]

zeby nie kombinowac za wiele:

1. wczytaj plik, jak napisal maverickkk, do tablicy

[PHP] pobierz, plaintext 
<?php
 
$baza = file('baza.txt');
 
?>
[PHP] pobierz, plaintext 

2. zaleznie od potrzeby odwroc sobie otrzymana tablice (nie pamietam w jakiej kolejnosci bedziesz mial dane wczytane przez file(), czy od konca czy poczatku pliku - sprawdz)

[PHP] pobierz, plaintext 
<?php
 
$baza = array_reverse($baza);
 
?>
[PHP] pobierz, plaintext 

3. dalej juz iteracja tablicy i wyswietlenie danych

[PHP] pobierz, plaintext 
<?php
 
 
for ($i = 0, $n=count($baza); $i  < $n; $i++)
{
// list($nick, $email, $wpis) = explode('|', $baza[$i]);
// itd.
}
 
?>
[PHP] pobierz, plaintext 

co musisz wiedziec?
file" title="Zobacz w manualu php" target="_manual, array_reverse" title="Zobacz w manualu php" target="_manual

dobrze by bylo sprawdzic na poczatku czy plik zostal wczytany i czy masz potrzebne dane, ale z tym z pewnoscia sobie poradzisz :)

pozdr.

[_bezimienny]

Wielkie dzięki, działa!
Na teraz wygląda od tak:

[PHP] pobierz, plaintext 
<?php
 
$filename = "baza.txt";
if(file_exists($filename)){
$baza = file("baza.txt");
$baza = array_reverse($baza);
for ($i = 0, $n=count($baza); $i  < $n; $i++)
{
list($nick, $email, $wpis) = explode('|', $baza[$i]);
echo "$nick<br>";
echo "$email<br>";
echo "$wpis<br><br>";
}
}
else { echo "Baza nie istnieje!"; }
 
?>
[PHP] pobierz, plaintext 

Na początku sprawdzam tylko czy plik istnieje. Dlaczego mam sprawdzać czy jest pusty albo czy się wczytał? Jest taka możliwość, żeby się nie wczytał?

.. aha i $i jest Dekrementowana nie inkrementowana

Tak wiem, pomyliłem się (IMG:http://forum.php.pl/style_emoticons/default/worriedsmiley.gif) (IMG:http://forum.php.pl/style_emoticons/default/rolleyes.gif)

Ten post edytował _bezimienny 19.04.2006, 19:39:35

[maverickkk]

"Tak wiem, pomyliłem się"
tak wiem poprawiłem cie (IMG:http://forum.php.pl/style_emoticons/default/snitch.gif)

"Dlaczego mam sprawdzać czy jest pusty"

Nigdzie niema warunku na sprawdzenie czy jest pusty? count($baza) w pętli służy do zliczenia ilości wierszy.

"albo czy się wczytał" to nie atari że coś się wczytuje. Jest plik to z niego korzystasz. nie ma to nie korzystasz.

[siemakuba]

dobrze by bylo sprawdzic na poczatku czy plik zostal wczytany i czy masz potrzebne dane

skrót myslowy - zostal wczytany == masz potrzebne dane


_bezimienny: mea culpa, nie zauwazylem file_exists na początku.

"albo czy się wczytał" to nie atari że coś się wczytuje. Jest plik to z niego korzystasz. nie ma to nie korzystasz.

drogi maverickkk, niestety mylisz się (bynajmniej w pewnych warunkach) :)

Kod

plik jest taki:
-rw-r-----    1 kuba     kuba            0 Apr 19 21:29 testfile.txt

[PHP] pobierz, plaintext 
<?php
 
$e = file_exists('testfile.txt');
var_dump($e);   
 
// zwraca true, znakiem to ze "Jest plik to z niego korzystasz"
 
 
$f = file('testfile.txt');
var_dump($f);   
 
// niestety, zwraca false, i do tego jeszcze:
// Warning: file(testfile.txt): failed to open stream: Permission denied
 
?>
[PHP] pobierz, plaintext 

to chyba dowodzi ze moze sie zdarzyc, ze jest plik, ale wcale z niego nie korzystasz :P

pozdr.

[_bezimienny]

Chciałem dorobić do tej księgi gości formularz dodawania wpisów, ale jakoś mi to nie wychodzi. Nie wiem co mam wpisać w <form action= > oraz jaką metodą, POST czy GET. Następna sprawa to jak zrobić, żeby dane, które wprowadzi użytkownik zostały przekazane jakimś zmiennym.
Z zapisaniem tych danych do pliku dałbym sobie już rade. Any ideas?
pozdr

[matrach]

$row masz zapisać do pliku

[PHP] pobierz, plaintext 
<?php
if (!isset($_POST['submit']))//jesli nie wysłane
{
//formularz
echo '
<form action="plik.php" method="POST">
User: <input type="text" name="user" value="" /><br />
E-mail: <input type="text" name="mail" value="" /><br />
Treść:<br />
<textarea name="tresc"></textarea><br />
<input type="submit" name="submit" value="Wyślij" />
</form>';
}
else
{
//user - user email - mail tresc - tresc posta
//zamiana znaczników
$user = htmlSpecialChars($_POST['user']);
$email = htmlSpecialChars($_POST['mail']);
//usuwanie nowych linii
$tresc = str_replace("\r\n", "<br />", htmlSpecialChars($_POST['tresc']));
//zmienna do zapisania w pliku
$row = $user . '|' . $email . '|' . $tresc . "\n";
 
 
}
?>
[PHP] pobierz, plaintext 

User - Nazwa usera podana w polu
Mail - E-mail podany w polu
Tresc - Tresc posta podany w textarea

Ten post edytował matrach 21.04.2006, 18:33:29

[_bezimienny]

Czy wtedy plik.php miałby wyglądać tak?

[PHP] pobierz, plaintext 
<?php
$filename = "baza.txt";
$fd = fopen($filename, "a");
if($fd == null){ echo "Blad: nie mozna otorzyc pliku: $filename";
return; }
fwrite($fd, $row);
fclose($fd);
?>
[PHP] pobierz, plaintext 

[matrach]

"Plik.php" cały odpowiada za zapisanie (formularz z zapisaniem).
Powinien wyglądać tak

[PHP] pobierz, plaintext 
<?php
 
/*Tamto poprzednie co dałem
...
i i teraz nie zamykaj ostatniego nawiasu z else*/
$filename = "baza.txt";
$fd = fopen($filename, "a") or die("Blad: nie mozna otorzyc pliku: $filename");
fwrite($fd, $row);
fclose($fd);
}
 
?>
[PHP] pobierz, plaintext 

[_bezimienny]

Tak jak podałeś to coś nie chciało działać. Zrobiłem tak:
Plik ksiega.php...

[PHP] pobierz, plaintext 
<?php
if (!isset($_POST['submit']))//jesli nie wysłane
{
echo '
<form action="plik.php" method="POST">
User: <input type="text" name="user" value="" /><br />
E-mail: <input type="text" name="mail" value="" /><br />
Treść:<br />
<textarea name="tresc"></textarea><br />
<input type="submit" name="submit" value="Wyślij" />
</form>';
}
else
{
 $user = $_POST['user'];
$email = $_POST['mail'];
$tresc = $_POST['tresc']; 
}
?>
[PHP] pobierz, plaintext 

I plik.php...

[PHP] pobierz, plaintext 
<?php
$filename = "baza.txt";
$fd = fopen($filename, "a");
if($fd == null){ echo "Blad: nie mozna otorzyc pliku: $filename";
return; }
fwrite($fd, $_POST['user'] . '|' . $_POST['mail'] . '|' . $_POST['tresc'] . "\n");
fclose($fd);
?>
[PHP] pobierz, plaintext 

Teraz pięknie śmiga (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) THX!
Jak czegoś nie będę wiedział, to już wiem gdzie mam się pytać (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
Mam jeszcze jedno pytanie, jak sprawdzić czy zmienna $email na prawdę zawiera mail? Jest jakaś funkcja żeby szukała @ :?:

Ten post edytował _bezimienny 22.04.2006, 17:29:01

[matrach]

Jest jakaś funkcja żeby szukała @ (IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)

Phpedia.pl - weryfikacja maila
A odnośnie skryptu, to użytkownik jest nie obliczalny
np
- jeśli zrobi nową linię w texarea
- wpisze |

to ci sie wszystko rozwali.
dlatego powinieneś użyć czegoś takiego

[PHP] pobierz, plaintext 
<?php
 
$zam['|'] = '|';
$zam["\r\n"] = "<br />";//Nowa linia zalerzy od systemu
$zam["\n"] = "<br />";
//wyszukuje w stringu klucza i zamienia na wartośc tablicy
$_POST['user'] = strtr($_POST['user'], $zam);
$_POST['mail'] = strtr($_POST['mail'], $zam);
$_POST['tresc'] = strtr($_POST['tresc'], $zam);
 
?>
[PHP] pobierz, plaintext 

chociażby ze względu bezpieczeństwa.
Aby nie rozwalić układu/nie pozwolić na znaczniki użyj htmlspecialchars(), choć jeśli nie chcesz nie będe cie namawiał.
funkcja strtr()
@down spróbuj jeszcze htmlentities()
---EDIT---
Nie zauważyłem że skrypt do wyświetlania poostów na tym forum zamienia entity na zwykłe znaki/nie zamienia '&' na '&#38;'

Ten post edytował matrach 22.04.2006, 19:28:39

[_bezimienny]

z htmlspecialchars() mi nie działało, ale jeszcze nad tym popracuję i dam znać co z tego wyszło

--EDIT--

Dzięki, skorzystałem z tego i teraz nie ma problemu.

[PHP] pobierz, plaintext 
<?php
zam['|'] = " ";
$zam["\r\n"] = " ";//Nowa linia zalerzy od systemu
$zam["\n"] = " ";
//wyszukuje w stringu klucza i zamienia na wartośc tablicy
$_POST['user'] = strtr($_POST['user'], $zam);
$_POST['mail'] = strtr($_POST['mail'], $zam);
$_POST['tresc'] = strtr($_POST['tresc'], $zam);
?>
[PHP] pobierz, plaintext 

Ten post edytował _bezimienny 24.04.2006, 16:22:37

[Maveilthain]

Witam.

Mam problem odnośnie księgi gości i update'u.
Wykorzystałem kod z książki "php 101 praktycznych skryptów" przerabiając go na Update do stronki aktualności. (tylko członkowie strony znają link do updatowania)

Struktura jest następująca:

- add.temp - formularz dodawania
- index.php - plik inkluduje 'main.temp'
- main.temp - cały wygląd stronki, który inkluduje na górze strony 'add.temp'a potem 'update.php'
- update.php - plik otwierający i wyświetlający wpisy
- update.txt - baza wpisów.

Wszystko działa nawet dobrze, za wyjątkiem odświerzania, bo gdy ktoś sobie wypełni formularz i klinknie na 'wyślij' to widzi nowy swój wpis, natomiast gdy klinkinie na 'odświerz' w swojej przeglądarce to kopiuje mu się ostatni wpis i jeżeli będzie tysiące razy klikał to powstanie tysiące takich samych wpisów - użyłem CHMODów i nic
może jakieś ?flock()? nie wiem jak zmodyfikować pomocy.
Oto kody:

add.temp

[HTML] pobierz, plaintext 
<STYLE TYPE="text/css">
    table {vertical-align: MIDDLE;}
  </STYLE>
 
<FORM   ACTION="../update/index.php"
        METHOD="post">
<CENTER><TABLE  WIDTH="556" BORDER="1" CELLPADDING="0" CELLSPACING="0">
<TR>
    <TD>News:
    </TD>
    <TD>
        <TEXTAREA style="overflow:hidden;" NAME="contents" ROWS="5" COLS="85"></TEXTAREA>
    </TD>
</TR>
<TR>
    <TD COLSPAN="2"><P ALIGN="RIGHT">
    <INPUT TYPE="text" NAME="who">
    </TD>
<TR>
    <TD>
        <INPUT TYPE="submit" VALUE="Dodaj">
    </TD>
</TR>
</TABLE></CENTER>
</FORM>
[HTML] pobierz, plaintext 

index.php

[PHP] pobierz, plaintext 
<?php
	 include("main.temp")
?>
[PHP] pobierz, plaintext 

main.temp

[PHP] pobierz, plaintext 
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01//EN">
<HTML>
<HEAD>
  <META HTTP-EQUIV="content-type" CONTENT="text/html; charset=ISO-8859-2">
  <META NAME="author" CONTENT="Michał Ligus">
  <META HTTP-EQUIV="reply-to" CONTENT>
  <META NAME="description" CONTENT>
  <META NAME="keywords" CONTENT>
  <META NAME="copyright" CONTENT="Michał Ligus">
  <META HTTP-EQUIV="content-language" CONTENT="pl">
  <META HTTP-EQUIV="creation-date" CONTENT="2006-03-04 18:53:10">
  <TITLE>Dodawanie Newsów</TITLE>
  <STYLE TYPE="text/css">
	table {color: #FFFFFF;}
  </STYLE>
</HEAD>
 
<BODY BGCOLOR="BLACK">
<CENTER>
<TABLE BORDER="0" WIDTH="780">
<TR>
	<TD WIDTH="755" ALIGN="center" VALIGN="TOP">
 
	Dodaj News
 
	</TD>
</TR>
<TR>
	<TD ALIGN="center" VALIGN="TOP">
	<?php
 
	  include('add.temp')
 
	?>
	</TD>
</TR>
</TABLE>
<TABLE BORDER="0" WIDTH="780">
	<?php
 
	  include('update.php')
 
	?>
	</TD>
</TABLE>
</BODY>
</HTML>
[PHP] pobierz, plaintext 

update.php

[PHP] pobierz, plaintext 
<?php
 
$contents = $_POST["contents"];
$who = $_POST["who"];
$data = date("d.m.");
$rok = date("Y");
$czas = date(" G:i");
$linia01 = "<TR><TD ALIGN=\"LEFT\"><P ALIGN=\"justify\"><FONT FACE=\"EngrvrsOldEng Bd BT\" COLOR=\"#ffff00\">R</FONT>ecz, ";
$linia02 = "<FONT FACE=\"EngrvrsOldEng Bd BT\"><FONT COLOR=\"#ffff00\">A.</FONT>D. ";
$linia03 = "</FONT><BR><HR COLOR=\"#7093db\">";
$linia04 = "<P ALIGN=\"RIGHT\"><SMALL>";
$linia05 = "</SMALL></P></P></TD></TR>";
 
function readNews()
{
	$contents = "";
	$fp = fopen("update.txt", "r");
	while(!feof($fp)){
		$contents .= fgets($fp) . "<BR>";
	}
	fclose($fp);
	return $contents;
}
function addNews($contents, $who, $data, $rok, $czas, $linia01, $linia02, $linia03, $linia04, $linia05)
{
	chmod("update.txt". "666");
	$fp = fopen("update.txt", "r");
	$tempC = fread($fp, filesize("update.txt"));
	fclose($fp);
	$fp = fopen("update.txt", "w");
	fputs($fp, $linia01. $data. $linia02. $rok. $czas. $linia03. $contents. $linia04. $who. $linia05. "\r");
	fputs($fp, $tempC);
	fclose($fp);
	}
if($contents == "" && $who == ""){
	print(readNews());
}
else{
	addNews($contents, $who, $data, $rok, $czas, $linia01, $linia02, $linia03, $linia04, $linia05);
	chmod("update.txt". "644");
	print(readNews());
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Maveilthain 4.05.2006, 11:05:19

[nospor]

tu masz male co nieco na temat odswierzania:
http://forum.php.pl/index.php?showtopic=44383