Kritika webu terciea.cekuj.net
http://terciea.cekuj.net
Michael Olšavský:6.2.2013 21:48
Hesla mám escapované a šifrované přes md5 
.
Zprávy do chatu mám přes soubor -> zápis přes php, pomocí jquery automatický update.
David Jančík:6.2.2013 21:55
md5 nepoužívej šáhni raději po dobře osolené
sha1 
Jsou servery
co generují databáze s heslama kódovanýma pod md5 a je tam už
hodně slov. Takže když se k těm datům někdo dostane, tak se to ve
většině případů dá dešifrovat, protože uživatelé většinou volí
obvyklá snadno zapamatovatelná hesla.
Jako vždy se dívám na kód - a toto je strašné...
<html5>
<!Doctype html>
<head>To jsem ještě neviděl...
Nějak takto by to mělo vypadat:
<!DOCTYPE html>
<html lang="cs-cz">
<head>
...
</head>
<body>
...
</body>
</html>V HTML5 ve formuláři nepoužívej neutrální span, ale k tomu určený label. JS předpokládám bude více - takže založ již teď soubor.
<span><i>(Michael Olšavský)</i></span>Zbytečné - jakož i mnohé jiné konstrukce - osekej to.
<script src="js/jquery.js"></script>Patří do head.
Tak malá stránka a 3 CSS soubory?
Moje vize:
<!DOCTYPE html>
<html lang="cs-cz">
<head>
<meta charset="UTF-8">
<title>Menu</title>
<link rel="stylesheet" type="text/css" href="index.css" />
<script type="text/javascript" src="js/jquery.js"></script>
<script type="text/javascript">
function Send()
{
if(document.forms["form"].password.value == "" || document.forms["form"].name.value == "")
{
alert("Chybí údaje!");
return false;
}
return true;
}
</script>
</head>
<body>
<header><img src="obrazky_WWW/3A.png" style="float:left; position: static;" height="140px"/></header>
<nav>
<ul>
<li><a href="index.php?id=index" id="Novinky" title="Novinky">Menu</a></li>
<li><a href="index.php?id=akce" id="Akce" title="Akce">Akce</a></li>
<li><a href="index.php?id=fotky" id="Fotky" title="Fotky">Fotky</a></li>
<li><a href="index.php?frame=View/Content/rozvrh.php" id="Rozvrh" title="Rozvrh">Rozvrh</a></li>
<li><a href="index.php?frame=http://kavalirka.cz/suplovani/suplovtr.htm" title="Suplování">Suplování</a></li>
</ul>
</nav>
<div id="login">
<form method="POST" class="formular" id="form">
<label for="email">Email:</span>
<input type="text" name="email" placeholder="Zde zadejte svůj e-mail" />
<label for="passvord">Heslo:</span>
<input type="password" name="password" placeholder="Zde zadejte své heslo" />
<br />
<input type="submit" class="loginButton" name="submit" value="Přihlásit" />
<button class="loginButton" onclick="Send()" />Registrovat</button>
</form>
</div>
<article>
<h2><a href=index.php?clanek=13>Lístky na maturitní ples</a></h2>
<i>(Michael Olšavský)</i>
<p>Pro lístky na maturitní ples si říkejte u našeho pokladníka.</p>
<p><i>2013-02-06</i></p>
</article>
</body>
</html>JS formuláře si nyní uprav - nemám čas - spím...
(Drobné chyby snad omluvíte, když to píši tak pozdě... )
matesax:7.2.2013 16:31
Další vyplavou, až to opravíš. Ano - měl jsi tam pár chybiček proti pravidlům HTML. Ale pak tam máš mnoho zbytečných konstrukcí, zbytečného JS - viz. placeholder atd... Máš tam JQuery, které nepoužiješ - nač? A i kvůli 3 věcem tahat celé JQuery - proč? Atd...
Přeci nehodláš dělat tak, že toto není závažné - k čemu to pak
děláš? (To si také mohu
jen tak něco vyplivnout...)
</html5>matesax:7.2.2013 19:22
Proč? Budou 2 - uděláš to jako control - partialview. A zobrazíš buď
jeden, nebo druhý - nač login na register stránce? A akce také odděl -
udělej to přes MVC a použij akci pouze jako akci - ne jako odkaz na stránku
- v PHP pak rozděl a panuj...
(Popis switch...)
Zobrazené 39 správy z 39.