Forum PHP.pl > [CSS][PHP]Formularz

Pomoc - Szukaj - Użytkownicy - Kalendarz

kaiba55

11.07.2012, 18:47:40

Witam.

Mój problem to data. Otóż jak odświerzam stronę, bez klikania wyslij w formularzu data i tak się pojawia. Próbowałem dać datę z ukrytym polem ale było podobnie.

Mój kod:

[PHP] pobierz, plaintext 
<html>
<head>
<link rel="stylesheet" type="text/css" href="styl.css" />
<title>Shoutbox</title>
</head>
 
<form action="index.php method="GET">
<input type=text name="nick"/><br>
<input type=text name="wpis"/><br>
<input type=submit value="wyslij"/>
</form>
<?php
$imie=$_GET['nick'].'<br>';
$wpis=$_GET['wpis'].'<br>';
$data=date("H:i:s").'<br>' ;
 
$imie3=fopen("imie.txt", "a");
fputs($imie3,$imie);
fclose($imie3);
 
$wpis3=fopen("wpis.txt","a");
fputs($wpis3,$wpis);
fclose($wpis3);
 
$data3=fopen("data.txt", "a");
fputs($data3, $data);
fclose($data3);
 
$imie2=fread(fopen("imie.txt", "r"), filesize("imie.txt"));
$wpis2=fread(fopen("wpis.txt", "r"), filesize("wpis.txt"));
$data2=fread(fopen("data.txt", "r"), filesize("data.txt"));
?>
 
    <div id="nick">
    <?php
    echo "nick".'<br>';
    echo $imie2;
    ?>
 
    </div>
 
 
    <div id="wpis">
    <?php
    echo "wpis".'<br>';
    echo $wpis2;
    ?>
    </div>
 
 
    <div id="data">
    <?php
    echo "data".'<br>';
    echo $data2;
    ?>
[PHP] pobierz, plaintext

Zedytowałem bo poprawiałem kod z powodu pewnej bzdury.

Co do css to już nieaktualne
Z góry dzięki za pomoc.
Pozdrawiam.

sobol6803

12.07.2012, 02:58:38

A o if'ach zapomniało się? Gdzie masz sprawdzanie, czy formularz został wysłany? Teraz wykonuje cały kod za każdym razem, nawet jak nie wyślesz formularza. Za zmienne, do których przypisujesz GET'y nie wstawia nic, za to wstawia " ". Datę tworzysz w zmiennej w php, więc ona pojawia się zawsze.

Zapoznaj się z:
if isset

Oraz dodaj do buttona "name", żeby sprawdzać wysłanie formularza w issecie.

Myślę, że sobie poradzisz.

kaiba55

12.07.2012, 08:06:29

dzięki wielkie, zamiast isset wykorzystałem empty i już sobie wszystko dzięki temu zrobiłem- datę także

mam jeszcze jeden problem, otóż na początku zmienne zawsze są puste, więc jak sprawdzam przez empty to przy 1 otwarciu strony zawsze wyskakuje że formularz nie do końca jest uzupełniony, co z tym zrobić? czy niezbędne jest wykorzystanie isset?

mam też drugi problem, na samym początku pliki tekstowe są puste i jak mamy:

[PHP] pobierz, plaintext 
$data2=fread(fopen("data.txt", "r"), filesize("data.txt"));
[PHP] pobierz, plaintext

To zawsze na wstępie wyrzuca błąd, że plik jest pusty. Da się coś z tym zrobić, czy muszę zamiast tego wszystkiego po prostu zaincludować plik i będzie?

Pozdro.

Tuminure

12.07.2012, 12:28:01

Problem tkwi w tym, że chyba źle stosujesz empty

.

Zmienne $imie, $wpis nie są puste nigdy. Wpisujesz do nich o czym wspomniał sobol6803.
Nie powinieneś wykorzystywać też empty do $_GET['nick'] lub $_GET['wpis'], ponieważ te indeksy mogą nie być zainicjowane.
Powinieneś zastosować isset do $_GET['nick'] i $_GET['wpis'].

Cytat

Oraz dodaj do buttona "name", żeby sprawdzać wysłanie formularza w issecie.

To akurat nic nie da. Wystarczy, że będzie sprawdzał $_GET['nick'] i $_GET['wpis'].

kaiba55

12.07.2012, 12:36:42

nie

przed dodaniem czegokolwiek pliki są na krótką chwile puste- nie ma tam nawet - z prostego powodu bo formularz nie został ani razu wypełniony.
Wykorzystałem empty dlatego, że czytałem że isset tylko sprawdza czy zmienna jest czy jej nie ma, natomiast empty sprawdzi czy jest ona pusta. Obawiam się, że wykorzystując isset sprawdze zmienną czy jest- a ona zawsze istnieje tyle, że może być pusta.

Wrzucam kod i proszę o porady:

[PHP] pobierz, plaintext 
<html>
<head>
<link rel="stylesheet" type="text/css" href="styl.css" />
<title>Shoutbox</title>
</head>
 
<form action="index.php method="GET">
<input type=text name="nick"/><br>
<input type=text name="wpis"/><br>
<input type=submit value="wyslij"/>
</form>
<?php
$imie=$_GET['nick'].'<br>';
$wpis=$_GET['wpis'].'<br>';
$data=date("H:i:s").'<br>' ;
 
 
if(empty($_GET['imie'])&&(empty($_GET['wpis'])))
echo "uzupelnij caly formularz".'<br>';
else
{ 
$imie3=fopen("imie.txt", "a");
fputs($imie3,$imie);
fclose($imie3);
 
$wpis3=fopen("wpis.txt","a");
fputs($wpis3,$wpis);
fclose($wpis3);
 
$data3=fopen("data.txt", "a");
fputs($data3, $data);
fclose($data3);
 
}
 
$plik1="imie.txt";
$plik2="wpis.txt";
$plik3="data.txt";
if((file_exists($plik1)==true)&&(file_exists($plik2)==true)&&(file_exists($plik3)==true))
{
$imie2=fread(fopen("imie.txt", "r"), filesize("imie.txt"));
$wpis2=fread(fopen("wpis.txt", "r"), filesize("wpis.txt"));
$data2=fread(fopen("data.txt", "r"), filesize("data.txt"));
}
else
echo "dokonaj instalacji skryptu, bo nie ma plikow tekstowych".'<br>';
?>
 
    <div id="nick">
    <?php
    echo "nick".'<br>';
    echo $imie2;
    ?>
 
    </div>
 
 
    <div id="wpis">
    <?php
    echo "wpis".'<br>';
    echo $wpis2;
    ?>
    </div>
 
 
    <div id="data">
    <?php
    echo "data".'<br>';
    echo $data2;
    ?>
 
    </div>
[PHP] pobierz, plaintext

Tuminure

12.07.2012, 13:13:18

Cytat

przed dodaniem czegokolwiek pliki są na krótką chwile puste

Pisałem o zmiennych (nie o plikach) $imie i $wpis, które nigdy nie są puste. Na początku kodu zmienne nie istnieją (czyli isset($imie) pokaże fałsz), potem zostają zainicjalizowane i zostaje do nich wpisane $_GET['nick'].' '. Jeżeli $_GET['nick'] jest puste, to do $imie, zostaje przy inicjalizacji wpisane . W skrócie $imie w żadnym momencie nie jest puste.

Te kawałki kodu powinny być dla Ciebie wystarczająco pomocne

[PHP] pobierz, plaintext 
if(isset($_GET['nick']) && isset($_GET['wpis']))
{
    $wpisuj_do_pliku = TRUE;
    $imie=$_GET['nick'].'<br>';
    $wpis=$_GET['wpis'].'<br>';           
}
else
{
    $wpisuj_do_pliku = FALSE;
}
[PHP] pobierz, plaintext

oraz

[PHP] pobierz, plaintext 
if($wpisuj_do_pliku)
{ 
    $imie3=fopen("imie.txt", "a");
    fputs($imie3,$imie);
    fclose($imie3);
 
    $wpis3=fopen("wpis.txt","a");
    fputs($wpis3,$wpis);
    fclose($wpis3);
 
    $data3=fopen("data.txt", "a");
    fputs($data3, $data);
    fclose($data3);
}
[PHP] pobierz, plaintext

PS.
Warunek

[PHP] pobierz, plaintext 
if(empty($_GET['imie'])&&(empty($_GET['wpis'])))
[PHP] pobierz, plaintext

będzie spełniony tylko, gdy będziesz miał w pasku adresu coś przypominającego "?imie=&wpis=" lub "?wpis=&imie=" (czyli puste brak wartości dla imie i wpis).

kaiba55

12.07.2012, 13:20:25

edit: albo jednak nie rozumiem

Zrobiłem tak:

[PHP] pobierz, plaintext 
<html>
<head>
<link rel="stylesheet" type="text/css" href="styl.css" />
<title>Shoutbox</title>
</head>
 
<form action="index.php method="GET">
<input type=text name="nick"/><br>
<input type=text name="wpis"/><br>
<input type=submit value="wyslij"/>
</form>
<?php
$imie=$_GET['nick'].'<br>';
$wpis=$_GET['wpis'].'<br>';
$data=date("H:i:s").'<br>' ;
 
 
if(isset($_GET['nick']) && isset($_GET['wpis']))
    {
$imie3=fopen("imie.txt", "a");
fputs($imie3,$imie);
fclose($imie3);
 
$wpis3=fopen("wpis.txt","a");
fputs($wpis3,$wpis);
fclose($wpis3);
 
$data3=fopen("data.txt", "a");
fputs($data3, $data);
fclose($data3);
    }
    else
    {
  echo "wypelnij caly formularz";
    }
 
 
 
 
 
$plik1="imie.txt";
$plik2="wpis.txt";
$plik3="data.txt";
if((file_exists($plik1)==true)&&(file_exists($plik2)==true)&&(file_exists($plik3)==true))
{
$imie2=fread(fopen("imie.txt", "r"), filesize("imie.txt"));
$wpis2=fread(fopen("wpis.txt", "r"), filesize("wpis.txt"));
$data2=fread(fopen("data.txt", "r"), filesize("data.txt"));
}
else
echo "dokonaj instalacji skryptu, bo nie ma plikow tekstowych".'<br>';
?>
 
    <div id="nick">
    <?php
    echo "nick".'<br>';
    echo $imie2;
    ?>
 
    </div>
 
 
    <div id="wpis">
    <?php
    echo "wpis".'<br>';
    echo $wpis2;
    ?>
    </div>
 
 
    <div id="data">
    <?php
    echo "data".'<br>';
    echo $data2;
    ?>
 
    </div>
[PHP] pobierz, plaintext

Ale to chyba w ogóle nie tak ma wyglądać.
Otóż jak sprawdzałem empty to sprawdzało czy nie jest pusto, a isset wskazuje tylko czy zmienna jest- jeśli jest, choćby pusta to formularz zostanie wysłany, nawet bez 1 pola, sprawdzając empty nie było takiej szansy- chyba że ja coś źle robię

sobol6803

12.07.2012, 14:36:13

A kto Ci każe wybierać pomiędzy empty a isset? Możesz to zrobić tak:

[PHP] pobierz, plaintext 
if (isset($_GET['imie']) && isset($_GET['wpis']) && !empty($_GET['imie']) && !empty($_GET['wpis'])) {
//formularz wysłany, pola nie są puste
}
else {
//formularz niewysłany
}
[PHP] pobierz, plaintext

Dodatkowo GET ma limit znaków, więc jak chcesz przekazywać przez niego jakieś dłuższe stringi, to lepiej to zrobić POST'em.

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.