document.getElementById('image').onclick=click ();
Здесь ты на событие "onclick" вешаешь не функцию "click", а то, что она вернет. Чтобы функции "click" что-нибудь вернула, она должна выполниться. Но функция "click" у тебя ничего не возвращает, а лишь вызывает "alert". Если хочется что-нибудь в твоем духе то вот:
var event_click = function click ()
{
alert('Click!');
}
document.getElementById('image').onclick = event_click;
Далее, на "onload" лучше (но не обязательно) вешать вызов какой-нибудь функции, которая и сделает все необходимые операции при загрузке. Например:
<html>
<head>
<script type="text/javascript">
var event_click = function click ()
{
alert('Click!');
}
function onLoadFunction()
{
document.getElementById('image').onclick = event_click;
}
</script>
</head>
<body onLoad="onLoadFunction()">
<img src="fire_100x100.png" id="image" />
</body>
</html>