Skip to content

Commit 63bbf63

Browse files
committed
fix: summarize Float and integer to number
1 parent 703fd78 commit 63bbf63

6 files changed

Lines changed: 22 additions & 20 deletions

File tree

src/main/java/de/vill/model/Attribute.java

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -67,18 +67,16 @@ public String getName() {
6767
* @return Name of the attribute (never null)
6868
*/
6969
public String getType() {
70-
if (value instanceof Integer) {
71-
return Constants.INTEGER;
72-
} else if (value instanceof Boolean) {
70+
if (value instanceof Boolean) {
7371
return Constants.BOOLEAN;
7472
} else if (value instanceof String) {
7573
return Constants.STRING;
7674
} else if (value instanceof AttributeList) {
7775
return Constants.ATTRIBUTE_LIST;
78-
} else if (value instanceof Float) {
79-
return Constants.FLOAT;
80-
}else {
81-
return Constants.UNDEF;
76+
} else if (value instanceof Number) {
77+
return Constants.NUMBER;
78+
} else {
79+
return Constants.UNDEF;
8280
}
8381
}
8482

src/main/java/de/vill/model/GlobalAttribute.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,7 @@
33
import de.vill.model.building.VariableReference;
44
import de.vill.util.Constants;
55

6-
import java.util.ArrayList;
76
import java.util.HashSet;
8-
import java.util.List;
97
import java.util.Set;
108

119
/**
@@ -14,10 +12,10 @@
1412
public class GlobalAttribute implements VariableReference {
1513

1614
public enum AttributeType {
15+
BOOLEAN(Constants.BOOLEAN),
1716
STRING(Constants.STRING),
18-
NUMBER(Constants.NUMBER),
19-
BOOLEAN(Constants.BOOLEAN);
20-
17+
ATTRIBUTE_LIST(Constants.ATTRIBUTE_LIST),
18+
NUMBER(Constants.NUMBER);
2119
private final String name;
2220

2321
AttributeType(String name) {

src/main/java/de/vill/model/expression/AggregateFunctionExpression.java

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
import de.vill.model.Feature;
66
import de.vill.model.GlobalAttribute;
77
import de.vill.model.building.VariableReference;
8-
import de.vill.util.Constants;
98
import java.util.Arrays;
109
import java.util.List;
1110
import java.util.Objects;
@@ -30,11 +29,7 @@ public String toString(boolean withSubmodels, String currentAlias) {
3029
return toString(withSubmodels, "aggregateFunction", currentAlias);
3130
}
3231

33-
@Override
34-
public String getReturnType() {
35-
// implement in children
36-
return Constants.NUMBER;
37-
}
32+
abstract public String getReturnType();
3833

3934
@Override
4035
public List<Expression> getExpressionSubParts() {

src/main/java/de/vill/model/expression/MaxAggregateFunctionExpression.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import de.vill.model.Attribute;
44
import de.vill.model.Feature;
55
import de.vill.model.GlobalAttribute;
6+
import de.vill.util.Constants;
67

78

89
import java.util.Arrays;
@@ -41,6 +42,11 @@ public String toString(boolean withSubmodels, String currentAlias) {
4142
return super.toString(withSubmodels, "max", currentAlias);
4243
}
4344

45+
@Override
46+
public String getReturnType() {
47+
return Constants.NUMBER;
48+
}
49+
4450
@Override
4551
public List<Expression> getExpressionSubParts() {
4652
return Arrays.asList();

src/main/java/de/vill/model/expression/MinAggregateFunctionExpression.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
import de.vill.model.Attribute;
44
import de.vill.model.Feature;
55
import de.vill.model.GlobalAttribute;
6+
import de.vill.util.Constants;
67

78

89
import java.util.Arrays;
@@ -42,6 +43,11 @@ public String toString(boolean withSubmodels, String currentAlias) {
4243
return super.toString(withSubmodels, "min", currentAlias);
4344
}
4445

46+
@Override
47+
public String getReturnType() {
48+
return Constants.NUMBER;
49+
}
50+
4551
@Override
4652
public List<Expression> getExpressionSubParts() {
4753
return Arrays.asList();

src/main/java/de/vill/util/Constants.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@
22

33
public class Constants {
44
// Literal Types
5-
public static final String INTEGER = "integer";
6-
public static final String FLOAT = "float";
75
public static final String ATTRIBUTE_LIST = "attribute_list";
86
public static final String STRING = "string";
97
public static final String BOOLEAN = "boolean";
108
public static final String TRUE = "true";
119
public static final String UNDEF = "undef";
10+
public static final String NUMBER = "number";
1211

1312
// Default attributes
1413
public static final String TYPE_LEVEL_VALUE = "type_level_value";

0 commit comments

Comments
 (0)