19-06-2013, 14:26
(Ten post był ostatnio modyfikowany: 19-06-2013, 14:28 przez Kartofelek.)
Nie Supermana a Fanthomasa. Kolega widać od niedawna na naszym forum.
1) Błędy składniowe w kodzie
2) Używaj id dla elementów na których będziesz skryptował. Np dla formularza
3) UŻYWAJ FORMATOWANIA KODU. Nie wszystko w 1 rządku, a w porządku
4) Pozdrów nauczyciela, bo uczy was technik sprzed 20 lat
(serio, serio!)
Twój kod:
Ogólnie powyższy kod jest idealnym przykładem kodu, który można zoptymalizować powiedzmy o 80%? Może nawet więcej. Tak naprawdę tam poza polem formularza, skryptem i 3 obrazkami nie powinno być nic więcej.
1) Błędy składniowe w kodzie
2) Używaj id dla elementów na których będziesz skryptował. Np dla formularza
3) UŻYWAJ FORMATOWANIA KODU. Nie wszystko w 1 rządku, a w porządku
4) Pozdrów nauczyciela, bo uczy was technik sprzed 20 lat
(serio, serio!)Twój kod:
Kod:
<html>
<head>
<title>Ryby</title>
<link rel="Stylesheet" type="text/css" href="text.css"/>
</head>
<body>
<h1>kl. I c </h1>
<h2>Ryby</h2>
<table align="center" border="0">
<tr>
<td width="150px" height="210px">
<img class="obr1" src="niebieska.jpg" />
</td>
</tr>
<tr>
<td width="150px" height="210px"></td>
<td width="150px" height="210px">
<img class="obr2" src="skrzydlica.jpg"/>
</td>
</tr>
<tr>
<td width="150px" height="210px"></td>
<td></td>
<td width="150px" height="210px">
<img class="obr3" src="blazenek.jpg"/>
</td>
</tr>
</table>
<br>
<form action="" id="formularz">
<div>
<input class="text" type="text" name="nazwa" value="" />
</div>
</form>
<script type="text/javascript">
var img = document.getElementsByTagName('img');
var form = document.getElementById('formularz');
for (i=0; i<img.length; i++) {
img[i].onclick = function() {
var src = this.getAttribute('src'); //pobieram atrybut src obrazka = jego nazwa
formularz.nazwa.value = src.substring(0, src.length-4); //przycinam nazwę by nie było w niej .jpg
}
}
</script>
</body>
</html>Ogólnie powyższy kod jest idealnym przykładem kodu, który można zoptymalizować powiedzmy o 80%? Może nawet więcej. Tak naprawdę tam poza polem formularza, skryptem i 3 obrazkami nie powinno być nic więcej.
