выглядит хорошо.Я думаю, что ваш getElementById не может получить то, что вы хотите.
Вот небольшой рабочий пример того, что вы хотите - но, как указано.Ваш синтаксис выглядит хорошо.
<script type="text/javascript">
function hello()
{
alert("hello");
}
</script>
<select id="Select1" onchange="hello()">
<option>1</option>
<option>2</option>
</select>