To ja również sobie ocenie :).
- Brak rozdzielenia warstw.
- Zmienne globalna zastosowana bez sensu , zagrożenie dla bezpieczeństwa. Brak zgodności zasadą SOLID.
-
var numer = Math.floor(Math.random()*5)+1;
Poziomu konsoli jestem wyczytać co się właściwie dzieje się z tą zmienną , mogę zmieniać jej stan, modyfikować jej właściwości. 2. Jeżeli mamy duży kod, może dojść do konfliktów. 3. Ten kod jest obecnej w formie nierozszerzalny po przez ustawienie stałego liczbowego górnego maksimum.
-
Przerost formy nad treścią (Nadmiarowość) . Nie właściwie użycie jQuery.
$("#slider").fadeOut(500);
Naprawdę musisz odpalać fabrykę by zmienić opacity?? I to jeszcze tą brzydką funkcję. Ten sam efekt da się uzyskać za pomocą CSS, a w razie potrzeby z pomocą JS jako kontrolera. (model MVC mówi Ci to coś? )
-
Błędna konwersja danych , brak optymalizacji. Nie wykorzystanie cache.
var plik="<img src=\"imgs/slajd" + numer + ".jpg\" width=1000 height=400/> ";
Załóżmy, że jestem nowym użytkownikiem i wchodzę pierwszy raz na tą stronę. Nagle zdjęcia mi się bugują. Dlaczego? Obecnie one uzyskały niski priorytet, przez co są przetwarzane przez pamięć zwykłą a nie szybką. Więcej na ten temat: http://fragged.org/preloading-images-using-javascript-the-right-way-and-without-frameworks_744.html.
Drobny tip: od mnie , pierw bym pobrał wszystkie img (które są wykorzystywane w JS), jeżeli ich nie ma dużo. Najwyżej bym rzucił jakiegoś placeholdera.
Obecnie twój plik to String, a nie typ docelowy jakim jest DOM Object. Szybki test:
https://jsfiddle.net/q1v1nb3e/1/
Mówiąc inaczej , karzesz biednej przeglądarce przekonwertować String na DOM przy jednym nie ma to różnicy, ale przy wielu no cóż, trzymajmy się dobrych praktyk! (Poczytaj o Nodelist, DOM)./
Reszty nie oceniam bo to kopiuj/wklej z jQuery (które dalej jest bezsensu wykorzystana).
Przejdżmy do dalszej części szablon :).
- Duża część została powiedziana. Zajmię się tylko jednym aspektem. Który, wprost mnie uderzył. Bootstrap... Przelatuje przez kod nagle, widzę button, ale jakoś tak dziwnie ustawiony. Nagle miga w umyśle, 'Zaraz, zaraz czy to nie bootstrapowy button!?'. Klasa btn-primary potwierdza mnie w swoim przekonaniu. No to szukam budowy szablonu Bootstrapa. I co nie ma... Gratuluje bardzo inteligetnego wykorzystania Bootstrapa. Dosłownie reguła, regułę marze. Jak już coś chcesz coś takiego bawić, no to chociaż munifikuj kod bootstrapa.
No to przejdżmy do ostatniego punktu CSS.
- Złe wykorzystanie style,
- Brak jakiejkolwiek porządku w CSS (reguły się powtarzają).
- Nie zastosowałeś mobile First , efekt niby są media-queries ale taki wyglądu strony nie chciałbym dostać).
btw: Menu się buguje przy przejściach.
btw2. Serio, mam strzałkę scroll w dół i on się przewija o 200 px? Nie mówiąc o tym, ze do góry nie ma.
Protip -> Wrucasz projekty. Spoko. Ale dawaj jakieś linki do ich kodu. Pracodawcy z chęcią patrzą na kod , a właściwie na jakość kodu :).
Ocena strony sławek coś tam...nie oceniam, poczekam na poprawki.
Z ciekawości przeszedłem na inne twoje strony. Spoko, że na jednej stronie się chwalisz, że masz wielu klientów. No to sobie myśle, że każdemu może wyjść kaszana no to zobaczmy wykonane zlecenia. I co widzę, te same strony.
Btw. W sekcji o klasie jumbotron na stronie blastar.pl, otworzenie wszystkich sekcji powoduje, że jedna ucieka na białe tło.
Btw2. Proszę przestań, nadużywać buttonów z Bootstrapa (W ogóle do tego...)!!! Nagle widzę, że to coś ma jakiś wygląd , sekcja 'aab' i nagle widzę, ten niebieski button dla ludzi upośledzonym wzrokiem. Aaa... sekcja 'aab' poważnie? Wróć do tej strony za X miesięcy , zobaczymy czy będziesz pamiętał co jest w aaa czy aab.
A i zdjęcia rozpikselowane w sliderze. No i odległość sekcji nie są równe. RWD dalej brak... A i obrazek w sliderze jest większy niż sekcja poniżej. Brzydko to wygląda.
Na więcej nie chcę patrzeć. blastar.pl ocena 3/10.
O tu też jest formularzyk, zobaczmy czy działa validacja :). Gratuluje PHP Injection jak najbardziej możliwy.
No cóz, z braku wyboru, patrzę na inne wykonanane projekty.
Blog: pasjainauka.pl
- Już widzę, pierwsze błędy. Strzelam, że mamy powtórkę. Więc, nie będę się powtarzał.
- Jak blog to myśle, że pewnie jakiś system CMS , a nie jednak jestem w błędzie, brak jakiejkolwiek wiązanki linków typowych np: dla wordpressa, utwierdza mnie w przekonaniu , że jest to jedynie wiązanka plików php bez jakiejkolwiek z sobą relacji. (Oprócz a href). To dlaczego to nie jest html?
Ocena 3/10
Kolejna strona , a właściwie sklep Engineland
- Boże, co ty masz ne tej stronie ładuje się z minutę (nie mówiąc o podstronach)
- Trochę pustawo...
- 3 jądra jQuery?!!!?!?!
-
$("a.koszyczeklink").hover(
function () {
$(".td").addClass("highlight");
},
function () {
$(".td").removeClass("highlight");
}
);
No poważnie?
-
Resztę skryptu nie oceniam bo to nie twój skrypt przede wszystkim + Te same błędy co w każdym.
Nie oceniam, wygląda mi to na wersję alpha.
No to teraz podsumowanie <3
Przede wszystkim wszystkim profesjonalizm
Milion błędów, nawet na tych projektach, które nie były robione jak twierdzisz 5 dni.
Sprawnie i skutecznie. Liczy sie dla mnie terminowość jak również każdy detal wykonywanej przeze mnie pracy.
Że tak brzydko powiem, wszystkie są zrobione na ... + plus kopiuj/wklej i zapomnij.
I dlatego mogę jasno stwierdzić, że to...
Jestem jednoosobową firmą.Posiadam kilkuletnie doświadczenie w branży IT.
... kłamstwo.
Sławku, jeżeli faktycznie siedzisz w tym X lat, to zmarnowałeś ten czas. Za bardzo rozszerzyłeś spektrum języków. Przez to, coś tam wiesz , ale jest to wiedza bardzo niska. Skup się na dwóch rzeczach max i twórz na podstawie tego swoje portfolio.
Możesz powiedzieć, że hejtuje cb. Nie Sławku, sam stwierdziłeś że masz kilkuletnie doświadczenie i z tego poziomu Cię oceniam.
I teraz ostatnia WAŻNA UWAGA.
Jeżeli twierdzisz, że znasz tyle języków programowania i standardów. Wiedz, że pracodawca będzie wymagał ich znajomości. Powiem jeszcze inaczej, jeżeli znasz jakiś język, poprzyj go projektem. Bo tak dla mnie to są puste słowa i dla pracodawcy też.
PS: Jak masz githuba to wrzuć go tutaj, chętnie popatrze na kod tej aplikacji.
A teraz przepraszam wracam do walki z DAO w JAVie <3.