Question

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Iterator;
import java.util.Map;
import java.util.AbstractMap;

/**
 * A simple model of a mail server. The server is able to receive
 * mail items for storage, and deliver them to clients on demand.
 */
 public class MailServer
 {
    // Storage for the arbitrary number of mail items to be stored
    // on the server.
    private HashMap<String,ArrayList<MailItem>> mailMap;

    /**
     * Constructor
     */
    public MailServer()
    {
        mailMap = new HashMap<String,ArrayList<MailItem>>();
    }

    /**
     * Return how many mail items are waiting for a user.
     */
    public int howManyMailItems(String who)
    {
        int count = 0;
        for(ArrayList<MailItem> array : mailMap.values()) {
            for (MailItem item : array) {
                if (item.getTo().equals(who)) {
                    count++;
                }
            }
        }
        return count;
    }

    //  public int howManyMailItems(String who) 
    //  {    
    //      return mailMap.get(who).size(); 
    //  }

    /**
     * Return the next mail item for a user or null if there
     * are none.
     */
    public MailItem getNextMailItems(String who, int howMany)
    {
        // Access the ArrayList for "who" remove and return the first element
        // Be careful what if "who" doesn't have an entry in the mailMap
        // Or what if "who" doesn't have any mail?
        Iterator<Map.Entry<String, ArrayList<MailItem>>> it = 
        mailMap.entrySet().iterator();
        while(it.hasNext()) {
            Map.Entry<String, ArrayList<MailItem>> entry = it.next();
            String key = entry.getKey();
            ArrayList<MailItem> value = entry.getValue();
            if (key.equals(who));
            {
                return value.remove(0);
            }
        }
        return null;
    }

    /**
     * Add the given mail item to the message list.
     */
    public void post(String who)
    {
        if (mailMap.containsKey(who)) {
            Map.put(who, Map.get(who) + 1);
        }

    }
}

以上代码适用于基本邮件服务器。我试图使用String键和ArrayList（MailItems）值在一个HashMap中存储一个MailItem（String recipient，String subject，String message）。

我遇到的问题是post（）方法。我无法弄清楚如何使它取消息的目标参数并将其存储在相应的ArrayList中。

我也遇到了getNextMailItems（）方法的问题。我无法弄清楚如何让它从收件人的ArrayList返回多个项目。我能够弄清楚的是添加一个参数，指定要返回多少个MailItems。

我对java非常缺乏经验，而且还在学习。请帮忙。谢谢大家。

Answer 1

由于您正在学习Java，我要指出一些事项：

请尝试番石榴MultiMap而不是更丑陋的事情：
```
private HashMap<String,ArrayList<MailItem>> mailMap;
```
还尝试使用接口而不是类，即Map<String,List<MailItem>会更好，因为它不会将您绑定到特定的实现。
这不是最佳的，我怀疑是错的：
```
int count = 0;
for(ArrayList<MailItem> array : mailMap.values()) {
  for (MailItem item : array) {
    if (item.getTo().equals(who)) {
      count++;
    }
  }
}
```
您应该在地图上调用get(who)，如果您获得非空列表，则会为MailItem提供who列表。然后调用List.size()将为您提供所需的计数（我怀疑）。

关于这段代码的两点说明：

Iterator<Map.Entry<String, ArrayList<MailItem>>> it = mailMap.entrySet().iterator();
while(it.hasNext()) {
  Map.Entry<String, ArrayList<MailItem>> entry = it.next();
  String key = entry.getKey();
  ArrayList<MailItem> value = entry.getValue();
  if (key.equals(who)); // the semicolon here means that if the condition is true, do nothing
  {
    return value.remove(0); // this block is ALWAYS EXECUTED
  }
}
return null;

如果if中的条件为真，则由于分号而不会执行任何操作，并且代码块中的代码将始终执行第一次时间。也就是说，您再次获得了密钥，因此您无需遍历整个地图（请参阅上一点）。但是，如果您想迭代地图，请使用更紧凑的习语：

for(Map.Entry<String,ArrayList<MailItem>> e : mailMap.entrySet()){
  // do something with the key and value, e.getKey(), e.getValue()
}

最后，这个街区令人费解：
```
if (mailMap.containsKey(who)) {
  Map.put(who, Map.get(who) + 1);
}
```
你想做什么？ Map是接口的名称，没有静态方法put或get，因此甚至无法编译。如果您尝试为mailMap键添加元素who，那么您可能需要类似（列表地图的典型习惯用法，再次参见Guava以获得更好的API）：< / p>
```
ArrayList<MailItem> l = mailMap.get(who);
if(l == null){ // this super verbose code will simplify with the Guava MultiMap
  l = new ArrayList<MailItem>();
  mailMap.put(who, l);
}
l.add(/* code to create a new MailItem */);
```

在arraylist中存储信息，这是hashmap中的值

1 个答案: