Performance Issue

[Nilesh Raje]

Hi Friends,

I came across this question and was unable to answer this. Can anyone help me regarding this question?

This is the given Class

public class SlowDictionary {
private final Map<String, String> dict = new HashMap<String, String>();

public synchronized String translate(String word)
throws IllegalArgumentException {
if (!dict.containsKey(word)) {
throw new IllegalArgumentException(word + " not found.");
}
return dict.get(word);
}

public synchronized void addToDictionary(String word, String translation)
throws IllegalArgumentException {
if (dict.containsKey(word)) {
throw new IllegalArgumentException(word + " already exists.");
}
dict.put(word, translation);
}

public synchronized Set<String> getAllWords() {
return dict.keySet();
}
}

Here were the questions asked.

1. This class has a serious flaw. What is it and how can it be fixed?.
2. Improve performance of the class in concurrent environment using the Java Concurrency package.
Explain why performance is better.
3. Write a test which demonstrates performance improvement in a concurrent environment.

Thanks

[Nitesh: Added CodeTags]

[Ulf Dittmer]

Help with what, specifically? This looks like homework that you should be doing in order to learn. Folks here are not going to do it for you.

So: What thoughts have you had on any of these? Where are you stuck making progress?

[Nilesh Raje]

Yes it is homework. But I am not good at Synchronize thingy.! My concepts are not clear about this. I did do my other homework . So please do tell me what would be your answers? I will make sure i shall study that later but the reason i posted is i am running short of time n i need this to be done asap.

Thanks

[Nitesh Kant]

Nilesh Raje wrote:I did do my other homework .

We have seen two of your assignments in this forum :twisted: so are you really sure you did the other homework

Anyways, going through this tutorial will not take much time and will make things clear for you to do this assignment.