java - 重构——结合PolygonClickListener和MarkerClickListener

Question

我试图通过删除 map.setOnPolygonClickListener 和 map.setOnMarkerClickListener 中的重复项来重构以下方法。

两个监听器执行完全相同的操作，我想将它们合并为一个，但由于调用来自不同的对象(Marker 或 Polygon)，我不是确定如何组合它们。

public class NavigationFragment extends Fragment implements OnMapReadyCallback,
    OnCameraIdleListener, Subject, SearchView.OnQueryTextListener, SearchView.OnCloseListener 
{

    /**
    * add markers and polygons overlay for each building
    */

    private void addBuildingMarkersAndPolygons() {
        List<Building> allBuildings = new ArrayList<>();
        allBuildings.addAll(Campus.SGW.getBuildings());
        allBuildings.addAll(Campus.LOY.getBuildings());

        for (Building building : allBuildings) {
            createBuildingMarkersAndPolygonOverlay(building);
        }

        map.setOnPolygonClickListener(new GoogleMap.OnPolygonClickListener() {
           @Override
           public void onPolygonClick(Polygon polygon) {
                setBottomSheetContent(polygon);
                setupOriginAndDestinationForOutdoorDirection(polygon);
            }
          });

       map.setOnMarkerClickListener(new GoogleMap.OnMarkerClickListener() {
            @Override
            public boolean onMarkerClick(Marker marker) {
                setBottomSheetContent(marker);
                setupOriginAndDestinationForOutdoorDirection(marker);
                return true;
           }
       });
    }
}

Answer 1

我本来想把它写成评论，但太长了。可能是我没有正确理解这个问题，标记和多边形没有共同的父级。我想，如果你创建一个嵌套类，即

private class PolygonAndMarkerClickListener implements GoogleMap.OnPolygonClickListener, GoogleMap.OnMarkerClickListener{

        @Override
        public boolean onMarkerClick(Marker marker) {
            //your code
            return false;
        }

        @Override
        public void onPolygonClick(Polygon polygon) {
            //your code
        }
    }

然后

PolygonAndMarkerClickListener polygonAndMarkerClickListener = new PolygonAndMarkerClickListener();
        map.setOnPolygonClickListener(polygonAndMarkerClickListener);
        map.setOnMarkerClickListener(polygonAndMarkerClickListener);

这会减少一些开销，而且看起来也不错。

如果您想创建一个方法，以便可以从两个监听器调用，那么您必须使用“instanceOf”，许多高级开发人员认为应该避免使用“instanceOf”。我想泛型在这里不会有太大帮助。