Changeset 493 in josm

Timestamp:

2007-12-17T01:35:58+01:00 (16 years ago)

Author:

gebner

Message:

Offscreen filtering; draw direction arrows on selection.
Patch by Petr Nejedly <Petr.Nejedly@…>

File:

• trunk/src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java (modified) (9 diffs)

trunk/src/org/openstreetmap/josm/data/osm/visitor/SimplePaintVisitor.java

@@ -7 +7 @@
 import java.awt.Point;
 import java.awt.Rectangle;
-import java.awt.Stroke;
 import java.awt.geom.GeneralPath;
-import java.awt.geom.Line2D;
-
+
+import java.util.Iterator;
 import org.openstreetmap.josm.Main;
 import org.openstreetmap.josm.data.osm.DataSet;
@@ -64 +63 @@
         protected GeneralPath currentPath = new GeneralPath();
 
+        private Rectangle screen;
+        Rectangle bbox = new Rectangle();
+
         public void visitAll(DataSet data) {
-                
                 inactiveColor = getPreferencesColor("inactive", Color.DARK_GRAY);
                 selectedColor = getPreferencesColor("selected", Color.WHITE);
@@ -97 +98 @@
                                 osm.visit(this);
                 displaySegments(null);
-        }
+        }
 
         /**
@@ -135 +136 @@
                 }
 
-                int orderNumber = 0;
-                Node lastN = null;
-                for (Node n : w.nodes) {
-                        if (lastN == null) {
-                                lastN = n;
-                                continue;
+                Iterator<Node> it = w.nodes.iterator();
+                if (it.hasNext()) {
+                        Point lastP = nc.getPoint(it.next().eastNorth);
+                        for (int orderNumber = 0; it.hasNext(); orderNumber++) {
+                                Point p = nc.getPoint(it.next().eastNorth);
+                                orderNumber++;
+                                drawSegment(lastP, p, w.selected && !inactive ? selectedColor : wayColor, showDirectionArrow || w.selected);
+                                if (showOrderNumber)
+                                        drawOrderNumber(lastP, p, orderNumber);
+                                lastP = p;
                         }
-                        orderNumber++;
-                        drawSegment(lastN, n, w.selected && !inactive ? selectedColor : wayColor, showDirectionArrow);
-                        if (showOrderNumber)
-                                drawOrderNumber(lastN, n, orderNumber);
-                        lastN = n;
                 }
         }
@@ -158 +158 @@
          * parents way
          */
-        protected void drawOrderNumber(Node n1, Node n2, int orderNumber) {
+        protected void drawOrderNumber(Point p1, Point p2, int orderNumber) {
                 int strlen = (""+orderNumber).length();
-                Point p1 = nc.getPoint(n1.eastNorth);
-                Point p2 = nc.getPoint(n2.eastNorth);
                 int x = (p1.x+p2.x)/2 - 4*strlen;
                 int y = (p1.y+p2.y)/2 + 4;
 
-                Rectangle screen = g.getClipBounds();
                 if (screen.contains(x,y)) {
                         Color c = g.getColor();
@@ -184 +181 @@
                 Point p = nc.getPoint(n.eastNorth);
                 g.setColor(color);
-                Rectangle screen = g.getClipBounds();
 
                 if (screen.contains(p.x, p.y))
@@ -198 +194 @@
          * Draw a line with the given color.
          */
-        protected void drawSegment(Node n1, Node n2, Color col, boolean showDirection) {
+        protected void drawSegment(Point p1, Point p2, Color col, boolean showDirection) {
 
                 if (col != currentColor) displaySegments(col);
                 
-                Point p1 = nc.getPoint(n1.eastNorth);
-                Point p2 = nc.getPoint(n2.eastNorth);
-                
-                Rectangle screen = g.getClipBounds();
-                Line2D line = new Line2D.Double(p1.x, p1.y, p2.x, p2.y);
-                if (screen.contains(p1.x, p1.y, p2.x, p2.y) || screen.intersectsLine(line))
-                {
+                if (onScreen(p1, p2)) {
                         currentPath.moveTo(p1.x, p1.y);
                         currentPath.lineTo(p2.x, p2.y);
@@ -220 +210 @@
                 }
         }
+        
+        private boolean onScreen(Point p1, Point p2) {
+                bbox.setBounds(p1.x, p1.y, 0, 0);
+                bbox.add(p2);
+                if (bbox.height + bbox.width < 1) return false;
+                bbox.width++;
+                bbox.height++;
+                return screen.intersects(bbox);
+        }
 
         public static Color getPreferencesColor(String colName, Color def) {
@@ -231 +230 @@
         
         public void setGraphics(Graphics g) {
-        this.g = g;
-    }
+                this.g = g;
+                screen = g.getClipBounds();
+        }
 
         public void setNavigatableComponent(NavigatableComponent nc) {