refactoring
This commit is contained in:
parent
92a7083b30
commit
2fd1a1d0f6
Binary file not shown.
|
@ -17,7 +17,6 @@ import static com.rusefi.ConfigFieldImpl.BOOLEAN_T;
|
||||||
*/
|
*/
|
||||||
public class ConfigStructureImpl implements ConfigStructure {
|
public class ConfigStructureImpl implements ConfigStructure {
|
||||||
public static final String ALIGNMENT_FILL_AT = "alignmentFill_at_";
|
public static final String ALIGNMENT_FILL_AT = "alignmentFill_at_";
|
||||||
public static final String UNUSED_ANYTHING_PREFIX = "unused";
|
|
||||||
public static final String UNUSED_BIT_PREFIX = "unusedBit_";
|
public static final String UNUSED_BIT_PREFIX = "unusedBit_";
|
||||||
|
|
||||||
private final String name;
|
private final String name;
|
||||||
|
|
|
@ -23,7 +23,7 @@ public class DataLogConsumer implements ConfigurationConsumer {
|
||||||
// https://github.com/rusefi/web_backend/issues/166
|
// https://github.com/rusefi/web_backend/issues/166
|
||||||
private static final int MSQ_LENGTH_LIMIT = 34;
|
private static final int MSQ_LENGTH_LIMIT = 34;
|
||||||
|
|
||||||
public static final String UNUSED = ConfigStructureImpl.UNUSED_ANYTHING_PREFIX;
|
public static final String UNUSED = ConfigStructure.UNUSED_ANYTHING_PREFIX;
|
||||||
private final String fileName;
|
private final String fileName;
|
||||||
private final StringBuilder tsWriter = new StringBuilder();
|
private final StringBuilder tsWriter = new StringBuilder();
|
||||||
private final TreeSet<String> comments = new TreeSet<>(String.CASE_INSENSITIVE_ORDER);
|
private final TreeSet<String> comments = new TreeSet<>(String.CASE_INSENSITIVE_ORDER);
|
||||||
|
|
|
@ -1,7 +1,6 @@
|
||||||
package com.rusefi.output;
|
package com.rusefi.output;
|
||||||
|
|
||||||
import com.rusefi.ConfigField;
|
import com.rusefi.ConfigField;
|
||||||
import com.rusefi.ConfigFieldImpl;
|
|
||||||
import com.rusefi.ReaderState;
|
import com.rusefi.ReaderState;
|
||||||
|
|
||||||
import java.io.IOException;
|
import java.io.IOException;
|
||||||
|
@ -26,7 +25,7 @@ public class SdCardFieldsContent {
|
||||||
private String processOutput(ConfigField configField, String prefix) {
|
private String processOutput(ConfigField configField, String prefix) {
|
||||||
if (configField.getName().startsWith(ConfigStructureImpl.ALIGNMENT_FILL_AT))
|
if (configField.getName().startsWith(ConfigStructureImpl.ALIGNMENT_FILL_AT))
|
||||||
return "";
|
return "";
|
||||||
if (configField.getName().startsWith(ConfigStructureImpl.UNUSED_ANYTHING_PREFIX))
|
if (configField.getName().startsWith(ConfigStructure.UNUSED_ANYTHING_PREFIX))
|
||||||
return "";
|
return "";
|
||||||
if (configField.isBit())
|
if (configField.isBit())
|
||||||
return "";
|
return "";
|
||||||
|
|
|
@ -51,7 +51,7 @@ public class TsOutput {
|
||||||
if (!usedNames.add(nameWithPrefix)
|
if (!usedNames.add(nameWithPrefix)
|
||||||
&& !isConstantsSection
|
&& !isConstantsSection
|
||||||
&& !configField.getName().startsWith(ConfigStructureImpl.ALIGNMENT_FILL_AT)
|
&& !configField.getName().startsWith(ConfigStructureImpl.ALIGNMENT_FILL_AT)
|
||||||
&& !configField.getName().startsWith(ConfigStructureImpl.UNUSED_ANYTHING_PREFIX)) {
|
&& !configField.getName().startsWith(ConfigStructure.UNUSED_ANYTHING_PREFIX)) {
|
||||||
throw new IllegalStateException(nameWithPrefix + " already present: " + configField);
|
throw new IllegalStateException(nameWithPrefix + " already present: " + configField);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -4,6 +4,158 @@ import com.rusefi.core.Pair;
|
||||||
import com.rusefi.output.ConfigStructure;
|
import com.rusefi.output.ConfigStructure;
|
||||||
|
|
||||||
public interface ConfigField {
|
public interface ConfigField {
|
||||||
|
ConfigField VOID = new ConfigField() {
|
||||||
|
@Override
|
||||||
|
public ConfigStructure getStructureType() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isArray() {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getArraySizeVariableName() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getTrueName() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getFalseName() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isBit() {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isDirective() {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int getSize(ConfigField next) {
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int[] getArraySizes() {
|
||||||
|
return new int[0];
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getComment() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getName() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getType() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int getElementSize() {
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isIterate() {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isHasAutoscale() {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public ReaderState getState() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getTsInfo() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isFsioVisible() {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String autoscaleSpec() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public double autoscaleSpecNumber() {
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Pair<Integer, Integer> autoscaleSpecPair() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String[] getTokens() {
|
||||||
|
return new String[0];
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getUnits() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public double getMin() {
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public double getMax() {
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int getDigits() {
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getIterateOriginalName() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public int getIterateIndex() {
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public boolean isFromIterate() {
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public String getCommentTemplated() {
|
||||||
|
return null;
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
ConfigStructure getStructureType();
|
ConfigStructure getStructureType();
|
||||||
|
|
||||||
boolean isArray();
|
boolean isArray();
|
||||||
|
|
|
@ -5,6 +5,8 @@ import com.rusefi.ConfigField;
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
|
|
||||||
public interface ConfigStructure {
|
public interface ConfigStructure {
|
||||||
|
String UNUSED_ANYTHING_PREFIX = "unused";
|
||||||
|
|
||||||
String getName();
|
String getName();
|
||||||
|
|
||||||
int getTotalSize();
|
int getTotalSize();
|
||||||
|
|
|
@ -2,7 +2,6 @@ package com.rusefi.output;
|
||||||
|
|
||||||
import com.rusefi.BitState;
|
import com.rusefi.BitState;
|
||||||
import com.rusefi.ConfigField;
|
import com.rusefi.ConfigField;
|
||||||
import com.rusefi.ConfigFieldImpl;
|
|
||||||
|
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
|
|
||||||
|
@ -13,7 +12,7 @@ import java.util.List;
|
||||||
public class FieldIterator {
|
public class FieldIterator {
|
||||||
private final List<ConfigField> fields;
|
private final List<ConfigField> fields;
|
||||||
BitState bitState = new BitState();
|
BitState bitState = new BitState();
|
||||||
private ConfigField prev = ConfigFieldImpl.VOID;
|
private ConfigField prev = ConfigField.VOID;
|
||||||
ConfigField next;
|
ConfigField next;
|
||||||
ConfigField cf;
|
ConfigField cf;
|
||||||
|
|
||||||
|
@ -33,7 +32,7 @@ public class FieldIterator {
|
||||||
while (nextIndex < fields.size() && fields.get(nextIndex).isDirective())
|
while (nextIndex < fields.size() && fields.get(nextIndex).isDirective())
|
||||||
nextIndex++;
|
nextIndex++;
|
||||||
|
|
||||||
next = nextIndex >= fields.size() ? ConfigFieldImpl.VOID : fields.get(nextIndex);
|
next = nextIndex >= fields.size() ? ConfigField.VOID : fields.get(nextIndex);
|
||||||
cf = fields.get(index);
|
cf = fields.get(index);
|
||||||
}
|
}
|
||||||
|
|
|
@ -1,7 +1,6 @@
|
||||||
package com.rusefi.output;
|
package com.rusefi.output;
|
||||||
|
|
||||||
import com.rusefi.ConfigField;
|
import com.rusefi.ConfigField;
|
||||||
import com.rusefi.ConfigFieldImpl;
|
|
||||||
import com.rusefi.ReaderState;
|
import com.rusefi.ReaderState;
|
||||||
|
|
||||||
import java.util.List;
|
import java.util.List;
|
|
@ -17,7 +17,7 @@ public abstract class JavaFieldsConsumer implements ConfigurationConsumer {
|
||||||
protected final ReaderState state;
|
protected final ReaderState state;
|
||||||
private final int baseOffset;
|
private final int baseOffset;
|
||||||
|
|
||||||
public JavaFieldsConsumer(ReaderStateImpl state, int baseOffset) {
|
public JavaFieldsConsumer(ReaderState state, int baseOffset) {
|
||||||
this.state = state;
|
this.state = state;
|
||||||
this.baseOffset = baseOffset;
|
this.baseOffset = baseOffset;
|
||||||
}
|
}
|
||||||
|
@ -74,7 +74,7 @@ public abstract class JavaFieldsConsumer implements ConfigurationConsumer {
|
||||||
String nameWithPrefix = prefix + configField.getName();
|
String nameWithPrefix = prefix + configField.getName();
|
||||||
|
|
||||||
if (configField.isBit()) {
|
if (configField.isBit()) {
|
||||||
if (!configField.getName().startsWith(DataLogConsumer.UNUSED)) {
|
if (!configField.getName().startsWith(ConfigStructure.UNUSED_ANYTHING_PREFIX)) {
|
||||||
writeJavaFieldName(nameWithPrefix, tsPosition);
|
writeJavaFieldName(nameWithPrefix, tsPosition);
|
||||||
content.append("FieldType.BIT, " + bitIndex + ")" + terminateField());
|
content.append("FieldType.BIT, " + bitIndex + ")" + terminateField());
|
||||||
}
|
}
|
Loading…
Reference in New Issue