GO: replace if-else-if by switch 96/13196/2
Calixte DENIZET [Fri, 22 Nov 2013 10:14:20 +0000 (11:14 +0100)]
Change-Id: I167c3d4f4e6b022e1b93f2b9d3013a084b508f25

scilab/modules/graphic_objects/src/cpp/NgonGeneralData.cpp
scilab/modules/graphic_objects/src/cpp/NgonGridData.cpp
scilab/modules/graphic_objects/src/cpp/NgonPolylineData.cpp
scilab/modules/graphic_objects/src/cpp/PolylineDecomposer.cpp
scilab/modules/graphic_objects/src/cpp/TriangleMeshData.cpp

index 53eb1b0..b8ea011 100644 (file)
@@ -72,34 +72,27 @@ int NgonGeneralData::getPropertyFromName(int propertyName)
 
 int NgonGeneralData::setDataProperty(int property, void const* value, int numElements)
 {
-
-    if (property == NUM_ELEMENTS_ARRAY)
-    {
-        return setNumElementsArray((int const*) value);
-    }
-    else if (property == COORDINATES)
-    {
-        setData((double const*) value, numElements);
-    }
-    else if (property == X_COORDINATES)
-    {
-        setDataX((double const*) value, numElements);
-    }
-    else if (property == Y_COORDINATES)
-    {
-        setDataY((double const*) value, numElements);
-    }
-    else if (property == Z_COORDINATES)
-    {
-        setDataZ((double const*) value, numElements);
-    }
-    else if (property == COLORS)
-    {
-        setColors((double const*) value, numElements);
-    }
-    else
+    switch (property)
     {
-        return NgonData::setDataProperty(property, value, numElements);
+        case NUM_ELEMENTS_ARRAY :
+            return setNumElementsArray((int const*) value);
+        case COORDINATES :
+            setData((double const*) value, numElements);
+            break;
+        case X_COORDINATES :
+            setDataX((double const*) value, numElements);
+            break;
+        case Y_COORDINATES :
+            setDataY((double const*) value, numElements);
+            break;
+        case Z_COORDINATES :
+            setDataZ((double const*) value, numElements);
+            break;
+        case COLORS :
+            setColors((double const*) value, numElements);
+            break;
+        default :
+            return NgonData::setDataProperty(property, value, numElements);
     }
 
     return 1;
@@ -107,40 +100,33 @@ int NgonGeneralData::setDataProperty(int property, void const* value, int numEle
 
 void NgonGeneralData::getDataProperty(int property, void **_pvData)
 {
-    if (property == NUM_ELEMENTS_ARRAY)
-    {
-        /* Not implemented yet */
-        //    getNumElementsArray();
-    }
-    else if (property == COORDINATES)
-    {
-        *_pvData = getData();
-    }
-    else if (property == X_COORDINATES)
-    {
-        *_pvData = getDataX();
-    }
-    else if (property == Y_COORDINATES)
-    {
-        *_pvData = getDataY();
-    }
-    else if (property == Z_COORDINATES)
-    {
-        *_pvData = getDataZ();
-    }
-    else if (property == COLORS)
-    {
-        *_pvData = getColors();
-    }
-    else if (property == NUM_COLORS)
-    {
-        ((int *) *_pvData)[0] = getNumColors();
-    }
-    else
+    switch (property)
     {
-        NgonData::getDataProperty(property, _pvData);
+        case NUM_ELEMENTS_ARRAY :
+            /* Not implemented yet */
+            //    getNumElementsArray();
+            break;
+        case COORDINATES :
+            *_pvData = getData();
+            break;
+        case X_COORDINATES :
+            *_pvData = getDataX();
+            break;
+        case Y_COORDINATES :
+            *_pvData = getDataY();
+            break;
+        case Z_COORDINATES :
+            *_pvData = getDataZ();
+            break;
+        case COLORS :
+            *_pvData = getColors();
+            break;
+        case NUM_COLORS :
+            ((int *) *_pvData)[0] = getNumColors();
+            break;
+        default :
+            NgonData::getDataProperty(property, _pvData);
     }
-
 }
 
 double* NgonGeneralData::getData()
index 463225d..4085b26 100644 (file)
@@ -93,29 +93,24 @@ int NgonGridData::getPropertyFromName(int propertyName)
 
 int NgonGridData::setDataProperty(int property, void const* value, int numElements)
 {
-    if (property == GRID_SIZE)
-    {
-        return setGridSize((int const*) value);
-    }
-    else if (property == X_COORDINATES)
-    {
-        setDataX((double const*) value, numElements);
-    }
-    else if (property == Y_COORDINATES)
-    {
-        setDataY((double const*) value, numElements);
-    }
-    else if (property == Z_COORDINATES)
-    {
-        setDataZ((double const*) value, numElements);
-    }
-    else if (property == Z_COORDINATES_SHIFT)
-    {
-        setZCoordinatesShift((double const*) value);
-    }
-    else
-    {
-        return NgonData::setDataProperty(property, value, numElements);
+    switch (property)
+    {
+        case GRID_SIZE :
+            return setGridSize((int const*) value);
+        case X_COORDINATES :
+            setDataX((double const*) value, numElements);
+            break;
+        case Y_COORDINATES :
+            setDataY((double const*) value, numElements);
+            break;
+        case Z_COORDINATES :
+            setDataZ((double const*) value, numElements);
+            break;
+        case Z_COORDINATES_SHIFT :
+            setZCoordinatesShift((double const*) value);
+            break;
+        default :
+            return NgonData::setDataProperty(property, value, numElements);
     }
 
     return 1;
@@ -123,47 +118,38 @@ int NgonGridData::setDataProperty(int property, void const* value, int numElemen
 
 void NgonGridData::getDataProperty(int property, void **_pvData)
 {
-    if (property == NUM_X)
-    {
-        ((int *) *_pvData)[0] = getNumX();
-    }
-    else if (property == NUM_Y)
-    {
-        ((int *) *_pvData)[0] = getNumY();
-    }
-    else if (property == NUM_Z)
-    {
-        ((int *) *_pvData)[0] = getNumZ();
-    }
-    else if (property == X_DIMENSIONS)
-    {
-        *_pvData = getXDimensions();
-    }
-    else if (property == Y_DIMENSIONS)
-    {
-        *_pvData = getYDimensions();
-    }
-    else if (property == X_COORDINATES)
-    {
-        *_pvData = getDataX();
-    }
-    else if (property == Y_COORDINATES)
-    {
-        *_pvData = getDataY();
-    }
-    else if (property == Z_COORDINATES)
-    {
-        *_pvData = getDataZ();
-    }
-    else if (property == Z_COORDINATES_SHIFT)
-    {
-        ((double *) *_pvData)[0] = getZCoordinatesShift();
-    }
-    else
-    {
-        NgonData::getDataProperty(property, _pvData);
+    switch (property)
+    {
+        case NUM_X:
+            ((int *) *_pvData)[0] = getNumX();
+            break;
+        case NUM_Y :
+            ((int *) *_pvData)[0] = getNumY();
+            break;
+        case NUM_Z :
+            ((int *) *_pvData)[0] = getNumZ();
+            break;
+        case X_DIMENSIONS :
+            *_pvData = getXDimensions();
+            break;
+        case Y_DIMENSIONS :
+            *_pvData = getYDimensions();
+            break;
+        case X_COORDINATES :
+            *_pvData = getDataX();
+            break;
+        case Y_COORDINATES :
+            *_pvData = getDataY();
+            break;
+        case Z_COORDINATES :
+            *_pvData = getDataZ();
+            break;
+        case Z_COORDINATES_SHIFT :
+            ((double *) *_pvData)[0] = getZCoordinatesShift();
+            break;
+        default :
+            NgonData::getDataProperty(property, _pvData);
     }
-
 }
 
 int NgonGridData::setGridSize(int const* gridSize)
@@ -229,7 +215,7 @@ int NgonGridData::setGridSize(int const* gridSize)
         }
     }
 
-    if (newXSize*newYSize != xSize * ySize)
+    if (newXSize * newYSize != xSize * ySize)
     {
         zModified = 1;
 
index 7ce7526..3d3d99c 100644 (file)
@@ -87,41 +87,30 @@ int NgonPolylineData::getPropertyFromName(int propertyName)
 
 int NgonPolylineData::setDataProperty(int property, void const* value, int numElements)
 {
-    if (property == NUM_ELEMENTS_ARRAY)
-    {
-        return setNumElementsArray((int const*) value);
-    }
-    else if (property == X_COORDINATES_SHIFT)
-    {
-        return setXCoordinatesShift((double const*) value, numElements);
-    }
-    else if (property == Y_COORDINATES_SHIFT)
-    {
-        return setYCoordinatesShift((double const*) value, numElements);
-    }
-    else if (property == Z_COORDINATES_SHIFT)
-    {
-        return setZCoordinatesShift((double const*) value, numElements);
-    }
-    else if (property == X_COORDINATES_SHIFT_SET)
-    {
-        setXCoordinatesShiftSet(*((int const*) value));
-    }
-    else if (property == Y_COORDINATES_SHIFT_SET)
-    {
-        setYCoordinatesShiftSet(*((int const*) value));
-    }
-    else if (property == Z_COORDINATES_SHIFT_SET)
-    {
-        setZCoordinatesShiftSet(*((int const*) value));
-    }
-    else if (property == Z_COORDINATES_SET)
-    {
-        setZCoordinatesSet(*((int const*) value));
-    }
-    else
-    {
-        return NgonGeneralData::setDataProperty(property, value, numElements);
+    switch (property)
+    {
+        case NUM_ELEMENTS_ARRAY :
+            return setNumElementsArray((int const*) value);
+        case X_COORDINATES_SHIFT :
+            return setXCoordinatesShift((double const*) value, numElements);
+        case Y_COORDINATES_SHIFT :
+            return setYCoordinatesShift((double const*) value, numElements);
+        case Z_COORDINATES_SHIFT :
+            return setZCoordinatesShift((double const*) value, numElements);
+        case X_COORDINATES_SHIFT_SET :
+            setXCoordinatesShiftSet(*((int const*) value));
+            break;
+        case Y_COORDINATES_SHIFT_SET :
+            setYCoordinatesShiftSet(*((int const*) value));
+            break;
+        case Z_COORDINATES_SHIFT_SET :
+            setZCoordinatesShiftSet(*((int const*) value));
+            break;
+        case Z_COORDINATES_SET :
+            setZCoordinatesSet(*((int const*) value));
+            break;
+        default :
+            return NgonGeneralData::setDataProperty(property, value, numElements);
     }
 
     return 1;
@@ -129,43 +118,35 @@ int NgonPolylineData::setDataProperty(int property, void const* value, int numEl
 
 void NgonPolylineData::getDataProperty(int property, void **_pvData)
 {
-    if (property == X_COORDINATES_SHIFT)
-    {
-        *_pvData = getXCoordinatesShift();
-    }
-    else if (property == Y_COORDINATES_SHIFT)
-    {
-        *_pvData = getYCoordinatesShift();
-    }
-    else if (property == Z_COORDINATES_SHIFT)
-    {
-        *_pvData = getZCoordinatesShift();
-    }
-    else if (property == NUM_ELEMENTS)
-    {
-        ((int *) *_pvData)[0] = getNumElements();
-    }
-    else if (property == X_COORDINATES_SHIFT_SET)
-    {
-        ((int *) *_pvData)[0] = getXCoordinatesShiftSet();
-    }
-    else if (property == Y_COORDINATES_SHIFT_SET)
-    {
-        ((int *) *_pvData)[0] = getYCoordinatesShiftSet();
-    }
-    else if (property == Z_COORDINATES_SHIFT_SET)
-    {
-        ((int *) *_pvData)[0] = getZCoordinatesShiftSet();
-    }
-    else if (property == Z_COORDINATES_SET)
-    {
-        ((int *) *_pvData)[0] = getZCoordinatesSet();
-    }
-    else
-    {
-        NgonGeneralData::getDataProperty(property, _pvData);
+    switch (property)
+    {
+        case X_COORDINATES_SHIFT :
+            *_pvData = getXCoordinatesShift();
+            break;
+        case Y_COORDINATES_SHIFT :
+            *_pvData = getYCoordinatesShift();
+            break;
+        case Z_COORDINATES_SHIFT :
+            *_pvData = getZCoordinatesShift();
+            break;
+        case NUM_ELEMENTS :
+            ((int *) *_pvData)[0] = getNumElements();
+            break;
+        case X_COORDINATES_SHIFT_SET :
+            ((int *) *_pvData)[0] = getXCoordinatesShiftSet();
+            break;
+        case Y_COORDINATES_SHIFT_SET :
+            ((int *) *_pvData)[0] = getYCoordinatesShiftSet();
+            break;
+        case Z_COORDINATES_SHIFT_SET :
+            ((int *) *_pvData)[0] = getZCoordinatesShiftSet();
+            break;
+        case Z_COORDINATES_SET :
+            ((int *) *_pvData)[0] = getZCoordinatesSet();
+            break;
+        default :
+            NgonGeneralData::getDataProperty(property, _pvData);
     }
-
 }
 
 int NgonPolylineData::getNumElements(void)
@@ -347,7 +328,7 @@ int NgonPolylineData::setNumElementsArray(int const* numElementsArray)
         return 1;
     }
 
-    if (numGons*numVerticesPerGon != newNumElements)
+    if (numGons * numVerticesPerGon != newNumElements)
     {
         double* newCoordinates = NULL;
         double* xShiftNew = NULL;
index a992fe7..6d9310b 100644 (file)
@@ -46,58 +46,46 @@ int PolylineDecomposer::getDataSize(int id)
         return 0;
     }
 
-    /* Segments */
-    if (polylineStyle == 1)
-    {
-        return nPoints;
-    }
-    /* Staircase */
-    else if (polylineStyle == 2)
+    switch (polylineStyle)
     {
-        if (closed)
-        {
+        case 1 :
+            /* Segments */
+            return nPoints;
+        case 2 :
+            /* Staircase */
+            if (closed)
+            {
+                return 2 * nPoints;
+            }
+            else
+            {
+                return (2 * nPoints) - 1;
+            }
+        case 3 :
+            /* Vertical segments plus segments */
             return 2 * nPoints;
-        }
-        else
+        case 4 :
+            /* Segments with arrow heads */
         {
-            return (2 * nPoints) - 1;
-        }
-    }
-    /* Vertical segments plus segments */
-    else if (polylineStyle == 3)
-    {
-        return 2 * nPoints;
-    }
-    /* Segments with arrow heads */
-    else if (polylineStyle == 4)
-    {
-        int nArrowVertices;
-        /* The numbers of arrow head vertices and indices are exactly the same */
-        nArrowVertices = PolylineDecomposer::getArrowTriangleIndicesSize(nPoints, closed);
+            int nArrowVertices;
+            /* The numbers of arrow head vertices and indices are exactly the same */
+            nArrowVertices = PolylineDecomposer::getArrowTriangleIndicesSize(nPoints, closed);
 
-        return nPoints + nArrowVertices;
-    }
-    /* Filled patch  */
-    else if (polylineStyle == 5)
-    {
-        return nPoints;
-    }
-    /* Vertical bars plus segments */
-    else if (polylineStyle == 6)
-    {
-        return 5 * nPoints;
-    }
-    /* Horizontal bars plus segments */
-    else if (polylineStyle == 7)
-    {
-        return 5 * nPoints;
-    }
-    /* To be done: remaining styles */
-    else
-    {
-        return 0;
+            return nPoints + nArrowVertices;
+        }
+        case 5 :
+            /* Filled patch  */
+            return nPoints;
+        case 6 :
+            /* Vertical bars plus segments */
+            return 5 * nPoints;
+        case 7 :
+            /* Horizontal bars plus segments */
+            return 5 * nPoints;
+        default :
+            /* To be done: remaining styles */
+            return 0;
     }
-
 }
 
 void PolylineDecomposer::fillVertices(int id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation, int logMask)
@@ -122,35 +110,30 @@ void PolylineDecomposer::fillVertices(int id, float* buffer, int bufferLength, i
 
     getGraphicObjectProperty(id, __GO_POLYLINE_STYLE__, jni_int, (void**) &piPolylineStyle);
 
-    if (polylineStyle == 1)
-    {
-        fillSegmentsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
-    }
-    else if (polylineStyle == 2)
-    {
-        fillStairDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
-    }
-    else if (polylineStyle == 3)
-    {
-        fillVerticalLinesDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+    switch (polylineStyle)
+    {
+        case 1 :
+            fillSegmentsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+            break;
+        case 2 :
+            fillStairDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+            break;
+        case 3 :
+            fillVerticalLinesDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+            break;
+        case 4 :
+            fillSegmentsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+            break;
+        case 5 :
+            fillSegmentsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+            break;
+        case 6 :
+            fillVerticalBarsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+            break;
+        case 7 :
+            fillHorizontalBarsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
+            break;
     }
-    else if (polylineStyle == 4)
-    {
-        fillSegmentsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
-    }
-    else if (polylineStyle == 5)
-    {
-        fillSegmentsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
-    }
-    else if (polylineStyle == 6)
-    {
-        fillVerticalBarsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
-    }
-    else if (polylineStyle == 7)
-    {
-        fillHorizontalBarsDecompositionVertices(id, buffer, bufferLength, elementsSize, coordinateMask, scale, translation, logMask, t, nPoints, xshift, yshift, zshift);
-    }
-
 }
 
 void PolylineDecomposer::fillSegmentsDecompositionVertices(int id, float* buffer, int bufferLength, int elementsSize, int coordinateMask, double* scale, double* translation,
@@ -992,26 +975,19 @@ int PolylineDecomposer::fillIndices(int id, int* buffer, int bufferLength, int l
         return 0;
     }
 
-    if (polylineStyle == 1)
+    switch (polylineStyle)
     {
-        return fillTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, fillMode, polylineStyle);
-    }
-    else if (polylineStyle == 4)
-    {
-        return fillArrowTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift);
-    }
-    else if (polylineStyle == 5)
-    {
-        /* Set fill mode to on, since patches are always filled whatever fill mode's value */
-        return fillTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, 1, polylineStyle);
-    }
-    else if (polylineStyle == 6)
-    {
-        return fillBarsDecompositionTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift);
-    }
-    else if (polylineStyle == 7)
-    {
-        return fillBarsDecompositionTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift);
+        case 1 :
+            return fillTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, fillMode, polylineStyle);
+        case 4 :
+            return fillArrowTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift);
+        case 5 :
+            /* Set fill mode to on, since patches are always filled whatever fill mode's value */
+            return fillTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, 1, polylineStyle);
+        case 6 :
+            return fillBarsDecompositionTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift);
+        case 7 :
+            return fillBarsDecompositionTriangleIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift);
     }
 
     return 0;
@@ -1341,46 +1317,32 @@ int PolylineDecomposer::getWireIndicesSize(int id)
         return 0;
     }
 
-    /* Segments */
-    if (polylineStyle == 1)
-    {
-        return getSegmentsDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
-    }
-    /* Staircase */
-    else if (polylineStyle == 2)
-    {
-        return getStairDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
-    }
-    /* Vertical segments plus segments */
-    else if (polylineStyle == 3)
-    {
-        return getVerticalLinesDecompositionSegmentIndicesSize(nPoints, lineMode);
-    }
-    /* Segments with arrow heads */
-    else if (polylineStyle == 4)
-    {
-        return getSegmentsDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
-    }
-    /* Filled patch  */
-    else if (polylineStyle == 5)
-    {
-        return getSegmentsDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
-    }
-    /* Vertical bars plus segments */
-    else if (polylineStyle == 6)
-    {
-        return getBarsDecompositionSegmentIndicesSize(nPoints, lineMode);
-    }
-    /* Horizontal bars plus segments */
-    else if (polylineStyle == 7)
-    {
-        return getBarsDecompositionSegmentIndicesSize(nPoints, lineMode);
-    }
-    else
-    {
-        return 0;
+    switch (polylineStyle)
+    {
+        case 1:
+            /* Segments */
+            return getSegmentsDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
+        case 2 :
+            /* Staircase */
+            return getStairDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
+        case 3 :
+            /* Vertical segments plus segments */
+            return getVerticalLinesDecompositionSegmentIndicesSize(nPoints, lineMode);
+        case 4 :
+            /* Segments with arrow heads */
+            return getSegmentsDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
+        case 5 :
+            /* Filled patch  */
+            return getSegmentsDecompositionSegmentIndicesSize(nPoints, lineMode, closed);
+        case 6 :
+            /* Vertical bars plus segments */
+            return getBarsDecompositionSegmentIndicesSize(nPoints, lineMode);
+        case 7 :
+            /* Horizontal bars plus segments */
+            return getBarsDecompositionSegmentIndicesSize(nPoints, lineMode);
     }
 
+    return 0;
 }
 
 int PolylineDecomposer::getSegmentsDecompositionSegmentIndicesSize(int nPoints, int lineMode, int closed)
@@ -1492,33 +1454,22 @@ int PolylineDecomposer::fillWireIndices(int id, int* buffer, int bufferLength, i
     getGraphicObjectProperty(id, __GO_LINE_MODE__, jni_bool, (void**) &piLineMode);
     getGraphicObjectProperty(id, __GO_CLOSED__, jni_bool, (void**) &piClosed);
 
-    if (polylineStyle == 1)
-    {
-        return fillSegmentsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
-    }
-    else if (polylineStyle == 2)
-    {
-        return fillStairDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
-    }
-    else if (polylineStyle == 3)
+    switch (polylineStyle)
     {
-        return fillVerticalLinesDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode);
-    }
-    else if (polylineStyle == 4)
-    {
-        return fillSegmentsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
-    }
-    else if (polylineStyle == 5)
-    {
-        return fillSegmentsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
-    }
-    else if (polylineStyle == 6)
-    {
-        return fillBarsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode);
-    }
-    else if (polylineStyle == 7)
-    {
-        return fillBarsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode);
+        case 1 :
+            return fillSegmentsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
+        case 2 :
+            return fillStairDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
+        case 3 :
+            return fillVerticalLinesDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode);
+        case 4 :
+            return fillSegmentsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
+        case 5 :
+            return fillSegmentsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode, closed);
+        case 6 :
+            return fillBarsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode);
+        case 7 :
+            return fillBarsDecompositionSegmentIndices(id, buffer, bufferLength, logMask, coordinates, nPoints, xshift, yshift, zshift, lineMode);
     }
 
     return 0;
@@ -1781,6 +1732,4 @@ void PolylineDecomposer::getShiftedPolylinePoint(double* coordinates, double* xs
     {
         *z += zshift[index];
     }
-
 }
-
index 87ed951..64ec89a 100644 (file)
@@ -86,41 +86,32 @@ int TriangleMeshData::getPropertyFromName(int propertyName)
 
 int TriangleMeshData::setDataProperty(int property, void const* value, int numElements)
 {
-    if (property == NUM_VERTICES)
+    switch (property)
     {
-        return setNumVertices(*((unsigned int const*) value));
-    }
-    else if (property == NUM_INDICES)
-    {
-        return setNumIndices(*((unsigned int const*) value));
-    }
-    else if (property == X_COORDINATES)
-    {
-        setDataX((double const*) value, numElements);
-    }
-    else if (property == Y_COORDINATES)
-    {
-        setDataY((double const*) value, numElements);
-    }
-    else if (property == Z_COORDINATES)
-    {
-        setDataZ((double const*) value, numElements);
-    }
-    else if (property == COORDINATES)
-    {
-        setVertices((double const*) value, numElements);
-    }
-    else if (property == INDICES)
-    {
-        setIndices((unsigned int const*) value, numElements);
-    }
-    else if (property == VALUES)
-    {
-        setValues((double const*) value, numElements);
-    }
-    else
-    {
-        return Data3D::setDataProperty(property, value, numElements);
+        case NUM_VERTICES :
+            return setNumVertices(*((unsigned int const*) value));
+        case NUM_INDICES :
+            return setNumIndices(*((unsigned int const*) value));
+        case X_COORDINATES :
+            setDataX((double const*) value, numElements);
+            break;
+        case Y_COORDINATES :
+            setDataY((double const*) value, numElements);
+            break;
+        case Z_COORDINATES :
+            setDataZ((double const*) value, numElements);
+            break;
+        case COORDINATES :
+            setVertices((double const*) value, numElements);
+            break;
+        case INDICES :
+            setIndices((unsigned int const*) value, numElements);
+            break;
+        case VALUES :
+            setValues((double const*) value, numElements);
+            break;
+        default :
+            return Data3D::setDataProperty(property, value, numElements);
     }
 
     return 1;
@@ -128,29 +119,25 @@ int TriangleMeshData::setDataProperty(int property, void const* value, int numEl
 
 void TriangleMeshData::getDataProperty(int property, void **_pvData)
 {
-    if (property == NUM_VERTICES)
-    {
-        ((int *)*_pvData)[0] = getNumVertices();
-    }
-    else if (property == NUM_INDICES)
-    {
-        ((int *)*_pvData)[0] = getNumIndices();
-    }
-    else if (property == COORDINATES)
+    switch (property)
     {
-        *_pvData = getVertices();
-    }
-    else if (property == INDICES)
-    {
-        *_pvData = getIndices();
-    }
-    else if (property == VALUES)
-    {
-        *_pvData = getValues();
-    }
-    else
-    {
-        Data3D::getDataProperty(property, _pvData);
+        case NUM_VERTICES :
+            ((int *)*_pvData)[0] = getNumVertices();
+            break;
+        case NUM_INDICES :
+            ((int *)*_pvData)[0] = getNumIndices();
+            break;
+        case COORDINATES :
+            *_pvData = getVertices();
+            break;
+        case INDICES :
+            *_pvData = getIndices();
+            break;
+        case VALUES :
+            *_pvData = getValues();
+            break;
+        default :
+            Data3D::getDataProperty(property, _pvData);
     }
 }