fix: code review

Author: Justin-ZS

src/component/visualMap/VisualMapModel.ts

@@ -261,7 +261,7 @@ class VisualMapModel<Opts extends VisualMapOption = VisualMapOption> extends Com
         if (seriesTargets) {
             // When seriesTargets is provided, collect all target series indices
             const indices: number[] = [];
-            for (const target of seriesTargets) {
+            each(seriesTargets, (target) => {
                 if (target.seriesIndex != null) {
                     indices.push(target.seriesIndex);
                 }
@@ -277,7 +277,7 @@ class VisualMapModel<Opts extends VisualMapOption = VisualMapOption> extends Com
                         indices.push(seriesModel.componentIndex);
                     }
                 }
-            }
+            });
             return indices;
         }
 

src/component/visualMap/preprocessor.ts

@@ -55,20 +55,18 @@ export default function visualMapPreprocessor(option) {
         }
 
         // Validate seriesTargets
-        const seriesTargets = opt.seriesTargets;
-        if (seriesTargets && zrUtil.isArray(seriesTargets)) {
-            each(seriesTargets, function (target) {
-                if (!zrUtil.isObject(target) || target.dimension == null) {
-                    if (__DEV__) {
+        if (__DEV__) {
+            const seriesTargets = opt.seriesTargets;
+            if (seriesTargets && zrUtil.isArray(seriesTargets)) {
+                each(seriesTargets, function (target) {
+                    if (!zrUtil.isObject(target) || target.dimension == null) {
                         console.warn('Each seriesTarget should have a dimension property');
                     }
-                }
-                if (target.seriesIndex == null && target.seriesId == null) {
-                    if (__DEV__) {
+                    if (target.seriesIndex == null && target.seriesId == null) {
                         console.warn('Each seriesTarget should have either seriesIndex or seriesId');
                     }
-                }
-            });
+                });
+            }
         }
     });
 }