hongshaorou/openrocket
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Instead of using Set<Manufacturer> for the backing store use ConcurrentHashMap<String,Manufacturer>. The greately reduces contention during startup with the multiple parallel calls to Manufacturer.get(String).

Browse Source
This commit is contained in:
Kevin Ruland 2012-08-23 19:55:20 +00:00
parent 54ea94f76b
commit d56298d296
1 changed files with 26 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

50
core/src/net/sf/openrocket/motor/Manufacturer.java
View File

@ -6,6 +6,7 @@ import java.util.HashSet;
import java.util.List; import java.util.List;
import java.util.Locale; import java.util.Locale;
import java.util.Set; import java.util.Set;
import java.util.concurrent.ConcurrentHashMap;
/** /**
* Class containing information about motor manufacturers. * Class containing information about motor manufacturers.
@ -13,8 +14,22 @@ import java.util.Set;
* @author Sampo Niskanen <sampo.niskanen@iki.fi> * @author Sampo Niskanen <sampo.niskanen@iki.fi>
*/ */
public class Manufacturer { public class Manufacturer {
private static Set<Manufacturer> manufacturers = new HashSet<Manufacturer>(); private static class ManufacturerList extends ConcurrentHashMap<String,Manufacturer> {
void add( Manufacturer m ) {
for( String s : m.getAllNames() ) {
Manufacturer previousRegistered;
if ( (previousRegistered = putIfAbsent( s, m )) != null ) {
throw new IllegalStateException("Manufacturer name clash between " +
"manufacturers " + previousRegistered + " and " + m + " name " + s);
}
}
}
}
private static ManufacturerList manufacturers = new ManufacturerList();
static { static {
// AeroTech has many name combinations... // AeroTech has many name combinations...
@ -99,20 +114,6 @@ public class Manufacturer {
"WECO", "WECO FEUERWERKS", "SF", "SACHSEN", "SACHSEN FEUERWERK", "WECO", "WECO FEUERWERKS", "SF", "SACHSEN", "SACHSEN FEUERWERK",
"SACHSEN FEUERWERKS")); "SACHSEN FEUERWERKS"));
// Check that no duplicates have appeared
for (Manufacturer m1 : manufacturers) {
for (Manufacturer m2 : manufacturers) {
if (m1 == m2)
continue;
for (String name : m1.getAllNames()) {
if (m2.matches(name)) {
throw new IllegalStateException("Manufacturer name clash between " +
"manufacturers " + m1 + " and " + m2 + " name " + name);
}
}
}
}
} }
@ -229,15 +230,16 @@ public class Manufacturer {
* @param name the manufacturer name to search for. * @param name the manufacturer name to search for.
* @return the Manufacturer object corresponding the name. * @return the Manufacturer object corresponding the name.
*/ */
public static synchronized Manufacturer getManufacturer(String name) { public static Manufacturer getManufacturer(String name) {
for (Manufacturer m : manufacturers) { Manufacturer m = manufacturers.get(name);
if (m.matches(name)) if ( m != null ) {
return m; return m;
} }
Manufacturer m = new Manufacturer(name.trim(), name.trim(), Motor.Type.UNKNOWN); m = new Manufacturer(name.trim(), name.trim(), Motor.Type.UNKNOWN);
manufacturers.add(m);
return m; Manufacturer oldManu = manufacturers.putIfAbsent(name, m);
return (oldManu != null) ? oldManu : m;
} }