-
Notifications
You must be signed in to change notification settings - Fork 1
HTML5 CSS JS valideerimine
HTML5 ja CSS koodi valideerimine tehti lehtedel http://validator.w3.org/ ning http://jigsaw.w3.org/css-validator/ validaatoritega. Javascripti koodi valideeriti JSLint (http://www.jslint.com/) validaatoriga.
HTML5 valideerimine
Alguses andis HTML5 validaator ainult 4 viga. Kui aga üks vigadest sai parandatud, tekkis juurde 32 viga. Seda seetõttu, et see üks viga takistas selle vea punktist dokumenti edasi valideerimast. Peale seda olid aga põhilised vead:
- Puuduvad property väärtused
- Pildid vajasid property "alt" väärtustamist
- Dropdown valikumenüü vajas property "label" väärtustamist
-
Pildi failidele mõõtmed valesti antud (vaja oli arvulist väärtust aga andsime sõnena)
-
Tagide vale kasutus ja lõpetamata jätmine
Meil oli alguses dokument defineeritud kui XHTML 1.0 Transitional, kuigi dokument ise oli HTML5. See sai ka ära parandatud.
Ühe vea jätsime ka sisse. See on seotud Facebooki login nupuga. Validaatorile ei meeldinud, kuidas me oleme nupu dokumenti lisanud. Samuti validaator hoiatas, et antud nuppu ei saa XML 1.0 näidata. Vea jätsime alles, sest kui seda parandama hakata, oleks olnud vaja üsna palju koodis ümber teha ning ka hakata paigutust uuesti korda seadma. Sisse jäi ka hoiatus, et fail kasutab UTF-8, võib mõnedele tekstiredaktoritele ning brauseritele probleeme tekitada.
CSS CSS validaator leidis kokku 7 viga. Põhiliselt ainult koodi kirjutamise käigus tekkinud typod.
- puudus elemendi laiuse ja kõrguse korrektne mõõde
- kasutatud propertyt mida ei eksisteeri
- väärtusel puuduv ühik
- parsing error property nimes oleva '*' tõttu
Kõik vead olid kergelt parandatavad ning CSS fail läbis CSS validaatori testi.
Javascript
Põhiline viga, mis javascripti validaator leidis, oli et koodis kasutatakse tühikute asemel tabulaatorit loetavuse parandamiseks. Lisaks
- massiivide defineerimisel ja väärtustamisel ei ole väärtused eraldatud ", " vaid lihtsalt ","
- muutujaid on kasutatud enne nende loomist
- tsüklites luuakse uusi muutujaid
- ei ole kasutatud 'use strict'
- väärtuste võrdlemisel harjutud Javas kirjutama (javas väärtuste võrdluseks "==", javascriptis aga peaks olema "===")
- funktsioonide välja kutsumisel ja väärtuste andmise käsu lõpetamisel puudus ";"
- funktsioonide defineerimisel ebavajalik ";"
- halb stiil (näiteks "var i = 0;" oli meil "var i=0;")
- kasutamata muutujad
- liiga palju muutujaid funktsioonides
- muutujaid ei kombineerita loomisel
Soovitustest võtsime kuulda näiteks, et ei loo muutujaid enam tsüklites, mõnes kohas kombineerisime muutujaid. Eemaldasime ebavajalikud ";" ning lisasime need kohtadesse, kust need puudusid. Parandasime ka natuke stiili.
Kuulda ei võtnud näieks võrdluste loogika muutmist, sest vastavalt dokumentatsioonile on "==" täiesti valiidne võrdlus javascriptis. Samuti ei võtnud kuulda muutujate kasutamise hoiatust enne nende loomist. Validaator hoiatas põhiliselt seda funktsioonide koha pealt. Osad kasutamata muutujad me kustutasime, aga lõpuks hakkas validaator näitama kasutamata muutujatena ka muutjaid, mida reaalselt kasutati, seetõttu üldiselt kasutamata muutujate soovitust kuulda ei võtnud. Ka ei võtnud kuulda liiga palju muutujaid soovitust, sest kõiki oli vaja.