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>

点击更改{svg元素的属性填充

1 个答案: