less crazy if elses

jasp-stats · Nov 28, 2024 · 0291089 · 0291089
1 parent 52eaf18
commit 0291089
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/CommonData/column.cpp b/CommonData/column.cpp
@@ -2381,12 +2381,13 @@ stringvec Column::previewTransform(columnType transformType)
 			for(Label * label : _labels)
 				if(!label->isEmptyValue() && !ColumnUtils::isDoubleValue(label->originalValueAsString()))
 				{
-					if(count < showThisMany)
+					if(count++ < showThisMany)
 						someImplicitEmptyValues << (count++ > 0 ? ", " : "") << '"' << label->originalValueAsString() << '"';
-					else if(count++ == showThisMany)
-						someImplicitEmptyValues << ", ...";
 					else
+					{
+						someImplicitEmptyValues << ", ...";
 						break; // Do not need to loop further over the labels.
+					}
 				}	
 		}