• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

question-closed Jakość kodu Java Script

VPS Starter Arubacloud
0 głosów
124 wizyt
pytanie zadane 23 czerwca 2020 w JavaScript przez patrick167 Początkujący (260 p.)
zamknięte 26 czerwca 2020 przez patrick167

Dzień dobry, 

proszę o ocenę jakości kodu oraz czy spełnia on standard czytelności dla innego programisty. Działanie programu polega na stworzeniu tracklisty, która wyświetla aktualnie odtwarzany utwór w zależności od czasu odtwarzacza na stronie. Nazwy utworów oraz miejsce w którym się on zaczyna jest umieszczony w pliku JSON. 

class Tracklist{
     constructor(className){
     
      this.tracklistSelector= document.querySelector("."+className) ;
      this.audio=document.getElementById("audio");
      this.songs=null;
      this.actualTime=null;
      this.span=document.createElement("span");
      this.tracklistGenerator();
      this.fileGET();
      this.timeUpdate();    
       
    };
    
    fileGET(){
        let request = new XMLHttpRequest();
             request.open('GET', 'tracklist.json');
             request.onload = function () {
             const data = JSON.parse(request.response);
             this.songs=data.songs;
             
                    

        }.bind(this)
        request.send();  
    }
   
    timeUpdate(){
                                    
     this.audio.addEventListener("timeupdate",function(){
        
        this.actualTime= this.audio.currentTime;
          this.songsCounter(this.songs);
     }.bind(this));
    }
    
    songsCounter(tracks){
            
        
            for(let i=0;i<tracks.length;i++){
                
                
                   if((parseInt(this.actualTime)==tracks[i].time) || ((parseInt(this.actualTime)>tracks[i].time) && (tracks[i+1]!=undefined)&&(parseInt(this.actualTime)<tracks[i+1].time) || (parseInt(this.actualTime)>tracks[tracks.length-1].time ))){
                       
                       if(tracks[i].name!=this.span.innerHTML){
                       this.span.innerHTML=tracks[i].name;
                       }
                      
                   
                   }
            }
        
    }
    tracklistGenerator(){
        
        this.span.classList.add("trackname");
        this.tracklistSelector.appendChild(this.span);
    } 

}

ob=new Tracklist("tracklist");


 

komentarz zamknięcia: Dostałem odpowiedź na pytanie

2 odpowiedzi

+2 głosów
odpowiedź 24 czerwca 2020 przez Comandeer Guru (599,730 p.)
wybrane 24 czerwca 2020 przez patrick167
 
Najlepsza
  • Czemu do konstruktora można podać klasę elementu? O wiele wygodniej byłoby podać cały selektor albo bezpośrednio element.
  • Czemu z kolei element #audio ma na sztywno ustawione [id]?
  • fileGET (które powinno się nazywać getFile; funkcje nazywa się "czasownikowo", czyli zróbCoś – pobierzPlik itd.) jest funkcją asynchroniczną, ale cała reszta kodu udaje, że nie. Ta metoda powinna zwracać Promise.
  • span absolutnie nic nie mówi i nie wiadomo, do czego służy ten element.
  • Linijkę 43 warto przenieść do pomocniczej funkcji i rozbić ten warunek.
komentarz 24 czerwca 2020 przez patrick167 Początkujący (260 p.)
Rozumiem, dzięki za pomoc.
+2 głosów
odpowiedź 23 czerwca 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)
tracklist -> trackList

metoda fileGET -> getFile()

timeUpdate -> updateTime()

linia 43 do poprawy

brakuje dobrego formatowania, z jakiego IDE korzystasz?
komentarz 24 czerwca 2020 przez patrick167 Początkujący (260 p.)
Brackets, dzięki za poprawki.
komentarz 24 czerwca 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)

zainteresuj się jakimś formaterrem kodu do tego IDE. Tutaj podsyłam pierwszy lepszy - https://github.com/brackets-beautify/brackets-beautify

Podobne pytania

+2 głosów
2 odpowiedzi 196 wizyt
pytanie zadane 22 listopada 2020 w JavaScript przez Chlipchlip Użytkownik (850 p.)
0 głosów
5 odpowiedzi 181 wizyt
pytanie zadane 4 listopada 2020 w C i C++ przez Garper Początkujący (360 p.)
0 głosów
1 odpowiedź 239 wizyt
pytanie zadane 31 maja 2022 w JavaScript przez olek1405 Nowicjusz (180 p.)

92,454 zapytań

141,262 odpowiedzi

319,089 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...