Просмотр исходного кода

Do not guard code that can not throw the caught exception

Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Daniel Calviño Sánchez 2 лет назад
Родитель
Сommit
ac4be52b84
1 измененных файлов с 30 добавлено и 27 удалено
  1. 30 27
      app/src/main/java/com/nextcloud/talk/webrtc/PeerConnectionWrapper.java

+ 30 - 27
app/src/main/java/com/nextcloud/talk/webrtc/PeerConnectionWrapper.java

@@ -320,37 +320,40 @@ public class PeerConnectionWrapper {
             String strData = new String(bytes);
             Log.d(TAG, "Got msg: " + strData + " over " + TAG + " " + sessionId);
 
+            DataChannelMessage dataChannelMessage;
             try {
-                DataChannelMessage dataChannelMessage = LoganSquare.parse(strData, DataChannelMessage.class);
-
-                if ("nickChanged".equals(dataChannelMessage.getType())) {
-                    String nick = null;
-                    if (dataChannelMessage.getPayload() instanceof String) {
-                        nick = (String) dataChannelMessage.getPayload();
-                    } else if (dataChannelMessage.getPayload() instanceof Map) {
-                        Map<String, String> payloadMap = (Map<String, String>) dataChannelMessage.getPayload();
-                        nick = payloadMap.get("name");
-                    }
+                dataChannelMessage = LoganSquare.parse(strData, DataChannelMessage.class);
+            } catch (IOException e) {
+                Log.d(TAG, "Failed to parse data channel message");
 
-                    if (nick != null) {
-                        EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
-                                .NICK_CHANGE, sessionId, nick, null, videoStreamType));
-                    }
-                } else if ("audioOn".equals(dataChannelMessage.getType())) {
-                    EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
-                            .AUDIO_CHANGE, sessionId, null, TRUE, videoStreamType));
-                } else if ("audioOff".equals(dataChannelMessage.getType())) {
-                    EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
-                            .AUDIO_CHANGE, sessionId, null, FALSE, videoStreamType));
-                } else if ("videoOn".equals(dataChannelMessage.getType())) {
-                    EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
-                            .VIDEO_CHANGE, sessionId, null, TRUE, videoStreamType));
-                } else if ("videoOff".equals(dataChannelMessage.getType())) {
+                return;
+            }
+
+            if ("nickChanged".equals(dataChannelMessage.getType())) {
+                String nick = null;
+                if (dataChannelMessage.getPayload() instanceof String) {
+                    nick = (String) dataChannelMessage.getPayload();
+                } else if (dataChannelMessage.getPayload() instanceof Map) {
+                    Map<String, String> payloadMap = (Map<String, String>) dataChannelMessage.getPayload();
+                    nick = payloadMap.get("name");
+                }
+
+                if (nick != null) {
                     EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
-                            .VIDEO_CHANGE, sessionId, null, FALSE, videoStreamType));
+                            .NICK_CHANGE, sessionId, nick, null, videoStreamType));
                 }
-            } catch (IOException e) {
-                Log.d(TAG, "Failed to parse data channel message");
+            } else if ("audioOn".equals(dataChannelMessage.getType())) {
+                EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
+                        .AUDIO_CHANGE, sessionId, null, TRUE, videoStreamType));
+            } else if ("audioOff".equals(dataChannelMessage.getType())) {
+                EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
+                        .AUDIO_CHANGE, sessionId, null, FALSE, videoStreamType));
+            } else if ("videoOn".equals(dataChannelMessage.getType())) {
+                EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
+                        .VIDEO_CHANGE, sessionId, null, TRUE, videoStreamType));
+            } else if ("videoOff".equals(dataChannelMessage.getType())) {
+                EventBus.getDefault().post(new PeerConnectionEvent(PeerConnectionEvent.PeerConnectionEventType
+                        .VIDEO_CHANGE, sessionId, null, FALSE, videoStreamType));
             }
         }
     }