有什么办法可以让这段代码更短、更高效？ [关闭]

Question

关闭。 这个问题不符合Stack Overflow 准则。 它目前不接受答案。

---

想改进这个问题？ 更新问题，使其成为 Stack Overflow的主题。

去年关闭。

改进这个问题

我正在学习 javascript，并尝试不同的项目/编码挑战。 写完这段代码后，我意识到它是 43 行（这是一个基本的图像滑块），这不是最有效的方法。 有什么办法可以缩短我的代码，或者提高效率？ 这个脚本说当你按右时，它转到下一个图像，当你按左时，它转到上一个图像

 // Try edit message const data = [ { image:"https://image.shutterstock.com/image-photo/bright-spring-view-cameo-island-260nw-1048185397.jpg", }, { image:"https://image.shutterstock.com/image-photo/mountains-during-sunset-beautiful-natural-260nw-407021107.jpg", }, ] let img = document.querySelector("img") let left = document.querySelector('#left') let right = document.querySelector('#right') let origin = 0 img.src = data[origin].image function incr(number){ if (number == 'left') { if (origin - 1 >= 0){ origin += -1 }else{ origin = data.length - 1 } }else if (number == 'right'){ if (origin + 1 <= data.length - 1){ origin += 1 }else{ origin = 0 } } return origin; } function moveleft(){ let value = incr('left') img.src = data[value].image } function moveright(){ let value = incr('right') img.src = data[value].image }

 <div> <img src="https://playcode.io/static/img/logo.png" width = 1280px height = 720px> <button onclick = "moveleft()" id = 'left'>left</button> <button onclick = "moveright()" id = 'right'>right</button> </div>

感谢所有帮助，谢谢。

Answer 1

你可以这样做：

function incr(offset) {
  // either +1 or +(-1)
  const incremented = origin + offset;
  // Take the smallest value out of the 2, effectively constrains to <= len - 1
  const constrainedUpper = Math.min(data.length - 1, incremented); 
  // Take the largest value out of the 2, effectively constrains to >= 0
  const constrainedLower = Math.max(0, constrainedUpper);
  origin = constrainedLower;
}

function moveleft(){
  incr(-1)
  img.src = data[origin].image;
}

function moveright(){
  incr(1)
  img.src = data[origin].image
}

删除'left'和'right'因为它们实际上只是您想要移动多少的指标。 然后，使用数学函数将结果限制为0和data.length - 1 。