你的代码语法上其实没什么大问题，但可能是一些细节导致没达到预期效果，我来帮你梳理排查并优化：

先给你一个更可靠的简化版本 #

你在onmouseover里重复通过ID查找元素完全没必要，直接用this指代当前的链接元素就好，这样不仅高效，还能避免ID重复之类的潜在问题：

<a id="link1" href="http://www.yahoo.com" onmouseover="this.style.color='yellow'">this is a link</a>

排查失效的可能原因 #

1. CSS优先级冲突：如果你的页面里写了全局的a:hover样式，或者有带!important的颜色规则，可能会覆盖内联style的设置。你可以按F12打开开发者工具，选中这个链接，在「样式」面板里看看color属性是不是被划掉了——如果是，就说明有更高优先级的样式在生效。
2. 缺少鼠标移出的恢复逻辑（可选）：如果你的需求是悬停变黄、移出恢复原颜色，记得加上onmouseout事件，不然鼠标移走后颜色会一直停留在黄色：

   <a id="link1" href="http://www.yahoo.com" onmouseover="this.style.color='yellow'" onmouseout="this.style.color=''">this is a link</a>
   

   把onmouseout的颜色设为空字符串，就能让链接恢复到原本的样式颜色。
3. 检查控制台报错：打开浏览器控制台（F12→控制台），看看有没有JS语法错误——比如拼写错color（写成colour）这种低级错误，也会导致代码失效。

更推荐的现代写法 #

从可维护性角度出发，尽量别用内联事件属性，把JS和HTML分开写会更规范：

<a id="link1" href="http://www.yahoo.com">this is a link</a>

<script>
  const linkElement = document.getElementById('link1');
  linkElement.addEventListener('mouseover', () => {
    linkElement.style.color = 'yellow';
  });
  linkElement.addEventListener('mouseout', () => {
    linkElement.style.color = '';
  });
</script>

内容的提问来源于stack exchange，提问作者Vaidehi Gupta