单击更改svg元素的属性填充

Question

我是JavaScript和编程的新手。在做一个小的学习项目时，我偶然发现了一个在单击时更改SVG元素颜色的小问题。我有一个由特定颜色和SVG元素组成的数组，每次单击它时，SVG都应该更改它的颜色。以下是我尝试的方法：http://jsfiddle.net/j0u3rman/有人能指出我的错误吗？

var coatColor = [
'rgb(101, 137, 164)',
'rgb(241, 43, 107)',
'rgb(146, 100, 161)',
'rgb(56, 159, 117)',
'rgb(215, 210, 55)',
'rgb(0, 0, 0)'
];
// Function for acquiring a random element from an array
var getRandomElement = function (arr) {
var randomElement = Math.floor(Math.random() * arr.length);
return randomElement;
};
var changeAppearance = userSetting.querySelector('.setup-player');
var wizardCoat = changeAppearance.querySelector('.setup-wizard .wizard- 
coat');
var changeCoat = function (arr) {
wizardCoat.style.fill = getRandomElement(arr);
}
wizardCoat.addEventListener('click', changeCoat(coatColor));

Answer 1

我删除了不需要的东西，简化了您的代码。此外，你必须知道，有时颜色会是相同的，因为随机函数可能会连续几次给你相同的颜色。

var coatColor = [
'rgb(101, 137, 164)',
'rgb(241, 43, 107)',
'rgb(146, 100, 161)',
'rgb(56, 159, 117)',
'rgb(215, 210, 55)',
'gold'
];
// Function for acquiring a random element from an array
var getRandomElement = function (arr) {
var randomElement = Math.floor(Math.random() * arr.length);
return randomElement;
};
var changeColor = function () {
this.style.fill = coatColor[getRandomElement(coatColor)];
}
wizard_head.addEventListener('click', changeColor);

svg{border:1px solid;}

<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 62 86" class="setup-wizard">
<defs>
<rect id="wizard_head" x="20" y="5" width="20" height="20" />
</defs>                          
<g class="wizard">
<use xlink:href="#wizard_head" class="wizard-head"></use>
</g>
</svg>