Witam,

to moja pierwsza strona , sporo się przy jej tworzeniu nauczyłem.

http://www.japierdziele.pl

Użytkownicy dodają na niej swoje historie z życia pod ocene i komentarz publiki.
Bardzo prosze o konstruktywną krytyke

[-] słabe wykonanie graficzne
[-] teksty z błędami ortograficznymi ("nie powiny też zawierać dużo ''byków''.")

podatności na różnych użyszkodników, którzy lubią pogrzebać w kodzie wolałem nie sprawdzać (sam formularz kontaktowy odstrasza ). krótko mówiąc, czeka cię jeszcze wiele nauki...

Bardzo kiepski pomysł na sprawdzanie, czy już głosowałeś. W chwile można ponabijać oceny.

dodawanie bez rejestracji i captcha ... prosisz się o spambota.
Przy rejestracji nie pisze ile co ma mieć znaków ...
strona podatna na xss

Wystarczy wyłączyć obsługę ciasteczek w przeglądarce i możemy głosować bez końca. Miłego votowania.

Do tego podatność na SQL Injection na stronie z komentarzami. Mały przykład:

http://www.japierdziele.pl/index.php?id=%2...201%20IS%20NULL
http://www.japierdziele.pl/index.php?id=%2...IS%20NOT%20NULL

BARDZO dziękuje za każdą krytyke jest bardzo pomocna!




Chciałem osiągnąc bardzo minimalistyczny wygląd , ale masz racje nie przyłożyłem się do layoutu jak powinienem.




Dzieki za cynk naprawiłem to takim kodem mam nadzieje ze wystarczy :

[PHP] pobierz, plaintext 
if(is_numeric($_GET[id])){
// wyswietlanie komentarzy
 
 
 
}else {
 $ip=$_SERVER['REMOTE_ADDR'];
 echo "Wszelkie próby SQL injection są monitorowane <br>
  Skrypt zapisał twoje IP : $ip , do bazy";
  $sql = "INSERT INTO `ajpi` VALUES ('$ip', '$_SESSION[name]', '$_GET')";
  mysql_query($sql); 
  die;
}
[PHP] pobierz, plaintext 

Gdy dodawałem ten skrypt wiedziałem że to będzie możliwe , ale pomyślałem że tak bedzie najlepiej powód : nieobciązanie servera zapytaniami

Captcha dodam jak tylko znajde jakąś minimalistyczna wersje
Z ilością znaków poprawiłem dzięki
A co do ataków xss musze się jeszcze o nich poduczyć i napewno zabespiecze strone

fajna nazwa - ja pierdziele - hehe dobre, potem będzie łatwo zapamiętać

nazwa super , grafika taka sobie

btw. widze, ze juz sql injection fixed , wszedlem w te linki Shipa , i bum moje Ip zostalo zapisane...wywal mnie , nic nie chcialem zrobic

Przez walidatory przechodzi poprawnie , ze tak powiem , pare malych bledow.

Hehe No grafika itp. według mnie na niskim poziomie Ale szczerze przeczytałem dwie pierwsze strony ... Niektóre wpisy są śmieszne Ale dla mnie trochę mało składu
Popracuj nad grafiką i innymi rzeczami a może stworzysz coś na styl demotywatorów Pozdrawiam

Troche małych błedów dokładnie 256 błedów i 73 ostrzeżenia ;P mógłbyś trochę poprawić ale nie patrz na w3c bo ten walidator doskonały nie jest

Też sprawdzałem SQL INJECTION i jak na razie działa

dzieki za pomoc!
zmieniłem wygląd strony i logo mam nadzieje że na lepsze, jakies porady dot kolorystyki?

Nie podoba mi się ten kontrast pomiędzy białym tłem, a blokami z tekstem, przyciemniłem tło i od razu lepiej jak dla mnie. Może by te bloki dać w jakiś nadrzędny nieco ciemniejszy od tła? Cały top jest za duży o przynajmniej połowę, za dużo nie wykorzystanego miejsca w nim. Moduł facebooka nie dostosowany kolorami / rozmiarami - defaultowy. W bloku z tekstami użytkowników brakuje paddingu z lewej strony. Przycisk Dodaj zbyt ciemny. Ikonki serwisów są brzydkie.

Pomysł strony - dobry.
Przydałoby się dopracowanie graficzne i estetyczne;) żeby nie było samych teł - jakieś urozmaicenia menu ładniejsze

Szkoda że zgapiony z http://www.fmylife.com/ prawie w 100%...

A jeżeli chodzi o stronę to poprzedni wygląd był lepszy obecny to koszmarek

Nie potrzebnie dajesz Facebook-a myślę ze to nie zwiększy popularności strony a paskudnie wygląda jeżeli już to daj wyłacznie na stronie głównej taki pasek bodajże na www.demotywatory.pl możesz zobaczyć przykład.

Niestety teraz robi sie tak ze prawie każda strona ma swój fansite na facebooku a co z tego wynika ze tylko największe mogą sie przebić z większa ilością fanów.

Kiepska strona, widać że zrobiona tylko i wyłączenie po to, aby zarabiać pieniądze na reklamach - niestety, ale przy czymś takim nie spodziewaj się kokosów ;-)

Brakuje walidacji na podstronach, np. nie trzeba wpisywać nick'a aby coś wysłać. Brak polskich znaków w rejestracji. Reszty już mi się nie chciało przeglądać, za dużo błędów. Ogólnie tego typu strony omijam z daleka, popracuj jeszcze nad tym projektem. Grafikę spróbuj zlecić osobie, która się tym zajmuje zawodowo, a mechanikę popraw, aby nie było błędów i może coś z tego wyjdzie.

Pozdrawiam.

Ile się uczysz PHP?

Kilka lat temu znałem się na poziomie zainstalowania jportala lub phpbb
później nic nie robiłem w php , przypomniałem sobie php jak zaczołem robić tą strone(kilka tyg temu)

Z jakich książek czy pomocy korzystałeś ?

1. Popracuj nad menu (czcionka)
2. To okienko od Facebook'a umieść w innym miejscu, wg mnie to trochę nieestetycznie wygląda w ten sposób, zasłania (oddala wzrok) logowanie i rejestrację
3. Ogólnie formularze też mógłbyś poprawić graficznie
4. Plusy i minusy tak średnio wyeksponowane, no ale może tak ma być.
5. Podoba mi się logo i tematyka ale jakbyś podrasował graficznie to by było Extra

Pozdro

http://helion.pl/ksiazki/cwph53.htm - z jakiegoś starszego wydania , ale to dawno już jej niemam , zgubiła sie.
mam jeszcze "PHP czarna księga" ale z niej nie korzystam.
Przypominając php ogładnołem sobie kurs php i mysql (podstawowy), i kilka filmików youtube(http://www.youtube.com/user/phpacademy).

A co to ma do tematu? Panowie takie sprawy załatwiajcie przez PW.

Co do strony ciesze sie bardzo ze dodales okienko facebook jak by jeszcze zaokrgolone rogi były i sama literka "f" bez tego paska byłoby super :D ale nie rozumiem po co dodałeś filmiki, czemu informacja kojarzy mi sie z twitterem :) lepiej daj jakies "i" oraz te ostro zerżniete z demotywatory.pl "podziel sie" nawet jest to ok ale nie lepiej to tekstowo zapisać jak graficznie masz brać od kogoś po prostu "Podziel Sie : Facebook - GG - Twitter - Wykop" i bedzie super :P

Aktualna wersja strony totalnie zgapiona z demotywatory.pl

Bo informacja jest z tweetera to ostatni tweet
podziel sie nie sa z demotywatorow zerzniete to darmowy skrypt "sharing is sexy" czy cos takiego.
filmiki dodałem żeby strona miala wiecej kontentu. nie wiem czy się sprawdza zawsze mozna usunac

Widocznie jezeli nie zaluwazyłem ze to twitter to zle jest przedstawione ale pomysł dobry muszę gdzies u siebie wykorzystać :D apropo "sharing is sexy" to Podziel Sie identycznie wyglada jak w demotywatorach ;) i oto mi chodzilo i widocznie u mnie na Google chrome źle wyświetla a pamietaj ze jak na chrome źle wyświetla to musisz to jak najszybciej poprawić :) bo popularność ciagle rosnie :]

Kurde mnie jeszcze przeraża jakoś ta czcionka, zero paddingów i estetyki a na dodatek problemy z kodowaniem Popracuj nad tym ...
Fajny pomysl, dosyc fajna domena ale wykonanie kiepskie Najlepiej zrób nowy Lay ... A tak w ogóle używałeś tu jakiegoś Photoshopa lub Gimpa ?
Bo logo wygląda jak byłoby w Paincie robione ... japierdziele

Nie widzę żadnego designu to odstrasza użytkowników do przeglądania dalej witryny design jest bardzo ważny pozostałe funkcji typu wyślij komuś link nie wszyscy umią ręcznie zaznaczać link więc dodaj jakąś funkcję typu magiczny button kopiuj dodaj jakiś atrybut hover np. z podkreśleniem do całego menu

jednym słowem największy minus brak designu byle szybko i zarabiało błędne myślenie ;/

http://demotywatory.pl/1295602/japierdziele.pl

Poza tym:

brak designu i wszystko ucieka na prawo poza okno przeglądarki.