Krótki kod(63) linijki ocena

Odpowiedz Nowy wątek
2019-02-19 14:01

Rejestracja: 2 lata temu

Ostatnio: 13 godzin temu

0

Witam ma prośbę jak w temacie proszę o ocenę kodu czy dobrze praktykuje czy może coś źle robię,na co zwracać uwagę , dziękuję za wskazówki. Chodzi mi tylko o js'a.
https://github.com/Raloseq/JS-Lotto

edytowany 1x, ostatnio: Raloseq, 2019-02-19 14:02
Zle praktykujesz bo jak masz katalog CSS to powinenes zrobic katalog JS i tam miec kod *.js - masterO 2019-02-19 14:27
no i wywal petle zamiast wypelniac tablice petla for uzyj mapera Array.apply(null, {length: 49}).map(Number.call, Number) - masterO 2019-02-19 14:34

Pozostało 580 znaków

2019-02-19 14:28

Rejestracja: 14 lat temu

Ostatnio: 1 rok temu

1

Nie przypisuj zmiennych jesli uzywasz ich tylko raz, wywal te const.
addEventListener tego sie juz nie stosuje

edytowany 2x, ostatnio: masterO, 2019-02-19 14:30

Pozostało 580 znaków

2019-02-19 14:36

Rejestracja: 2 lata temu

Ostatnio: 13 godzin temu

0

Co zamiast addEventListenera polecasz ?

moze $(document).on('click', '#id', function(){}); ? - masterO 2019-02-19 16:16
Nie korzystam na razie z jQuery - Raloseq 2019-02-19 16:28
@masterO: Jak już chcesz polecać jQuery to poleć dobrze: $("#id").click( - TomRiddle 2019-02-27 13:06
@masterO: Poza tym, co, niby jQuery w on() nie robi addEventListener()? Proszę Cię. - TomRiddle 2019-02-27 13:07

Pozostało 580 znaków

2019-02-19 19:38

Rejestracja: 14 lat temu

Ostatnio: 1 rok temu

0

no to

document.getElementById('calc').onclick = function() {

}
Pokaż pozostałe 3 komentarze
@masterO: od kiedy lepsza? Coś mnie ominęło? - hauleth 2019-02-26 10:04
od 2003 roku - masterO 2019-02-26 10:08
@masterO: what? Nawet dokumentacja pisze, że "no kurde lepiej nie" https://developer.mozilla.org[...]I/GlobalEventHandlers/onclick - hauleth 2019-02-26 13:58
A no widzisz bo dokumentacje pisza takie ludzie co sie nie znaja :) i reszta to bierze za prawde. Ja ci powiadam, ze lepiej uzyc tego co ja mowie niz wierzyc dokumentacji. Kiedys tez byla dokumentacja do Angulara 1.0 :D a teraz najchetniej by o tym zapomnieli. - masterO 2019-02-26 14:07

Pozostało 580 znaków

2019-02-25 21:01

Rejestracja: 3 lata temu

Ostatnio: 1 tydzień temu

1

Ja sugerowałbym zamiast stosowania komentarzy objąć te bloki kodu w funkcje o konkretnych nazwach. Koniecznie w języku angielskim!

Pokaż pozostałe 4 komentarze
portfolio - adams0 2019-02-25 22:05
To śmiało własnie jestem w trakcie tworzenia - Raloseq 2019-02-25 22:05
To może na pw bo tutaj i tak postu to nie dotyczy ? - Raloseq 2019-02-25 22:19
Na PW nikt inny się nie nauczy. Jak mi ktoś usunie będzie na mnie ;-) - adams0 2019-02-25 22:21
Ok to czekam :D - Raloseq 2019-02-25 22:24

Pozostało 580 znaków

2019-02-25 22:33

Rejestracja: 3 lata temu

Ostatnio: 1 tydzień temu

0

@Raloseq Oto moje spostrzeżenia na temat twojej strony portfolio:

  • 2 zdjęcia
  • Co node_modules robi na githubie ? (użyj gitignore)
  • "Chciałbym dążyć do" && "Napotkane problemy przeze mnie lubię rozwiązywać"
  • import {
    isInViewport
    } from './isInViewport.js';

    można w jednej linijce

Pozostało 580 znaków

2019-02-25 22:55

Rejestracja: 2 lata temu

Ostatnio: 13 godzin temu

0

co do zdjęć to jest funkcja która sprawdza czy jest w viewporcie i wtedy wyskakuje co do importa to beaufity jakos tak działa, node_modules będzie w .gitignore a do tego tekstu to nie wiem

Jeśli pisząc "beaufity" masz na myśli wtyczkę do VS Code, to myślę, że powinno dać się zmienić jej konfigurację, gdyby Ci przeszkadzało, jak działa. - Silv 2019-02-25 22:58
Mam na myśli wtyczkę ale jakoś nie przykładałem do tego większej uwagi. - Raloseq 2019-02-25 22:58
'Dążę do' 'Lubię rozwiązywać napotkane problemy' jest prostsze i bardziej treściwe. Moim zdaniem. - adams0 2019-02-26 08:37
Aa o to chodzi ok - Raloseq 2019-02-26 11:31

Pozostało 580 znaków

2019-02-26 00:39

Rejestracja: 2 lata temu

Ostatnio: 3 godziny temu

4
masterO napisał(a):

no to

document.getElementById('calc').onclick = function()

A czy e.onclick pozwala na swobodne przypisywanie do zdarzenia na elemencie dowolnej liczby różnych funkcji oraz ich dowolne usuwanie?

Bo e.addEventListener na to pozwala. Pozwala też sterować kolejnością wykonywania funkcji przypisanych do akcji na elemencie i akcji przypisanych do jego przodków, patrz: bubbling. Jak dla mnie addEventListener jest bardziej elastyczne, co nie znaczy, że koniecznie trzeba zawsze akurat jego używać.


edytowany 1x, ostatnio: Freja Draco, 2019-02-26 00:39
Pokaż pozostałe 7 komentarzy
Ty nie znasz to nie znaczy ze inni takiej nie używają. - masterO 2019-02-27 15:28
Kto mnie wołał :o - MasterOf 2019-02-27 16:05
@MasterOf: Przepraszam, wpisałem zły link. - TomRiddle 2019-02-27 16:20
@masterO: Podaj przykład. Nie wiem czy trollujesz a ja Cię feeduję, czy może na prawdę wierzysz w to że niskopoziomowe property jest lepszym rozwiązaniem niż wyżej poziomowe funkcje, ale nie chce mi się już z Tobą kłócić. Proszę, podaj jedną przeglądarkę która nie wie co to jest addEventListener(). - TomRiddle 2019-02-27 16:21
Dobra Voldemord wygrałeś, poza tym nie chce sie kłócic z najpotezniejszym czarnoksieznikiem masz wieksza różdżkę - masterO 2019-02-28 10:31

Pozostało 580 znaków

2019-02-27 12:55

Rejestracja: 10 lat temu

Ostatnio: 7 godzin temu

1

Porób z tego funkcje.


Pozostało 580 znaków

2019-02-27 13:03

Rejestracja: 10 lat temu

Ostatnio: 7 godzin temu

1

I nie słuchaj @masterO : addEventListener() jest spoko :)


jest spoko ale onClick jest lepsze - masterO 2019-02-27 13:45
Bo tam jest diabeł - masterO 2019-02-27 15:28

Pozostało 580 znaków

2019-02-27 15:25

Rejestracja: 10 lat temu

Ostatnio: 7 godzin temu

1

@Raloseq jak Ci się tego nie uda poprzemieniać na funkcje to napisz, pokażę Ci jak.


Pozostało 580 znaków

Odpowiedz

1 użytkowników online, w tym zalogowanych: 0, gości: 1, botów: 0