Problem with closing RAMDictionary.

[Kleanthi]

Here is my code:

String wordNetFolder = System.getProperty("user.dir") + "/src/main/resources/wordnet/dict/";
File exFile = new File(wordNetFolder + "JWI_Export_.wn");
IRAMDictionary dictionary;
dictionary = new RAMDictionary(dictionaryFolder);
dictionary.setLoadPolicy(ILoadPolicy.IMMEDIATE_LOAD);
try {
    dictionary.open();
    dictionary.export(new FileOutputStream(exFile));
    dictionary.close();
} catch (IOException e1) {
    e1.printStackTrace();
}

When I do
assert(dictionary.isOpen() == false);

I get java.lang.AssertionError because the dictionary seems to still be open.
Why is this happening?

I am using the 2.4.0 version.

[rm-hull]

I would refactor your code as follows:

String wordNetFolder = System.getProperty("user.dir") + "/src/main/resources/wordnet/dict/";
File exFile = new File(wordNetFolder + "JWI_Export_.wn");
IRAMDictionary dictionary;
dictionary = new RAMDictionary(dictionaryFolder);
dictionary.setLoadPolicy(ILoadPolicy.IMMEDIATE_LOAD);
try {
    dictionary.open();
     dictionary.export(new FileOutputStream(exFile));
} catch (IOException e1) {
    e1.printStackTrace();
} finally {
    if (dictionary != null) {
        dictionary.close();
    }
}

[Kleanthi]

Thanks for you quick reply. I have done exactly what you suggested, but I also added a print to check if the dictionary is actually not null, after if (dictionary != null) . It seems that the dictionary is not null but it is still open after I call the function to close it.

[Kleanthi]

Sorry to bother again in such a short notice, but I have been doing some digging inside your classes and apparently in https://github.com/rm-hull/jwi/blob/master/src/edu/mit/jwi/RAMDictionary.java#L448 the state != LifecycleState.CLOSING should be state == LifecycleState.CLOSING, otherwise the close() function returns without actually setting the loader and the data into null, and without closing the backing and without changing the state of the dictionary in CLOSE. As a result, the dictionary stays loaded in memory.