- Вы не останавливаете функцию
printColorAndGroup
с помощью }
.
- Вы должны использовать объект для пар ключ / значение, а не массив.
- Возможно, вы захотите использовать буквенную запись для объекта.
- Внутри функции
onchange
вы можете передать выбор с помощью this
. Так что select.options[...]
тогда будет возможно и чище.
- Вместо имен вы также можете использовать идентификаторы, чтобы вам не приходилось каждый раз беспокоиться о
[0]
.
Измененная версия: http://jsfiddle.net/pimvdb/RemPF/1/
Это буквенное обозначение объекта:
var colors = { apple: "red",
grape: "purple",
milk: "white",
cheese: "yellow",
chicken: "white",
beef: "red" };
Вот как работают идентификаторы:
<input type="text" id="food_group" ...>
И вы можете получить элемент с помощью:
document.getElementById('food_group')
Вот как работает передача select:
<select name="food" onchange="printColorAndGroup(this)">
со следующим JavaScript:
function printColorAndGroup(select){
var text = select.options[select.selectedIndex].value;
...