Ignore:
Timestamp:
2015-08-02T23:21:30+02:00 (9 years ago)
Author:
donvip
Message:

[jmapviewer] fix more Javadoc/Sonar issues

Location:
applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer
Files:
4 edited

Legend:

Unmodified
Added
Removed

  • applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/JMapViewer.java

    r31438 r31439  
    250250    /**
    251251     * Sets the displayed map pane and zoom level so that all chosen map elements are visible.
     252     * @param markers whether to consider markers
     253     * @param rectangles whether to consider rectangles
     254     * @param polygons whether to consider polygons
    252255     */
    253256    public void setDisplayToFitMapElements(boolean markers, boolean rectangles, boolean polygons) {
    … …  
    262265            return;
    263266
    264         int x_min = Integer.MAX_VALUE;
    265         int y_min = Integer.MAX_VALUE;
    266         int x_max = Integer.MIN_VALUE;
    267         int y_max = Integer.MIN_VALUE;
     267        int xMin = Integer.MAX_VALUE;
     268        int yMin = Integer.MAX_VALUE;
     269        int xMax = Integer.MIN_VALUE;
     270        int yMax = Integer.MIN_VALUE;
    268271        int mapZoomMax = tileController.getTileSource().getMaxZoom();
    269272
    … …  
    273276                    if (marker.isVisible()) {
    274277                        Point p = tileSource.latLonToXY(marker.getCoordinate(), mapZoomMax);
    275                         x_max = Math.max(x_max, p.x);
    276                         y_max = Math.max(y_max, p.y);
    277                         x_min = Math.min(x_min, p.x);
    278                         y_min = Math.min(y_min, p.y);
     278                        xMax = Math.max(xMax, p.x);
     279                        yMax = Math.max(yMax, p.y);
     280                        xMin = Math.min(xMin, p.x);
     281                        yMin = Math.min(yMin, p.y);
    279282                    }
    280283                }
    … …  
    288291                        Point bottomRight = tileSource.latLonToXY(rectangle.getBottomRight(), mapZoomMax);
    289292                        Point topLeft = tileSource.latLonToXY(rectangle.getTopLeft(), mapZoomMax);
    290                         x_max = Math.max(x_max, bottomRight.x);
    291                         y_max = Math.max(y_max, topLeft.y);
    292                         x_min = Math.min(x_min, topLeft.x);
    293                         y_min = Math.min(y_min, bottomRight.y);
     293                        xMax = Math.max(xMax, bottomRight.x);
     294                        yMax = Math.max(yMax, topLeft.y);
     295                        xMin = Math.min(xMin, topLeft.x);
     296                        yMin = Math.min(yMin, bottomRight.y);
    294297                    }
    295298                }
    … …  
    303306                        for (ICoordinate c : polygon.getPoints()) {
    304307                            Point p = tileSource.latLonToXY(c, mapZoomMax);
    305                             x_max = Math.max(x_max, p.x);
    306                             y_max = Math.max(y_max, p.y);
    307                             x_min = Math.min(x_min, p.x);
    308                             y_min = Math.min(y_min, p.y);
     308                            xMax = Math.max(xMax, p.x);
     309                            yMax = Math.max(yMax, p.y);
     310                            xMin = Math.min(xMin, p.x);
     311                            yMin = Math.min(yMin, p.y);
    309312                        }
    310313                    }
    … …  
    316319        int width = Math.max(0, getWidth());
    317320        int newZoom = mapZoomMax;
    318         int x = x_max - x_min;
    319         int y = y_max - y_min;
     321        int x = xMax - xMin;
     322        int y = yMax - yMin;
    320323        while (x > width || y > height) {
    321324            newZoom--;
    … …  
    323326            y >>= 1;
    324327        }
    325         x = x_min + (x_max - x_min) / 2;
    326         y = y_min + (y_max - y_min) / 2;
     328        x = xMin + (xMax - xMin) / 2;
     329        y = yMin + (yMax - yMin) / 2;
    327330        int z = 1 << (mapZoomMax - newZoom);
    328331        x /= z;
    … …  
    416419        p.translate(-(center.x - getWidth() / 2), -(center.y - getHeight() /2));
    417420
    418         if (checkOutside) {
    419             if (p.x < 0 || p.y < 0 || p.x > getWidth() || p.y > getHeight())
    420                 return null;
     421        if (checkOutside && (p.x < 0 || p.y < 0 || p.x > getWidth() || p.y > getHeight())) {
     422            return null;
    421423        }
    422424        return p;
     425    }
     426
     427    /**
     428     * Calculates the position on the map of a given coordinate
     429     *
     430     * @param lat latitude
     431     * @param lon longitude
     432     * @return point on the map or <code>null</code> if the point is not visible
     433     */
     434    public Point getMapPosition(double lat, double lon) {
     435        return getMapPosition(lat, lon, true);
    423436    }
    424437
    … …  
    435448        Point p = tileSource.latLonToXY(lat, lon, zoom);
    436449        int y = p.y - center.y - getHeight() / 2;
    437         if (checkOutside) {
    438             if (y < 0 || y > getHeight())
    439                 return null;
     450        if (checkOutside && (y < 0 || y > getHeight())) {
     451            return null;
    440452        }
    441453        return y;
    … …  
    455467        int y = tileSource.latToY(lat + offset, zoom);
    456468        y -= center.y - getHeight() / 2;
    457         if (checkOutside) {
    458             if (y < 0 || y > getHeight())
    459                 return null;
     469        if (checkOutside && (y < 0 || y > getHeight())) {
     470            return null;
    460471        }
    461472        return y;
    … …  
    465476     * Calculates the position on the map of a given coordinate
    466477     *
    467      * @param lat latitude
    468      * @param lon longitude
    469      * @return point on the map or <code>null</code> if the point is not visible
    470      */
    471     public Point getMapPosition(double lat, double lon) {
    472         return getMapPosition(lat, lon, true);
    473     }
    474 
    475     /**
    476      * Calculates the position on the map of a given coordinate
    477      *
    478478     * @param marker MapMarker object that define the x,y coordinate
     479     * @param p coordinate
    479480     * @return Integer the radius in pixels
    480481     */
    … …  
    669670    /**
    670671     * Paint a single marker.
     672     * @param g Graphics used for painting
     673     * @param marker marker to paint
    671674     */
    672675    protected void paintMarker(Graphics g, MapMarker marker) {
    … …  
    702705    /**
    703706     * Paint a single rectangle.
     707     * @param g Graphics used for painting
     708     * @param rectangle rectangle to paint
    704709     */
    705710    protected void paintRectangle(Graphics g, MapRectangle rectangle) {
    … …  
    732737                        rectangle.paint(g, pTopLeft, pBottomRight);
    733738                    }
    734 
    735739                }
    736740            }
    … …  
    740744    /**
    741745     * Paint a single polygon.
     746     * @param g Graphics used for painting
     747     * @param polygon polygon to paint
    742748     */
    743749    protected void paintPolygon(Graphics g, MapPolygon polygon) {
    … …  
    10871093    /**
    10881094     * Return tile information caching class
     1095     * @return tile cache
    10891096     * @see TileController#getTileCache()
    10901097     */

  • applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/Tile.java

    r31438 r31439  
    9090     * Tries to get tiles of a lower or higher zoom level (one or two level
    9191     * difference) from cache and use it as a placeholder until the tile has been loaded.
     92     * @param cache Tile cache
    9293     */
    9394    public void loadPlaceholderFromCache(TileCache cache) {

  • applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/interfaces/TileJob.java

    r31429 r31439  
    2626    /**
    2727     * submits download job to backend.
    28      * @param force true if the load should skip all the caches (local & remote)
     28     * @param force true if the load should skip all the caches (local &amp; remote)
    2929     */
    3030    void submit(boolean force);

  • applications/viewer/jmapviewer/src/org/openstreetmap/gui/jmapviewer/tilesources/BingAerialTileSource.java

    r31434 r31439  
    5959    /**
    6060     * Constructs a new {@code BingAerialTileSource}.
     61     * @param info imagery info
    6162     */
    6263    public BingAerialTileSource(TileSourceInfo info) {
Note: See TracChangeset for help on using the changeset viewer.