Code: Selecteer alles
ArrayList<Order> orders;
Order ord1;
HashTable<Order, Vector<Order>> supports;
...
...
for(Order ord2 : orders){
if(ord1.equals(ord2)){
if(supports.get(ord1) == null){
System.out.println("something went wrong");
}
if(supports.get(ord2) == null){
System.out.println("something went wrong2");
}
}
}
a)Daar heb je je probleem. De equals-methode van Object vergelijkt pointers, zoals Bart al aangaf, en mag je dus NOOIT gebruiken (idem met de hashcode). Dus object.clone().equals(object)==falseohja, vergeten erbij te vermelden: de class Order implementeert geen nieuwe equals() methode, dus het gaat hier gewoon om de equals() methode die is overgeërfd van Object.
ja, dat snap ik, maar allebei de variabelen zijn niet null, dat heb ik gechecked.Bart schreef:equals() vergelijkt references en geen values. Als obj1 en obj2 beide een null reference hebben, dan zal obj1.equals(obj2) true terug geven
De standaard equals() methode in Java is gelijk aan "=="
Het is überhaupt nooit de bedoeling geweest om de equals() methode te gebruiken. Het probleem was dat supports.get(ord1) null geeft, terwijl supports.get(ord2) dat niet doet, terwijl ik dacht dat ord1 en ord2 toch echt naar hetzelfde object refereerden. De equals() methode heb ik hier dus alleen gebruikt om te checken of mijn hypothese klopte. En inderdaad blijken beide pointers naar hetzelfde object te verwijzen. Dus begrijp ik het probleem nog steeds niet.317070 schreef:a)Daar heb je je probleem. De equals-methode van Object vergelijkt pointers, zoals Bart al aangaf, en mag je dus NOOIT gebruiken (idem met de hashcode). Dus object.clone().equals(object)==false
b)als je hashen wil gebruiken, MOET je dus zelf een methode equals en een methode hashcode implementeren.
c) hashtable mag je niet gebruiken, hij wordt deprecated. HashMap is het juiste alternatief.
Bedankt voor de tip!c) hashtable mag je niet gebruiken, hij wordt deprecated. HashMap is het juiste alternatief.
Ik weet veel van Java, ik heb al op mijn eentje een full-blown media-managment programma gemaakt (lees: mp3 afspelen, binnen Java videos afspelen met de hulp van VLC-dll files die ik binnenhaalde via JNA, youtube-crawler en videos downloaden van youtube, dynamisch klassen binnenhalen die niet met mijn programma gecompiled zijn om als plugin te gebruiken, ...) Zie: http://sourceforge.net/projects/lyricscatcher/Hangt hier werkelijk niemand rond die genoeg van java weet om mij dit probleem uit te leggen???
In jouw implementatie zijn de 2 bovenstaande regels dus niet vervuld. Als je de functies equals() en hashcode() niet geïmplementeerd hebt, is het nogal vanzelfsprekend dat het niet werkt.http://www.ifs.tuwien.ac.at/ifs/lehre/eprog/javadoc/jdk/docs/api/java/lang/Object.html#hashCode%28%29 schreef:# If two objects are equal according to the equals(Object) method, then calling the hashCode method on each of the two objects must produce the same integer result.
# It is not required that if two objects are unequal according to the equals(java.lang.Object) method, then calling the hashCode method on each of the two objects must produce distinct integer results. However, the programmer should be aware that producing distinct integer results for unequal objects may improve the performance of hashtables.
http://mindprod.com/jgloss/hashcode.htmlThe default hashCode() method uses the 32-bit internal JVM address of the Object as its hashCode. However, if the Object is moved in memory during garbage collection, the hashCode stays constant. This default hashCode is not very useful, since to look up an Object in a HashMap, you need the exact same key Object by which the key/value pair was originally filed. Normally, when you go to look up, you don’t have the original key Object itself, just some data for a key. So, unless your key is a String, nearly always you will need to implement a hashCode and equals method on your key class. Object.hashCode in a native method.
hoe doe je dit in java? Bedoel je het geheugenadres waar ze naar verwijzen afdrukken?Heb je ord1 en ord2 al eens gewoon afgedrukt en gekeken of ze dezelfde referentie hebben?
Meuh, dat is toch geen werk om dat zelf te implementeren.Math-E-Mad-X schreef:@317070: Dus om HashTable / HashMap te gebruiken moet je eerst helemaal zelf een eigen HashCode gaan lopen definiëren? Niet echt praktisch....
Is er dan geen andere datastructuur die ik kan gebruiken om simpelweg objecten van klasse A naar objecten van klasse B te mappen? Ik kan wel wat manieren verzinnen om zoiets zelf te implementeren, maar ik had toch wel gedacht dat er een ready-to-use datastructuur was die dat voor je doet.
Code: Selecteer alles
public int hashCode()
{
int hash = 0;
hash = 31 * hash + field1.hashcode();
hash = 31 * hash + field2.hashcode();
hash = 31 * hash + field3.hashcode();
hash = 31 * hash + field4.hashcode();
return hash;
}Sorry, dit snap ik niet.In jouw implementatie zijn de 2 bovenstaande regels dus niet vervuld.
Ja, maar het probleem is dat ik deze klassen niet zelf geschreven heb, ze komen uit een framework.Nu als ik het goed begrijp gebruik je toch identieke objecten om dan terug in je hashtable op te zoeken. Is het dan niet logischer om gewoon een veld met object B binnen je object A op te slaan?
Code: Selecteer alles
ArrayList<Order> orders;
Order ord1;
HashTable<Order, Vector<Order>> supports;
...
...
for(Order ord2 : orders){
if(ord1 == ord2){
if(supports.get(ord1) == null){
System.out.println("something went wrong");
}
if(supports.get(ord2) == null){
System.out.println("something went wrong2");
}
}
}