जावा कैश्ड वस्तु पुराने मूल्य

Question

मैं सूची कैश्ड है लौटने के लिए, मेरे कोड इस

public class MyList { 
private final List<String> cache = new ArrayList<String>(); 

private List<String> loadMyList() { 
    // HEAVY OPERATION TO LOAD DATA 
} 

public List<String> list() { 
    synchronized (cache) { 
     if(cache.size() == 0) { 
      cache.addAll(loadMyList()); 
     } 
     return Collections.unmodifiableList(cache); 
    } 
} 

public void invalidateCache() { 
    synchronized (cache) { 
     cache.clear(); 
    } 
} 
} 

तरह लग रहा है क्योंकि सूची लोड बहुत भारी मैं एक अनुरोध मिला है कि अगर सूची लोड प्रगति पर है मैं वापस "पुराने" कैश्ड है डेटा ...

कि संभव है, और कोई मुझे कैसे यहां से आगे बढ़ने के लिए पर संकेत दे सकते हैं

संपादित करें: एडम होर्वाथ और bayou.io इस

की तरह कुछ sugested

public class MyList 
{ 
private final List<String> cache = new ArrayList<String>(); 
private final List<String> oldCache = new ArrayList<String>(); 
private volatile boolean loadInProgress = false; 

private List<String> loadMyList() 
{ 
    // HEAVY OPERATION TO LOAD DATA 
} 

public List<String> list() 
{ 
    synchronized (cache) 
    { 
     if(loadInProgress) 
      return Collections.unmodifiableList(oldCache); 
     else 
      return Collections.unmodifiableList(cache); 
    } 
} 

public void invalidateCache() 
{ 
    synchronized (cache) 
    { 
     // copy to old cache 
     oldCache = new ArrayList<String>(cache); 
     // set flag that load is in progress 
     loadInProgress = true; 
     // clear cache 
     cache.clear(); 

     // initialize load in new thread 
     Thread t = new Thread(new Runnable() 
     { 
      public void run() 
      { 
       cache.addAll(loadMyList()); 
       // set flag that load is finished 
       loadInProgress = false; 
      } 
     }); 
     t.start(); 


    } 

    } 
} 

क्या इस संपादित कोड में कोई समस्या है ?? जब से मैं बहु सूत्रण और/या अनुकूलन भुना में अनुभवी नहीं कर रहा हूँ मैं किसी भी और सभी प्रदर्शन संबंधी सुझाव देने

Answer 1

"क्योंकि सूची लोड बहुत भारी मैं एक अनुरोध मिला है कि अगर सूची लोड प्रगति पर है मैं वापस है" पुराने "कैश्ड की सराहना करेंगे डेटा ... "

यह आपके" सिंक्रनाइज़ (कैश) "ब्लॉक के कारण नहीं होगा। आपको एक सूची बुलाने के लिए आपको अस्थिर बुलियन ध्वज (म्यूटेक्स) की आवश्यकता है। जब कोई थ्रेड सूची प्राप्त करने का प्रयास करता है() और म्यूटेक्स सत्य है, तो उसे कैश किया जाएगा। LoadMyList() पूरा होने पर इसे गलत सेट करें।

तो, सिंक्रनाइज़ किए गए ब्लॉक को हटाएं और अपनी सूची को एक अलग थ्रेड में लोड करना प्रारंभ करें।

public class MyList { 
    private List<String> cache = new ArrayList<String>(); 
    private volatile boolean loadInProgress = false; 

    private List<String> loadMyList() { 
     // HEAVY OPERATION TO LOAD DATA 
    } 

    public List<String> list() { 
     // Whatever is in cache, you can always return it 
     return Collections.unmodifiableList(cache); 
    } 

    /** 
    * Starts the loader-thread and then continues. 
    */ 
    public void invalidateCache() { 
     // Next two lines make sure only one Loader-thread can be started at the same time 
     synchronized (cache) { 
      if (!loadInProgress) { 
       // initialize load in new thread 
       Thread t = new Thread("Loader-thread") { 
        @Override 
        public void run() { 
         List<String> toAssign = loadMyList(); 
         // You can simply assign instead of copying 
         cache = toAssign; 
         // cache now "points to" refreshed list 
         loadInProgress = false; 
        } 
       }; 
       loadInProgress = true; 
       t.start(); 
       // Now let's exit the synchronized block. Hopefully the Thread will start working soon 
      } else { 
       // A Thread is already working or about to start working, don't bother him 
      } 
     } 
    } 
}