Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

2 Stron V   1 2 >  
Reply to this topicStart new topic
> Serwis Ogłoszeniowy -> Zwierzęta
LowiczakPL
post 5.05.2016, 08:06:51
Post #1





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

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


Witam

Prośba o ocenę serwisu ogłoszeniowego, rejestracja może być z 5 minutowych e-maii, przed finalnym uruchomieniem wyzeruję bazę, czyli za około tydzień.

Chciałbym go jeszcze dopieścić przed oddaniem dlatego wstawiam tu link i proszę o ocenę, sugestie, info co sądzicie o obsłudze, wyglądzie.

serwis http://pupilaki.pl


Proszę o wyrozumiałość ponieważ mogą pojawić się błędy, nie jestem w stanie sam przewidzieć wszystkich zachowań użytkowników a nie napisałem testów.



--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
gitbejbe
post 5.05.2016, 09:36:54
Post #2





Grupa: Zarejestrowani
Postów: 515
Pomógł: 63
Dołączył: 27.08.2012

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


Cytat
przed finalnym uruchomieniem wyzeruję bazę, czyli za około tydzień.


Cytat
Chciałbym go jeszcze dopieścić przed oddaniem
.

Jeśli zrobiłeś to dla kogoś za kasę, to lepiej oddaj te pieniądze i ściemnij że spalił Ci się komp na którym robiłeś projekt.
Co do moich dodatkowych spostrzeżeń, szacuje że mniej więcej od tygodnia w końcu masz w domu internet - to by tłumaczyło ogólną jakość tego co zrobiłeś.
Go to the top of the page
+Quote Post
nospor
post 5.05.2016, 10:50:02
Post #3





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6290
Dołączył: 27.12.2004




http://pupilaki.pl/ogloszenia/psrry/
Cytat
An error occured, The error has been reported.


Adres email "walidujesz" jedynie w przegladarce przez podanie TYPE=email dla pola... Pierwszy lepszy "haker" moze tam wstawic co tylko zechce

pupilaki.pl/informacje/ostronieNO-I-WLASNIE-ZHAKOWALEM-WYSWIETLANA-TRESC-NA-TWOJEJ-STRONIE.html


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
phpion
post 5.05.2016, 10:52:02
Post #4





Grupa: Moderatorzy
Postów: 6 070
Pomógł: 860
Dołączył: 10.12.2003
Skąd: Dąbrowa Górnicza




http://pupilaki.pl/ogloszenia/tu-moge-wpis...taj-tez,19.html <- spójrz na segmenty linku
Go to the top of the page
+Quote Post
nospor
post 5.05.2016, 10:56:37
Post #5





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6290
Dołączył: 27.12.2004




pupilaki.pl/ogloszenia/koty/uslugi/Ktos zyczliwy moze ci te strone troche zindeksowac inaczej jakbys chcial,19.html

@up haha, o tym samym pomyslelismy wink.gif


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
LowiczakPL
post 5.05.2016, 11:33:16
Post #6





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

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


dzięki za info, biorę się za robotę wink.gif

powyższe błędy poprawione z wyjątkiem Walidacji e-maila aczkolwiek każda zmienna wpadająca do strony jest validowana przez filter_var
ale mimo wszystko nie sprawdzam poprawności e-maila po stronie serwera

Ten post edytował LowiczakPL 5.05.2016, 12:07:32


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
nospor
post 5.05.2016, 12:22:07
Post #7





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6290
Dołączył: 27.12.2004




http://pupilaki.pl/ogloszenia/koty/uslugi/...upilem,190.html

Teraz juz tak poprawiles ze linki ktore dzialaly, juz teraz maja 404.... tongue.gif
http://pupilaki.pl/informacje/ostronie.html


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
aras785
post 5.05.2016, 12:50:07
Post #8





Grupa: Zarejestrowani
Postów: 859
Pomógł: 177
Dołączył: 29.10.2009

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


Cytat(gitbejbe @ 5.05.2016, 10:36:54 ) *
.

Jeśli zrobiłeś to dla kogoś za kasę, to lepiej oddaj te pieniądze i ściemnij że spalił Ci się komp na którym robiłeś projekt.
Co do moich dodatkowych spostrzeżeń, szacuje że mniej więcej od tygodnia w końcu masz w domu internet - to by tłumaczyło ogólną jakość tego co zrobiłeś.


Swoje mądrości zostaw w domu
Go to the top of the page
+Quote Post
LowiczakPL
post 5.05.2016, 13:23:33
Post #9





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

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


Cytat(nospor @ 5.05.2016, 13:22:07 ) *
http://pupilaki.pl/ogloszenia/koty/uslugi/...upilem,190.html

Teraz juz tak poprawiles ze linki ktore dzialaly, juz teraz maja 404.... tongue.gif
http://pupilaki.pl/informacje/ostronie.html



no właśnie te linki nie działały, dzięki Wam 404 teraz jest prawidłową reakcją wink.gif


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
nospor
post 5.05.2016, 13:30:59
Post #10





Grupa: Moderatorzy
Postów: 36 440
Pomógł: 6290
Dołączył: 27.12.2004




No przeciez ten link:
http://pupilaki.pl/informacje/ostronie.html
jest jak najbardziej prawidlowy.... Nawet sam do niego linkujesz na stronie... Czytaj troche uwazniej


Zas ten link nadal masz zly
http://pupilaki.pl/ogloszenia/koty/uslugi/...upilem,190.html
podalem 190 i ci sie appka wykrzacza. Po raz kolejny - czytaj uwaznie co ci piszemy


--------------------

"Myśl, myśl, myśl..." - Kubuś Puchatek || "Manual, manual, manual..." - Kubuś Programista
"Szukaj, szukaj, szukaj..." - Kubuś Odkrywca || "Debuguj, debuguj, debuguj..." - Kubuś Developer

Go to the top of the page
+Quote Post
LowiczakPL
post 5.05.2016, 13:54:00
Post #11





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

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


Cytat(nospor @ 5.05.2016, 14:30:59 ) *
No przeciez ten link:
http://pupilaki.pl/informacje/ostronie.html
jest jak najbardziej prawidlowy.... Nawet sam do niego linkujesz na stronie... Czytaj troche uwazniej


Zas ten link nadal masz zly
http://pupilaki.pl/ogloszenia/koty/uslugi/...upilem,190.html
podalem 190 i ci sie appka wykrzacza. Po raz kolejny - czytaj uwaznie co ci piszemy


sorki zasugerowałem się pierwszym linkiem, on jest w stopce ale ta strona nie istnieje w systemie (nie ma jeszcze klasy do jej obsługi)

co do 2 linka to już poprawiłem błąd przekierowania

Ten post edytował LowiczakPL 5.05.2016, 13:55:59


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
Pyton_000
post 5.05.2016, 14:02:52
Post #12





Grupa: Zarejestrowani
Postów: 8 068
Pomógł: 1414
Dołączył: 26.10.2005

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


Zmień grafikę. Jest okropna.
Nie wiadomo co to za serwis jak się wchodzi pierwszy raz.
Takie to z .... wszystko.

Masz, oglądaj... https://www.webresourcesfree.com/best-class...site-templates/
Go to the top of the page
+Quote Post
LowiczakPL
post 5.05.2016, 14:10:23
Post #13





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

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


Co do szaty to zdaję sobie z tego sprawę, nie jestem w tym mocny, nie jestem w tym nawet słaby, gdzieś tam na dole listy poniżej Sida się znajduję.

Co do linka to ciekawe inspiracje, dzięki.

Ten post edytował LowiczakPL 5.05.2016, 14:20:43


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
Comandeer
post 5.05.2016, 16:41:34
Post #14





Grupa: Zarejestrowani
Postów: 1 268
Pomógł: 254
Dołączył: 11.06.2009
Skąd: Świętochłowice

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


Serwis jest całkowicie niedostępny z powodu całkowicie bezsensownego HTML-a… a > button – jaki to ma sens? Tym sposobem dla każdego linku mamy podwójny tab stop, z czego jeden jest anonsowany jako link, a drugi – jako przycisk. No i tym samym obydwa tab stopy różnie działają, co de facto uniemożliwia sensowną nawigację klawiaturą…

[alt] dla obrazków w ogłoszeniach są na tyle powtarzalne, że zastanowiłbym się nad wyłączeniem całkowicie slidera dla czytników ekranowych i zostawieniu samej nazwy produktu. Zwłaszcza, że te opisy jedyne, co robią, to tę nazwę duplikują.

Tradycyjny błąd dostępności: formularze bez label… Polecam poczytać specyfikację HTML5 w tym zakresie, bo [placeholder] NIE zastępuje label!

Czemu do walidacji formularza używasz standardowego alerta browsera, skoro Bootstrap dostarcza ładniejszego?

Po wysłaniu pustego formularza rejestracji, zwracana jest pusta strona…

Zamiast tworzyć milion funkcji do każdego formularza osobno, warto zrobić po prostu klasę walidatora. No i form[name] stosowało się 10 lat temu.

Strony błędów są tak bardzo nieprzyjazne, że aż nasuwają podejrzenie, że nie ma tam żadnej obsługi błędów wink.gif

Checkboxy bez klikalnej etykiety? Sorry, ale to już jest przegięcie – zarówno od strony a11y, jak i UX.


--------------------
Go to the top of the page
+Quote Post
LowiczakPL
post 5.05.2016, 17:39:30
Post #15





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

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


Co do rejestracji i braku Label, to tak jakoś mi się to podobało na innych stronach i uznałem że label jest w tym momencie zbędny, że wystarczy placeholder, po co powielać 2 razy tą samą nazwę.

Z altem obrazków nie miałem pojęcia co zrobić, brak to błąd walidacji dlatego powieliłem

Co do alerta JS to był zrobiony na szybko ponieważ nie opracowałem jeszcze klasy do Modala

Pusty formularz się wysyła ale podczas dodania ogłoszenia, ponieważ nie waliduję go na razie oczywiście, zapomniałem tego zrobić, dzięki Tobie błąd zostanie naprawiony.

404 było robione w kilka minut na sztuke dlatego jest takie surowe

Label podczas dodawania ogłoszenia był poprawiony ale zapomniałem wgrać na serwer, już jest zaktualizowane.

Ale co do BUTTONÓW i A to może faktycznie przegiąłem nie miałem weny jak mam to zrobić. Pyton_000 podesłał linka do szablonów więc coś wybiorę i zaimplementuję inne rozwiązanie. Na razie jest to wersja rozwojowa.
-------------
Form logowania ma już label, dodałem też Modal

Ten post edytował LowiczakPL 5.05.2016, 18:40:10


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
Comandeer
post 5.05.2016, 20:28:53
Post #16





Grupa: Zarejestrowani
Postów: 1 268
Pomógł: 254
Dołączył: 11.06.2009
Skąd: Świętochłowice

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


Co do label, jak już naprawdę nie pasują do designu, to jest .visuallyhiddenhttp://a11yproject.com/posts/how-to-hide-content


--------------------
Go to the top of the page
+Quote Post
LowiczakPL
post 6.05.2016, 06:58:38
Post #17





Grupa: Zarejestrowani
Postów: 531
Pomógł: 55
Dołączył: 3.01.2016
Skąd: Łowicz

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


Cytat(Comandeer @ 5.05.2016, 21:28:53 ) *
Co do label, jak już naprawdę nie pasują do designu, to jest .visuallyhidden ? http://a11yproject.com/posts/how-to-hide-content



Zaczynam analizować stronę z przesłanego linku, dzięki serdeczne za te wskazówki, nie do końca rozumiałem o co Ci chodzi do momentu kiedy nie dałeś mi tego linku.

Od tej pory będę starał się robić strony przyjazne dla czytników ekranowych.

------

Pokombinowałem i ciekaw jestem czy teraz czytniki poradzą sobie z serwisem.

Ten post edytował LowiczakPL 6.05.2016, 08:24:39


--------------------
Szukam zleceń Symfony, Laravel, Back-End, Front-End, PHP, MySQL ...
Go to the top of the page
+Quote Post
daro0
post 17.05.2016, 09:48:03
Post #18





Grupa: Zarejestrowani
Postów: 88
Pomógł: 12
Dołączył: 17.09.2014
Skąd: Krasnystaw

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


Cytat(LowiczakPL @ 5.05.2016, 12:33:16 ) *
dzięki za info, biorę się za robotę wink.gif

powyższe błędy poprawione z wyjątkiem Walidacji e-maila aczkolwiek każda zmienna wpadająca do strony jest validowana przez filter_var
ale mimo wszystko nie sprawdzam poprawności e-maila po stronie serwera


No cóż, nie wiem na czym jest Twoj serwis, jeśli jednak nie na jakimś frameworku, to biorąc pod uwagę obecne rozwiązania, sprawa jest trochę bardziej złożona:

https://kohanaframework.org/3.2/guide/api/Valid#email

Generalnie chodzi o to:

  1. $expression = '/^[-_a-z0-9\'+*$^&%=~!?{}]++(?:\.[-_a-z0-9\'+*$^&%=~!?{}]+)*+@(?:(?![-.])[-a-z0-9.]+(?<![-.])\.[a-z]{2,6}|\d{1,3}(?:\.\d{1,3}){3})$/iD';
  2.  
  3. return (bool) preg_match($expression, (string) $email);


Weź pod uwagę jeszcze jedną z takich opcji, że ktoś może wykorzystać choćby CURL i POST-em dodać np. jakieś nowe ogłoszenie, no chyba że masz jakieś zabezpieczenie przed tym. Więc można by się zastanawiać nad tym czy sama tylko walidacja po stronie przeglądarki jest wystarczającym (bezpiecznym) mechanizmem...
Go to the top of the page
+Quote Post
Spawnm
post 17.05.2016, 10:21:51
Post #19





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




Hasło może mieć tak 1 znak jak i milion. Wprowadź ograniczenia.
Go to the top of the page
+Quote Post
daro0
post 17.05.2016, 10:27:51
Post #20





Grupa: Zarejestrowani
Postów: 88
Pomógł: 12
Dołączył: 17.09.2014
Skąd: Krasnystaw

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


A i jeszcze brakuje automatycznego przekierowania 301 z www na nie-www albo na odwrót

https://support.tigertech.net/redirect-www

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: 28.03.2024 - 10:16