Changeset 872 for cpp/frams/model/similarity/simil_model.cpp

Timestamp:

05/04/19 23:39:39 (6 years ago)

Author:

Maciej Komosinski

Message:

Fixed error messages, changed int to paInt (for 32/64-bit safety), changed two int variables to bool as they should be

File:

• cpp/frams/model/similarity/simil_model.cpp (modified) (13 diffs)

cpp/frams/model/similarity/simil_model.cpp

@@ -75 +75 @@
         //Determines whether "fuzzy vertex degree" should be used.
         //Currently "fuzzy vertex degree" is inactive.
-        isFuzzy = 0;
+        isFuzzy = false;
         fuzzyDepth = 10;
 
@@ -81 +81 @@
         wMDS = 0;
         //Determines whether best matching should be saved using hungarian similarity measure.
-        saveMatching = 0;
+        saveMatching = false;
 }
 
@@ -105 +105 @@
         {
                 logPrintf("ModelSimil", "EvaluateDistanceGreedy", LOG_ERROR, "NULL or invalid model pointer(s)");
-                        return 0.0;
+                return 0.0;
         }
 
@@ -139 +139 @@
         if ((this->*pfMatchingFunction)() == 0)
         {
-                logPrintf("ModelSimil", "EvaluateDistanceGreedy", LOG_ERROR, "Matching function error");
-                        return 0.0;
+                logPrintf("ModelSimil", "EvaluateDistanceGreedy", LOG_ERROR, "The matching function returned 0");
+                return 0.0;
         }
 
@@ -151 +151 @@
                 if (CountPartsDistance() == 0)
                 {
-                        logPrintf("ModelSimil", "EvaluateDistanceGreedy", LOG_ERROR, "CountPartDistance() error");
-                                return 0.0;
+                        logPrintf("ModelSimil", "EvaluateDistanceGreedy", LOG_ERROR, "CountPartDistance()==0");
+                        return 0.0;
                 }
 
@@ -562 +562 @@
                 }
 
-                if (m_aDegrees[i] != NULL && (isFuzzy != 1 || (m_Neighbours[i] != NULL && m_fuzzyNeighb[i] != NULL)))
+                if (m_aDegrees[i] != NULL && ((!isFuzzy) || (m_Neighbours[i] != NULL && m_fuzzyNeighb[i] != NULL)))
                 {
                         // wypelnij tablice zgodnie z sensem TDN[0] - orginalny index
@@ -600 +600 @@
                 else
                 {
-                        logPrintf("ModelSimil", "EvaluateDistanceGreedy", LOG_ERROR, "Memory error");
-                                return 0;
+                        logPrintf("ModelSimil", "CreatePartInfoTables", LOG_ERROR, "Not enough memory?");
+                        return 0;
                 }
                 // utworz tablice dla pozycji 3D Parts (wielkosc tablicy: liczba Parts organizmu)
@@ -931 +931 @@
         int i;
         int(*pfDegreeFunction) (const void*, const void*) = NULL;
-        pfDegreeFunction = (isFuzzy == 1) ? &CompareFuzzyDegrees : &CompareDegrees;
+        pfDegreeFunction = isFuzzy ? &CompareFuzzyDegrees : &CompareDegrees;
         // sortowanie obu tablic wg stopni punktów - TDN[1]
         for (i = 0; i < 2; i++)
@@ -943 +943 @@
         // sprawdzenie wartosci parametru
         DB(i = sizeof(TDN);)
-                int degreeType = (isFuzzy == 1) ? FUZZ_DEG : DEGREE;
+                int degreeType = isFuzzy ? FUZZ_DEG : DEGREE;
 
         // sortowanie obu tablic m_aDegrees wedlug liczby neuronów i
@@ -2079 +2079 @@
         {
                 logPrintf("ModelSimil", "EvaluateDistanceHungarian", LOG_ERROR, "NULL genotype pointer(s)");
-                        return 0.0;
+                return 0.0;
         }
         // create models of objects to compare
@@ -2089 +2089 @@
         {
                 logPrintf("ModelSimil", "EvaluateDistanceHungarian", LOG_ERROR, "NULL or invalid model pointer(s)");
-                        return 0.0;
+                return 0.0;
         }
 
@@ -2165 +2165 @@
                         {
                                 dMinSimValue = dCurrentSim;
-                                if (saveMatching == 1)
+                                if (saveMatching)
                                 {
                                         minAssignment.clear();
@@ -2174 +2174 @@
 
                 dResult = dMinSimValue;
-                if (saveMatching == 1)
+                if (saveMatching)
                         std::copy(minAssignment.begin(), minAssignment.end(), assignment);
         }