aboutsummaryrefslogtreecommitdiff
path: root/src/main/java/org/traccar
diff options
context:
space:
mode:
authorYuriy Piskarev <yuriy.piskarev@gmail.com>2022-03-23 15:46:37 +0300
committerYuriy Piskarev <yuriy.piskarev@gmail.com>2022-03-23 15:46:37 +0300
commit93e9d449d187bc3947ca76a1f3bdd4f02b37752c (patch)
treef280259892d45a15a56a4f48730ed0421f15c8b5 /src/main/java/org/traccar
parent2fcf8ea7ed086175658d05e536d4ce071cf17dda (diff)
downloadtrackermap-server-93e9d449d187bc3947ca76a1f3bdd4f02b37752c.tar.gz
trackermap-server-93e9d449d187bc3947ca76a1f3bdd4f02b37752c.tar.bz2
trackermap-server-93e9d449d187bc3947ca76a1f3bdd4f02b37752c.zip
- fixed warnings from checkstyle;
- rename tmp variable; - added use of BitUtil; - redesigned parameter checking;
Diffstat (limited to 'src/main/java/org/traccar')
-rw-r--r--src/main/java/org/traccar/protocol/NavtelecomProtocolDecoder.java148
1 files changed, 56 insertions, 92 deletions
diff --git a/src/main/java/org/traccar/protocol/NavtelecomProtocolDecoder.java b/src/main/java/org/traccar/protocol/NavtelecomProtocolDecoder.java
index 72a74a534..d4026353d 100644
--- a/src/main/java/org/traccar/protocol/NavtelecomProtocolDecoder.java
+++ b/src/main/java/org/traccar/protocol/NavtelecomProtocolDecoder.java
@@ -196,7 +196,7 @@ public class NavtelecomProtocolDecoder extends BaseProtocolDecoder {
for (int j = 0; j < bits.length(); j++) {
if (bits.get(j)) {
- int tmp = 0; // To parse a value by multiple conditions
+ int value = 0;
switch (j + 1) {
case 1:
@@ -209,9 +209,9 @@ public class NavtelecomProtocolDecoder extends BaseProtocolDecoder {
position.setDeviceTime(new Date(buf.readUnsignedIntLE() * 1000));
break;
case 8:
- tmp = buf.readUnsignedByte();
- position.setValid((tmp & 0b00000010) == 0b00000010);
- position.set(Position.KEY_SATELLITES, ((tmp & 0b11111100) >> 2));
+ value = buf.readUnsignedByte();
+ position.setValid(BitUtil.check(value, 1));
+ position.set(Position.KEY_SATELLITES, ((value & 0b11111100) >> 2));
break;
case 9:
position.setFixTime(new Date(buf.readUnsignedIntLE() * 1000));
@@ -232,154 +232,118 @@ public class NavtelecomProtocolDecoder extends BaseProtocolDecoder {
position.setCourse(buf.readUnsignedShortLE());
break;
case 15:
- position.set(Position.KEY_ODOMETER,buf.readFloatLE());
+ position.set(Position.KEY_ODOMETER, buf.readFloatLE());
break;
case 19:
- position.set(Position.KEY_POWER,buf.readShortLE() * 0.001);
+ position.set(Position.KEY_POWER, buf.readShortLE() * 0.001);
break;
case 20:
- position.set(Position.KEY_BATTERY,buf.readShortLE() * 0.001);
+ position.set(Position.KEY_BATTERY, buf.readShortLE() * 0.001);
break;
case 21:
- position.set(Position.PREFIX_ADC + 1,buf.readUnsignedShortLE() * 0.001);
+ position.set(Position.PREFIX_ADC + 1, buf.readUnsignedShortLE() * 0.001);
break;
case 22:
- position.set(Position.PREFIX_ADC + 2,buf.readUnsignedShortLE() * 0.001);
+ position.set(Position.PREFIX_ADC + 2, buf.readUnsignedShortLE() * 0.001);
break;
case 23:
- position.set(Position.PREFIX_ADC + 3,buf.readUnsignedShortLE() * 0.001);
+ position.set(Position.PREFIX_ADC + 3, buf.readUnsignedShortLE() * 0.001);
break;
case 24:
- position.set(Position.PREFIX_ADC + 4,buf.readUnsignedShortLE() * 0.001);
+ position.set(Position.PREFIX_ADC + 4, buf.readUnsignedShortLE() * 0.001);
break;
case 25:
- position.set(Position.PREFIX_ADC + 5,buf.readUnsignedShortLE() * 0.001);
+ position.set(Position.PREFIX_ADC + 5, buf.readUnsignedShortLE() * 0.001);
break;
case 26:
- position.set(Position.PREFIX_ADC + 6,buf.readUnsignedShortLE() * 0.001);
+ position.set(Position.PREFIX_ADC + 6, buf.readUnsignedShortLE() * 0.001);
break;
case 29:
- tmp = buf.readUnsignedByte();
- position.set(Position.PREFIX_IN + 1, (tmp & 0b00000001) == 0b00000001);
- position.set(Position.PREFIX_IN + 2, (tmp & 0b00000010) == 0b00000010);
- position.set(Position.PREFIX_IN + 3, (tmp & 0b00000100) == 0b00000100);
- position.set(Position.PREFIX_IN + 4, (tmp & 0b00001000) == 0b00001000);
- position.set(Position.PREFIX_IN + 5, (tmp & 0b00010000) == 0b00010000);
- position.set(Position.PREFIX_IN + 6, (tmp & 0b00100000) == 0b00100000);
- position.set(Position.PREFIX_IN + 7, (tmp & 0b01000000) == 0b01000000);
- position.set(Position.PREFIX_IN + 8, (tmp & 0b10000000) == 0b10000000);
+ value = buf.readUnsignedByte();
+ for (int k = 0; k <= 7; k++) {
+ position.set(Position.PREFIX_IN + (k + 1), BitUtil.check(value, k) ? 1 : 0);
+ }
break;
case 31:
- tmp = buf.readUnsignedByte();
- position.set(Position.PREFIX_OUT + 1, (tmp & 0b00000001) == 0b00000001);
- position.set(Position.PREFIX_OUT + 2, (tmp & 0b00000010) == 0b00000010);
- position.set(Position.PREFIX_OUT + 3, (tmp & 0b00000100) == 0b00000100);
- position.set(Position.PREFIX_OUT + 4, (tmp & 0b00001000) == 0b00001000);
+ value = buf.readUnsignedByte();
+ for (int k = 0; k <= 3; k++) {
+ position.set(Position.PREFIX_OUT + (k + 1), BitUtil.check(value, k) ? 1 : 0);
+ }
break;
case 33:
- position.set(Position.PREFIX_COUNT + 1,buf.readUnsignedIntLE());
+ position.set(Position.PREFIX_COUNT + 1, buf.readUnsignedIntLE());
break;
case 34:
- position.set(Position.PREFIX_COUNT + 2,buf.readUnsignedIntLE());
+ position.set(Position.PREFIX_COUNT + 2, buf.readUnsignedIntLE());
break;
case 35:
- position.set("freq" + 1,buf.readUnsignedShortLE());
+ position.set("freq" + 1, buf.readUnsignedShortLE());
break;
case 36:
- position.set("freq" + 2,buf.readUnsignedShortLE());
+ position.set("freq" + 2, buf.readUnsignedShortLE());
break;
case 37:
- position.set("engine_hours",buf.readUnsignedIntLE());
+ position.set("engine_hours", buf.readUnsignedIntLE());
break;
case 38:
- tmp = buf.readUnsignedShortLE();
- if (tmp < 65500) { // Do not write error codes
- position.set("rs485fuel_level" + 1,tmp);
- }
+ value = buf.readUnsignedShortLE();
+ position.set("rs485fuel_level" + 1, (value < 65500) ? value : null);
break;
case 39:
- tmp = buf.readUnsignedShortLE();
- if (tmp < 65500) { // Do not write error codes
- position.set("rs485fuel_level" + 2,tmp);
- }
+ value = buf.readUnsignedShortLE();
+ position.set("rs485fuel_level" + 2, (value < 65500) ? value : null);
break;
case 40:
- tmp = buf.readUnsignedShortLE();
- if (tmp < 65500) { // Do not write error codes
- position.set("rs485fuel_level" + 3,tmp);
- }
+ value = buf.readUnsignedShortLE();
+ position.set("rs485fuel_level" + 3, (value < 65500) ? value : null);
break;
case 41:
- tmp = buf.readUnsignedShortLE();
- if (tmp < 65500) { // Do not write error codes
- position.set("rs485fuel_level" + 4,tmp);
- }
+ value = buf.readUnsignedShortLE();
+ position.set("rs485fuel_level" + 4, (value < 65500) ? value : null);
break;
case 42:
- tmp = buf.readUnsignedShortLE();
- if (tmp < 65500) { // Do not write error codes
- position.set("rs485fuel_level" + 5,tmp);
- }
+ value = buf.readUnsignedShortLE();
+ position.set("rs485fuel_level" + 5, (value < 65500) ? value : null);
break;
case 43:
- tmp = buf.readUnsignedShortLE();
- if (tmp < 65500) { // Do not write error codes
- position.set("rs485fuel_level" + 6,tmp);
- }
+ value = buf.readUnsignedShortLE();
+ position.set("rs485fuel_level" + 6, (value < 65500) ? value : null);
break;
case 44:
- tmp = buf.readUnsignedShortLE();
- if (tmp < 65500) { // Do not write error codes
- position.set("rs232fuel_level",tmp);
- }
+ value = buf.readUnsignedShortLE();
+ position.set("rs232fuel_level", (value < 65500) ? value : null);
break;
case 45:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 1,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 1, (value != 0x80) ? value : null);
break;
case 46:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 2,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 2, (value != 0x80) ? value : null);
break;
case 47:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 3,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 3, (value != 0x80) ? value : null);
break;
case 48:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 4,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 4, (value != 0x80) ? value : null);
break;
case 49:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 5,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 5, (value != 0x80) ? value : null);
break;
case 50:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 6,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 6, (value != 0x80) ? value : null);
break;
case 51:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 7,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 7, (value != 0x80) ? value : null);
break;
case 52:
- tmp = buf.readByte();
- if (tmp != 0x80) { // Do not write error codes
- position.set(Position.PREFIX_TEMP + 8,tmp);
- }
+ value = buf.readByte();
+ position.set(Position.PREFIX_TEMP + 8, (value != 0x80) ? value : null);
break;
default:
buf.skipBytes(getItemLength(j + 1));