Refactoring work.

git-svn-id: http://svn.igniterealtime.org/svn/repos/messenger/trunk@584 b35dd754-fafc-0310-a699-88a17e54d16e

src/java/org/jivesoftware/database/DbConnectionManager.java

@@ -160,6 +160,26 @@ public class DbConnectionManager {
 
     }
 
+    /**
+     * Close the prepared statement and connection
+     * @param pstmt the <code>PreparedStatement</code> to close.
+     * @param con the <code>Connection</code> to close.
+     */
+    public static void close(PreparedStatement pstmt, Connection con) {
+        try {
+            if (pstmt != null) pstmt.close();
+        }
+        catch (Exception e) {
+            Log.error(e);
+        }
+        try {
+            if (con != null) con.close();
+        }
+        catch (Exception e) {
+            Log.error(e);
+        }
+    }
+
     /**
      * Creates a scroll insensitive Statement if the JDBC driver supports it, or a normal
      * Statement otherwise.

src/java/org/jivesoftware/messenger/user/CachedRoster.java

@@ -11,7 +11,6 @@
 
 package org.jivesoftware.messenger.user;
 
-import org.jivesoftware.messenger.Presence;
 import org.jivesoftware.messenger.auth.UnauthorizedException;
 import org.jivesoftware.util.Cacheable;
 import org.xmpp.packet.Presence;

src/java/org/jivesoftware/messenger/user/spi/CachedRosterItemImpl.java

@@ -11,14 +11,12 @@
 
 package org.jivesoftware.messenger.user.spi;
 
-import org.jivesoftware.util.CacheSizes;
-import org.jivesoftware.messenger.XMPPAddress;
+import java.util.List;
 import org.jivesoftware.messenger.user.BasicRosterItem;
 import org.jivesoftware.messenger.user.CachedRosterItem;
 import org.jivesoftware.messenger.user.RosterItem;
 import org.jivesoftware.util.CacheSizes;
-
-import java.util.List;
+import org.xmpp.packet.JID;
 
 /**
  * In-memory implementation of a roster item. The ID of the roster item is it's roster ID.
@@ -28,7 +26,7 @@ import java.util.List;
 public class CachedRosterItemImpl extends BasicRosterItem implements CachedRosterItem {
 
     public CachedRosterItemImpl(long id,
-                                XMPPAddress jid,
+                                JID jid,
                                 SubType subStatus,
                                 AskType askStatus,
                                 RecvType recvStatus,
@@ -38,7 +36,7 @@ public class CachedRosterItemImpl extends BasicRosterItem implements CachedRoste
         this.rosterID = id;
     }
 
-    public CachedRosterItemImpl(long id, XMPPAddress jid) {
+    public CachedRosterItemImpl(long id, JID jid) {
         this(id,
                 jid,
                 RosterItem.SUB_NONE,
@@ -48,7 +46,7 @@ public class CachedRosterItemImpl extends BasicRosterItem implements CachedRoste
                 null);
     }
 
-    public CachedRosterItemImpl(long id, XMPPAddress jid, String nickname, List groups) {
+    public CachedRosterItemImpl(long id, JID jid, String nickname, List groups) {
         this(id,
                 jid,
                 RosterItem.SUB_NONE,
@@ -90,7 +88,7 @@ public class CachedRosterItemImpl extends BasicRosterItem implements CachedRoste
     }
 
     public int getCachedSize() {
-        int size = jid.getCachedSize();
+        int size = jid.toBareJID().length();
         size += CacheSizes.sizeOfString(nickname);
         size += CacheSizes.sizeOfList(groups);
         size += CacheSizes.sizeOfInt(); // subStatus

src/java/org/jivesoftware/messenger/user/spi/DbRosterItemProvider.java

@@ -11,22 +11,25 @@
 
 package org.jivesoftware.messenger.user.spi;
 
-import org.jivesoftware.database.DbConnectionManager;
-import org.jivesoftware.database.SequenceManager;
-import org.jivesoftware.util.JiveConstants;
-import org.jivesoftware.util.LocaleUtils;
-import org.jivesoftware.util.Log;
-import org.jivesoftware.messenger.XMPPAddress;
-import org.jivesoftware.messenger.user.*;
-
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.util.ArrayList;
 import java.util.Iterator;
 import java.util.LinkedList;
-import java.util.ArrayList;
 import java.util.List;
+import org.jivesoftware.database.DbConnectionManager;
+import org.jivesoftware.database.SequenceManager;
+import org.jivesoftware.messenger.user.CachedRosterItem;
+import org.jivesoftware.messenger.user.RosterItem;
+import org.jivesoftware.messenger.user.RosterItemProvider;
+import org.jivesoftware.messenger.user.UserAlreadyExistsException;
+import org.jivesoftware.messenger.user.UserNotFoundException;
+import org.jivesoftware.util.JiveConstants;
+import org.jivesoftware.util.LocaleUtils;
+import org.jivesoftware.util.Log;
+import org.xmpp.packet.JID;
 
 /**
  * <p>Implements the roster item provider against the jiveRoster table
@@ -52,7 +55,7 @@ public class DbRosterItemProvider implements RosterItemProvider {
             pstmt = con.prepareStatement(CREATE_ROSTER_ITEM);
             pstmt.setString(1, username);
             pstmt.setLong(2, rosterID);
-            pstmt.setString(3, item.getJid().toBareString());
+            pstmt.setString(3, item.getJid().toBareJID());
             pstmt.setInt(4, item.getSubStatus().getValue());
             pstmt.setInt(5, item.getAskStatus().getValue());
             pstmt.setInt(6, item.getRecvStatus().getValue());
@@ -71,13 +74,10 @@ public class DbRosterItemProvider implements RosterItemProvider {
             insertGroups(rosterID, item.getGroups().iterator(), pstmt, con);
         }
         catch (SQLException e) {
-            throw new UserAlreadyExistsException(item.getJid().toStringPrep());
+            throw new UserAlreadyExistsException(item.getJid().toBareJID());
         }
         finally {
-            try { if (pstmt != null) { pstmt.close(); } }
-            catch (Exception e) { Log.error(e); }
-            try { if (con != null) { con.close(); } }
-            catch (Exception e) { Log.error(e); }
+            DbConnectionManager.close(pstmt, con);
         }
         return cachedItem;
     }
@@ -115,10 +115,7 @@ public class DbRosterItemProvider implements RosterItemProvider {
             Log.error(LocaleUtils.getLocalizedString("admin.error"), e);
         }
         finally {
-            try { if (pstmt != null) { pstmt.close(); } }
-            catch (Exception e) { Log.error(e); }
-            try { if (con != null) { con.close(); } }
-            catch (Exception e) { Log.error(e); }
+            DbConnectionManager.close(pstmt, con);
         }
     }
 
@@ -183,10 +180,22 @@ public class DbRosterItemProvider implements RosterItemProvider {
             Log.error(LocaleUtils.getLocalizedString("admin.error"), e);
         }
         finally {
-            try { if (pstmt != null) { pstmt.close(); } }
-            catch (Exception e) { Log.error(e); }
-            try { if (con != null) { con.close(); } }
-            catch (Exception e) { Log.error(e); }
+            try {
+                if (pstmt != null) {
+                    pstmt.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
+            try {
+                if (con != null) {
+                    con.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
         }
     }
 
@@ -210,10 +219,22 @@ public class DbRosterItemProvider implements RosterItemProvider {
             Log.error(LocaleUtils.getLocalizedString("admin.error"), e);
         }
         finally {
-            try { if (pstmt != null) { pstmt.close(); } }
-            catch (Exception e) { Log.error(e); }
-            try { if (con != null) { con.close(); } }
-            catch (Exception e) { Log.error(e); }
+            try {
+                if (pstmt != null) {
+                    pstmt.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
+            try {
+                if (con != null) {
+                    con.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
         }
         return answer.iterator();
     }
@@ -238,10 +259,22 @@ public class DbRosterItemProvider implements RosterItemProvider {
             Log.error(LocaleUtils.getLocalizedString("admin.error"), e);
         }
         finally {
-            try { if (pstmt != null) { pstmt.close(); } }
-            catch (Exception e) { Log.error(e); }
-            try { if (con != null) { con.close(); } }
-            catch (Exception e) { Log.error(e); }
+            try {
+                if (pstmt != null) {
+                    pstmt.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
+            try {
+                if (con != null) {
+                    con.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
         }
         return count;
     }
@@ -262,7 +295,7 @@ public class DbRosterItemProvider implements RosterItemProvider {
             ResultSet rs = pstmt.executeQuery();
             while (rs.next()) {
                 CachedRosterItem item = new CachedRosterItemImpl(rs.getLong(2),
-                        XMPPAddress.parseJID(rs.getString(1)),
+                        new JID(rs.getString(1)),
                         RosterItem.SubType.getTypeFromInt(rs.getInt(3)),
                         RosterItem.AskType.getTypeFromInt(rs.getInt(4)),
                         RosterItem.RecvType.getTypeFromInt(rs.getInt(5)),
@@ -280,10 +313,22 @@ public class DbRosterItemProvider implements RosterItemProvider {
                     itemList.add(item);
                 }
                 finally {
-                    try {if (gs != null) { gs.close(); } }
-                    catch (Exception e) { Log.error(e); }
-                    try {if (gstmt != null) { gstmt.close(); } }
-                    catch (Exception e) { Log.error(e); }
+                    try {
+                        if (gs != null) {
+                            gs.close();
+                        }
+                    }
+                    catch (Exception e) {
+                        Log.error(e);
+                    }
+                    try {
+                        if (gstmt != null) {
+                            gstmt.close();
+                        }
+                    }
+                    catch (Exception e) {
+                        Log.error(e);
+                    }
                 }
             }
         }
@@ -291,10 +336,22 @@ public class DbRosterItemProvider implements RosterItemProvider {
             Log.error(LocaleUtils.getLocalizedString("admin.error"), e);
         }
         finally {
-            try { if (pstmt != null) { pstmt.close(); } }
-            catch (Exception e) { Log.error(e); }
-            try { if (con != null) { con.close(); } }
-            catch (Exception e) { Log.error(e); }
+            try {
+                if (pstmt != null) {
+                    pstmt.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
+            try {
+                if (con != null) {
+                    con.close();
+                }
+            }
+            catch (Exception e) {
+                Log.error(e);
+            }
         }
         return itemList.iterator();
     }