Javascript 拖放闪烁效果带网格

Question

我正在尝试为 tilemap 图像创建一个选择系统。 添加图像，每当用户选择一个图块并将鼠标移动到 select 多个图块时，选择容器的大小会随着图块大小的增加而增加。

不知何故，当按住鼠标按钮并将其移动时，这会产生闪烁效果。 是什么导致了这个问题？

var tilesetImageContainer = document.getElementById("tileMapContainer");
var tileSize = 32;
let startSelection = [];
var selectionWidth = 32;
var selectionHeight = 32;
let endSelection = [];

function getCoordinates(e) {
  const { x, y } = e.target.getBoundingClientRect();
  const mouseX = e.clientX - x;
  const mouseY = e.clientY - y;
  return [Math.floor(mouseX / tileSize), Math.floor(mouseY / tileSize)];
}

const mouseDownHandler = function (e) {

if (e.which === 1) {
  e.preventDefault();
  selectedTileMapPositions = [];
  tileMapSelector.style.width = tileSize + "px";
  tileMapSelector.style.height = tileSize + "px";
  tilesetImageContainer.style.cursor = 'crosshair';
  tilesetImageContainer.style.userSelect = 'none';

  startSelection = getCoordinates(e);
  tileMapSelector.style.left = startSelection[0] * tileSize + "px";
  tileMapSelector.style.top = startSelection[1] * tileSize + "px";

  tilesetImageContainer.addEventListener('mousemove', mouseMoveSelectionHandler);
  tilesetImageContainer.addEventListener('mouseup', mouseUpSelectionHandler);
}

};

const mouseMoveSelectionHandler = function (e) {

currentPositionCoordinates = getCoordinates(e);
endSelection = getCoordinates(e);

if (currentPositionCoordinates[0] > startSelection[0] || currentPositionCoordinates[1] < startSelection[1]) {
  // selected more then 1 tile

  selectionWidth = ((currentPositionCoordinates[0] - startSelection[0]) * tileSize);
  selectionHeight = ((currentPositionCoordinates[1] - startSelection[1]) * tileSize);
}


tileMapSelector.style.width = selectionWidth + tileSize + "px";
tileMapSelector.style.height = selectionHeight + tileSize + "px";

console.log("start: " + startSelection);
console.log("current: " + currentPositionCoordinates);
};

const mouseUpSelectionHandler = function (e) {

for (var x = startSelection[0]; x <= endSelection[0]; x++) {
  for (var y = startSelection[1]; y <= endSelection[1]; y++) {
    selectedTileMapPositions.push([x,y]);
  }
}


console.log(selectedTileMapPositions);

tilesetImageContainer.removeEventListener('mousemove', mouseMoveSelectionHandler);
tilesetImageContainer.removeEventListener('mouseup', mouseUpSelectionHandler);

tilesetImageContainer.style.cursor = 'default';
tilesetImageContainer.style.removeProperty('user-select');

startSelection = [];
endSelection = [];
};

tilesetImageContainer.addEventListener('mousedown', mouseDownHandler);

这是一个 JsFiddle： https://jsfiddle.net/6kv3pj1n/19/

Answer 1

如果没有必要，不要重新分配变量。 声明 currentPositionCoordinates on top fix getCoordinates function 不要使用 x 和 y

let currentPositionCoordinates

function getCoordinates(e) {
  const mouseX = e.clientX;
  const mouseY = e.clientY;
  return [Math.floor(mouseX / tileSize), Math.floor(mouseY / tileSize)];
}

if (currentPositionCoordinates) {
  if (currentPositionCoordinates[0] === getCoordinates(e)[0] && currentPositionCoordinates[1] === getCoordinates(e)[1]) {
  return
  }
}

if (selectionWidth + tileSize + "px" !== tileMapSelector.style.width) {
    tileMapSelector.style.width = selectionWidth + tileSize + "px"
}
if (selectionHeight + tileSize + "px" !== tileMapSelector.style.height) {
    tileMapSelector.style.height = selectionHeight + tileSize + "px"
}

https://jsfiddle.net/2Lx740vg/2/

最新编辑应该完全按照您的预期工作