Kritika webu http://zekr.8u.cz/

HTML a CSS Kritika webů - ukažte se! http://zekr.8u.cz/ American English version English version

Jak se ti líbí web?
Celkové hodnocení (4 hlasů):
44444
Přejít na web:
http://zekr.8u.cz/
Aktivity (1)
Avatar
Frunta
Redaktor
Avatar
Frunta:21.10.2012 21:09

Zdravím,
přibližně v březnu tohoto roku jsem "položil základní kámen" mého webu. Tehdy skutečně jeho vzhled připomínal kámen a radost naň pohledět neexistovala. Nyní jsem už asi počtvrté přehodnotil design a bylo by dobré, kdyby jej někdo "poplival" či pochválil, děkuji za každý komentář.
S pozdravem, Frunta

 
Odpovědět 21.10.2012 21:09
Avatar
jakub
Redaktor
Avatar
jakub:21.10.2012 21:40

Web dobrý,obsah taky,není co vytknout.

Nahoru Odpovědět 21.10.2012 21:40
Uvnitř jsou všichni stejní...
Avatar
Kit
Redaktor
Avatar
Odpovídá na Frunta
Kit:21.10.2012 22:35

Máš tam bezpečnostní díru. Podařilo se mi ten server zacyklit, takže mi poslal 50 MB textu.

Jsem alergický na větu "Vítám Vás na mých stránkách." Ta věta je úplně zbytečná a na web nepatří. Tvá stránka není o vítání, není o stránkách, ale je o tvé tvorbě.

Nahoru Odpovědět  +1 21.10.2012 22:35
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Frunta
Redaktor
Avatar
Frunta:22.10.2012 14:14

Pokusím se to napravit, ale jak se Ti to podařilo?

 
Nahoru Odpovědět 22.10.2012 14:14
Avatar
Kit
Redaktor
Avatar
Odpovídá na Frunta
Kit:22.10.2012 14:46

Opravdu to mám zveřejnit? Tak jen kousek:

?stranka=../index
Nahoru Odpovědět 22.10.2012 14:46
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Frunta
Redaktor
Avatar
Frunta:22.10.2012 14:52

Děkuji, mělo by to být spraveno.

 
Nahoru Odpovědět 22.10.2012 14:52
Avatar
Frunta
Redaktor
Avatar
Frunta:22.10.2012 15:03
if(strpos ($_GET["stranka"], "index") == true){
  $_GET["stranka"] = "uvod";
}

Tohle se zdá jako řešení.

 
Nahoru Odpovědět 22.10.2012 15:03
Avatar
Kit
Redaktor
Avatar
Odpovídá na Frunta
Kit:22.10.2012 15:15

Ne, to není řešení. Tohle je řešení:

$stranka=basename($_GET["stranka"]);

Nikdy, opakuji, nikdy nepřepisuj obsah pole $__GET, $__POST a dalších superglobálních proměnných, které k tomu nejsou určeny. Existuje sice pár výjimek, ale asi je nepoužíváš.

Editováno 22.10.2012 15:18
Nahoru Odpovědět 22.10.2012 15:15
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Kit
Redaktor
Avatar
Odpovídá na Frunta
Kit:22.10.2012 15:32

Aha, koukám, že v adresáři "web" máš ještě jeden index.php, ale ten už nezlobí.

Zato ti tam zlobí chybějící soubor "stranky/uvod2­.php".

Nahoru Odpovědět 22.10.2012 15:32
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
Frunta
Redaktor
Avatar
Frunta:22.10.2012 15:53

Dobře, děkuji.

 
Nahoru Odpovědět 22.10.2012 15:53
Avatar
David Čápka
Tým ITnetwork
Avatar
Odpovídá na Frunta
David Čápka:22.10.2012 19:36

Hezký web a i má nějaký obsah :) V sekci Aplikace ti vypadl obrázek.

Nahoru Odpovědět 22.10.2012 19:36
Miluji svou práci a zdejší komunitu, baví mě se rozvíjet, děkuji každému členovi za to, že zde působí.
Avatar
jan.vencl
Redaktor
Avatar
jan.vencl:3.11.2012 9:57

Ahoj hezký to je menu, pozadí super jen mi tam barevně nesedne ta zelená k tomu menu :)ale super

 
Nahoru Odpovědět 3.11.2012 9:57
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 12 zpráv z 12.