github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-12-25 11:11:16 +00:00
Code Issues Projects Releases Packages Wiki Activity

sonar cleanup in Sessionhandler

Browse source
This commit is contained in:
Ingmar Goudt 2019-03-10 22:04:27 +01:00
parent dab481ee17
commit 7a5920322d
1 changed files with 13 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
Mage.Client/src/main/java/mage/client/SessionHandler.java
View file

@ -14,6 +14,7 @@ import mage.remote.MageRemoteException;
import mage.remote.Session; import mage.remote.Session;
import mage.remote.SessionImpl; import mage.remote.SessionImpl;
import mage.view.*; import mage.view.*;
import org.apache.log4j.Logger;
import java.util.*; import java.util.*;
@ -22,9 +23,15 @@ import java.util.*;
*/ */
public final class SessionHandler { public final class SessionHandler {
private static final Logger logger = Logger.getLogger(SessionHandler.class);
private static Session session; private static Session session;
private static String lastConnectError = ""; private static String lastConnectError = "";
private SessionHandler(){
}
public static void startSession(MageFrame mageFrame) { public static void startSession(MageFrame mageFrame) {
session = new SessionImpl(mageFrame); session = new SessionImpl(mageFrame);
@ -181,7 +188,7 @@ public final class SessionHandler {
try { try {
return session.getTournament(tournamentId); return session.getTournament(tournamentId);
} catch (MageRemoteException e) { } catch (MageRemoteException e) {
e.printStackTrace(); logger.info(e);
return null; return null;
} }
@ -275,7 +282,7 @@ public final class SessionHandler {
try { try {
return session.getRoomUsers(roomId); return session.getRoomUsers(roomId);
} catch (MageRemoteException e) { } catch (MageRemoteException e) {
e.printStackTrace(); logger.info(e);
return Collections.emptyList(); return Collections.emptyList();
} }
} }
@ -284,8 +291,8 @@ public final class SessionHandler {
try { try {
return session.getFinishedMatches(roomId); return session.getFinishedMatches(roomId);
} catch (MageRemoteException e) { } catch (MageRemoteException e) {
e.printStackTrace(); logger.info(e);
return new ArrayList<>(); return Collections.emptyList();
} }
} }
@ -301,8 +308,8 @@ public final class SessionHandler {
try { try {
return session.getTables(roomId); return session.getTables(roomId);
} catch (MageRemoteException e) { } catch (MageRemoteException e) {
e.printStackTrace(); logger.info(e);
return new ArrayList<>(); return Collections.emptyList();
} }
} }