Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
Reply to this topicStart new topic
> Sklep internetowy
kujol
post 2.11.2013, 19:18:44
Post #1





Grupa: Zarejestrowani
Postów: 324
Pomógł: 27
Dołączył: 21.07.2013

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


Witam.
Dziś chciałbym zaprezentować mój pierwszy cms sklepu internetowego w dużej wersji beta.
W tej chwili sklep posiada najważniejsze funkcje.
Użytkownicy:

- Dodawanie do koszyka
- Zamawianie

Administrator:

- Podgląd na sklep
- Dodawanie/usuwanie produktu
- Zarządzanie treściami na stronie
- Dodawanie/usuwanie administratora
- Usuwanie użytkownika
- Ustawienie gł. adresu email
- Realizacja zamówień

Cms z czasem będzie wzbogacany o nowe funkcje, lecz teraz wystawiam go do oceny w pomocy znalezienia błędów, propozycji o co może zostać rozbudowany i oceny ogólnej.
Grafikiem nie jestem, więc dlatego to tak wygląda jak wygląda biggrin.gif, ale to i tak się zmieni.
W przyszłości chcę dodać:

- Zmianę tła, i grafiki na całej stronie
- Ustawienia kąta (zmiana hasła itp.)
- System prywatnych wiadomości
- System komentarzy

www.zul.cba.pl/SklepNew/
Login: Uzytkownik
Hasło: Uzytkownik

www.zul.cba.pl/SklepNew/Admin/
Login: Administrator
Hasło: Administrator




--------------------
;)
Go to the top of the page
+Quote Post
Evinek
post 2.11.2013, 20:01:26
Post #2





Grupa: Zarejestrowani
Postów: 280
Pomógł: 46
Dołączył: 23.03.2010

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


No dobra, nie jesteś dobrym grafikiem, ale mogłeś normalny szablon zrobić lub nawet użyć gotowego. Na to nie da rady nawet patrzeć.
- Obrazki w produktach nie działają.
- Przy dodawaniu produktów TRZEBA dodać wszystkie zdjęcia - a jak chce dodać jedno? Albo jak chce dodać 5? Najłatwiej zrobić dodatkową podstronę i wysyłać po kolei zdjęcia do danego produktu. Wtedy będziesz miał możliwość dodawania ile chcesz zdjęć.
- Nie jestem dobrym testerem zabezpieczeń, ale masz: http://www.zul.cba.pl/SklepNew/Uzytkownik/...0World%3C/h1%3E

Co mnie zdziwiło to to, że nie można usunąć ostatniego Admina. Tu masz u mnie plus.
Go to the top of the page
+Quote Post
valkirek
post 2.11.2013, 21:06:18
Post #3





Grupa: Zarejestrowani
Postów: 117
Pomógł: 6
Dołączył: 24.07.2013

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


Cytat
Grafikiem nie jestem, więc dlatego to tak wygląda jak wygląda

Wow jak do tego doszedłeś?

No faktycznie nie jesteś a ja myślałem, że u mnie kiepsko z grafiką. Oceniałem swoją grafikę 2/10 ale u Ciebie ocena idzie w minusy...

Tragedia ten wygląd, nawet nie mam chęci tego testować...
Go to the top of the page
+Quote Post
shmit
post 2.11.2013, 21:25:04
Post #4





Grupa: Zarejestrowani
Postów: 7
Pomógł: 1
Dołączył: 29.10.2013
Skąd: Kielce/Katowice

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


grafika to pierwsze, na co klient zwraca uwagę...
czerwony wyróżnia,ale często też drażni...
spróbuj zneutralizować go jakimś tłem, może jakiś metaliczny szary albo po prostu zamień na inny....
myślę,że panowie wtedy chętniej ocenią Twoją prace smile.gif
w zakładce produkty prace może ułatwić podgląd na cały obecny asortyment,dodatkowo dzielony na kategorię

ze strony adobe pobierzesz sobie oryginalnego photoshopa 2 (jesli nie masz)
miłego kolorowania tongue.gif

Ten post edytował shmit 2.11.2013, 21:53:14
Go to the top of the page
+Quote Post
Damonsson
post 2.11.2013, 22:02:18
Post #5





Grupa: Zarejestrowani
Postów: 2 355
Pomógł: 533
Dołączył: 15.01.2010
Skąd: Bydgoszcz

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


SQL injection, chyba we wszystkich możliwych zapytaniach.
Go to the top of the page
+Quote Post
pedro84
post 2.11.2013, 22:02:53
Post #6





Grupa: Nieautoryzowani
Postów: 2 249
Pomógł: 305
Dołączył: 2.10.2006

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


Templatka na ThemeForest to ok $15. Nie ma co oszczędzać, aż oczy bolą wink.gif

Na szybko:
- fotki produktów nie działają
- brak strony błędu: http://www.zul.cba.pl/SklepNew/Uzytkownik/....php?id=%27xxxx
- brak walidacji pola ilość przy dodawaniu do koszyka
- brak przyjaznych URLi
- przy zamówieniu można wpisać dowolny nick - po co to w ogóle dla zalogowanego usera?
- przy userze niezalogowanym brak jakichkolwiek komunikatów.

Ogólnie, to masakra wink.gif


--------------------
Google knows the answer...
Go to the top of the page
+Quote Post
aniolekx
post 3.11.2013, 09:11:35
Post #7





Grupa: Zarejestrowani
Postów: 340
Pomógł: 46
Dołączył: 31.07.2009
Skąd: A

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


aby poprawic wyglad tego sklepu uzyj twitter bootstrap, bedzie znacznie lepiej wygladac.

masz problem z kodowaniem polskich znakow,

widzialem tez gdzies jakis php error

Sa tez takie niedociagniecia jak np gdy w kategorii nie ma produktow to powinienes o tym poinformowac, a aktualnie dalej wyswietlana jest paginacja.

Raczej to wyglada jak jakas bardzo wczesna alfa.
Go to the top of the page
+Quote Post
pyro
post 3.11.2013, 12:34:54
Post #8





Grupa: Zarejestrowani
Postów: 2 148
Pomógł: 230
Dołączył: 26.03.2008

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


Wow, tragedia chyba pod każdym względem:

1.) Grafika i wygląd - no comments
2.) Kod - brak nawet takich podstawowych rzeczy jak doctype albo używanie skryptów już za </html>, walidator leży i kwiczy
3.) Bezpieczeństwo - skrypt chyba podatny na wszystkie możliwe sposoby tongue.gif
4.) Funkcjonalność - wyświetla się dość różnie w różnych przeglądarkach, czasem jakieś errory, nie wyświetlają się polskie znaki itp...

Wymieniać można długo. Nadawać to się do niczego nie nadaje, natomiast jeśli jest to pierwszy skrypt jaki kiedykolwiek zrobiłeś, to jesteś usprawiedliwiony wink.gif .

Ten post edytował pyro 3.11.2013, 12:36:13


--------------------
ET LINGUA EIUS LOQUETUR IUDICIUM
Go to the top of the page
+Quote Post
kujol
post 3.11.2013, 16:44:13
Post #9





Grupa: Zarejestrowani
Postów: 324
Pomógł: 27
Dołączył: 21.07.2013

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


Cytat(Evinek @ 2.11.2013, 20:01:26 ) *
- Obrazki w produktach nie działają.


Obrazki działały, to były puste img, do których dopiero pobierałem zdjęcia z bazy, ale już to naprawiłem.

Cytat(valkirek @ 2.11.2013, 21:06:18 ) *
Tragedia ten wygląd, nawet nie mam chęci tego testować...


Na czymś musiałem to zamontować biggrin.gif, ale jak już wspomniałem, cały wygląd się zmieni.

Cytat(Damonsson @ 2.11.2013, 22:02:18 ) *
SQL injection, chyba we wszystkich możliwych zapytaniach.


Na razie jest to kompletni bez jakichkolwiek zabezpieczeń i filtracji danych pochodzących o użytkownika.

Cytat(pedro84 @ 2.11.2013, 22:02:53 ) *
- przy zamówieniu można wpisać dowolny nick - po co to w ogóle dla zalogowanego usera?


Będzie to bardziej rozbudowany formularz z danymi, które były podane podczas rejestracji.
Można tam będzie zmienić adres zamieszkania klienta lub dane kontaktowe.

Cytat(aniolekx @ 3.11.2013, 09:11:35 ) *
masz problem z kodowaniem polskich znakow,


Na tym poziomie jest to pisane bardziej dla testów, ale i to się zmieni wink.gif.

Cytat(pyro @ 3.11.2013, 12:34:54 ) *
2.) (...) walidator leży i kwiczy


Chociaż walidator Css nie wypadł tak źle- tylko 2 błędy biggrin.gif.




--------------------
;)
Go to the top of the page
+Quote Post
toffiak
post 3.11.2013, 16:52:24
Post #10





Grupa: Zarejestrowani
Postów: 395
Pomógł: 80
Dołączył: 24.08.2009

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


Co prawda strona jest do oceny ale tak naprawdę nie ma co tu oceniać widać że to bardzo wczesna wersja alpha, z oceną poczekam chociażby do bety.

PS Pierwszego skryptu nie powinno się pokazywać publicznie, za rok wstyd będzie Ci się do niego przyznać.


--------------------
Go to the top of the page
+Quote Post
Turson
post 3.11.2013, 22:22:38
Post #11





Grupa: Zarejestrowani
Postów: 4 291
Pomógł: 829
Dołączył: 14.02.2009
Skąd: łódź

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


Cytat(kujol @ 3.11.2013, 16:44:13 ) *
Na razie jest to kompletni bez jakichkolwiek zabezpieczeń i filtracji danych pochodzących o użytkownika.

To po co w ogóle dajesz to do testów? facepalmxd.gif
Go to the top of the page
+Quote Post
freemp3
post 4.11.2013, 10:04:46
Post #12





Grupa: Zarejestrowani
Postów: 467
Pomógł: 77
Dołączył: 6.09.2008
Skąd: Miechów / Kraków

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


Cytat
Na razie jest to kompletni bez jakichkolwiek zabezpieczeń i filtracji danych pochodzących o użytkownika.

Dlaczego nie zrobiłeś tego na samym początku? Jak później dodasz mechanizm filtracji to będzesz szukał po kodzie, gdzie należy go wstawić? Niepotrzebna robota wykonywana dwa razy, a co za tym idzie strata czasu. Już pomijając fakt straty czasu możesz coś pominąć i będziesz miał fajną dziurę. Dodając to od razu masz problem z głowy.
Rozumiem, że fajnie się kodzi i patrzy jak coś działa i "rośnie", ale uwierz mi, że lepiej na początku poświęcić trochę więcej czasu na takie podstawowe mechanizmy. Im lepiej sobie je przygotujesz tym lepiej i szybciej będzie się pisało smile.gif


--------------------
Niemożliwym jest stworzenie czegokolwiek idiotoodpornego, ponieważ idioci są wyjątkowo pomysłowi.

https://www.aroch.pl
https://themeforest.net/user/aroch
https://www.astroblog.aroch.pl
https://www.4geeks.pl
Go to the top of the page
+Quote Post
kujol
post 4.11.2013, 10:20:51
Post #13





Grupa: Zarejestrowani
Postów: 324
Pomógł: 27
Dołączył: 21.07.2013

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


Dobra, dzięki za wszystkie opinie. Biorę się do roboty, możecie mi jeszcze doradzić jak ze strony klienta powinna wyglądać grafika + przykłady gotowych szablonów (jeśli ktoś ma) wink.gif.


--------------------
;)
Go to the top of the page
+Quote Post
Turson
post 4.11.2013, 11:49:26
Post #14





Grupa: Zarejestrowani
Postów: 4 291
Pomógł: 829
Dołączył: 14.02.2009
Skąd: łódź

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


Jak nie masz zmysłu graficznego, a zależy Ci na napisaniu samego kodu PHP, to weź jakiś darmowy szablon -> http://www.freewebsitetemplates.com
Go to the top of the page
+Quote Post
kujol
post 6.11.2013, 11:28:38
Post #15





Grupa: Zarejestrowani
Postów: 324
Pomógł: 27
Dołączył: 21.07.2013

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


Zrobiłem taki szablon strony głównej:
szablon
Jak się podoba, coś zmienić ? wink.gif


--------------------
;)
Go to the top of the page
+Quote Post
pedro84
post 6.11.2013, 11:37:05
Post #16





Grupa: Nieautoryzowani
Postów: 2 249
Pomógł: 305
Dołączył: 2.10.2006

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


Cytat(kujol @ 6.11.2013, 11:28:38 ) *
Zrobiłem taki szablon strony głównej:
szablon
Jak się podoba, coś zmienić ? wink.gif

Nie chcę być niemiły, ani zniechęcić Cię do roboty, ale weź sobie parę darmówek, albo szablonów z TF, popatrz, a potem zobacz na swój twór. Wnioski wyciągniesz sam smile.gif


--------------------
Google knows the answer...
Go to the top of the page
+Quote Post
kujol
post 6.11.2013, 11:43:53
Post #17





Grupa: Zarejestrowani
Postów: 324
Pomógł: 27
Dołączył: 21.07.2013

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


Ale teraz wydaje się mi, że jest lepiej w porównaniu z poprzednim smile.gif. Będę dalej kombinował wink.gif.


--------------------
;)
Go to the top of the page
+Quote Post
pedro84
post 6.11.2013, 11:58:15
Post #18





Grupa: Nieautoryzowani
Postów: 2 249
Pomógł: 305
Dołączył: 2.10.2006

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


Cytat(kujol @ 6.11.2013, 11:43:53 ) *
Ale teraz wydaje się mi, że jest lepiej w porównaniu z poprzednim smile.gif. Będę dalej kombinował wink.gif.

Może i lepiej, ale w dalszym ciągu tragedia.

Naprawdę, szkoda Twojego czasu, lepiej się PHP poucz, grafik z Ciebie będzie żaden. Im szybciej to do Ciebie dotrze, tym lepiej (do mnie na przykład szybko dorarło) smile.gif


--------------------
Google knows the answer...
Go to the top of the page
+Quote Post

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: 19.04.2024 - 22:01