-
Fototrend
JavaScript != Java (A JavaScript nem összekeverendő a Javával, két különböző programozási nyelvről van szó!)
Új hozzászólás Aktív témák
-
Jim-Y
veterán
válasz
nymarti
#3552
üzenetére
Én egy ilyet csináltam: http://jsfiddle.net/Jim_Y/Ls55L/1/
Próbáltam a Sk8erPeter által korábban linkelt konvenciókat figyelembe venni, mint:
ne nagyon legyen javascript a html kódban
és most is mint mindig azért várom a kritikát ^^ -
Sk8erPeter
nagyúr
válasz
nymarti
#3550
üzenetére
"van valami tippetek, hogy mit ronthattam el?"
Hát elég sok mindent.
- nem raktad fel a kódodat jsFiddle-re, hogy könnyebben és gyorsabban tudjuk debuggolni
- kezdjük ott, hogy a select15() és a magas() nem túl beszédes nevek, egy év múlva esélyes, hogy fogalmad sem lesz róla a kód megnézése nélkül, hogy az a két függvény mit csinál. Kerülendő. Inkább legyenek jó hosszú változó- és függvény-/metódusneveid, mint hogy később nehézkes legyen rájönni, mi a feladatuk (ahogy az a,b,c,s, x,y,z és hasonló nevek is kerülendők).
- onChange="magas();" --> ezzel tehát azt mondtad, hogy a select listából való válogatás során, ha történik bármi változás, akkor a magas() függvény hívódjon meg; a select15() egyáltalán nem hívódik meg, szóval azt hiába is várod.
- nem tudjuk, mi van a magas() függvényedben, de a disabled-re állítás amúgy sem történik meg, mivel a select15() nem hívódik meg
- mi a frász az az s[16]?
Egyáltalán hol van itt az az s tömb? Mondjuk feltételezem, azt gondoltad, hogy így eléred a következő select elemet, aminek sel16 az id-je, de így nem fog menni. Vannak ilyenek, mint a document.getElementById, nextElementSibling, children, és így tovább.
- na, egyelőre elég infó.
Új hozzászólás Aktív témák
- Samsung Galaxy A56 - megbízható középszerűség
- exHWSW - Értünk mindenhez IS
- Audi, Cupra, Seat, Skoda, Volkswagen topik
- Revolut
- Samsung Galaxy A55 - új év, régi stratégia
- Samsung Galaxy S24 Ultra - ha működik, ne változtass!
- AMD Ryzen 9 / 7 / 5 / 3 5***(X) "Zen 3" (AM4)
- Fejhallgatós találkozó
- Kertészet, mezőgazdaság topik
- Elemlámpa, zseblámpa
- További aktív témák...
- Gamer PC - i5 / GTX 960/12 Gb Ram (Erősebb, mint a 1050 Ti!)
- Eladó AKG Ara, dupla kapszulás mikrofon! Bontatlan, garanciás! Több darab is elérhető!
- Logitech Superstrike x2
- D-link 16 és 24 portos, sima és POE, gigabites managelhető switchek
- Gigabyte H510M PRO-E alaplap + Intel Core i5 10400F CPU (+ram, táp, vga igény szerint)
- PC Architech Shop Számítógép és hálózat szerviz
- 27% - ÚJ Samsung Odyssey G5 S27FG506SU QD-OLED Monitor! 2560x1440 / 180Hz / G-Sync / FreeSync
- 238 - Lenovo Legion Pro 7 (16IRX8) - Intel Core i9-13900HX, RTX 4070
- Dell Wyse 5470,14",FHD,Touch,N4100 CPU,8GB DDR4,128GB SSD,WIN11,IR KAMERA
- Eladó Apple iPhone 14 128GB fekete / 12 hó jótállás
Állásajánlatok
Cég: Laptopműhely Bt.
Város: Budapest
és most is mint mindig azért várom a kritikát ^^

