From 3b7148075815aa7f6cd650013949934a50a573f7 Mon Sep 17 00:00:00 2001
From: Anton Tananaev <anton.tananaev@gmail.com>
Date: Fri, 7 Apr 2017 23:03:46 +1200
Subject: Optimize minifinder decoder class

---
 .../protocol/MiniFinderProtocolDecoder.java        | 24 +++++++++-------------
 1 file changed, 10 insertions(+), 14 deletions(-)

(limited to 'src/org/traccar/protocol')

diff --git a/src/org/traccar/protocol/MiniFinderProtocolDecoder.java b/src/org/traccar/protocol/MiniFinderProtocolDecoder.java
index db01eaa9f..c7b0aecb2 100644
--- a/src/org/traccar/protocol/MiniFinderProtocolDecoder.java
+++ b/src/org/traccar/protocol/MiniFinderProtocolDecoder.java
@@ -142,18 +142,12 @@ public class MiniFinderProtocolDecoder extends BaseProtocolDecoder {
         String sentence = (String) msg;
 
         if (sentence.startsWith("!1,")) {
-
             getDeviceSession(channel, remoteAddress, sentence.substring(3, sentence.length()));
-
             return null;
         }
 
         DeviceSession deviceSession = getDeviceSession(channel, remoteAddress);
-        if (deviceSession == null) {
-            return null;
-        }
-
-        if (!sentence.matches("![A-D],.*")) {
+        if (deviceSession == null || !sentence.matches("![A-D],.*")) {
             return null;
         }
 
@@ -161,10 +155,11 @@ public class MiniFinderProtocolDecoder extends BaseProtocolDecoder {
         position.setProtocol(getProtocolName());
         position.setDeviceId(deviceSession.getDeviceId());
 
-        String recordType = sentence.substring(1, 2);
-        position.set(Position.KEY_TYPE, recordType);
+        String type = sentence.substring(1, 2);
+        position.set(Position.KEY_TYPE, type);
+
+        if (type.equals("B") || type.equals("D")) {
 
-        if (recordType.matches("[BD]")) {
             Parser parser = new Parser(PATTERN_BD, sentence);
             if (!parser.matches()) {
                 return null;
@@ -175,9 +170,9 @@ public class MiniFinderProtocolDecoder extends BaseProtocolDecoder {
             decodeGPSPrecision(position, parser);
 
             return position;
-        }
 
-        if (recordType.matches("C")) {
+        } else if (type.equals("C")) {
+
             Parser parser = new Parser(PATTERN_C, sentence);
             if (!parser.matches()) {
                 return null;
@@ -187,9 +182,9 @@ public class MiniFinderProtocolDecoder extends BaseProtocolDecoder {
             decodeState(position, parser);
 
             return position;
-        }
 
-        if (recordType.matches("A")) {
+        } else if (type.equals("A")) {
+
             Parser parser = new Parser(PATTERN_A, sentence);
             if (!parser.matches()) {
                 return null;
@@ -198,6 +193,7 @@ public class MiniFinderProtocolDecoder extends BaseProtocolDecoder {
             decodeFix(position, parser);
 
             return position;
+
         }
 
         return null;
-- 
cgit v1.2.3