在做性能测试的时候，发现一个MD5的工具类有问题，在大压力的情况下，会出现计算出错的问题，分析了一下代码，找到了原因所在。

这是原来的代码：

 

  1. public class MD5 { 
  2.  
  3.     // ------------------------------------------------------------ Private data 
  4.     private static SecureRandom random = null; 
  5.     private static MessageDigest md    = null; 
  6.  
  7.     // ------------------------------------------------------------ Constructors 
  8.  
  9.     /** Creates a new instance of MD5 */ 
  10.     protected MD5() { 
  11.     } 
  12.  
  13.     // ---------------------------------------------------------- Public methods 
  14.  
  15.     /** 
  16.      * Returns a new nonce to be used for MD5 authentication. The nonce bytes 
  17.      * are guaranteed to be in the printable range from ascii 32 to 128. 
  18.      * 
  19.      * @return a new 16 bytes long nonce 
  20.      */ 
  21.     public static byte[] getNextNonce() { 
  22.         byte[] nextNonce = new byte[16]; 
  23.         random.nextBytes(nextNonce); 
  24.  
  25.         int i; 
  26.         for (int j=0; j<nextNonce.length; ++j) { 
  27.             i = nextNonce[j] & 0x000000ff; 
  28.             if ((i<32) || (i>128)) { 
  29.                 nextNonce[j] = (byte)(32 + (i % 64)); 
  30.             } 
  31.         } 
  32.  
  33.         return nextNonce; 
  34.     } 
  35.  
  36.     /** 
  37.      * MD5ies the given content 
  38.      * 
  39.      * @param data the data to be digested 
  40.      * 
  41.      */ 
  42.     public static byte[] digest(byte[] data) { 
  43.         md.reset(); 
  44.         return md.digest(data); 
  45.     } 
  46.  
  47.     // --------------------------------------------------------- Private methods 
  48.  
  49.     /** 
  50.      * Creates and initialize the random generator. Called ad class loading 
  51.      * time. 
  52.      */ 
  53.     private static void randomGeneratorInit() 
  54.     throws java.security.NoSuchAlgorithmException { 
  55.         random = SecureRandom.getInstance("SHA1PRNG"); 
  56.     } 
  57.  
  58.     // ------------------------------------------------------------- Static code 
  59.  
  60.     static { 
  61.         try { 
  62.             randomGeneratorInit(); 
  63.             md = MessageDigest.getInstance("MD5"); 
  64.         } catch(Exception e) { 
  65.             e.printStackTrace(); 
  66.         } 
  67.     } 
  68. } 

  69. 这是我修改后的代码：
  70.  
  1. public class MD5 { 
  2.  
  3.     // ------------------------------------------------------------ Private data 
  4.     private static SecureRandom random = null; 
  5.     private static MessageDigest md    = null; 
  6.  
  7.     // ------------------------------------------------------------ Constructors 
  8.  
  9.     /** Creates a new instance of MD5 */ 
  10.     protected MD5() { 
  11.     } 
  12.  
  13.     // ---------------------------------------------------------- Public methods 
  14.  
  15.     /** 
  16.      * Returns a new nonce to be used for MD5 authentication. The nonce bytes 
  17.      * are guaranteed to be in the printable range from ascii 32 to 128. 
  18.      * 
  19.      * @return a new 16 bytes long nonce 
  20.      */ 
  21.     public static byte[] getNextNonce() { 
  22.         byte[] nextNonce = new byte[16]; 
  23.         random.nextBytes(nextNonce); 
  24.  
  25.         int i; 
  26.         for (int j=0; j<nextNonce.length; ++j) { 
  27.             i = nextNonce[j] & 0x000000ff; 
  28.             if ((i<32) || (i>128)) { 
  29.                 nextNonce[j] = (byte)(32 + (i % 64)); 
  30.             } 
  31.         } 
  32.  
  33.         return nextNonce; 
  34.     } 
  35.  
  36.     /** 
  37.      * MD5ies the given content 
  38.      * 
  39.      * @param data the data to be digested 
  40.      * 
  41.      */ 
  42.     public static byte[] digest(byte[] data) { 
  43.         synchronized (md) { 
  44.             md.reset(); 
  45.             return md.digest(data); 
  46.         } 
  47.  
  48.     } 
  49.  
  50.     // --------------------------------------------------------- Private methods 
  51.  
  52.     /** 
  53.      * Creates and initialize the random generator. Called ad class loading 
  54.      * time. 
  55.      */ 
  56.     private static void randomGeneratorInit() 
  57.     throws java.security.NoSuchAlgorithmException { 
  58.         random = SecureRandom.getInstance("SHA1PRNG"); 
  59.     } 
  60.  
  61.     // ------------------------------------------------------------- Static code 
  62.  
  63.     static { 
  64.         try { 
  65.             randomGeneratorInit(); 
  66.             md = MessageDigest.getInstance("MD5"); 
  67.         } catch(Exception e) { 
  68.             e.printStackTrace(); 
  69.         } 
  70.     } 
  71. } 
主要区别在这里：
  1.         synchronized (md) { 
  2.             md.reset(); 
  3.             return md.digest(data); 
  4.         }