我来帮你分析问题出在哪，以及怎么修复——你的两个尝试分别踩了不同的坑，我会逐个解释，再给你可行的解决方案，最后还能扩展到多图切换的场景。

问题1：直接比较src属性的路径匹配坑 #

你第一个版本的代码里，current__img.src返回的是浏览器解析后的完整绝对路径（比如http://your-domain.com/Images/image1.jpg），而你用来比较的是相对路径Images/image1.jpg，这两个字符串永远不相等，所以每次都会执行current__img.src = image2，自然切不回图1。

问题2：计数器定义在函数内部的生命周期问题 #

第二个版本里，firstcounter是在changeimage函数内部声明的，每次点击按钮调用函数时，这个变量都会被重新初始化为0，所以永远只会触发if (firstcounter == 0)的分支，切到图2后，下次点击又会重置计数器，根本不会执行else逻辑。

修复方案：两种可行的实现方式 #

方式1：用dataset存储当前状态（推荐，支持多图扩展） #

我们可以给图片元素加一个自定义属性记录当前显示的是第几张图，后续加再多图都能轻松扩展：

<div class="main">
  <div class="main__container">
    <div class="main_content">
      <!-- 新增自定义属性记录当前图片索引，初始为0 -->
      <img src="Images/image1.jpg" alt="pic" class="enrollment__img" id="enrollment__img1" data-current-index="0">
      <button id="clickme" onclick="changeimage()">click me</button>
      <h1>insert 1st text here</h1>
      <h2>insert 2nd text here</h2>
      <h3>insert 3rd text here</h3>
    </div>
  </div>
</div>

const current__img = document.getElementById("enrollment__img1");
// 把所有图片路径存在数组里，后续加图直接往数组里追加即可
const imageList = [
  "Images/image1.jpg",
  "Images/image2.jpg"
  // 扩展多图示例："Images/image3.jpg", "Images/image4.jpg"
];

function changeimage() {
  // 获取当前索引并转为数字类型
  let currentIndex = Number(current__img.dataset.currentIndex);
  // 计算下一个索引：循环切换（最后一张后回到第一张）
  currentIndex = (currentIndex + 1) % imageList.length;
  // 更新图片源
  current__img.src = imageList[currentIndex];
  // 把新索引存回自定义属性，保持状态
  current__img.dataset.currentIndex = currentIndex;
}

方式2：用外部变量存储状态（适合简单两图切换） #

如果只需要两图切换，也可以把计数器定义在函数外部，避免每次调用函数时重置变量：

const current__img = document.getElementById("enrollment__img1");
// 计数器定义在函数外部，保持状态不丢失
let firstcounter = 0;
// 提前缓存图片路径，避免每次函数调用重复声明
const image1 = "Images/image1.jpg";
const image2 = "Images/image2.jpg";

function changeimage() {
  if (firstcounter === 0) {
    current__img.src = image2;
    firstcounter++;
  } else {
    current__img.src = image1;
    firstcounter--;
  }
}

额外优化建议 #

1. 把图片路径、计数器等变量定义在函数外部，避免每次调用函数重复创建变量，养成性能优化的小习惯
2. 如果是多图场景，优先选择方式1的数组+自定义属性方案，扩展性极强，后续加图完全不需要修改切换逻辑
3. 可以提前预加载所有图片，避免切换时的加载空白：

   // 预加载所有图片的小技巧
   imageList.forEach(src => {
     const img = new Image();
     img.src = src;
   });
   

这样应该就彻底解决你的切换问题了，你可以根据自己的需求选择对应的实现方式~