Closed
Description
maptalks's version and what browser you use?
Issue description
- not use forEach
for (let i = 0, len = this.layer._geoList.length; i < len; i++) {
this.checkGeo(this.layer._geoList[i]);
}
- It's too time-consuming,I've even found that it's faster without field filtering
const extent2D = painter.get2DExtent(this.resources, TEMP_EXTENT);
if (!extent2D || !extent2D.intersects(this._displayExtent)) {
return;
}
if (painter.hasPoint()) {
this._hasPoint = true;
}
maptalks.js/src/renderer/geometry/Painter.js
Line 406 in c647ad2
this.containerOffset = offset || layer.containerOffset || map._pointToContainerPoint(renderer.southWest)._add(0, -map.height);
- Do not use intrinsic functions
maptalks.js/src/core/Canvas.js
Line 841 in c647ad2
maptalks.js/src/core/Canvas.js
Line 796 in c647ad2
maptalks.js/src/core/Canvas.js
Line 608 in c647ad2
maptalks.js/src/core/Canvas.js
Line 636 in c647ad2
maptalks.js/src/core/Canvas.js
Line 420 in c647ad2
Please provide a reproduction URL (on any jsfiddle like site)
Metadata
Metadata
Assignees
Labels
No labels