![]() |
![]() ![]() |
![]() |
![]()
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. -------------------- |
|
|
![]()
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
![]() -------------------- |
|
|
![]()
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
![]() -------------------- 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
|
|
|
![]()
Post
#4
|
|
![]() Grupa: Zarejestrowani Postów: 869 Pomógł: 53 Dołączył: 20.10.2003 Skąd: Przeworsk Ostrzeżenie: (0%) ![]() ![]() |
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 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. -------------------- |
|
|
![]()
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ć? |
|
|
![]()
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
|
|
|
![]()
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 ...
|
|
|
![]()
Post
#8
|
|
![]() Grupa: Zarejestrowani Postów: 1 178 Pomógł: 51 Dołączył: 7.01.2009 Skąd: Gdańsk Ostrzeżenie: (0%) ![]() ![]() |
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
![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() ![]() |
|
|
![]()
Post
#9
|
|
![]() Grupa: Zarejestrowani Postów: 869 Pomógł: 53 Dołączył: 20.10.2003 Skąd: Przeworsk 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 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.tak ma być? 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ą. 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 -------------------- |
|
|
![]()
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 ![]() Cytat Notice: Array to string conversion in /home/krystian/wiemjem/carrot.php on line 12 Wbrew pozorom taki komunikat to sporo informacji ![]() 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
|
|
|
![]()
Post
#11
|
|
![]() Grupa: Zarejestrowani Postów: 869 Pomógł: 53 Dołączył: 20.10.2003 Skąd: Przeworsk Ostrzeżenie: (0%) ![]() ![]() |
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. ![]() Jedyne do czego mógłbym się przyczepić to włączone Notice na serwerze produkcyjnym Ok, wyświetlanie błędów wyłączę. W 12 linijce znajduje się funkcja session_start().![]() Wbrew pozorom taki komunikat to sporo informacji ![]() 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.EDIT: dodaj overflow w kilku miejscach bo mój "login" rozwala design lekko :/ ups... Ok, nie pomyślałem o tym. Thx.-------------------- |
|
|
![]()
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 ![]() -------------------- Warsztat: Kubuntu, PhpStorm, Opera
|
|
|
![]()
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 ![]() |
|
|
![]()
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. |
|
|
![]()
Post
#15
|
|
![]() Grupa: Zarejestrowani Postów: 23 Pomógł: 1 Dołączył: 4.02.2010 Ostrzeżenie: (0%) ![]() ![]() |
Pomysl naprawde dobry
![]() Zycze powodzenia ![]() |
|
|
![]()
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!
![]() ![]() -------------------- |
|
|
![]()
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 -------------------- |
|
|
![]()
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?" ![]() |
|
|
![]()
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 -------------------- |
|
|
![]()
Post
#20
|
|
![]() Grupa: Moderatorzy Postów: 4 069 Pomógł: 497 Dołączył: 11.05.2007 Skąd: Warszawa ![]() |
Zbyt pixelowaty.
|
|
|
![]() ![]() |
![]() |
Wersja Lo-Fi | Aktualny czas: 14.06.2025 - 17:51 |