使用 HashMap 更新具有相同键的多个值的问题

Question

我遇到了一个问题，即一个键/值对正在正确更新，但具有相同键的其他值没有得到更新。我正在使用一个 HashMap，它由一个字符串列表（项目 Nbr）作为键和存储为字符串列表（PO Nbr）的值组成，因此在这方面它本质上是一对多的。下面是我实现这个 HashMap 的代码，但我只是卡住了。任何帮助将不胜感激。

public void process(List<PoBean> excelData,List<String> errors) throws CustomException 
{       
    Map<String,ArrayList<String>> poNbrItemNbrMap = new HashMap<String, ArrayList<String>>();
    ArrayList<String> poNbrList = new ArrayList<String>();

    int recordCount = excelData.size();

    for(PoBean bean : excelData)
    {                                       
        if(rowValidator.validate(bean, errors))
        {
            if (!newErrorInRow(errors)) 
            {
                poNbrList.add(bean.getPoId());
                poNbrItemNbrMap.put(bean.getItemNbr(),bean.getPoId());
            }           
        }
    }

    List<DutyFees> dutyFeesTempDAO = myService.getAssocIdByPOAndItemNbr(new ArrayList<String>(poNbrItemNbrMap.keySet()),new ArrayList<String>(poNbrItemNbrMap.values()));           

    for(DutyFees duty : dutyFeesTempDAO)
    {   
        Double dutyFees = new Double(0);
        if(poNbrItemNbrMap.containsKey(duty.getItemNbr()) && poNbrItemNbrMap.containsValue(duty.getPoId())) 
        {               
            for(PoBean dutyBean : excelData)
            {
                if(duty.getPoId().equals(dutyBean.getPoId()) && duty.getItemNbr().equals(dutyBean.getItemNbr()))
                {
                    dutyFees = new Double(dutyBean.getDutyFees());
                    break;
                }
            }
            Boolean result = myService.updateDutyFeesByAssocID(duty.getAssocId(), dutyFees);            
        }                               
    }       
}

Answer 1

你说你想要一对多，但你的代码是一对一的

HashMap<String,String> poNbrItemNbrMap = new HashMap<String,String>();

一个字符串键对应一个字符串值

斜体部分与您的问题中的粗体部分形成对比。

我正在使用由字符串列表（项目 Nbr）组成的 HashMap 作为 存储为字符串列表 (PO Nbr) 的键和值， 所以它是 在这方面基本上是一对多。

斜体表示多对多。 粗体表示一对多。

无论如何，让我给你两个实现。

public class Test {
  public static void main(String[] args) {

    System.out.println("= One to Many =");
    Map<String, List<String>> stringListMap = new HashMap<>();
    String key = "list of 4 strings";
    List<String> valuesList = Arrays.asList("one","two","three","four");
    stringListMap.put(key,valuesList);
    stringListMap.forEach((strings, strings2) -> System.out.println(strings.toString() + strings2.toString()));

    System.out.println("= Many to Many =");
    Map<List<String>, List<String>> listListMap = new HashMap<>();
    List<String> keys = Arrays.asList("1","2","3","4");
    List<String> values = Arrays.asList("one","two","three","four");
    listListMap.put(keys,values);
    listListMap.forEach((strings, strings2) -> System.out.println(strings.toString() + strings2.toString()));


  }
}

---

EDIT 回复您的评论。 不确定您在那条评论中的意思，但我确实看到了几个问题。首先，当您输入地图时，您输入的内容似乎是项目编号和 po id，因此它们似乎都是单个字符串类型而不是 ArrayList 类型。所以我什至不确定你如何编译它。除非您的 getPoId 实际上是 getPoIds 并返回一个列表。我说的是这条线

poNbrItemNbrMap.put(bean.getItemNbr(),bean.getPoId());

现在关于

new ArrayList<String>(poNbrItemNbrMap.values())

你不能这样初始化数组列表。

我不知道 getAssocIdByPoAndItemNbr 函数的签名是什么，所以无法给出答案。但是，如果您只想发送映射值，那么直接发送它，为什么将其作为构造函数参数放入 new ArrayList()？

Answer 2

process 的第一部分的目的很明确：将PoBean 对象列表转换为Map<String, List<String>>，将对象按itemNbr 分组为一个列表。这是执行此操作的代码：

// One-to-many relationship requires the value to be a List
// Better not say ArrayList here because then you put unnecessary
// Restrictions on the code.
Map<String, List<String>> poNbrItemNbrMap = new HashMap<>();

for(PoBean bean : excelData)
{                                       
    if(rowValidator.validate(bean, errors))
    {
        if (!newErrorInRow(errors)) 
        {
            String key = bean.getItemNbr();
            List<String> value = poNbrItemNbrMap.get(key);
            if (value == null) {
                // This is first time we see this item nbr,
                // Create a new list to hold the values
                value = new ArrayList<>();
                poNbrItemNbrMap.put(key, value);
            }
            // Just add the value to the list
            value.add(bean.getPoId());
        }           
    }
}

现在第二部分令人困惑，尤其是这一行：

List<DutyFees> dutyFeesTempDAO = myService.getAssocIdByPOAndItemNbr(new ArrayList<String>(poNbrItemNbrMap.keySet()),new ArrayList<String>(poNbrItemNbrMap.values()));

第一个参数几乎没有意义，它以List 的形式传递键；没关系。第二个参数无法编译，因为您需要new ArrayList<List<String>>(poNbrItemNbrMap.values())。是的，您需要一个字符串列表。但这仅仅是开始。 这样你就基本消除了键和值之间的关系，getAssocIdByPOAndItemNbr 方法不能再关联键和它们各自的值了。它不能假设键中的第一项映射到值中的第一项。这意味着我们在第一部分所做的任何事情都是无用的。