Diskuze: file
Kit:7.9.2013 15:37
Asi obojí 
Nějak mi ten tvůj obrázek nepasoval s výrokem "pokud si pamatuju taky jsem mýval".
Snorlax:7.9.2013 15:47
ikdyž to byl vtip na
můj účet tak i přesto se směju jak magor v blázinci
matesax:7.9.2013 15:49
No nazdar - chci tě vidět...
$fileLineCount tu je, abych pořád neusel tahat countem... Unescape je tu proto, abych mohl používat quote znaky... Díra tam není - na server nikdo nezná heslo a jméno... (Každý kdo zná heslo a jméno si může číst jakýkoliv script...) Neznám jiný příkaz, který zabije proces a přitom vypíše hlášku... Na jeden krátký řádek to opravdu nejde...
Neaktivní uživatel:7.9.2013 16:14
i kdyby ten text byl třeba chlupatý, tak i tak se to dá uložit do
databáze
David Čápka:7.9.2013 18:22
Rád ti vysvětlím co je špatně.
case "goTo":GoTo je velmi špatně zvolený název návěstí, pokud se v návěstí parsuje, mělo by se jmenovat asi jinak, ne? Vkládat tak dlouhý kód do větve switche je blbost, měla by se tam volat metoda. Pomocí reflexe můžeš dát ten switch tam, kam patří - pryč.
$value = "[\"".$_GET["time"]."\",\"";Co jsou ta lomítka? Co třeba otevřít jednoduché uvozovky místo dvojitých?
$file = file("History\\".$_GET["file"]);To jako vážně? Necháš si zpracovat libovolný soubor co uživatel zadá? Asi chceš, aby ti to někdo hacknul.
$noFound = true;To je názvosloví a navíc k ošetření chyb jsou výjimky.
$index = 0;
while($index < $fileLineCount)
{
if(strpos($file[$index], $value) === 0)
{
$noFound = false;
break;
}
$index++;
}Tohle bych nahradil jednou řádkou:
$array = explode("\n", file_get_contents('file.txt'));a potom hledal hodnotu v tom poli.
if($noFound)
die("alert('Nenalezeno!');");Tady jsem málem spadl pod stůl, smrtící kombinace die alert je opravda skvělý
způsob pro zpracování chyby s kombinací s proměnnou noFound (ano, je to
ironie).
Ten zbylý bordel jsem ani nepochopil.
Tu URL nikdo nikdy neuvidí. To souvisí s tím pojmenováním - nebude takto lidské, ale takové, aby si tu URL nikdo nemohl ani tipnout... Navíc je uvnitř AJAXu obfuscovaného JS... Dále explode("\n", file_get_contents('file.txt')) - a co asi udělá file??? (s mnohem kratším kódem) Jaká výjimka mi vznikne? Musel bych ji udělat, jenže nemám pod kontrolou interpreter - takže stejně bych to nechytal tam, kde mám. Stejně se nezbavím větvení - výjimku bych musel vytvářet - to věc moc nezachraňuje... Prostě hledáš jen kraviny...
Ale nejvtipnější na tvém komentu je, že jsi nahradil část kódu jen
jeho částí - a napsal jsi, že ať se zbytek udělá...
explode("\n", file_get_contents('file.txt')) == file("file.txt")A pod tím bude zbytek mého kódu...
Zobrazeno 54 zpráv z 54.