这里需要 eval 函数吗？

Question

我想问你，在这段代码中省略 eval () 函数是否合理。特别是如何

  <script>
...
...

        function addGeoJson (geoJsonPath, iconPath = "leaflet-2/images/marker-icon.png", iconSize = [30,50], popUpContent, sideBarContent)
         
        {$.getJSON(geoJsonPath,function(data){
        
          var geoIcon = L.icon({
                        iconUrl: iconPath,
                        iconSize: iconSize
                        });
      
        L.geoJson(data,{
          pointToLayer: function(feature,latlng){
            var marker = L.marker(latlng,{icon: geoIcon, riseOnHover: true});
            
            marker.bindPopup(eval(popUpContent));
            
            marker.on('mouseover',function() {
              document.getElementById('sidebar').innerHTML = eval(sideBarContent);
            });
            
            marker.on('mouseout', function(){
              document.getElementById('sidebar').innerHTML = " ";
            });
            
            return marker;
          }
           
        }).addTo(map);
      });
      };

var layer1pop = "feature.properties.name + '<br/>' + feature.properties.info";
var layer1side = "feature.properties.price + '<br/>' + feature.properties.web";

addGeoJson("points.geojson", undefined, undefined, layer1pop, layer1side);

...
...
</script>

我想重用带有多个 geoJSON 文件的 addGeoJson 函数，并且为每个文件使用不同的标记模板、弹出窗口、侧边栏等。在我不知道 geoJSON 可以有多少属性以及是否我需要更改弹出窗口中属性项的顺序，例如我可以编辑 addGeoJson 基金的参数。如果我试图通过，就会发生错误

var layer1pop = feature.properties.name + '<br/>' + feature.properties.info;
var layer1side = feature.properties.price + '<br/>' + feature.properties.web;

直接因为不可能到达本地feature 并且在传递参数时它甚至不存在。 我尝试用new Function 替换它，但也无法访问feature。

你有什么想法吗？ 谢谢。

Answer 1

为什么不换行marker.bindPopup(eval(popUpContent)); 给marker.bindPopup(feature.properties.name + '<br/>' + feature.properties.info);

并将相同的逻辑应用于第二个函数。
如果有什么东西阻止了你，请重新编写它以使其正常工作。
eval 是一种邪恶的代码气味

如果映射不断变化（即您将使用 feature 的哪一部分），请尝试传递一个知道要提取什么的函数

var layer1pop = function(feature){
 return feature.properties.name + '<br/>' + feature.properties.info;
}

然后marker.bindPopup(eval(popUpContent)); 变成marker.bindPopup(popUpContent(feature));