import static com.google.common.base.Preconditions.checkNotNull;
import java.util.Collection;
+import java.util.Collections;
import java.util.Iterator;
import java.util.Map;
/** The network this bot is on. */
private final Network network;
+ private final String channel;
/** The packs this bot carries. */
private final Map<String, Pack> packs = Maps.newHashMap();
+ private final Map<String, Pack> packsByName = Maps.newHashMap();
/** The current name of the bot. */
private String name;
- /**
- * Creates a new bot.
- *
- * @param network
- * The network the bot is on
- * @param name
- * The name of the bot
- */
- public Bot(Network network, String name) {
+ public Bot(Network network, String channel, String name) {
this.network = checkNotNull(network, "network must not be null");
+ this.channel = checkNotNull(channel, "channel must not be null");
this.name = checkNotNull(name, "name must not be null");
}
return network;
}
+ public String channel() {
+ return channel;
+ }
+
/**
* Returns the current name of this bot.
*
* @return The packs this bot carries
*/
public Collection<Pack> packs() {
- return packs.values();
+ synchronized (packs) {
+ return Collections.unmodifiableCollection(packs.values());
+ }
}
//
* The pack to add
*/
public void addPack(Pack pack) {
- packs.put(pack.id(), pack);
+ synchronized (this) {
+ if (packsByName.containsKey(pack.name())) {
+ Pack oldPack = packsByName.remove(pack.name());
+ packs.remove(oldPack.id());
+ }
+ packs.put(pack.id(), pack);
+ packsByName.put(pack.name(), pack);
+ }
}
//
@Override
public Iterator<Pack> iterator() {
- return packs.values().iterator();
+ return packs().iterator();
}
//