добавление двух объектов в hashMap?

Question

Правильно ли я добавил элемент в hashTable?

Flows flows = new Flows(sIP,dIP); 
FlowStatics flowStatics = new FlowStatics(packetLength,timeArrival); 

HashMap myHashMap = new HashMap<Flows, FlowStatics>(); 
myHashMap.put(flows, flowStatics); 

Answer 1

Чтобы избежать предупреждений компилятора, кода это следующим образом:

HashMap<Flows, FlowStatics> myHashMap = new HashMap<Flows, FlowStatics>(); 
myHashMap.put(flows, flowStatics); 

Если вы не Параметризуем переменную myHashMap , то вы не можете добавить во вторую строку без предупреждения.

---

Здесь работает пример о том, как 'печать' некоторые статистические данные Hashmap:

HashMap<Flows, FlowStatics> myHashMap = new HashMap<Flows, FlowStatics>(); 
for (int i = 0; i < 10; i++) { 
    // OP commented that the map is populated in a loop 
    myHashMap.put(createNewFlow(), createNewFlowStatistics()); // populate map 
} 
System.out.printf("Number of items in Map: %s%n", myHashMap.keyset().size()); 

(попросил ОП для adivice в комментарии к другому ответу)

Answer 2

Код выглядит нормально.

Однако вы должны убедиться, что Flows переопределение equals и hashCode

Answer 3

Я хотел бы изменить HashMap myHashMap = new HashMap<Flows, FlowStatics>(); к HashMap<Flows, FlowStatics> myHashMap = new HashMap<Flows, FlowStatics>(); но addig Абсолютно нормально.

Answer 4

Замените эту строку

HashMap myHashMap = new HashMap<Flows, FlowStatics>(); 

с этим одним

Map<Flows, FlowStatics> myHashMap = new HashMap<Flows, FlowStatics>(); 

Answer 5

@Shervin

не так ли?

package myclassifier; 


public class Flows implements Comparable<Flows> { 

    String srcAddr, dstAddr, srcPort, dstPort, protocol; 

    public Flows(String sIP, String dIP){ 
     this.srcAddr = sIP; 
     this.dstAddr = dIP; 
    } 

    public int compareTo(Flows other) { 
      int res = this.srcAddr.compareTo(other.srcAddr); 
      if (res != 0) { 
       return res; 
      } 
      res = this.dstAddr.compareTo(other.dstAddr); 
      if (res != 0) { 
       return res; 
      } 
      res = this.srcPort.compareTo(other.srcPort); 
      if (res != 0) { 
       return res; 
      } 
      res = this.dstPort.compareTo(other.dstPort); 
      if (res != 0) { 
       return res; 
      } 
      return this.protocol.compareTo(other.protocol); 
} 

    @Override 
    public int hashCode() { 

     final int prime = 31; 
     int result = 1; 
     result = prime * result + ((dstAddr == null) ? 0 : dstAddr.hashCode()); 
     result = prime * result + ((dstPort == null) ? 0 : dstPort.hashCode()); 
     result = prime * result + ((srcAddr == null) ? 0 : srcAddr.hashCode()); 
     result = prime * result + ((srcPort == null) ? 0 : srcPort.hashCode()); 
     return result; 

    } 


    @Override 
    public boolean equals(Object obj) { 
     if (this == obj) 
      return true; 
     if (obj == null) 
      return false; 

     if (getClass() != obj.getClass()) 
      return false; 

     Flows other = (Flows) obj; 

     if (dstAddr == null) { 
      if (other.dstAddr != null) 
       return false; 
     } else if (!dstAddr.equals(other.dstAddr)) 
      return false; 

     if (dstPort == null) { 
      if (other.dstPort != null) 
       return false; 
     } else if (!dstPort.equals(other.dstPort)) 
      return false; 

     if (srcAddr == null) { 
      if (other.srcAddr != null) 
       return false; 
     } else if (!srcAddr.equals(other.srcAddr)) 
      return false; 

     if (srcPort == null) { 
      if (other.srcPort != null) 
       return false; 
     } else if (!srcPort.equals(other.srcPort)) 
      return false; 

     return true; 
    } 

} 

Answer 6

Если вы не собираетесь снова использовать flows и flowStatics переменные -

HashMap<Flows, FlowStatics> myHashMap = new HashMap<Flows, FlowStatics>(); 
myHashMap.put(new Flows(sIP,dIP), new FlowStatics(packetLength,timeArrival));