ThinkGeo.com    |     Documentation    |     Premium Support

Shape validation

#1

Hello,


in our application users can modify the vertices collection of line/polygon shape. To find out if the modified shape is valid I use the shapes validate() method. But if you can see in the picture above the validate() method still returns 'True' although the last vertex of polygon shape has been deleted.


Is this a bug. How can I check wether the shape is valid?



Thomas



ShapeValidation.png (30 KB)

#2

Thomas,


Thanks for your post and questions.
 
Yes, as you already have noticed, the validation will still pass even though the last vertex was removed from the ring of the polygon, like the following code snippet shows:

 

RingShape ringShape = new RingShape();
ringShape.Vertices.Add(new Vertex(0, 0));
ringShape.Vertices.Add(new Vertex(1, 0));
ringShape.Vertices.Add(new Vertex(1, 1));
ringShape.Vertices.Add(new Vertex(0, 1));

PolygonShape polygonShape = new PolygonShape();
polygonShape.OuterRing = ringShape;
double area = polygonShape.GetArea(GeographyUnit.Meter, AreaUnit.SquareMeters);
ShapeValidationResult result = polygonShape.Validate(ShapeValidationMode.Simple);

 
The reason for this is that we only did very simple validation on the shapes, for the lineshape, we only check the vertex number exceeds 2 and for the RingShape we only check the vertex number is greater than 3. We reverse the complex logic to advanced validation mode. For now, we can need to write the validate logic as we want, following is some code snippet which add the vertex checking for the out ring of the polgyonshape.


ShapeValidationResult newResult = ValidatePolygon(polygonShape);
private static ShapeValidationResult ValidatePolygon(PolygonShape polygonShape)
{
     ShapeValidationResult result = polygonShape.Validate(ShapeValidationMode.Simple);

     int ourRingVertexCount = polygonShape.OuterRing.Vertices.Count;
     if (polygonShape.OuterRing.Vertices[0] != polygonShape.OuterRing.Vertices[ourRingVertexCount - 1])
     {
         result = new ShapeValidationResult(false, "The first vertex is not the last vertex");
     }

     return result;
}

 
Any more questions just feel free to let me know.
 
Thanks.
 
Yale

#3

Hello Yale,


as I can see I can implement my own validation logic but this is not what I want. I do not know what parameters have to be checked to decide a shape is valid. Valid for me means that your drawing logic or the ThinkGeo methods they work with shape did not throw an Exception e.g. "points must form a closed linestring".


I think you have to improve your validation method to check all the parameters of the shpe to avoid an Exception is throw in your core methods.


Thanks Thomas



#4

Thomas,


Thanks for your post and feedback.
 
I agree with you that we may need to enhance the validation for the shapes. The non-closed ring shape passed our validation while the NTS will be throwing exception when we try to create Geometry from it. The exception “points must form a closed linestring” is thrown out by nts GisSharpBlog.NetTopologySuite.IO.WKTReader when we try to use it to create Geometry.
 
I have added this enhancement to our working list and hope we can get it done soon.
 
Any more questions just feel free to let me know.
 
Thanks.
 
Yale

#5

Hello Yale, 
  
 just want to ask the status on enhacing the validation algorithm. 
  
 Thomas

#6

Thomas, 
  
 The enhancement is not finished yet, because recently we are busy working on another higher priority task. If we are finish it I will let you know. 
  
 Sorry for inconvenience 
  
 James