Вы не должны встраивать свой JS в свой код, как это. Хотя я советую использовать такую библиотеку, как jQuery (которая облегчит вашу жизнь), я объясню без нее.
Не вставляйте свой JS в код. Если вам действительно нужно, пусть она вызовет такую функцию:
<img id="bullet1" name="bullet1" height="30px" width="30px" src="img/bulletwhite.png" onmousedown="bulletClicked()" style="opacity:0.4;filter:alpha(opacity=40)"/>
Затем в разделе заголовка между тегами скрипта вы запустите свой javascript:
function bulletClicked() {
this.src='img/bulletwhite.png';
document.images['bullet2'].src='img/bullet.png';
document.images['bullet3'].src='img/bullet.png';
document.images['bullet4'].src='img/bullet.png';
}
Судя по тому, что вы делаете это неправильно, вы, вероятно, помещаете этот код onclikc в каждое изображение маркера, слегка измененное для каждого. Вместо этого, если бы вы просто использовали события, вы бы сильно упростили.
Если вы сделали что-то подобное ... (и указали свой рост, ширину и другие CSS в разделе стилей, где они принадлежат, не делайте то, что вы делали, когда-либо снова).
<img id="bullet1" name="bullet1" src="img/bulletwhite.png" onmousedown="bulletClicked(this)"/>
Тогда ваш javascript может быть ...
function bulletClicked(e) {
document.images['bullet1'].src='img/bullet.png';
document.images['bullet2'].src='img/bullet.png';
document.images['bullet3'].src='img/bullet.png';
document.images['bullet4'].src='img/bullet.png';
e.src='img/bulletwhite.png';
}
Есть гораздо лучшие способы решения этой проблемы, и я настоятельно рекомендую вам взять jQuery и поработать с разделением ваших HTML, JavaScript и CSS компонентов на ваших страницах.