-
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
- Acer Chromebase All-in-One PC 23.8" Touchscreen
- ÁRGARANCIA!Épített KomPhone Ryzen 7 7700X 32/64GB RAM RX 9070 16GB GAMER PC termékbeszámítással
- Bomba ár! HP ProBook X360 435 G7 - Ryzen 3 I 8GB I 256SSD I 13,3" FHD Touch I W11 I Cam I Gari!
- Telefon felvásárlás!! iPhone 16/iPhone 16 Plus/iPhone 16 Pro/iPhone 16 Pro Max
- Dell Latitude 7210 2-in-1 12" FHD touch, i7 10610U, 16GB RAM, 512GB SSD, jó akku, számla, 6 hó gar
Állásajánlatok
Cég: BroadBit Hungary Kft.
Város: Budakeszi
Cég: Laptopszaki Kft.
Város: Budapest
és most is mint mindig azért várom a kritikát ^^


