Kritika webu: www.zekr.8u.cz

Frantislav · 2012-05-05 20:14:05

Pomohli jste mně vylepšit můj web, za to Vám velmi děkuji.

kksmirice · 2012-05-05 20:35:34

Až budeš opět něco vylepšovat, můžeš se inspirovat třeba tady
Jednoduché, efektivní, účelné.

Na tom fóru mi trošku nesedí ten text pod příspěvkem.
Nejsou vyplněna všechna pole, text je příliš dlouhý, nebo bylo špatně opsáno číslo.
A to jsem ještě nic nenapsal ani neodeslal - je to takové ne moc profi...

Frantislav · 2012-05-05 21:50:53

Tak tedy jsem to upravil. Ale nevím, z jakého důvodu mně nejde namísto metody "GET" metoda "POST".

Martin · 2012-05-05 21:57:53

Frantislav napsal:

Tak tedy jsem to upravil. Ale nevím, z jakého důvodu mně nejde namísto metody "GET" metoda "POST".

Co znamená, že to nejde?

kksmirice · 2012-05-05 22:03:25

dej sem soubor forum.php
nebo na e-mail.

Frantislav · 2012-05-05 22:04:29

Je to tak, že když se pokusím odeslat příspěvek, neodešle se a bude vypsána chybová hláška.
Odesláno na email.

kksmirice · 2012-05-05 22:06:32

Chybné vyhodnocení proměnných - tipuji

kksmirice · 2012-05-05 22:12:12

zaměň v forum.php
$_GET za $_POST

Frantislav · 2012-05-05 22:15:31

Funguje. Ale jaký to má tedy význam?

kksmirice · 2012-05-05 22:17:58

http://php.vrana.cz/http-metody-get-a-post.php
http://cs.wikipedia.org/wiki/GET
http://cs.wikipedia.org/wiki/POST

Super,

Frantislav · 2012-05-05 22:18:51

Už chápu.

l564 · 2012-05-06 09:55:03

Frantislav napsal:

O většině z toho vím. Předtím jsem neměl příliš času a účelem bylo zpřístupnit fórum. Teď ovšem upravím vzhled fóra, nastavím maximální délku příspěvku a opravím chybu 404.

Ověřujěš délku též na serveru? Protože zda-li to máš pouze jako javascipt či přímo v elementu, tak jednoduše zapnu firebug a odeberu ti ten rušivý element, a mám již neomezenou délku příspěvku.

Frantislav · 2012-05-06 10:38:49

Mám to pomocí podmínky v PHP, takto:

 if (strlen($_POST["obsah"]) > 0 && strlen($_POST["jmeno"]) > 0 && strlen($_POST["nadpis"]) > 0 && strlen($_POST["obsah"]) < 794 && strlen($_POST["jmeno"]) < 20 && strlen($_POST["nadpis"]) < 33)

Po odeslání příspěvku proběhne kontrola, pokud neobstojí, příspěvek se neodešle a zobrazí se zpráva v JavaScriptu.

Martin · 2012-05-06 11:03:16

Neodpustím si k tomu něco napsat. Jestli chcete uživatele nasrat, nechte to tak. Zřejmě jste ještě nikdy nepsal příspěvek a po odeslání jste nezjistil, že je moc dlouhý a šmitec. Nevím jak to máte celé řešené, každopádně to na mě nepůsobí dobře, spíše jako podpásovka a to se nedělá. Položte si otázku jako například co se stane, když někdo odešle moc dlouhý text? (Zavře kartu prohlížeče a už se na web nikdy nevrátí stejně tak jako tam nepřidá komentář.)
Nemluvě o tom, že řešíte nějaké hloupé délky příspěvků a zapomínáte na atribut maxlength?
(A když už, nepřijde vám 800 znaků jako směšné číslo? Jen tento příspěvek má přes 600 a mohlo by to být o dost delší!)

Frantislav · 2012-05-06 11:39:52

Ano, máte pravdu. Bohužel nevím, jak to spravit.

kksmirice · 2012-05-06 11:45:54

V tomhle má Martin naprostou pravdu. Řešení problému se mi nezdá složité.

Osobně bych nejprve ošetřil vstupy:

if (!isset($_POST["jmeno"])):
  $prjmeno="";
else:
  $prjmeno=htmlspecialchars($_POST["jmeno"]);
endif;
if (!isset($_POST["nadpis"])):
  $prnadpis="";
else:
  $prnadpis=htmlspecialchars($_POST["nadpis"]);
endif;
if (!isset($_POST["text"])):
  $prtext="";
else:
  $probsah=htmlspecialchars($_POST["obsah"]);
endif;
// nastavení maximální délek vstupů
$jmeno_max=20;
$nadpis_max=30;
$obsah_max=1000;

Tedy buď tento způsob, nebo pak ošetřit přes $GLOBALS.
Takto stačí, pokud je "jen" uloženo do obsahu souboru.
V dalším bych pak pracoval už jen s těmito proměnnými.
1. každou zvlášť ošetřit, zda splňuje
a) není prázdná,
b) nepřesahuje maximální povolenou délku.
2. pokud nesplňuje podmínky, pak uživateli sdělit co je špatně.

$chyba=0;
if ($prjmeno=="" OR strlen($prjmeno)>$jmeno_max):
  // vypsat hlášku o nesplnění podmínky (třeba před příspěvek)
  $chyba=1;
endif;
if ($prnadpis=="" OR strlen($prnadpis)>$nadpis_max):
  // vypsat hlášku o nesplnění podmínky (třeba před příspěvek)
  $chyba=1;
endif;
if ($probsah=="" OR strlen($probsah)>$obsah_max):
  // vypsat hlášku o nesplnění podmínky (třeba před příspěvek)
  $chyba=1;
endif;
if (!$chyba):
  // zpracování bezchybného příspěvku
...
endif;

3. vrátit se na původní stránku s předvyplňeným textem tak, aby uživatel mohl nedostatek opravit a znovu odeslat.

Frantislav · 2012-05-06 11:50:08

Děkuji.

kksmirice · 2012-05-06 11:55:38

Samozřejmě je to jen narychlo náčrt možného řešení, každý má trochu jiný pohled a také systém.

Možná by i stálo za úvahu, zda pro fórum nepoužít databázi. Je to pak jednodušší na hledání, výpisy, třídění, ...

Frantislav · 2012-05-06 12:01:02

S MySQL ještě neumím.

kksmirice · 2012-05-06 12:29:21

Oprav si v tom druhém kódu co jsem psal text -> obsah (a v textu slovo dlohý -> dlouhý)
Nějak mi to ulítlo...

Teď je v příspěvku opraven správně.

EDIT:
Ještě popřemýšlej, jak zobrazit uživatelem vyplněné údaje, tak aby je mohl opravovat.....

EDIT2:
Budu kolem 18:00. Pak na to mrknem...

l564 · 2012-05-06 14:39:14

Jak psal Martin, to by mě též naštvalo.

Doporučuji ještě aby jsi pomocí JavaScriptu zobrazoval počet aktuálních písmen, aby vědeli kolik ještě zbývá.

Náhled, Zdrojový kód

Frantislav · 2012-05-06 14:50:02

Teď jsem objevil řešení a už se obsah při nevyplnění dalších polí nesmaže.
Pomocí cookies.

l564 · 2012-05-06 15:04:29

nebo pomocí Sessions

kksmirice · 2012-05-06 17:34:12

Frantislav napsal:

... už se obsah při nevyplnění dalších polí nesmaže ...

Nesmaže se pouze obsah text ($obsah), jméno a nadpis se mažou.

Osobně bych to řešil předáním parametrů $_POST <input type="hidden" name="$promenna"> a výpis hodnot <input value="$promenna"> ve formuláři

l564 · 2012-05-06 20:23:19

A když by se mělo rozhodnout k sušenkám tak by nebylo od věci nastavit expiraci na 1 hodinu

#51 2012-05-05 20:14:05

Re: Kritika webu: www.zekr.8u.cz

#52 2012-05-05 20:35:34

Re: Kritika webu: www.zekr.8u.cz

#53 2012-05-05 21:50:53

Re: Kritika webu: www.zekr.8u.cz

#54 2012-05-05 21:57:53

Re: Kritika webu: www.zekr.8u.cz

#55 2012-05-05 22:03:25

Re: Kritika webu: www.zekr.8u.cz

#56 2012-05-05 22:04:29

Re: Kritika webu: www.zekr.8u.cz

#57 2012-05-05 22:06:32

Re: Kritika webu: www.zekr.8u.cz

#58 2012-05-05 22:12:12

Re: Kritika webu: www.zekr.8u.cz

#59 2012-05-05 22:15:31

Re: Kritika webu: www.zekr.8u.cz

#60 2012-05-05 22:17:58

Re: Kritika webu: www.zekr.8u.cz

#61 2012-05-05 22:18:51

Re: Kritika webu: www.zekr.8u.cz

#62 2012-05-06 09:55:03

Re: Kritika webu: www.zekr.8u.cz

#63 2012-05-06 10:38:49

Re: Kritika webu: www.zekr.8u.cz

#64 2012-05-06 11:03:16

Re: Kritika webu: www.zekr.8u.cz

#65 2012-05-06 11:39:52

Re: Kritika webu: www.zekr.8u.cz

#66 2012-05-06 11:45:54

Re: Kritika webu: www.zekr.8u.cz

#67 2012-05-06 11:50:08

Re: Kritika webu: www.zekr.8u.cz

#68 2012-05-06 11:55:38

Re: Kritika webu: www.zekr.8u.cz

#69 2012-05-06 12:01:02

Re: Kritika webu: www.zekr.8u.cz

#70 2012-05-06 12:29:21

Re: Kritika webu: www.zekr.8u.cz

#71 2012-05-06 14:39:14

Re: Kritika webu: www.zekr.8u.cz

#72 2012-05-06 14:50:02

Re: Kritika webu: www.zekr.8u.cz

#73 2012-05-06 15:04:29

Re: Kritika webu: www.zekr.8u.cz

#74 2012-05-06 17:34:12

Re: Kritika webu: www.zekr.8u.cz

#75 2012-05-06 20:23:19

Re: Kritika webu: www.zekr.8u.cz

Zápatí