Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

2 Stron V   1 2 >  
Reply to this topicStart new topic
> Wiem-Jem.pl - Cała prawda o jedzeniu
Kildyt
post 4.03.2010, 22:36:20
Post #1





Grupa: Zarejestrowani
Postów: 869
Pomógł: 53
Dołączył: 20.10.2003
Skąd: Przeworsk

Ostrzeżenie: (0%)
-----


Cześć!

Proszę o ocenę mojej świeżej strony. Strona Wiem-Jem.pl - Cała prawda o jedzeniu powstała aby pomóc w rozszyfrowaniu składników popularnych produktów spożywczych.

Ocenie podlega wszystko.

Dzięki za wszystkie wypowiedzi.


--------------------
Go to the top of the page
+Quote Post
piotrooo89
post 4.03.2010, 22:41:25
Post #2


Newsman


Grupa: Moderatorzy
Postów: 4 005
Pomógł: 548
Dołączył: 7.04.2008
Skąd: Trzebinia/Kraków




jak wszystko dopracowane i dopięte na ostatni guzik smile.gif mnie się naprawde podoba. nie walidowałem, muszę?



--------------------
Go to the top of the page
+Quote Post
thek
post 4.03.2010, 23:08:46
Post #3





Grupa: Moderatorzy
Postów: 4 362
Pomógł: 714
Dołączył: 12.02.2009
Skąd: Jak się położę tak leżę :D




Graficznie jest dobrze, ale zauważyłem, że paginacja Ci szwankuje. Przykładowo poprzez katalog wybrałem sobie Wody i ma ona minimum 4 podstrony, ale linki paginatora pokazują na główny katalog. Tak jakby tworzenie linka SEO-friendly Ci nawaliło. Widzę także, że breadcrumb z główną kategorią (Woda & napoje) gdy jesteśmy w podkategorii (Woda) także prowadzi do głównej strony katalogu, ale możliwe, że to na przyszłość, gdy kategorii więcej będzie, takie "zabezpieczenie", które obecnie zamiast do strony głównej kategorii prowadzi do katalogu produktów wszystkich. Poza tym nie testowałem strony pod katem ewentualnych ataków. Od tego są odpowiednie strony i programy (także pluginy przeglądarek) więc możesz ich sam użyć dla pewności. Oceniając jednak layout uważam, że jest prosty, ale funkcjonalny. A to moim zdaniem najbardziej oprócz bezpieczeństwa się liczy smile.gif Zastanawialbym się tylko ewentualnie nad sensem autocomplete'a z grafikami. To może po pewnym wypełnieniu się produktami bazy, zmulić serwis poprzez ciągłe przesyłanie grafik do userów. Odczujesz to na transferze niestety wtedy.


--------------------
Najpierw był manual... Jeśli tam nie zawarto słów mądrości to zapytaj wszechwiedzącego Google zadając mu własciwe pytania. A jeśli i on milczy to Twój problem nie istnieje :D
Go to the top of the page
+Quote Post
Kildyt
post 4.03.2010, 23:22:16
Post #4





Grupa: Zarejestrowani
Postów: 869
Pomógł: 53
Dołączył: 20.10.2003
Skąd: Przeworsk

Ostrzeżenie: (0%)
-----


Cytat(thek @ 5.03.2010, 01:08:46 ) *
Graficznie jest dobrze, ale zauważyłem, że paginacja Ci szwankuje. Przykładowo poprzez katalog wybrałem sobie Wody i ma ona minimum 4 podstrony, ale linki paginatora pokazują na główny katalog. Tak jakby tworzenie linka SEO-friendly Ci nawaliło. Widzę także, że breadcrumb z główną kategorią (Woda & napoje) gdy jesteśmy w podkategorii (Woda) także prowadzi do głównej strony katalogu, ale możliwe, że to na przyszłość, gdy kategorii więcej będzie, takie "zabezpieczenie", które obecnie zamiast do strony głównej kategorii prowadzi do katalogu produktów wszystkich. Poza tym nie testowałem strony pod katem ewentualnych ataków. Od tego są odpowiednie strony i programy (także pluginy przeglądarek) więc możesz ich sam użyć dla pewności. Oceniając jednak layout uważam, że jest prosty, ale funkcjonalny. A to moim zdaniem najbardziej oprócz bezpieczeństwa się liczy smile.gif Zastanawialbym się tylko ewentualnie nad sensem autocomplete'a z grafikami. To może po pewnym wypełnieniu się produktami bazy, zmulić serwis poprzez ciągłe przesyłanie grafik do userów. Odczujesz to na transferze niestety wtedy.
Błąd z paginacją już naprawiony. Nie tworzyłem osobnych podstron dla podkategorii. Może gdy strona będzie się rozwijała taki krok będzie wymagany.

@piotrooo89 sprawdzałem i walidator nic nie wypluwał (możliwe, że jakąś podstronę przegapiłem). Walidator CSS-a trochę wypluwa bo używam kilku nowości z CSS3 w mniejszych elementach.


--------------------
Go to the top of the page
+Quote Post
Rymar
post 5.03.2010, 01:25:20
Post #5





Grupa: Zarejestrowani
Postów: 33
Pomógł: 2
Dołączył: 18.09.2008

Ostrzeżenie: (0%)
-----


Narazie z tego co widze to strona psuje się całkowicie pod IE 6 i tak trochę pod IE 7 js nie działa przy komentarzach, a tekst sie rozjeżdza gdzieniegdzie

tak ma być?
Go to the top of the page
+Quote Post
SHiP
post 5.03.2010, 12:52:41
Post #6





Grupa: Zarejestrowani
Postów: 697
Pomógł: 47
Dołączył: 19.12.2003
Skąd: Lublin

Ostrzeżenie: (0%)
-----


Fajna ta strona. Jak to jest z tym E330? Jest rakotwórczy czy nie? Bo są sprzeczne informacje na jego temat na twojej stronie.


--------------------
Warsztat: Kubuntu, PhpStorm, Opera
Go to the top of the page
+Quote Post
Spawnm
post 5.03.2010, 13:29:28
Post #7





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




denerwujący jest autosuggest, najpierw pojawia się wielki pusty box potem po ok 1-2s pojawiają się obrazki/linki w nim ... dodatkowo box ma zawsze ten sam height bez względu na ilość znalezionych produktów ...
Go to the top of the page
+Quote Post
thomson89
post 5.03.2010, 14:48:22
Post #8





Grupa: Zarejestrowani
Postów: 1 178
Pomógł: 51
Dołączył: 7.01.2009
Skąd: Gdańsk

Ostrzeżenie: (0%)
-----


Cytat(Spawnm @ 5.03.2010, 13:29:28 ) *
denerwujący jest autosuggest, najpierw pojawia się wielki pusty box potem po ok 1-2s pojawiają się obrazki/linki w nim ... dodatkowo box ma zawsze ten sam height bez względu na ilość znalezionych produktów ...


Podpinam się pod wypowiedź poprzednika. Trochę to irytuje...


--------------------
Sklep 70%
Go to the top of the page
+Quote Post
Kildyt
post 5.03.2010, 21:29:56
Post #9





Grupa: Zarejestrowani
Postów: 869
Pomógł: 53
Dołączył: 20.10.2003
Skąd: Przeworsk

Ostrzeżenie: (0%)
-----


Cytat(Rymar @ 5.03.2010, 03:25:20 ) *
Narazie z tego co widze to strona psuje się całkowicie pod IE 6 i tak trochę pod IE 7 js nie działa przy komentarzach, a tekst sie rozjeżdza gdzieniegdzie

tak ma być?
Sprawdź czy w meta-tagach jest podpięty specjalny styl (oddzielny dla IE6, oddzielny dla IE7 i oddzielny dla IE8). Jakiego używasz os-a? Wyślij screenshoty.

Cytat(SHiP @ 5.03.2010, 14:52:41 ) *
Fajna ta strona. Jak to jest z tym E330? Jest rakotwórczy czy nie? Bo są sprzeczne informacje na jego temat na twojej stronie.
Wyrzuciłem informację o rakotwórczości E330 z niepożądanych następstw, a w informacji zastrzegłem, że informacja jest sprzeczna. Uznałem ta informację za błąd, gdyż inne źródła nic o tym nie mówią. Dzięki za cynk.
A propos, znalazłeś może jakieś luki w zabezpieczeniach? Zauważyłem, że interesujesz się tą tematyką.

Cytat(Spawnm @ 5.03.2010, 15:29:28 ) *
denerwujący jest autosuggest, najpierw pojawia się wielki pusty box potem po ok 1-2s pojawiają się obrazki/linki w nim ... dodatkowo box ma zawsze ten sam height bez względu na ilość znalezionych produktów ...
Racja, przydałby się jakiś loader. Co do stałego height'a to sądzę, że obecny jest wystarczający. Nie mam za bardzo pomysłu jak można by to inaczej zaprojektować. Automatyczny height raczej nie wchodzi w grę.

Edit:
W ATOM-ach mam kilka błędów. Za niedługo poprawię.

Ten post edytował Kildyt 5.03.2010, 21:33:03


--------------------
Go to the top of the page
+Quote Post
SHiP
post 5.03.2010, 22:33:57
Post #10





Grupa: Zarejestrowani
Postów: 697
Pomógł: 47
Dołączył: 19.12.2003
Skąd: Lublin

Ostrzeżenie: (0%)
-----


Cytat
A propos, znalazłeś może jakieś luki w zabezpieczeniach? Zauważyłem, że interesujesz się tą tematyką.

To była pierwsza rzecz jaką zacząłem sprawdzać =). Na szczęście istnieją jeszcze dobrzy programiści i Ty jesteś tego przykładem. Ładnie filtrujesz znaki w loginach, mod_rewrite wstępnie przecedza śmieci. Zmieniasz id sesji przy każdym nowym generowaniu etc.

Jedyne do czego mógłbym się przyczepić to włączone Notice na serwerze produkcyjnym winksmiley.jpg. Obsługę błędów powinno się wyłączać ponieważ pomaga to na zbadanie struktury strony. I tak np w 12 linijce w carrot.php nie filtrujesz danych przez co łatwo wygenerować błąd podając tablicę jako argument
Cytat
Notice: Array to string conversion in /home/krystian/wiemjem/carrot.php on line 12

Wbrew pozorom taki komunikat to sporo informacji winksmiley.jpg.

Nie mam też czasu się zagłębiać w obrazki, bo musiałbym wiedzieć do jakiego katalogu są wrzucane, a zanim bym to obczaił troszkę by z tym zeszło. Zreszta zapewne sam je moderujesz zanim gdziekolwiek trafią.

EDIT: dodaj overflow w kilku miejscach bo mój "login" rozwala design lekko :/ ups...

EDIT2: Ojj wiecej jest takich miejsc

Cytat
Notice: Array to string conversion in /home/krystian/wiemjem/page.php on line 6
Notice: Array to string conversion in /home/krystian/wiemjem/page.php on line 9
Notice: Array to string conversion in /home/krystian/wiemjem/carrot.php on line 12
Warning: htmlspecialchars() expects parameter 1 to be string, array given in /home/krystian/wiemjem/engine/page.php on line 143

jem-wiem.pl/engine/ - zmien prawa dostępu aby nie dało się przeglądać tego katalogu

Ten post edytował SHiP 5.03.2010, 22:51:36


--------------------
Warsztat: Kubuntu, PhpStorm, Opera
Go to the top of the page
+Quote Post
Kildyt
post 5.03.2010, 22:50:20
Post #11





Grupa: Zarejestrowani
Postów: 869
Pomógł: 53
Dołączył: 20.10.2003
Skąd: Przeworsk

Ostrzeżenie: (0%)
-----


Cytat(SHiP @ 6.03.2010, 00:33:57 ) *
To była pierwsza rzecz jaką zacząłem sprawdzać =). Na szczęście istnieją jeszcze dobrzy programiści i Ty jesteś tego przykładem. Ładnie filtrujesz znaki w loginach, mod_rewrite wstępnie przecedza śmieci. Zmieniasz id sesji przy każdym nowym generowaniu etc.
Z pewnością są to pochwały nad wyraz ponieważ zbyt mało czasu poświęcam na kształtowaniu się w zakresie bezpieczeństwa, a obecną wiedzę załapałem "przypadkiem", ale i tak dzięki. winksmiley.jpg

Cytat(SHiP @ 6.03.2010, 00:33:57 ) *
Jedyne do czego mógłbym się przyczepić to włączone Notice na serwerze produkcyjnym winksmiley.jpg. Obsługę błędów powinno się wyłączać ponieważ pomaga to na zbadanie struktury strony. I tak np w 12 linijce w carrot.php nie filtrujesz danych przez co łatwo wygenerować błąd podając tablicę jako argument

Wbrew pozorom taki komunikat to sporo informacji winksmiley.jpg.
Ok, wyświetlanie błędów wyłączę. W 12 linijce znajduje się funkcja session_start().

Cytat(SHiP @ 6.03.2010, 00:33:57 ) *
Nie mam też czasu się zagłębiać w obrazki, bo musiałbym wiedzieć do jakiego katalogu są wrzucane, a zanim bym to obczaił troszkę by z tym zeszło. Zreszta zapewne sam je moderujesz zanim gdziekolwiek trafią.
Wszystkie obrazki zapisuję przez GD, więc chyba nie powinno być problemów.


Cytat(SHiP @ 6.03.2010, 00:33:57 ) *
EDIT: dodaj overflow w kilku miejscach bo mój "login" rozwala design lekko :/ ups...
Ok, nie pomyślałem o tym. Thx.


--------------------
Go to the top of the page
+Quote Post
SHiP
post 5.03.2010, 23:37:46
Post #12





Grupa: Zarejestrowani
Postów: 697
Pomógł: 47
Dołączył: 19.12.2003
Skąd: Lublin

Ostrzeżenie: (0%)
-----


Nie chodzi o to jaka funkcja się znajduje ale to, że zwraca ona bład przy przekazaniu tablicy. Musisz wyłączyć błedy lub sprawdzać jakiego typu dane przychodzą z ciastek. Chodzi o to, że potencjalny atakujący ma nie znać żadnych informacji na temat strony(np. dokładnego położenia na serwerze), serwera oraz ich konfiguracji ponieważ w przypadku wykrycia jakiejs dziury(nie koniecznie na stronie, moze być w apache lub php) przeprowadzenie ataku jest juz dużo łatwiejsze.

Pozdrawiam winksmiley.jpg


--------------------
Warsztat: Kubuntu, PhpStorm, Opera
Go to the top of the page
+Quote Post
Rymar
post 6.03.2010, 10:38:31
Post #13





Grupa: Zarejestrowani
Postów: 33
Pomógł: 2
Dołączył: 18.09.2008

Ostrzeżenie: (0%)
-----


Cytat
Sprawdź czy w meta-tagach jest podpięty specjalny styl (oddzielny dla IE6, oddzielny dla IE7 i oddzielny dla IE8). Jakiego używasz os-a? Wyślij screenshoty.


Wina po mojej stronie, IE Tester coś wariuje smile.gif
Go to the top of the page
+Quote Post
athabus
post 6.03.2010, 13:16:46
Post #14





Grupa: Zarejestrowani
Postów: 898
Pomógł: 48
Dołączył: 2.11.2005
Skąd: Poznań

Ostrzeżenie: (0%)
-----


Ja coś napiszę od strony użyszkodnika.

Pomysł dla mnie bomba - mam nawet za sprawą żony w domu taką książeczkę o tych wszystkich dziwnych rzeczach co dodają do jedzenia ;-) Strach się bać ;-)

Fajnie, że zrobiłeś taką wersję elektroniczną z konkretnymi produktami ale, 90% produktów w które wszedłem nie ma informacji o składnikach. Tj. jest wymieniony składnik ale nie ma linku co to robi i czemu jest złe albo dobre. Bez tego komponentu strona jest bezużyteczna. Jeśli uzupełnisz te informacje to będzie to świetna strona. Pokażę ją żonie bo ona uwielbia takie klimaty jedzeniowe.

Zanim jednak stronę upublicznisz dla szerokiego grona ludzi, to warto ją dopracować pod względem ilości informacji.

Swoją drogą to znalazłeś fajną niszę - myślę, że możesz z tego mieć niezłe pieniądze (o ile nie robisz tego na zlecenie) z reklam i polecania produktów.

Grafika - dla mnie bomba - dokładnie tak jak lubię, czyli ładna, nienarzucająca się i dostosowana do tematyki.
Go to the top of the page
+Quote Post
Ja_ck
post 6.03.2010, 17:42:23
Post #15





Grupa: Zarejestrowani
Postów: 23
Pomógł: 1
Dołączył: 4.02.2010

Ostrzeżenie: (0%)
-----


Pomysl naprawde dobry smile.gif

Zycze powodzenia smile.gif
Go to the top of the page
+Quote Post
bim2
post 7.03.2010, 16:15:04
Post #16





Grupa: Zarejestrowani
Postów: 1 873
Pomógł: 152
Dołączył: 9.04.2006
Skąd: Berlin

Ostrzeżenie: (0%)
-----


Jak już pisałem na gg, pomysł oraz opracowanie świetne! biggrin.gif Może uda ci sie i ten serwis będzie źródłem twoich przyszłych dochodów smile.gif


--------------------
Go to the top of the page
+Quote Post
frytek92
post 7.03.2010, 18:46:39
Post #17





Grupa: Zarejestrowani
Postów: 163
Pomógł: 10
Dołączył: 9.11.2007
Skąd: Goleniów

Ostrzeżenie: (0%)
-----


Jestem pod wielkim wrażeniem naprawdę super zrobiony i co najważniejsze będę z niego korzystał. A przyszłości jak pisał mój poprzednik możesz spokojnie na nim zarabiać bo dietetyka to naprawdę rozbudowany rynek.

Ten post edytował frytek92 7.03.2010, 18:52:34


--------------------
Go to the top of the page
+Quote Post
zwierzołak
post 7.03.2010, 21:11:50
Post #18





Grupa: Zarejestrowani
Postów: 78
Pomógł: 1
Dołączył: 25.03.2008

Ostrzeżenie: (0%)
-----


Dziwnie to wygląda, że są tylko 3 losowe produkty na stronie głównej. Robi to wrażenie, że strona ma jeszcze śladowe ilości treści i niema czym wypełnić tego boxa w którym wydaje się (z rozmiaru), że powinno być 6 produktów.

Swoją drogą mam dla Ciebie propozycję alternatywnego sloganu przewodniego serwisu: "Czy to jeszcze pieczywo, czy sztuczne tworzywo?" smile.gif
Go to the top of the page
+Quote Post
Kildyt
post 5.05.2010, 12:51:58
Post #19





Grupa: Zarejestrowani
Postów: 869
Pomógł: 53
Dołączył: 20.10.2003
Skąd: Przeworsk

Ostrzeżenie: (0%)
-----


Dzięki wielkie wszystkim za wypowiedzi. Pomoże to w zbudowaniu lepszej strony. Na flakerze można znaleźć historyjkę gdzie publikowana jest aktualizowana lista TODO.

Na deser: strona błędy na wiem-jem. Wygląda lepiej na nowszych przeglądarkach.


New:

Chwilka czasu już minęła i chciałbym zaprezentować odświeżony theme i prosić was o ocenę nowego logotypu (obecny jest do luftu).

Ten post edytował Kildyt 5.05.2010, 12:52:34


--------------------
Go to the top of the page
+Quote Post
Spawnm
post 5.05.2010, 13:07:49
Post #20





Grupa: Moderatorzy
Postów: 4 069
Pomógł: 497
Dołączył: 11.05.2007
Skąd: Warszawa




Zbyt pixelowaty.
Go to the top of the page
+Quote Post

2 Stron V   1 2 >
Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 



RSS Wersja Lo-Fi Aktualny czas: 14.06.2025 - 17:51