W dniu 2012-11-03 20:49, Jasio pisze:
> W dniu 2012-11-03 19:20, Cezary Tomczyk pisze:
>> W dniu 2012-11-02 22:41, Jasio pisze:
>>> BTW. tak czytając Twój kod na githubie, to Ty się lepiej weź za lepienie
>>> pierogów...
>>
>> Co z nim jest nie tak? Jakieś bardziej konkretne uwagi bym poznał.
>>
>
> np. to:

> if( !empty($_GET['section']) )
> {
> foreach ($_GET['section'] as $value)
> {
> if ($value)
> {
> array_push($sections, $value);
> }
> }
> }
>
> foreach( $sections as $value )
> {
> $filename = 'jscode.'.strtolower( $value ).'.js';
> $jsCodeSource .= file_get_contents(
> dirname(__FILE__).DS.$filename ).'
> ';
> }
>
> podstawiasz do file_get_contents, to co dostałeś w GET bez żadnej
> weryfikacji

Owszem, nie sprawdzam i wiem, jakie to nosi za sobą ryzyko. Napisałem
ten fragment jako coś początkowego, co miało mi pomóc zebrać potrzebne
pliki w jedną całość. Jasne, że trzeba będzie to usprawnić.

No i jak coś krytykujesz to przynajmniej napisz który fragment i
dlaczego. Będzie lepiej prowadzić rozmowę. Takie ogólne stwierdzenia,
jak "[...] tak czytając Twój kod na githubie, to Ty się lepiej weź za
lepienie pierogów...[...]" nie niesie ze sobą żadnej sensownej treści.

--
Cezary Tomczyk
http://www.ctomczyk.pl/