Как можно оптимизировать данный код?

Question

[tostershmoster]

private isThumbsCollision(): boolean {
  let fromEdge: number;
  let toEdge: number;
  let thumbSize: number;

  if (this.settings.isVertical) {
    thumbSize = this.to.tooltip.element.getBoundingClientRect().height;
    fromEdge = this.from.element.getBoundingClientRect().top;
    toEdge = this.to.element.getBoundingClientRect().top;
  } else {
    thumbSize = this.to.tooltip.element.getBoundingClientRect().width;
    fromEdge = this.from.element.getBoundingClientRect().right;
    toEdge = this.to.element.getBoundingClientRect().right;
  }
  return toEdge - fromEdge <= thumbSize;
}

Comments

[acwartz]

Тут нечего оптимизировать потому что не обычная функция это кусок класса, и пока останое не видно то и оптимизировать нечего т.к. нет пищи для ума.

Answer 1

[Сергей Соколов]

Если оптимизировать = «чтобы без повторений», можно потерять читаемость:

private isThumbsCollision(): boolean {
  const prop = [
    {size: 'width', side: 'right'},
    {size: 'height', side: 'top'},
  ][+this.settings.isVertical];

  return this.to.tooltip.element.getBoundingClientRect()[prop.size] 
    >= this.to.element.getBoundingClientRect()[prop.side]
    - this.from.element.getBoundingClientRect()[prop.side];
}

Comments

[tostershmoster]

в данном случае важна читаемость

Answer 2

[mn3m0n1c_3n3m1]

const getRect = (el) => el.tooltip.element.getBoundingClientRect();

function isThumbsCollision({from, to, settings}) {
    const fromRect = getRect(from);
    const {top, right, height, width} = getRect(to);

    if (settings.isVertical)
        return top - fromRect.top <= height;

    return right - fromRect.right <= width
}

Comments

[tostershmoster]

там разные элементы to.element и to.tooltip.element