Получение нулевого значения, когда я вызываю hashTable.get (key)

Question

В моем проекте есть класс Pair, и я использую хеш-таблицу в своем приложении.После создания моей хеш-таблицы я проверяю, что объекты Pair создаются и правильно хранятся в хеш-таблице, печатая содержимое хеш-таблицы, и сразу же я пытаюсь получить одно из значений с помощью метода get (key), и он всегда даетme null.

Это весь мой класс, Mapping, который имеет закрытый объект типа hashtable package metastore;

import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Enumeration;
import java.util.Hashtable;
import java.util.StringTokenizer;
import org.apache.hadoop.hive.ql.parse.ASTNode;
import preprocessingQuery.Pair;

public class Mapping {
private Hashtable<Pair, Pair> hashTable ;

public Mapping(){
    hashTable= new Hashtable<Pair, Pair>();
}


public Hashtable<Pair, Pair> getHashTable() {
    return hashTable;
}


public void setHashTable(Hashtable<Pair, Pair> hashTable) {
    this.hashTable = hashTable;
}


public Pair getMapping( Pair originalPair) {
    Pair mappedPair=(hashTable.get(originalPair));
    return mappedPair;
}
public ArrayList<Mapping> getPairs(ASTNode an){
    ArrayList<Mapping> pairs=new ArrayList<Mapping>();
    return pairs;
}

public void print() {
    Enumeration<Pair> contentOfHT;
    contentOfHT = hashTable.keys(); 
    while(contentOfHT.hasMoreElements()) { 
    Object str =  contentOfHT.nextElement(); 
    System.out.println(str + "\tis mapped to " + 
            hashTable.get(str)); 
    } 
}


public void loadMappingTable() {
    String originalTable;
    String originalCol;
    String mappedTable;
    String mappedCol;
    Pair originalPair;
    Pair mappedPair;
    BufferedReader in = null;

    try {
        in = new BufferedReader(
                new FileReader(
                        "D:\\Documents and Settings\\QUAdmin.STAFF\\Desktop\\mapping.txt"));
        String line ;
        while ((line = in.readLine()) != null) {
            StringTokenizer stok = new StringTokenizer(line, "\t");
            originalTable= stok.nextToken();
            originalCol= stok.nextToken();
            mappedTable= stok.nextToken();
            mappedCol= stok.nextToken();
            originalPair=new Pair(originalTable,originalCol);
            mappedPair=new Pair(mappedTable,mappedCol);
            hashTable.put(originalPair, mappedPair);

        }
    } catch (Exception ex) {
        // catch all exceptions as one. This is bad form imho
        ex.printStackTrace();
    } finally {
        try {
            if (in != null)
                in.close();
        } catch (IOException ex) {
        }
    }
}

public static void main(String[] args)
{
    Mapping map=new Mapping();
    map.loadMappingTable();
    System.out.println("Size: "+ map.getHashTable().size());

    System.out.println("The content of the hash table");
    map.print();
    System.out.println("Testing the mapping");
    Pair originalPair=new Pair("table1","table1_name");
    System.out.println(map.getMapping(originalPair));
    System.out.println(map.getHashTable().get(originalPair));
    System.out.println(map.getHashTable());

}
}//end of Mapping Class

И это вывод

Size: 3

The content of the hash table

[table=table1, col=table1_age]  is mapped to [table=table1_SNT, col=table1_SNT_age]

[table=table1, col=table1_name] is mapped to [table=table1_SNT, col=table1_SNT_name]

[table=table1, col=table1_id]   is mapped to [table=table1_SNT, col=table1_SNT_id]

Testing the mapping

null

null

{[table=table1, col=table1_age]=[table=table1_SNT, col=table1_SNT_age], [table=table1, col=table1_name]=[table=table1_SNT, col=table1_SNT_name], [table=table1, col=table1_id]=[table=table1_SNT, col=table1_SNT_id]}

Спасибо

Answer 1

Мне нужно увидеть вашу реализацию Pair. Я предполагаю, что вы не правильно используете equals и hashcode.

---

[Изменить]

Учитывая вашу реализацию Pair (взято из комментариев)

package preprocessingQuery; 
public class Pair { 
    private String table; 
    private String col; 
    public Pair(String table, String col) { 
        super(); 
        this.table = table; 
        this.col = col; 
    }

    public String getTable() { 
        return table; 
    }

    public void setTable(String table) { 
        this.table = table; 
    }

    public String getCol() { 
        return col; 
    }

    public void setCol(String col) { 
        this.col = col; 
    } 

    @Override public String toString() { 
        return "[table=" + table + ", col=" + col + "]"; 
    } 
}

Вам действительно не хватает равенства и хэш-кода. Немного предыстории: реализация по умолчанию Object.equals и Object.hashCode основана на адресе памяти объекта (ссылка на объект). С этой точки зрения все ваши пары разные, так как они разные объекты.

Для правильной работы любой реализации коллекции необходимо переопределить реализацию по умолчанию для equals и hashCode объектов, которые будут храниться в коллекции.

Для вашего класса Pair это должно выглядеть примерно так:

@Override
public boolean equals(Object other) {
    if (this == other) {
        return true; // shortcut for referential equality
    }
    if (other == null) {
        return false; // by definition, 'this' object is not null
    }
    if (!(other instanceof Pair)) {
        return false;
    }
    Pair otherPair  = (Pair) other; // Cast to the known type
    // check equality of the members
    if (this.table == null) {  
        if (otherPair.table != null) {
            return false;
        }
    } else if (!this.table.equals(otherPair.table)) {
        return false;
    }
    if (this.col == null) {  
        if (otherPair.col != null) {
            return false;
        }
    } else if (!this.col.equals(otherPair.col)) {
        return false;
    }
    return true;
}

HashCode следует за набором. Вы должны понимать и следовать генеральному контракту хэш-кода .

@Override
public int hashCode() {
    int hash = this.table==null?0:table.hashCode();
    hash += 41 * this.col==null?0:col.hashCode();
    return hash;
 }

Answer 2

Это связано с тем, что вы не переопределили методы equals и hashCode в классе Pair или, по крайней мере, они не переопределены должным образом. Когда вы вызываете 'get' для хеш-таблицы, она сначала вызывает метод hashCode, чтобы найти запись в своей таблице. Если hashCode не переопределен должным образом, то hashtable не найдет вашу запись. Во-вторых, когда hashtable найдет запись, он проверит, что ключ записи равен предоставленному вами (в случае столкновения hashCode). Вы можете переопределить эти методы следующим образом:

public int hashCode {
   return table.hashCode()+tableName.hashCode();
}

public boolean equals(Object o) {
   if (o==this)
       return true;
   if (o instanceof Pair) {
       Pair p = (Pair) o;
       return this.table.equals(p.table) && this.tableName.equals(p.tableName);
   }
   return false;
}

Наконец, когда вы выполняете итерацию по Hashtable (и, в более общем смысле, по карте), вы не должны вызывать ключи и делать get (ключ), но вместо этого вы должны выполнять итерации непосредственно по записям

for(Entry<K,V> e: map.entrySet()) {
   System.err.println(e.getKey+" is mapped to "+e.getValue());
}

Это гораздо эффективнее, потому что это не вызовет hashCode и равнозначно методам (как объяснено выше), которые могут быть дорогостоящими операциями.

Answer 3

Переопределить equals и hashcode в классе Pair.